Title: [91513] branches/safari-534.51-branch/Source/WebKit2
Revision
91513
Author
[email protected]
Date
2011-07-21 14:59:09 -0700 (Thu, 21 Jul 2011)

Log Message

Merge r89720.

Modified Paths

Diff

Modified: branches/safari-534.51-branch/Source/WebKit2/ChangeLog (91512 => 91513)


--- branches/safari-534.51-branch/Source/WebKit2/ChangeLog	2011-07-21 21:58:10 UTC (rev 91512)
+++ branches/safari-534.51-branch/Source/WebKit2/ChangeLog	2011-07-21 21:59:09 UTC (rev 91513)
@@ -1,5 +1,24 @@
 2011-07-21  Lucas Forschler  <[email protected]>
 
+    Merged 89720.
+
+    2011-07-21  Lucas Forschler  <[email protected]>
+
+        Reviewed by John Sullivan.
+
+        Drag-and-drop tab in same window makes web process falsely report unresponsiveness
+        https://bugs.webkit.org/show_bug.cgi?id=63369
+        <rdar://problem/9673120>
+
+        If the view becomes invisible, stop the responsiveness timer, because we might not get any
+        messages that will cause it to stop (such as painting notifications since we don't paint
+        when we're not visible).
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::viewStateDidChange):
+
+2011-07-21  Lucas Forschler  <[email protected]>
+
     Merged 88667.
 
     2011-06-13  Sam Weinig  <[email protected]>

Modified: branches/safari-534.51-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp (91512 => 91513)


--- branches/safari-534.51-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp	2011-07-21 21:58:10 UTC (rev 91512)
+++ branches/safari-534.51-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp	2011-07-21 21:59:09 UTC (rev 91513)
@@ -631,6 +631,13 @@
             m_isVisible = isVisible;
             m_drawingArea->visibilityDidChange();
             m_drawingArea->setPageIsVisible(isVisible);
+
+            if (!m_isVisible) {
+                // If we've started the responsiveness timer as part of telling the web process to update the backing store
+                // state, it might not send back a reply (since it won't paint anything if the web page is hidden) so we
+                // stop the unresponsiveness timer here.
+                process()->responsivenessTimer()->stop();
+            }
         }
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to