Title: [202574] trunk/Source/WebKit2
- Revision
- 202574
- Author
- [email protected]
- Date
- 2016-06-28 11:07:05 -0700 (Tue, 28 Jun 2016)
Log 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):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (202573 => 202574)
--- trunk/Source/WebKit2/ChangeLog 2016-06-28 17:23:43 UTC (rev 202573)
+++ trunk/Source/WebKit2/ChangeLog 2016-06-28 18:07:05 UTC (rev 202574)
@@ -1,3 +1,19 @@
+2016-06-28 Chris Dumez <[email protected]>
+
+ [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):
+
2016-06-27 Anders Carlsson <[email protected]>
WebKit::WebPaymentCoordinator leak
Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (202573 => 202574)
--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-06-28 17:23:43 UTC (rev 202573)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-06-28 18:07:05 UTC (rev 202574)
@@ -2317,9 +2317,11 @@
process().send(Messages::WebPage::RestoreSession(m_backForwardList->itemStates()), m_pageID);
- // The back / forward list was restored from a sessionState so we don't want to snapshot the current
- // page when navigating away. Suppress navigation snapshotting until the next load has committed.
- m_suppressNavigationSnapshotting = true;
+ if (navigate) {
+ // The back / forward list was restored from a sessionState so we don't want to snapshot the current
+ // page when navigating away. Suppress navigation snapshotting until the next load has committed
+ m_suppressNavigationSnapshotting = true;
+ }
}
// FIXME: Navigating should be separate from state restoration.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
