Remove unused ChromeClient::layoutUpdated().
authorakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Feb 2014 22:29:34 +0000 (22:29 +0000)
committerakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Feb 2014 22:29:34 +0000 (22:29 +0000)
<https://webkit.org/b/128470>

Nobody listens for this callback anymore so remove it.

Reviewed by Anders Carlsson.

* page/Chrome.cpp:
* page/Chrome.h:
* page/ChromeClient.h:
* page/FrameView.cpp:
(WebCore::FrameView::layout):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@163726 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/Chrome.cpp
Source/WebCore/page/Chrome.h
Source/WebCore/page/ChromeClient.h
Source/WebCore/page/FrameView.cpp

index 8c3f147f5f3971e6a0cd9192a9306b3079ce6d8e..336b70bf1a91e728a12b570cca183da085bf1bec 100644 (file)
@@ -1,3 +1,18 @@
+2014-02-08  Andreas Kling  <akling@apple.com>
+
+        Remove unused ChromeClient::layoutUpdated().
+        <https://webkit.org/b/128470>
+
+        Nobody listens for this callback anymore so remove it.
+
+        Reviewed by Anders Carlsson.
+
+        * page/Chrome.cpp:
+        * page/Chrome.h:
+        * page/ChromeClient.h:
+        * page/FrameView.cpp:
+        (WebCore::FrameView::layout):
+
 2014-02-08  Dan Bernstein  <mitz@apple.com>
 
         Stop using PLATFORM(MAC) in WebCore except where it means “OS X but not iOS”
 2014-02-08  Dan Bernstein  <mitz@apple.com>
 
         Stop using PLATFORM(MAC) in WebCore except where it means “OS X but not iOS”
index 20f9d8e4a6df5b88f9982cd813875fdd50543bce..800f06a055dc0bab4613547b8aa48ef5a5a8414e 100644 (file)
@@ -125,11 +125,6 @@ void Chrome::contentsSizeChanged(Frame* frame, const IntSize& size) const
     m_client.contentsSizeChanged(frame, size);
 }
 
     m_client.contentsSizeChanged(frame, size);
 }
 
-void Chrome::layoutUpdated(Frame* frame) const
-{
-    m_client.layoutUpdated(frame);
-}
-
 void Chrome::scrollRectIntoView(const IntRect& rect) const
 {
     m_client.scrollRectIntoView(rect);
 void Chrome::scrollRectIntoView(const IntRect& rect) const
 {
     m_client.scrollRectIntoView(rect);
index 45f43cc71fb65ad588e70e01f966da513410a294..9c47d21559fe90f3289a3cd9c0d3b4f0b63f20f2 100644 (file)
@@ -93,7 +93,6 @@ public:
     void scrollRectIntoView(const IntRect&) const;
 
     void contentsSizeChanged(Frame*, const IntSize&) const;
     void scrollRectIntoView(const IntRect&) const;
 
     void contentsSizeChanged(Frame*, const IntSize&) const;
-    void layoutUpdated(Frame*) const;
 
     void setWindowRect(const FloatRect&) const;
     FloatRect windowRect() const;
 
     void setWindowRect(const FloatRect&) const;
     FloatRect windowRect() const;
index edf74c8bd5db28f60b656e3f262b844dab0f0940..5242a9b21984512bf65cf8c5dbb0fe8094ef74be 100644 (file)
@@ -179,7 +179,6 @@ public:
     virtual void dispatchViewportPropertiesDidChange(const ViewportArguments&) const { }
 
     virtual void contentsSizeChanged(Frame*, const IntSize&) const = 0;
     virtual void dispatchViewportPropertiesDidChange(const ViewportArguments&) const { }
 
     virtual void contentsSizeChanged(Frame*, const IntSize&) const = 0;
-    virtual void layoutUpdated(Frame*) const { }
     virtual void scrollRectIntoView(const IntRect&) const { }; // Currently only Mac has a non empty implementation.
 
     virtual bool shouldUnavailablePluginMessageBeButton(RenderEmbeddedObject::PluginUnavailabilityReason) const { return false; }
     virtual void scrollRectIntoView(const IntRect&) const { }; // Currently only Mac has a non empty implementation.
 
     virtual bool shouldUnavailablePluginMessageBeButton(RenderEmbeddedObject::PluginUnavailabilityReason) const { return false; }
index 498332838cbddcc994a1d3df0d1c15f3db96768a..89bb15941cda9818f1777b56edca216cb694c45e 100644 (file)
@@ -1340,12 +1340,6 @@ void FrameView::layout(bool allowSubtree)
     InspectorInstrumentation::didLayout(cookie, root);
 
     --m_nestedLayoutCount;
     InspectorInstrumentation::didLayout(cookie, root);
 
     --m_nestedLayoutCount;
-
-    if (m_nestedLayoutCount)
-        return;
-
-    if (Page* page = frame().page())
-        page->chrome().client().layoutUpdated(&frame());
 }
 
 RenderBox* FrameView::embeddedContentBox() const
 }
 
 RenderBox* FrameView::embeddedContentBox() const