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