Title: [204637] trunk/Source
Revision
204637
Author
[email protected]
Date
2016-08-19 11:10:23 -0700 (Fri, 19 Aug 2016)

Log Message

Rename didLayout(LayoutMilestones) to didReachLayoutMilestone(), and related WK2 functions
https://bugs.webkit.org/show_bug.cgi?id=160923

Reviewed by Tim Horton.

Source/WebCore:

didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)

* dom/Document.cpp:
(WebCore::Document::setVisualUpdatesAllowed):
* loader/EmptyClients.h:
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::didReachLayoutMilestone):
(WebCore::FrameLoader::didLayout): Deleted.
* loader/FrameLoader.h:
* loader/FrameLoaderClient.h:
* page/FrameView.cpp:
(WebCore::FrameView::fireLayoutRelatedMilestonesIfNeeded):
(WebCore::FrameView::firePaintRelatedMilestonesIfNeeded):
* page/LayoutMilestones.h: Formatting
* page/Page.cpp:
(WebCore::Page::addRelevantRepaintedObject):

Source/WebKit/mac:

* WebCoreSupport/WebFrameLoaderClient.h:
* WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
(WebFrameLoaderClient::dispatchDidLayout): Deleted.

Source/WebKit/win:

didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)

* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
(WebFrameLoaderClient::dispatchDidLayout): Deleted.
* WebCoreSupport/WebFrameLoaderClient.h:

Source/WebKit2:

didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)

Avoided changing functions that are exposed as API/SPI.

* UIProcess/API/APILoaderClient.h:
(API::LoaderClient::didReachLayoutMilestone):
(API::LoaderClient::didLayout): Deleted.
* UIProcess/API/C/WKPage.cpp:
(WKPageSetPageLoaderClient):
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didLayoutForCustomContentProvider):
(WebKit::WebPageProxy::didReachLayoutMilestone):
(WebKit::WebPageProxy::didLayout): Deleted.
* UIProcess/WebPageProxy.h:
* UIProcess/WebPageProxy.messages.in:
* UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::didCommitLayerTree):
* UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm:
(WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
* WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm:
(didReachLayoutMilestone):
(setUpPageLoaderClient):
(didLayout): Deleted.
* WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
(WebKit::InjectedBundlePageLoaderClient::didReachLayoutMilestone):
(WebKit::InjectedBundlePageLoaderClient::didLayout): Deleted.
* WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
(WebKit::WebFrameLoaderClient::dispatchDidLayout): Deleted.
* WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
* WebProcess/WebPage/DrawingArea.h:
(WebKit::DrawingArea::dispatchDidReachLayoutMilestone):
(WebKit::DrawingArea::dispatchDidLayout): Deleted.
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::dispatchDidReachLayoutMilestone):
(WebKit::WebPage::dispatchDidLayout): Deleted.
* WebProcess/WebPage/WebPage.h:
* WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h:
* WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::dispatchDidReachLayoutMilestone):
(WebKit::RemoteLayerTreeDrawingArea::dispatchDidLayout): Deleted.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (204636 => 204637)


--- trunk/Source/WebCore/ChangeLog	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/ChangeLog	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1,3 +1,28 @@
+2016-08-16  Simon Fraser  <[email protected]>
+
+        Rename didLayout(LayoutMilestones) to didReachLayoutMilestone(), and related WK2 functions
+        https://bugs.webkit.org/show_bug.cgi?id=160923
+
+        Reviewed by Tim Horton.
+
+        didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
+        dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)
+
+        * dom/Document.cpp:
+        (WebCore::Document::setVisualUpdatesAllowed):
+        * loader/EmptyClients.h:
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::didReachLayoutMilestone):
+        (WebCore::FrameLoader::didLayout): Deleted.
+        * loader/FrameLoader.h:
+        * loader/FrameLoaderClient.h:
+        * page/FrameView.cpp:
+        (WebCore::FrameView::fireLayoutRelatedMilestonesIfNeeded):
+        (WebCore::FrameView::firePaintRelatedMilestonesIfNeeded):
+        * page/LayoutMilestones.h: Formatting
+        * page/Page.cpp:
+        (WebCore::Page::addRelevantRepaintedObject):
+
 2016-08-19  Chris Dumez  <[email protected]>
 
         DumpRenderTree crashed in com.apple.WebCore: WebCore::DOMWindow::resetDOMWindowProperties + 607

Modified: trunk/Source/WebCore/dom/Document.cpp (204636 => 204637)


--- trunk/Source/WebCore/dom/Document.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/dom/Document.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1305,7 +1305,7 @@
         if (frame()->isMainFrame()) {
             frameView->addPaintPendingMilestones(DidFirstPaintAfterSuppressedIncrementalRendering);
             if (page->requestedLayoutMilestones() & DidFirstLayoutAfterSuppressedIncrementalRendering)
-                frame()->loader().didLayout(DidFirstLayoutAfterSuppressedIncrementalRendering);
+                frame()->loader().didReachLayoutMilestone(DidFirstLayoutAfterSuppressedIncrementalRendering);
         }
     }
 

Modified: trunk/Source/WebCore/loader/EmptyClients.h (204636 => 204637)


--- trunk/Source/WebCore/loader/EmptyClients.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/loader/EmptyClients.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -300,7 +300,7 @@
     void dispatchDidFailLoad(const ResourceError&) override { }
     void dispatchDidFinishDocumentLoad() override { }
     void dispatchDidFinishLoad() override { }
-    void dispatchDidLayout(LayoutMilestones) override { }
+    void dispatchDidReachLayoutMilestone(LayoutMilestones) override { }
 
     Frame* dispatchCreatePage(const NavigationAction&) override { return nullptr; }
     void dispatchShow() override { }

Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (204636 => 204637)


--- trunk/Source/WebCore/loader/FrameLoader.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -2383,11 +2383,11 @@
         request.setFirstPartyForCookies(originalURL);
 }
 
-void FrameLoader::didLayout(LayoutMilestones milestones)
+void FrameLoader::didReachLayoutMilestone(LayoutMilestones milestones)
 {
     ASSERT(m_frame.isMainFrame());
 
-    m_client.dispatchDidLayout(milestones);
+    m_client.dispatchDidReachLayoutMilestone(milestones);
 }
 
 void FrameLoader::didFirstLayout()

Modified: trunk/Source/WebCore/loader/FrameLoader.h (204636 => 204637)


--- trunk/Source/WebCore/loader/FrameLoader.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/loader/FrameLoader.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -190,7 +190,7 @@
 
     CachePolicy subresourceCachePolicy() const;
 
-    void didLayout(LayoutMilestones);
+    void didReachLayoutMilestone(LayoutMilestones);
     void didFirstLayout();
 
     void loadedResourceFromMemoryCache(CachedResource*, ResourceRequest& newRequest);

Modified: trunk/Source/WebCore/loader/FrameLoaderClient.h (204636 => 204637)


--- trunk/Source/WebCore/loader/FrameLoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/loader/FrameLoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -174,7 +174,7 @@
 #endif
 
         virtual void dispatchDidLayout() { }
-        virtual void dispatchDidLayout(LayoutMilestones) { }
+        virtual void dispatchDidReachLayoutMilestone(LayoutMilestones) { }
 
         virtual Frame* dispatchCreatePage(const NavigationAction&) = 0;
         virtual void dispatchShow() = 0;

Modified: trunk/Source/WebCore/page/FrameView.cpp (204636 => 204637)


--- trunk/Source/WebCore/page/FrameView.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/page/FrameView.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -4849,7 +4849,7 @@
     }
 
     if (milestonesAchieved && frame().isMainFrame())
-        frame().loader().didLayout(milestonesAchieved);
+        frame().loader().didReachLayoutMilestone(milestonesAchieved);
 }
 
 void FrameView::firePaintRelatedMilestonesIfNeeded()
@@ -4874,7 +4874,7 @@
     m_milestonesPendingPaint = 0;
 
     if (milestonesAchieved)
-        page->mainFrame().loader().didLayout(milestonesAchieved);
+        page->mainFrame().loader().didReachLayoutMilestone(milestonesAchieved);
 }
 
 void FrameView::setVisualUpdatesAllowedByClient(bool visualUpdatesAllowed)

Modified: trunk/Source/WebCore/page/LayoutMilestones.h (204636 => 204637)


--- trunk/Source/WebCore/page/LayoutMilestones.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/page/LayoutMilestones.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -32,13 +32,13 @@
 // We should either re-name them to something more generic, or split them into
 // two enums -- one for painting and one for layout.
 enum LayoutMilestoneFlag {
-    DidFirstLayout = 1 << 0,
-    DidFirstVisuallyNonEmptyLayout = 1 << 1,
-    DidHitRelevantRepaintedObjectsAreaThreshold = 1 << 2,
-    DidFirstFlushForHeaderLayer = 1 << 3,
-    DidFirstLayoutAfterSuppressedIncrementalRendering = 1 << 4,
-    DidFirstPaintAfterSuppressedIncrementalRendering = 1 << 5,
-    ReachedSessionRestorationRenderTreeSizeThreshold = 1 << 6 // FIXME: only implemented by WK2 currently.
+    DidFirstLayout                                      = 1 << 0,
+    DidFirstVisuallyNonEmptyLayout                      = 1 << 1,
+    DidHitRelevantRepaintedObjectsAreaThreshold         = 1 << 2,
+    DidFirstFlushForHeaderLayer                         = 1 << 3,
+    DidFirstLayoutAfterSuppressedIncrementalRendering   = 1 << 4,
+    DidFirstPaintAfterSuppressedIncrementalRendering    = 1 << 5,
+    ReachedSessionRestorationRenderTreeSizeThreshold    = 1 << 6 // FIXME: only implemented by WK2 currently.
 };
 
 typedef unsigned LayoutMilestones;

Modified: trunk/Source/WebCore/page/Page.cpp (204636 => 204637)


--- trunk/Source/WebCore/page/Page.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebCore/page/Page.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1729,7 +1729,7 @@
         m_isCountingRelevantRepaintedObjects = false;
         resetRelevantPaintedObjectCounter();
         if (Frame* frame = &mainFrame())
-            frame->loader().didLayout(DidHitRelevantRepaintedObjectsAreaThreshold);
+            frame->loader().didReachLayoutMilestone(DidHitRelevantRepaintedObjectsAreaThreshold);
     }
 }
 

Modified: trunk/Source/WebKit/mac/ChangeLog (204636 => 204637)


--- trunk/Source/WebKit/mac/ChangeLog	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/mac/ChangeLog	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1,3 +1,15 @@
+2016-08-16  Simon Fraser  <[email protected]>
+
+        Rename didLayout(LayoutMilestones) to didReachLayoutMilestone(), and related WK2 functions
+        https://bugs.webkit.org/show_bug.cgi?id=160923
+
+        Reviewed by Tim Horton.
+
+        * WebCoreSupport/WebFrameLoaderClient.h:
+        * WebCoreSupport/WebFrameLoaderClient.mm:
+        (WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
+        (WebFrameLoaderClient::dispatchDidLayout): Deleted.
+
 2016-08-18  Daniel Bates  <[email protected]>
 
         Ld warns of non-existent PrivateFrameworks directory when building WebKit with the public iOS 9.3 SDK

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.h (204636 => 204637)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -116,7 +116,7 @@
     void dispatchDidFailLoad(const WebCore::ResourceError&) override;
     void dispatchDidFinishDocumentLoad() override;
     void dispatchDidFinishLoad() override;
-    void dispatchDidLayout(WebCore::LayoutMilestones) override;
+    void dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones) override;
 
     WebCore::Frame* dispatchCreatePage(const WebCore::NavigationAction&) override;
     void dispatchShow() override;

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm (204636 => 204637)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm	2016-08-19 18:10:23 UTC (rev 204637)
@@ -784,7 +784,7 @@
     [m_webFrame->_private->internalLoadDelegate webFrame:m_webFrame.get() didFinishLoadWithError:nil];
 }
 
-void WebFrameLoaderClient::dispatchDidLayout(LayoutMilestones milestones)
+void WebFrameLoaderClient::dispatchDidReachLayoutMilestone(LayoutMilestones milestones)
 {
     WebView *webView = getWebView(m_webFrame.get());
     WebFrameLoadDelegateImplementationCache* implementations = WebViewGetFrameLoadDelegateImplementations(webView);

Modified: trunk/Source/WebKit/win/ChangeLog (204636 => 204637)


--- trunk/Source/WebKit/win/ChangeLog	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/win/ChangeLog	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1,3 +1,18 @@
+2016-08-16  Simon Fraser  <[email protected]>
+
+        Rename didLayout(LayoutMilestones) to didReachLayoutMilestone(), and related WK2 functions
+        https://bugs.webkit.org/show_bug.cgi?id=160923
+
+        Reviewed by Tim Horton.
+
+        didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
+        dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)
+
+        * WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
+        (WebFrameLoaderClient::dispatchDidLayout): Deleted.
+        * WebCoreSupport/WebFrameLoaderClient.h:
+
 2016-08-18  Anders Carlsson  <[email protected]>
 
         Move an NPAPI-only function from WebCore to WebKit

Modified: trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.cpp (204636 => 204637)


--- trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -469,7 +469,7 @@
         frameLoadDelegate->didFinishLoadForFrame(webView, m_webFrame);
 }
 
-void WebFrameLoaderClient::dispatchDidLayout(LayoutMilestones milestones)
+void WebFrameLoaderClient::dispatchDidReachLayoutMilestone(LayoutMilestones milestones)
 {
     WebView* webView = m_webFrame->webView();
 

Modified: trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.h (204636 => 204637)


--- trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit/win/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -91,7 +91,7 @@
     void dispatchDidFailLoad(const WebCore::ResourceError&) override;
     void dispatchDidFinishDocumentLoad() override;
     void dispatchDidFinishLoad() override;
-    void dispatchDidLayout(WebCore::LayoutMilestones) override;
+    void dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones) override;
 
     void dispatchDecidePolicyForResponse(const WebCore::ResourceResponse&, const WebCore::ResourceRequest&, WebCore::FramePolicyFunction) override;
     void dispatchDecidePolicyForNewWindowAction(const WebCore::NavigationAction&, const WebCore::ResourceRequest&, PassRefPtr<WebCore::FormState>, const WTF::String& frameName, WebCore::FramePolicyFunction) override;

Modified: trunk/Source/WebKit2/ChangeLog (204636 => 204637)


--- trunk/Source/WebKit2/ChangeLog	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1,3 +1,54 @@
+2016-08-16  Simon Fraser  <[email protected]>
+
+        Rename didLayout(LayoutMilestones) to didReachLayoutMilestone(), and related WK2 functions
+        https://bugs.webkit.org/show_bug.cgi?id=160923
+
+        Reviewed by Tim Horton.
+
+        didLayout(LayoutMilestones) -> didReachLayoutMilestone(LayoutMilestones)
+        dispatchDidLayout(LayoutMilestones) -> dispatchDidReachLayoutMilestone(LayoutMilestones)
+
+        Avoided changing functions that are exposed as API/SPI.
+
+        * UIProcess/API/APILoaderClient.h:
+        (API::LoaderClient::didReachLayoutMilestone):
+        (API::LoaderClient::didLayout): Deleted.
+        * UIProcess/API/C/WKPage.cpp:
+        (WKPageSetPageLoaderClient):
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::didLayoutForCustomContentProvider):
+        (WebKit::WebPageProxy::didReachLayoutMilestone):
+        (WebKit::WebPageProxy::didLayout): Deleted.
+        * UIProcess/WebPageProxy.h:
+        * UIProcess/WebPageProxy.messages.in:
+        * UIProcess/ios/WebPageProxyIOS.mm:
+        (WebKit::WebPageProxy::didCommitLayerTree):
+        * UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm:
+        (WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
+        * WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm:
+        (didReachLayoutMilestone):
+        (setUpPageLoaderClient):
+        (didLayout): Deleted.
+        * WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
+        (WebKit::InjectedBundlePageLoaderClient::didReachLayoutMilestone):
+        (WebKit::InjectedBundlePageLoaderClient::didLayout): Deleted.
+        * WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::dispatchDidReachLayoutMilestone):
+        (WebKit::WebFrameLoaderClient::dispatchDidLayout): Deleted.
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
+        * WebProcess/WebPage/DrawingArea.h:
+        (WebKit::DrawingArea::dispatchDidReachLayoutMilestone):
+        (WebKit::DrawingArea::dispatchDidLayout): Deleted.
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::dispatchDidReachLayoutMilestone):
+        (WebKit::WebPage::dispatchDidLayout): Deleted.
+        * WebProcess/WebPage/WebPage.h:
+        * WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h:
+        * WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
+        (WebKit::RemoteLayerTreeDrawingArea::dispatchDidReachLayoutMilestone):
+        (WebKit::RemoteLayerTreeDrawingArea::dispatchDidLayout): Deleted.
+
 2016-08-18  Daniel Bates  <[email protected]>
 
         Ld warns of non-existent PrivateFrameworks directory when building WebKit with the public iOS 9.3 SDK

Modified: trunk/Source/WebKit2/UIProcess/API/APILoaderClient.h (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/API/APILoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/API/APILoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -75,7 +75,7 @@
     virtual void didRunInsecureContentForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, API::Object*) { }
     virtual void didDetectXSSForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, API::Object*) { }
 
-    virtual void didLayout(WebKit::WebPageProxy&, WebCore::LayoutMilestones) { }
+    virtual void didReachLayoutMilestone(WebKit::WebPageProxy&, WebCore::LayoutMilestones) { }
     
     virtual bool canAuthenticateAgainstProtectionSpaceInFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, WebKit::WebProtectionSpace*) { return false; }
     virtual void didReceiveAuthenticationChallengeInFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, WebKit::AuthenticationChallengeProxy*) { }

Modified: trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/API/C/WKPage.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1087,7 +1087,7 @@
             m_client.didFirstVisuallyNonEmptyLayoutForFrame(toAPI(&page), toAPI(&frame), toAPI(userData), m_client.base.clientInfo);
         }
 
-        void didLayout(WebPageProxy& page, LayoutMilestones milestones) override
+        void didReachLayoutMilestone(WebPageProxy& page, LayoutMilestones milestones) override
         {
             if (!m_client.didLayout)
                 return;

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -3469,10 +3469,10 @@
 
 void WebPageProxy::didLayoutForCustomContentProvider()
 {
-    didLayout(DidFirstLayout | DidFirstVisuallyNonEmptyLayout | DidHitRelevantRepaintedObjectsAreaThreshold);
+    didReachLayoutMilestone(DidFirstLayout | DidFirstVisuallyNonEmptyLayout | DidHitRelevantRepaintedObjectsAreaThreshold);
 }
 
-void WebPageProxy::didLayout(uint32_t layoutMilestones)
+void WebPageProxy::didReachLayoutMilestone(uint32_t layoutMilestones)
 {
     PageClientProtector protector(m_pageClient);
 
@@ -3479,7 +3479,7 @@
     if (m_navigationClient)
         m_navigationClient->renderingProgressDidChange(*this, static_cast<LayoutMilestones>(layoutMilestones));
     else
-        m_loaderClient->didLayout(*this, static_cast<LayoutMilestones>(layoutMilestones));
+        m_loaderClient->didReachLayoutMilestone(*this, static_cast<LayoutMilestones>(layoutMilestones));
 }
 
 void WebPageProxy::didDisplayInsecureContentForFrame(uint64_t frameID, const UserData& userData)

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.h (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -1100,7 +1100,7 @@
     void clearWheelEventTestTrigger();
     void callAfterNextPresentationUpdate(std::function<void (CallbackBase::Error)>);
 
-    void didLayout(uint32_t layoutMilestones);
+    void didReachLayoutMilestone(uint32_t layoutMilestones);
 
     void didRestoreScrollPosition();
 

Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.messages.in (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/WebPageProxy.messages.in	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.messages.in	2016-08-19 18:10:23 UTC (rev 204637)
@@ -132,7 +132,7 @@
     DidFinishLoadForFrame(uint64_t frameID, uint64_t navigationID, WebKit::UserData userData)
     DidFirstLayoutForFrame(uint64_t frameID, WebKit::UserData userData)
     DidFirstVisuallyNonEmptyLayoutForFrame(uint64_t frameID, WebKit::UserData userData)
-    DidLayout(uint32_t type)
+    DidReachLayoutMilestone(uint32_t type)
     DidReceiveTitleForFrame(uint64_t frameID, String title, WebKit::UserData userData)
     DidDisplayInsecureContentForFrame(uint64_t frameID, WebKit::UserData userData)
     DidRunInsecureContentForFrame(uint64_t frameID, WebKit::UserData userData)

Modified: trunk/Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/ios/WebPageProxyIOS.mm	2016-08-19 18:10:23 UTC (rev 204637)
@@ -364,7 +364,7 @@
     if (m_wantsSessionRestorationRenderTreeSizeThresholdEvent && !m_hitRenderTreeSizeThreshold
         && exceedsRenderTreeSizeSizeThreshold(m_sessionRestorationRenderTreeSize, layerTreeTransaction.renderTreeSize())) {
         m_hitRenderTreeSizeThreshold = true;
-        didLayout(WebCore::ReachedSessionRestorationRenderTreeSizeThreshold);
+        didReachLayoutMilestone(WebCore::ReachedSessionRestorationRenderTreeSizeThreshold);
     }
 
     if (m_hasDeferredStartAssistingNode) {

Modified: trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm (204636 => 204637)


--- trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm	2016-08-19 18:10:23 UTC (rev 204637)
@@ -236,7 +236,7 @@
 #endif
 
     if (auto milestones = layerTreeTransaction.newlyReachedLayoutMilestones())
-        m_webPageProxy.didLayout(milestones);
+        m_webPageProxy.didReachLayoutMilestone(milestones);
 
     for (auto& callbackID : layerTreeTransaction.callbackIDs()) {
         if (auto callback = m_callbacks.take<VoidCallback>(callbackID))

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm	2016-08-19 18:10:23 UTC (rev 204637)
@@ -182,7 +182,7 @@
         [loadDelegate webProcessPlugInBrowserContextController:pluginContextController didLayoutForFrame:wrapper(*toImpl(frame))];
 }
 
-static void didLayout(WKBundlePageRef page, WKLayoutMilestones milestones, WKTypeRef* userData, const void *clientInfo)
+static void didReachLayoutMilestone(WKBundlePageRef page, WKLayoutMilestones milestones, WKTypeRef* userData, const void *clientInfo)
 {
     WKWebProcessPlugInBrowserContextController *pluginContextController = (WKWebProcessPlugInBrowserContextController *)clientInfo;
     auto loadDelegate = pluginContextController->_loadDelegate.get();
@@ -249,7 +249,7 @@
     client.userAgentForURL = userAgentForURL;
 
     client.didLayoutForFrame = didLayoutForFrame;
-    client.didLayout = didLayout;
+    client.didLayout = didReachLayoutMilestone;
 
     page.initializeInjectedBundleLoaderClient(&client.base);
 }

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -252,7 +252,7 @@
     m_client.didLayoutForFrame(toAPI(page), toAPI(frame), m_client.base.clientInfo);
 }
 
-void InjectedBundlePageLoaderClient::didLayout(WebPage* page, LayoutMilestones milestones, RefPtr<API::Object>& userData)
+void InjectedBundlePageLoaderClient::didReachLayoutMilestone(WebPage* page, LayoutMilestones milestones, RefPtr<API::Object>& userData)
 {
     if (!m_client.didLayout)
         return;

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -84,7 +84,7 @@
     void didFirstLayoutForFrame(WebPage*, WebFrame*, RefPtr<API::Object>& userData);
     void didFirstVisuallyNonEmptyLayoutForFrame(WebPage*, WebFrame*, RefPtr<API::Object>& userData);
     void didLayoutForFrame(WebPage*, WebFrame*);
-    void didLayout(WebPage*, WebCore::LayoutMilestones, RefPtr<API::Object>& userData);
+    void didReachLayoutMilestone(WebPage*, WebCore::LayoutMilestones, RefPtr<API::Object>& userData);
 
     void didClearWindowObjectForFrame(WebPage*, WebFrame*, WebCore::DOMWrapperWorld&);
     void didCancelClientRedirectForFrame(WebPage*, WebFrame*);

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -566,7 +566,7 @@
     m_didCompletePageTransition = true;
 }
 
-void WebFrameLoaderClient::dispatchDidLayout(LayoutMilestones milestones)
+void WebFrameLoaderClient::dispatchDidReachLayoutMilestone(LayoutMilestones milestones)
 {
     WebPage* webPage = m_frame->page();
     if (!webPage)
@@ -595,7 +595,7 @@
     }
 
     // Send this after DidFirstLayout-specific calls since some clients expect to get those messages first.
-    webPage->dispatchDidLayout(milestones);
+    webPage->dispatchDidReachLayoutMilestone(milestones);
 
     if (milestones & DidFirstVisuallyNonEmptyLayout) {
         if (m_frame->isMainFrame() && !m_didCompletePageTransition && !webPage->corePage()->settings().suppressesIncrementalRendering()) {

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.h (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -102,7 +102,7 @@
     void dispatchDidFinishDocumentLoad() override;
     void dispatchDidFinishLoad() override;
 
-    void dispatchDidLayout(WebCore::LayoutMilestones) override;
+    void dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones) override;
     void dispatchDidLayout() override;
 
     WebCore::Frame* dispatchCreatePage(const WebCore::NavigationAction&) override;

Modified: trunk/Source/WebKit2/WebProcess/WebPage/DrawingArea.h (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebPage/DrawingArea.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebPage/DrawingArea.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -131,7 +131,7 @@
 
     virtual void setShouldScaleViewToFitDocument(bool) { }
 
-    virtual bool dispatchDidLayout(WebCore::LayoutMilestones) { return false; }
+    virtual bool dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones) { return false; }
 
 #if PLATFORM(COCOA)
     // Used by TiledCoreAnimationDrawingArea.

Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp	2016-08-19 18:10:23 UTC (rev 204637)
@@ -5435,19 +5435,19 @@
     m_userContentController->removeAllUserContent();
 }
 
-void WebPage::dispatchDidLayout(WebCore::LayoutMilestones milestones)
+void WebPage::dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones milestones)
 {
     RefPtr<API::Object> userData;
-    injectedBundleLoaderClient().didLayout(this, milestones, userData);
+    injectedBundleLoaderClient().didReachLayoutMilestone(this, milestones, userData);
 
     // Clients should not set userData for this message, and it won't be passed through.
     ASSERT(!userData);
 
     // The drawing area might want to defer dispatch of didLayout to the UI process.
-    if (m_drawingArea && m_drawingArea->dispatchDidLayout(milestones))
+    if (m_drawingArea && m_drawingArea->dispatchDidReachLayoutMilestone(milestones))
         return;
 
-    send(Messages::WebPageProxy::DidLayout(milestones));
+    send(Messages::WebPageProxy::DidReachLayoutMilestone(milestones));
 }
 
 void WebPage::didRestoreScrollPosition()

Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -937,7 +937,7 @@
     void addUserStyleSheet(const String& source, WebCore::UserContentInjectedFrames);
     void removeAllUserContent();
 
-    void dispatchDidLayout(WebCore::LayoutMilestones);
+    void dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones);
 
     void didRestoreScrollPosition();
 

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h	2016-08-19 18:10:23 UTC (rev 204637)
@@ -110,7 +110,7 @@
 
     bool adjustLayerFlushThrottling(WebCore::LayerFlushThrottleState::Flags) override;
 
-    bool dispatchDidLayout(WebCore::LayoutMilestones) override;
+    bool dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones) override;
 
     void updateScrolledExposedRect();
     void updateRootLayers();

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm (204636 => 204637)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm	2016-08-19 17:34:58 UTC (rev 204636)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm	2016-08-19 18:10:23 UTC (rev 204637)
@@ -511,7 +511,7 @@
     scheduleCompositingLayerFlush();
 }
 
-bool RemoteLayerTreeDrawingArea::dispatchDidLayout(WebCore::LayoutMilestones layoutMilestones)
+bool RemoteLayerTreeDrawingArea::dispatchDidReachLayoutMilestone(WebCore::LayoutMilestones layoutMilestones)
 {
     m_pendingNewlyReachedLayoutMilestones |= layoutMilestones;
     return true;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to