[SCM] WebKit Debian packaging branch, webkit-1.3, updated. upstream/1.3.7-4207-g178b198

mjs at apple.com mjs at apple.com
Mon Feb 21 00:16:39 UTC 2011


The following commit has been merged in the webkit-1.3 branch:
commit e687428e45954472b9828f9d12145597573d555e
Author: mjs at apple.com <mjs at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Sat Jan 29 03:45:17 2011 +0000

    2011-01-28  Maciej Stachowiak  <mjs at 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 ea43df2..b517bac 100644
--- a/Source/WebKit2/ChangeLog
+++ b/Source/WebKit2/ChangeLog
@@ -1,3 +1,13 @@
+2011-01-28  Maciej Stachowiak  <mjs at 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 at 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 7e31882..0ca1cbd 100644
--- 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);
 }
 

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list