2010-08-12 Sheriff Bot <webkit.review.bot@gmail.com>
authorhamaji@chromium.org <hamaji@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Aug 2010 12:36:52 +0000 (12:36 +0000)
committerhamaji@chromium.org <hamaji@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Aug 2010 12:36:52 +0000 (12:36 +0000)
        Unreviewed, rolling out r65181.
        http://trac.webkit.org/changeset/65181
        https://bugs.webkit.org/show_bug.cgi?id=43909

        It broke 35 layout tests fail on chromium-win (Requested by
        hamaji on #webkit).

        * public/WebFrameClient.h:
        * src/FrameLoaderClientImpl.cpp:
        (WebKit::FrameLoaderClientImpl::detachedFromParent3):

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

WebKit/chromium/ChangeLog
WebKit/chromium/public/WebFrameClient.h
WebKit/chromium/src/FrameLoaderClientImpl.cpp

index eb24668..6631b0b 100644 (file)
@@ -1,5 +1,18 @@
 2010-08-12  Sheriff Bot  <webkit.review.bot@gmail.com>
 
+        Unreviewed, rolling out r65181.
+        http://trac.webkit.org/changeset/65181
+        https://bugs.webkit.org/show_bug.cgi?id=43909
+
+        It broke 35 layout tests fail on chromium-win (Requested by
+        hamaji on #webkit).
+
+        * public/WebFrameClient.h:
+        * src/FrameLoaderClientImpl.cpp:
+        (WebKit::FrameLoaderClientImpl::detachedFromParent3):
+
+2010-08-12  Sheriff Bot  <webkit.review.bot@gmail.com>
+
         Unreviewed, rolling out r65221.
         http://trac.webkit.org/changeset/65221
         https://bugs.webkit.org/show_bug.cgi?id=43896
index 91f0e38..2b7ee0d 100644 (file)
@@ -90,12 +90,6 @@ public:
 
     // General notifications -----------------------------------------------
 
-    // This frame has been detached from the view.
-    //
-    // FIXME: Do not use this in new code. Currently this is used by code in
-    // Chromium that errantly caches WebKit objects.
-    virtual void frameDetached(WebFrame*) { }
-
     // This frame is about to be closed.
     virtual void willClose(WebFrame*) { }
 
index 74186bf..db74080 100644 (file)
@@ -248,12 +248,7 @@ void FrameLoaderClientImpl::detachedFromParent3()
     // go to a page and then navigate to a new page without getting any asserts
     // or crashes.
     m_webFrame->frame()->script()->proxy()->clearForClose();
-
-    // Alert the client that the frame is being detached. This is the last
-    // chance we have to communicate with the client.
-    if (m_webFrame->client())
-        m_webFrame->client()->frameDetached(m_webFrame);
-
+    
     // Stop communicating with the WebFrameClient at this point since we are no
     // longer associated with the Page.
     m_webFrame->setClient(0);