Changeset 54380 in webkit

Timestamp:

Feb 4, 2010 3:43:20 PM (14 years ago)

Author:

eric@webkit.org

Message:

2010-02-04 Drew Wilson <​atwilson@chromium.org>

Reviewed by David Levin.

WorkerContext.close() does not work in the chromium port
​https://bugs.webkit.org/show_bug.cgi?id=34551

Test: Adding new downstream test.

• src/WebWorkerBase.cpp: (WebKit::WebWorkerBase::workerContextClosedTask): Now shuts down the worker thread when WorkerContext::close() is invoked.

Location:

trunk/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebWorkerBase.cpp (modified) (1 diff)

trunk/WebKit/chromium/ChangeLog

@@ +1 @@
+2010-02-04  Drew Wilson  <atwilson@chromium.org>
+
+        Reviewed by David Levin.
+
+        WorkerContext.close() does not work in the chromium port
+        https://bugs.webkit.org/show_bug.cgi?id=34551
+
+        Test: Adding new downstream test.
+
+        * src/WebWorkerBase.cpp:
+        (WebKit::WebWorkerBase::workerContextClosedTask):
+        Now shuts down the worker thread when WorkerContext::close() is invoked.
+
 2010-02-04  Chris Guillory  <chris.guillory@google.com>
 

trunk/WebKit/chromium/src/WebWorkerBase.cpp

@@ -272 +272 @@
     if (thisPtr->commonClient())
         thisPtr->commonClient()->workerContextClosed();
+
+    thisPtr->stopWorkerThread();
 }