Changeset 202574 in webkit


Ignore:
Timestamp:
Jun 28, 2016 11:07:05 AM (8 years ago)
Author:
Chris Dumez
Message:

[WK2] Do not suppress navigation snapshotting on session restore if we are not navigating right away
https://bugs.webkit.org/show_bug.cgi?id=159216
<rdar://problem/27058360>

Reviewed by Andreas Kling.

Do not suppress navigation snapshotting on session restore if we are not navigating
right away. The snapshot we already have may be outdated by the time we actually
navigate (e.g. Because the user scrolled) so we don't want to suppress snapshotting
in this case.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::restoreFromSessionState):

Location:
trunk/Source/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r202570 r202574  
     12016-06-28  Chris Dumez  <cdumez@apple.com>
     2
     3        [WK2] Do not suppress navigation snapshotting on session restore if we are not navigating right away
     4        https://bugs.webkit.org/show_bug.cgi?id=159216
     5        <rdar://problem/27058360>
     6
     7        Reviewed by Andreas Kling.
     8
     9        Do not suppress navigation snapshotting on session restore if we are not navigating
     10        right away. The snapshot we already have may be outdated by the time we actually
     11        navigate (e.g. Because the user scrolled) so we don't want to suppress snapshotting
     12        in this case.
     13
     14        * UIProcess/WebPageProxy.cpp:
     15        (WebKit::WebPageProxy::restoreFromSessionState):
     16
    1172016-06-27  Anders Carlsson  <andersca@apple.com>
    218
  • trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp

    r202540 r202574  
    23182318        process().send(Messages::WebPage::RestoreSession(m_backForwardList->itemStates()), m_pageID);
    23192319
    2320         // The back / forward list was restored from a sessionState so we don't want to snapshot the current
    2321         // page when navigating away. Suppress navigation snapshotting until the next load has committed.
    2322         m_suppressNavigationSnapshotting = true;
     2320        if (navigate) {
     2321            // The back / forward list was restored from a sessionState so we don't want to snapshot the current
     2322            // page when navigating away. Suppress navigation snapshotting until the next load has committed
     2323            m_suppressNavigationSnapshotting = true;
     2324        }
    23232325    }
    23242326
Note: See TracChangeset for help on using the changeset viewer.