Title: [87030] branches/safari-534-branch/Source/WebKit2
Revision
87030
Author
[email protected]
Date
2011-05-22 09:27:50 -0700 (Sun, 22 May 2011)

Log Message

https://bugs.webkit.org/show_bug.cgi?id=61244
<rdar://problem/9482856>
Undo operations for a web page linger after page is destroyed

Reviewed by Darin Adler.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::closePage):
Clear editing commands before telling UI client to close the page (this must be done
while the page is still in a window so its undo manager can be accessed, so waiting
for WebPageProxy::close() is too late).

Modified Paths

Diff

Modified: branches/safari-534-branch/Source/WebKit2/ChangeLog (87029 => 87030)


--- branches/safari-534-branch/Source/WebKit2/ChangeLog	2011-05-22 15:56:02 UTC (rev 87029)
+++ branches/safari-534-branch/Source/WebKit2/ChangeLog	2011-05-22 16:27:50 UTC (rev 87030)
@@ -1,3 +1,17 @@
+2011-05-21  John Sullivan  <[email protected]>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=61244
+        <rdar://problem/9482856>
+        Undo operations for a web page linger after page is destroyed
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::closePage):
+        Clear editing commands before telling UI client to close the page (this must be done
+        while the page is still in a window so its undo manager can be accessed, so waiting
+        for WebPageProxy::close() is too late).
+
 2011-05-19  Lucas Forschler  <[email protected]
 
     Merged r86851.

Modified: branches/safari-534-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp (87029 => 87030)


--- branches/safari-534-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp	2011-05-22 15:56:02 UTC (rev 87029)
+++ branches/safari-534-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp	2011-05-22 16:27:50 UTC (rev 87030)
@@ -1928,6 +1928,7 @@
     if (stopResponsivenessTimer)
         process()->responsivenessTimer()->stop();
 
+    m_pageClient->clearAllEditCommands();
     m_uiClient.close(this);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to