Title: [111140] trunk/Source/WebCore
Revision
111140
Author
[email protected]
Date
2012-03-18 16:39:57 -0700 (Sun, 18 Mar 2012)

Log Message

Unreviewed, rolling out r111086.
http://trac.webkit.org/changeset/111086
https://bugs.webkit.org/show_bug.cgi?id=81374

Chromium crash

* notifications/DOMWindowNotifications.cpp:
(WebCore::DOMWindowNotifications::disconnectFrame):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (111139 => 111140)


--- trunk/Source/WebCore/ChangeLog	2012-03-18 23:07:27 UTC (rev 111139)
+++ trunk/Source/WebCore/ChangeLog	2012-03-18 23:39:57 UTC (rev 111140)
@@ -1,3 +1,14 @@
+2012-03-18  Kentaro Hara  <[email protected]>
+
+        Unreviewed, rolling out r111086.
+        http://trac.webkit.org/changeset/111086
+        https://bugs.webkit.org/show_bug.cgi?id=81374
+
+        Chromium crash
+
+        * notifications/DOMWindowNotifications.cpp:
+        (WebCore::DOMWindowNotifications::disconnectFrame):
+
 2012-03-18  David Reveman  <[email protected]>
 
         Invalidation issue when creating backing layer for fixed positioned element.

Modified: trunk/Source/WebCore/notifications/DOMWindowNotifications.cpp (111139 => 111140)


--- trunk/Source/WebCore/notifications/DOMWindowNotifications.cpp	2012-03-18 23:07:27 UTC (rev 111139)
+++ trunk/Source/WebCore/notifications/DOMWindowNotifications.cpp	2012-03-18 23:39:57 UTC (rev 111140)
@@ -87,13 +87,10 @@
 
 void DOMWindowNotifications::disconnectFrame()
 {
-    // FIXME: We should support keep the old m_notificationCenter so we can support reconnectFrame().
-    // See DOMWindowIndexedDatabase::disconnectFrame() for an example of how to do this.
     if (m_notificationCenter) {
         m_notificationCenter->disconnectFrame();
         m_notificationCenter = 0;
     }
-    DOMWindowProperty::disconnectFrame();
 }
 
 } // namespace WebCore
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to