Changeset 247223 in webkit

Timestamp:

Jul 8, 2019 12:03:42 PM (5 years ago)

Author:

Wenson Hsieh

Message:

[iPadOS] Viewport pops while loading sohu.com, xinhuanet.com, and various other websites
​https://bugs.webkit.org/show_bug.cgi?id=199575
<rdar://problem/51842220>

Reviewed by Simon Fraser.

Source/WebKit:

Currently, the shrink-to-fit-content heuristic added to scale desktop sites fit within the viewport on iPadOS
only runs during two intervals: after document load, and after page load. On very script-heavy websites, this
may cause a visible jump, as rendering may commence well before the document is finished parsing.

To mitigate this, we move the first opportunity for the shrink-to-fit heuristic from after document, to right
after the page transition has completed (before we unfreeze the layer tree).

• WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::dispatchDidFinishDocumentLoad):

• WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::didCompletePageTransition):
(WebKit::WebPage::didFinishDocumentLoad): Deleted.

Remove this didFinishDocumentLoad hook, which was only used to schedule the shrink-to-fit timer.

• WebProcess/WebPage/WebPage.h:
• WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::WebPage::immediatelyShrinkToFitContent):

m_mainFrame might be null after page transition completes, so we need to ensure immediatelyShrinkToFitContent is
robust in this scenario.

LayoutTests:

Adjusts an existing layout test, and introduces a new test to verify that the shrink-to-fit heuristic is given a
chance to run before document load. See WebKit ChangeLog for more details.

• fast/viewport/ios/shrink-to-fit-content-before-document-load-expected.txt: Added.
• fast/viewport/ios/shrink-to-fit-content-before-document-load.html: Added.
• fast/viewport/ios/shrink-to-fit-content-temporary-overflow.html:

Tweak this layout test to add the temporarily overflowing element after document load.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/fast/viewport/ios/shrink-to-fit-content-before-document-load-expected.txt (added)
• LayoutTests/fast/viewport/ios/shrink-to-fit-content-before-document-load.html (added)
• LayoutTests/fast/viewport/ios/shrink-to-fit-content-temporary-overflow.html (modified) (2 diffs)
• Source/WebKit/ChangeLog (modified) (1 diff)
• Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (modified) (1 diff)
• Source/WebKit/WebProcess/WebPage/WebPage.cpp (modified) (2 diffs)
• Source/WebKit/WebProcess/WebPage/WebPage.h (modified) (1 diff)
• Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2019-07-08  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iPadOS] Viewport pops while loading sohu.com, xinhuanet.com, and various other websites
+        https://bugs.webkit.org/show_bug.cgi?id=199575
+        <rdar://problem/51842220>
+
+        Reviewed by Simon Fraser.
+
+        Adjusts an existing layout test, and introduces a new test to verify that the shrink-to-fit heuristic is given a
+        chance to run before document load. See WebKit ChangeLog for more details.
+
+        * fast/viewport/ios/shrink-to-fit-content-before-document-load-expected.txt: Added.
+        * fast/viewport/ios/shrink-to-fit-content-before-document-load.html: Added.
+        * fast/viewport/ios/shrink-to-fit-content-temporary-overflow.html:
+
+        Tweak this layout test to add the temporarily overflowing element after document load.
+
 2019-07-08  Wenson Hsieh  <wenson_hsieh@apple.com>
 

trunk/LayoutTests/fast/viewport/ios/shrink-to-fit-content-temporary-overflow.html

@@ -49 +49 @@
 description("This test verifies that a temporary change in content width does not cause the viewport width to permanently expand to try and accomodate the content. To test manually, load the page and check that the box below reads 'PASS'. This test is only intended to run on devices with less than 1200px screen width.");
 
+function forceLayoutByScrollingTo(y) {
+    document.scrollingElement.scrollTo(0, y);
+    return document.scrollingElement.scrollTop;
+}
+
+addEventListener("DOMContentLoaded", () => {
+    forceLayoutByScrollingTo(1);
+    const bar = document.createElement("div");
+    bar.classList.add("bar");
+    document.body.appendChild(bar);
+    forceLayoutByScrollingTo(0);
+    bar.remove();
+});
+
 addEventListener("load", async () => {
     if (!window.testRunner)
@@ -63 +77 @@
 <div id="description"></div>
 <div class="square"></div>
-<script>
-const bar = document.createElement("div");
-bar.classList.add("bar");
-document.body.appendChild(bar);
-document.scrollingElement.scrollTo(0, 1);
-document.scrollingElement.scrollTo(0, 0);
-document.scrollingElement.scrollTop;
-bar.remove();
-</script>
 </body>
 </html>

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2019-07-08  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        [iPadOS] Viewport pops while loading sohu.com, xinhuanet.com, and various other websites
+        https://bugs.webkit.org/show_bug.cgi?id=199575
+        <rdar://problem/51842220>
+
+        Reviewed by Simon Fraser.
+
+        Currently, the shrink-to-fit-content heuristic added to scale desktop sites fit within the viewport on iPadOS
+        only runs during two intervals: after document load, and after page load. On very script-heavy websites, this
+        may cause a visible jump, as rendering may commence well before the document is finished parsing.
+
+        To mitigate this, we move the first opportunity for the shrink-to-fit heuristic from after document, to right
+        after the page transition has completed (before we unfreeze the layer tree).
+
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::dispatchDidFinishDocumentLoad):
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::didCompletePageTransition):
+        (WebKit::WebPage::didFinishDocumentLoad): Deleted.
+
+        Remove this didFinishDocumentLoad hook, which was only used to schedule the shrink-to-fit timer.
+
+        * WebProcess/WebPage/WebPage.h:
+        * WebProcess/WebPage/ios/WebPageIOS.mm:
+        (WebKit::WebPage::immediatelyShrinkToFitContent):
+
+        m_mainFrame might be null after page transition completes, so we need to ensure immediatelyShrinkToFitContent is
+        robust in this scenario.
+
 2019-07-08  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp

@@ -615 +615 @@
     // Notify the UIProcess.
     webPage->send(Messages::WebPageProxy::DidFinishDocumentLoadForFrame(m_frame->frameID(), navigationID, UserData(WebProcess::singleton().transformObjectsToHandles(userData.get()).get())));
-
-    webPage->didFinishDocumentLoad(*m_frame);
 }
 

trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp

@@ -3271 +3271 @@
 void WebPage::didCompletePageTransition()
 {
+#if ENABLE(VIEWPORT_RESIZING)
+    if (immediatelyShrinkToFitContent())
+        viewportConfigurationChanged(ZoomToInitialScale::Yes);
+#endif
+
     unfreezeLayerTree(LayerTreeFreezeReason::PageTransition);
 
@@ -5796 +5801 @@
 }
 
-void WebPage::didFinishDocumentLoad(WebFrame& frame)
-{
-    if (!frame.isMainFrame())
-        return;
-
-#if ENABLE(VIEWPORT_RESIZING)
-    scheduleShrinkToFitContent();
-#endif
-}
-
 void WebPage::didFinishLoad(WebFrame& frame)
 {

trunk/Source/WebKit/WebProcess/WebPage/WebPage.h

@@ -360 +360 @@
     void willReplaceMultipartContent(const WebFrame&);
     void didReplaceMultipartContent(const WebFrame&);
-    void didFinishDocumentLoad(WebFrame&);
     void didFinishLoad(WebFrame&);
     void show();

trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm

@@ -3321 +3321 @@
         return false;
 
+    if (!m_mainFrame)
+        return false;
+
     if (!m_page->settings().allowViewportShrinkToFitContent())
         return false;