Title: [119740] trunk/Source/WebCore
Revision
119740
Author
[email protected]
Date
2012-06-07 11:31:50 -0700 (Thu, 07 Jun 2012)

Log Message

Worker tear-down can re-enter JSC during GC finalization
https://bugs.webkit.org/show_bug.cgi?id=88449

Reviewed by Geoffrey Garen.

No new tests.

This is the first of two patches to fix this issue with Workers.

* workers/AbstractWorker.cpp:
(WebCore::AbstractWorker::~AbstractWorker): We don't need to call onDestroyWorker() here, it
will be called elsewhere in contextDestroyed().

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (119739 => 119740)


--- trunk/Source/WebCore/ChangeLog	2012-06-07 18:18:37 UTC (rev 119739)
+++ trunk/Source/WebCore/ChangeLog	2012-06-07 18:31:50 UTC (rev 119740)
@@ -1,3 +1,18 @@
+2012-06-07  Mark Hahnenberg  <[email protected]>
+
+        Worker tear-down can re-enter JSC during GC finalization
+        https://bugs.webkit.org/show_bug.cgi?id=88449
+
+        Reviewed by Geoffrey Garen.
+
+        No new tests. 
+
+        This is the first of two patches to fix this issue with Workers.
+
+        * workers/AbstractWorker.cpp:
+        (WebCore::AbstractWorker::~AbstractWorker): We don't need to call onDestroyWorker() here, it 
+        will be called elsewhere in contextDestroyed().
+
 2012-06-07  Jer Noble  <[email protected]>
 
         sometimes all slaved videos don't start playing

Modified: trunk/Source/WebCore/workers/AbstractWorker.cpp (119739 => 119740)


--- trunk/Source/WebCore/workers/AbstractWorker.cpp	2012-06-07 18:18:37 UTC (rev 119739)
+++ trunk/Source/WebCore/workers/AbstractWorker.cpp	2012-06-07 18:31:50 UTC (rev 119740)
@@ -53,7 +53,6 @@
 
 AbstractWorker::~AbstractWorker()
 {
-    onDestroyWorker();
 }
 
 void AbstractWorker::onDestroyWorker()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to