<rdar://problem/5424801> REGRESSION (r25151): Web page area of window doesn...

        Don't bypass WebView WM_PAINT if the WebView has never committed a page.
        Added getter for committedFirstRealDocumentLoad.

        Reviewed by Maciej.

        * WebView.cpp:
        (WebViewWndProc): Don't skip painting if we haven't committed any pages in this view.
        * loader/FrameLoader.h: Added committedFirstRealDocumentLoad.
        (WebCore::FrameLoader::committedFirstRealDocumentLoad): Added.

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 0d6bf44360118ee3424e7eca1170029460ee9bd3..2028686dbb0a32b4d6557e054b4414af6b47366d 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,12 @@
+2007-08-27  Steve Falkenburg  <sfalken@apple.com>
+
+        Added getter for committedFirstRealDocumentLoad.
+        
+        Reviewed by Maciej.
+
+        * loader/FrameLoader.h: Added committedFirstRealDocumentLoad.
+        (WebCore::FrameLoader::committedFirstRealDocumentLoad): Added.
+
 2007-08-27  Antti Koivisto  <antti@apple.com>
 
         Reviewed by Maciej.

diff --git a/WebCore/loader/FrameLoader.h b/WebCore/loader/FrameLoader.h
index 8a1c04dcd97f6541c1caa1ca72a624978cb76d06..ef284ff82e34fd1a806daedded5da9bcb0ae5456 100644 (file)
--- a/WebCore/loader/FrameLoader.h
+++ b/WebCore/loader/FrameLoader.h
@@ -178,6 +178,7 @@ namespace WebCore {
         bool isLoadingMainResource() const { return m_isLoadingMainResource; }
         bool isLoading() const;
         bool frameHasLoaded() const;
+        bool committedFirstRealDocumentLoad() const { return m_committedFirstRealDocumentLoad; }
 
         int numPendingOrLoadingRequests(bool recurse) const;
         bool isReloading() const;

diff --git a/WebKit/win/ChangeLog b/WebKit/win/ChangeLog
index 7145ec332c3658e7d656cf3497925fe2f49816f3..4e6084eb4d65022e7aead2acb607aa77247778ed 100644 (file)
--- a/WebKit/win/ChangeLog
+++ b/WebKit/win/ChangeLog
@@ -1,3 +1,14 @@
+2007-08-27  Steve Falkenburg  <sfalken@apple.com>
+
+        <rdar://problem/5424801> REGRESSION (r25151): Web page area of window doesn't redraw if page isn't loaded
+        
+        Don't bypass WebView WM_PAINT if the WebView has never committed a page.
+        
+        Reviewed by Maciej.
+
+        * WebView.cpp:
+        (WebViewWndProc): Don't skip painting if we haven't committed any pages in this view.
+
 2007-08-25  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Adam and Sam

diff --git a/WebKit/win/WebView.cpp b/WebKit/win/WebView.cpp
index 7aef477585cb45c6caa737d91b354d828337ea8d..44c57bca89d4198d225d7e0534b62c993056465b 100644 (file)
--- a/WebKit/win/WebView.cpp
+++ b/WebKit/win/WebView.cpp
@@ -1165,7 +1165,7 @@ static LRESULT CALLBACK WebViewWndProc(HWND hWnd, UINT message, WPARAM wParam, L
             COMPtr<IWebDataSource> dataSource;
             mainFrameImpl->dataSource(&dataSource);
             Frame* coreFrame = core(mainFrameImpl);
-            if (!webView->isPainting() && (!dataSource || coreFrame && coreFrame->view()->didFirstLayout()))
+            if (!webView->isPainting() && (!dataSource || coreFrame && (coreFrame->view()->didFirstLayout() || !coreFrame->loader()->committedFirstRealDocumentLoad())))
                 webView->paint(0, 0);
             else
                 ValidateRect(hWnd, 0);