Remove invalidateContents, it isn't used and it never makes sense to only invalidate...
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Mar 2010 00:09:17 +0000 (00:09 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Mar 2010 00:09:17 +0000 (00:09 +0000)
Reviewed by David Hyatt.

WebCore:

* loader/EmptyClients.h:
* page/Chrome.cpp:
* page/Chrome.h:
* page/ChromeClient.h:
* platform/HostWindow.h:

WebKit/chromium:

* src/ChromeClientImpl.cpp:
* src/ChromeClientImpl.h:

WebKit/gtk:

* WebCoreSupport/ChromeClientGtk.cpp:
* WebCoreSupport/ChromeClientGtk.h:

WebKit/haiku:

* WebCoreSupport/ChromeClientHaiku.cpp:
* WebCoreSupport/ChromeClientHaiku.h:

WebKit/mac:

* WebCoreSupport/WebChromeClient.h:
* WebCoreSupport/WebChromeClient.mm:

WebKit/qt:

* WebCoreSupport/ChromeClientQt.cpp:
* WebCoreSupport/ChromeClientQt.h:

WebKit/win:

* WebCoreSupport/WebChromeClient.cpp:
* WebCoreSupport/WebChromeClient.h:

WebKit/wx:

* WebKitSupport/ChromeClientWx.cpp:
* WebKitSupport/ChromeClientWx.h:

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

27 files changed:
WebCore/ChangeLog
WebCore/loader/EmptyClients.h
WebCore/page/Chrome.cpp
WebCore/page/Chrome.h
WebCore/page/ChromeClient.h
WebCore/platform/HostWindow.h
WebKit/chromium/ChangeLog
WebKit/chromium/src/ChromeClientImpl.cpp
WebKit/chromium/src/ChromeClientImpl.h
WebKit/gtk/ChangeLog
WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp
WebKit/gtk/WebCoreSupport/ChromeClientGtk.h
WebKit/haiku/ChangeLog
WebKit/haiku/WebCoreSupport/ChromeClientHaiku.cpp
WebKit/haiku/WebCoreSupport/ChromeClientHaiku.h
WebKit/mac/ChangeLog
WebKit/mac/WebCoreSupport/WebChromeClient.h
WebKit/mac/WebCoreSupport/WebChromeClient.mm
WebKit/qt/ChangeLog
WebKit/qt/WebCoreSupport/ChromeClientQt.cpp
WebKit/qt/WebCoreSupport/ChromeClientQt.h
WebKit/win/ChangeLog
WebKit/win/WebCoreSupport/WebChromeClient.cpp
WebKit/win/WebCoreSupport/WebChromeClient.h
WebKit/wx/ChangeLog
WebKit/wx/WebKitSupport/ChromeClientWx.cpp
WebKit/wx/WebKitSupport/ChromeClientWx.h

index b5b8324..2b1259d 100644 (file)
@@ -1,3 +1,15 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * loader/EmptyClients.h:
+        * page/Chrome.cpp:
+        * page/Chrome.h:
+        * page/ChromeClient.h:
+        * platform/HostWindow.h:
+
 2010-03-11  Nate Chapin  <japhet@chromium.org>
 
         Reviewed by Dimitri Glazkov.
index 6a2ecd5..a267374 100644 (file)
@@ -114,7 +114,6 @@ public:
 
     virtual IntRect windowResizerRect() const { return IntRect(); }
 
-    virtual void invalidateContents(const IntRect&, bool) { }
     virtual void invalidateWindow(const IntRect&, bool) { }
     virtual void invalidateContentsAndWindow(const IntRect&, bool) { }
     virtual void invalidateContentsForSlowScroll(const IntRect&, bool) {};
index 771df87..2f3078c 100644 (file)
@@ -67,11 +67,6 @@ Chrome::~Chrome()
     m_client->chromeDestroyed();
 }
 
-void Chrome::invalidateContents(const IntRect& updateRect, bool immediate)
-{
-    m_client->invalidateContents(updateRect, immediate);
-}
-
 void Chrome::invalidateWindow(const IntRect& updateRect, bool immediate)
 {
     m_client->invalidateWindow(updateRect, immediate);
index 279a7f8..3adf25d 100644 (file)
@@ -61,7 +61,6 @@ namespace WebCore {
 
         // HostWindow methods.
 
-        virtual void invalidateContents(const IntRect&, bool);
         virtual void invalidateWindow(const IntRect&, bool);
         virtual void invalidateContentsAndWindow(const IntRect&, bool);
         virtual void invalidateContentsForSlowScroll(const IntRect&, bool);
index fcc8770..e4bc8d1 100644 (file)
@@ -129,7 +129,6 @@ namespace WebCore {
         virtual IntRect windowResizerRect() const = 0;
 
         // Methods used by HostWindow.
-        virtual void invalidateContents(const IntRect&, bool) = 0;
         virtual void invalidateWindow(const IntRect&, bool) = 0;
         virtual void invalidateContentsAndWindow(const IntRect&, bool) = 0;
         virtual void invalidateContentsForSlowScroll(const IntRect&, bool) = 0;
index 97abf87..dc681a1 100644 (file)
@@ -35,9 +35,6 @@ class HostWindow : public Noncopyable {
 public:
     virtual ~HostWindow() { }
 
-    // Requests the host invalidate the contents, not the window.  If immediate is true do so synchronously, otherwise async.
-    virtual void invalidateContents(const IntRect& updateRect, bool immediate) = 0;
-
     // Requests the host invalidate the window, not the contents.  If immediate is true do so synchronously, otherwise async.
     virtual void invalidateWindow(const IntRect& updateRect, bool immediate) = 0;
 
index 44a93ee..8bbdb4e 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * src/ChromeClientImpl.cpp:
+        * src/ChromeClientImpl.h:
+
 2010-03-10  Darin Fisher  <darin@chromium.org>
 
         Reviewed by Jeremy Orlow.
index efd2d38..6e899bd 100644 (file)
@@ -466,11 +466,6 @@ IntRect ChromeClientImpl::windowResizerRect() const
     return result;
 }
 
-void ChromeClientImpl::invalidateContents(const IntRect&, bool)
-{
-    notImplemented();
-}
-
 void ChromeClientImpl::invalidateWindow(const IntRect&, bool)
 {
     notImplemented();
index a369f49..cb53f69 100644 (file)
@@ -96,7 +96,6 @@ public:
     virtual bool shouldInterruptJavaScript();
     virtual bool tabsToLinks() const;
     virtual WebCore::IntRect windowResizerRect() const;
-    virtual void invalidateContents(const WebCore::IntRect&, bool);
     virtual void invalidateWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsAndWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsForSlowScroll(const WebCore::IntRect&, bool);
index 9add6e2..2a289e2 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebCoreSupport/ChromeClientGtk.cpp:
+        * WebCoreSupport/ChromeClientGtk.h:
+
 2010-03-09  Philippe Normand  <pnormand@igalia.com>
 
         Reviewed by Gustavo Noronha Silva.
index 514b09a..c63a03b 100644 (file)
@@ -342,11 +342,6 @@ IntRect ChromeClient::windowResizerRect() const
     return IntRect();
 }
 
-void ChromeClient::invalidateContents(const IntRect&, bool)
-{
-    notImplemented();
-}
-
 void ChromeClient::invalidateWindow(const IntRect&, bool)
 {
     notImplemented();
index 503f66f..90f2d48 100644 (file)
@@ -87,7 +87,6 @@ namespace WebKit {
 
         virtual WebCore::IntRect windowResizerRect() const;
 
-        virtual void invalidateContents(const WebCore::IntRect&, bool);
         virtual void invalidateWindow(const WebCore::IntRect&, bool);
         virtual void invalidateContentsAndWindow(const WebCore::IntRect&, bool);
         virtual void invalidateContentsForSlowScroll(const WebCore::IntRect&, bool);
index 0b57a59..1c6870a 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebCoreSupport/ChromeClientHaiku.cpp:
+        * WebCoreSupport/ChromeClientHaiku.h:
+
 2010-03-02  Adam Treat  <atreat@rim.com>
 
         Reviewed by Dave Hyatt.
index cd8f1f4..172137d 100644 (file)
@@ -253,11 +253,6 @@ IntRect ChromeClientHaiku::windowResizerRect() const
     return IntRect();
 }
 
-void ChromeClientHaiku::invalidateContents(const IntRect&, bool)
-{
-    notImplemented();
-}
-
 void ChromeClientHaiku::invalidateWindow(const IntRect&, bool)
 {
     notImplemented();
index 990f3f0..838a579 100644 (file)
@@ -102,7 +102,6 @@ namespace WebCore {
         bool tabsToLinks() const;
         IntRect windowResizerRect() const;
 
-        void invalidateContents(const IntRect&, bool);
         void invalidateWindow(const IntRect&, bool);
         void invalidateContentsAndWindow(const IntRect&, bool);
         void invalidateContentsForSlowScroll(const IntRect&, bool);
index b96208b..97dd5ae 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebCoreSupport/WebChromeClient.h:
+        * WebCoreSupport/WebChromeClient.mm:
+
 2010-03-11  Alexey Proskuryakov  <ap@apple.com>
 
         Reviewed by Geoff Garen.
index 85267d5..730d0ee 100644 (file)
@@ -91,7 +91,6 @@ public:
     
     virtual WebCore::IntRect windowResizerRect() const;
 
-    virtual void invalidateContents(const WebCore::IntRect&, bool);
     virtual void invalidateWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsAndWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsForSlowScroll(const WebCore::IntRect&, bool);
index da6c15c..8d65f49 100644 (file)
@@ -439,10 +439,6 @@ IntRect WebChromeClient::windowResizerRect() const
     return enclosingIntRect([m_webView convertRect:rect fromView:nil]);
 }
 
-void WebChromeClient::invalidateContents(const IntRect&, bool)
-{
-}
-
 void WebChromeClient::invalidateWindow(const IntRect&, bool immediate)
 {
     if (immediate) {
index 4c2f4e1..81f91c3 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebCoreSupport/ChromeClientQt.cpp:
+        * WebCoreSupport/ChromeClientQt.h:
+
 2010-03-10  Robert Hogan  <robert@webkit.org>
 
         Reviewed by Simon Hausmann.
index 6f682c3..d58e5eb 100644 (file)
@@ -327,11 +327,6 @@ IntRect ChromeClientQt::windowResizerRect() const
     return IntRect();
 }
 
-void ChromeClientQt::invalidateContents(const IntRect&, bool)
-{
-    notImplemented();
-}
-
 void ChromeClientQt::invalidateWindow(const IntRect&, bool)
 {
     notImplemented();
index cb8a996..2751342 100644 (file)
@@ -105,7 +105,6 @@ namespace WebCore {
         virtual bool tabsToLinks() const;
         virtual IntRect windowResizerRect() const;
 
-        virtual void invalidateContents(const IntRect&, bool);
         virtual void invalidateWindow(const IntRect&, bool);
         virtual void invalidateContentsAndWindow(const IntRect&, bool);
         virtual void invalidateContentsForSlowScroll(const IntRect&, bool);
index c927d2a..bf37332 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebCoreSupport/WebChromeClient.cpp:
+        * WebCoreSupport/WebChromeClient.h:
+
 2010-03-10  Eric Uhrhane  <ericu@chromium.org>
 
         Reviewed by David Levin.
index aca2d59..47d2279 100644 (file)
@@ -454,12 +454,6 @@ IntRect WebChromeClient::windowResizerRect() const
     return IntRect();
 }
 
-void WebChromeClient::invalidateContents(const IntRect& windowRect, bool)
-{
-    ASSERT(core(m_webView->topLevelFrame()));
-    m_webView->repaint(windowRect, true /*contentChanged*/, false /*immediate*/, true /*repaintContentOnly*/);
-}
-
 void WebChromeClient::invalidateWindow(const IntRect& windowRect, bool immediate)
 {
     ASSERT(core(m_webView->topLevelFrame()));
index 9510fc4..8f3c986 100644 (file)
@@ -93,7 +93,6 @@ public:
     virtual bool tabsToLinks() const;
     virtual WebCore::IntRect windowResizerRect() const;
 
-    virtual void invalidateContents(const WebCore::IntRect&, bool);
     virtual void invalidateWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsAndWindow(const WebCore::IntRect&, bool);
     virtual void invalidateContentsForSlowScroll(const WebCore::IntRect&, bool);
index 4ac7609..df2a89f 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-11  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by David Hyatt.
+
+        Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.
+
+        * WebKitSupport/ChromeClientWx.cpp:
+        * WebKitSupport/ChromeClientWx.h:
+
 2010-03-02  Adam Treat  <atreat@rim.com>
 
         Reviewed by Dave Hyatt.
index 94b01c9..243f97c 100644 (file)
@@ -336,11 +336,6 @@ IntRect ChromeClientWx::windowResizerRect() const
     return IntRect();
 }
 
-void ChromeClientWx::invalidateContents(const IntRect& rect, bool)
-{
-    notImplemented();
-}
-
 void ChromeClientWx::invalidateWindow(const IntRect& rect, bool immediate)
 {
     if (immediate)
index c670f99..71da63d 100644 (file)
@@ -102,7 +102,6 @@ public:
     virtual void scrollBackingStore(int dx, int dy, const IntRect& scrollViewRect, const IntRect& clipRect);
     virtual void updateBackingStore();
     
-    virtual void invalidateContents(const IntRect&, bool);
     virtual void invalidateWindow(const IntRect&, bool);
     virtual void invalidateContentsAndWindow(const IntRect&, bool);
     virtual void invalidateContentsForSlowScroll(const IntRect&, bool);