2011-01-28 Maciej Stachowiak <mjs@apple.com>

        Reviewed by Dan Bernstein.

        WKPageGetEstimatedProgress returns wrong value after a mainframe provisional load has started
        https://bugs.webkit.org/show_bug.cgi?id=53358

        * UIProcess/WebPageProxy.cpp:
        (WebKit::WebPageProxy::didStartProgress): Start progress at the magic initial value, not 0.

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

diff --git a/Source/WebKit2/ChangeLog b/Source/WebKit2/ChangeLog
index ea43df2d541aeb66b51b905e77ee6828ad9ec751..b517bac38952ccacff55d2d06036a272e82ba023 100644 (file)
--- a/Source/WebKit2/ChangeLog
+++ b/Source/WebKit2/ChangeLog
@@ -1,3 +1,13 @@
+2011-01-28  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Dan Bernstein.
+
+        WKPageGetEstimatedProgress returns wrong value after a mainframe provisional load has started
+        https://bugs.webkit.org/show_bug.cgi?id=53358
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::didStartProgress): Start progress at the magic initial value, not 0.
+
 2011-01-28  Anders Carlsson  <andersca@apple.com>
 
         Yet another build fix (What was I thinking?)

diff --git a/Source/WebKit2/UIProcess/WebPageProxy.cpp b/Source/WebKit2/UIProcess/WebPageProxy.cpp
index 7e318828e7632e55fdb5c21d15782cdf31953045..0ca1cbd498420e50df1ea2fc6fd7c848b0d99a5c 100644 (file)
--- a/Source/WebKit2/UIProcess/WebPageProxy.cpp
+++ b/Source/WebKit2/UIProcess/WebPageProxy.cpp
@@ -1174,17 +1174,23 @@ void WebPageProxy::didRestoreFrameFromPageCache(uint64_t frameID, uint64_t paren
     parentFrame->appendChild(subframe);
 }
 
+
+// Always start progress at initialProgressValue. This helps provide feedback as
+// soon as a load starts.
+
+static const double initialProgressValue = 0.1;
+
 void WebPageProxy::didStartProgress()
 {
-    m_estimatedProgress = 0.0;
-    
+    m_estimatedProgress = initialProgressValue;
+
     m_loaderClient.didStartProgress(this);
 }
 
 void WebPageProxy::didChangeProgress(double value)
 {
     m_estimatedProgress = value;
-    
+
     m_loaderClient.didChangeProgress(this);
 }