Reviewed by Anders Carlsson.
authorggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Nov 2006 22:14:22 +0000 (22:14 +0000)
committerggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Nov 2006 22:14:22 +0000 (22:14 +0000)
        Fixed http://bugs.webkit.org/show_bug.cgi?id=11712
        REGRESSION: Crash when clicking JS link on crateandbarrel.com (window.open("http[...]"))

        This was an accidental commit by Anders.

        Chrome can't use the FrameView at window.open time, because it doesn't
        exist yet.

        * page/Chrome.cpp:
        (WebCore::Chrome::pageRect):

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

WebCore/ChangeLog
WebCore/page/Chrome.cpp

index f12ad243e1fecbbdbc95f7be92d18031d013ac50..c9241a58664c747947bc5e131ebdd5456da3fb42 100644 (file)
@@ -1,3 +1,18 @@
+2006-11-29  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Anders Carlsson.
+        
+        Fixed http://bugs.webkit.org/show_bug.cgi?id=11712
+        REGRESSION: Crash when clicking JS link on crateandbarrel.com (window.open("http[...]"))
+        
+        This was an accidental commit by Anders.
+
+        Chrome can't use the FrameView at window.open time, because it doesn't
+        exist yet.
+        
+        * page/Chrome.cpp:
+        (WebCore::Chrome::pageRect):
+
 2006-11-29  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Mitz Pettel.
index e3950bbc8428b959a64d60b217ed138ad6932b65..eacfda61126d343819089bf2ea8f25dae64ba89f 100644 (file)
@@ -23,8 +23,6 @@
 
 #include "ChromeClient.h"
 #include "FloatRect.h"
-#include "Frame.h"
-#include "FrameView.h"
 #include "Page.h"
 #include "ResourceLoader.h"
 #include <wtf/PassRefPtr.h>
@@ -57,7 +55,7 @@ FloatRect Chrome::windowRect() const
 
 FloatRect Chrome::pageRect() const
 {
-    return m_page->mainFrame()->view()->visibleContentRect();
+    return m_client->pageRect();
 }
         
 float Chrome::scaleFactor()