Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Unified Diff: test/cctest/test-threads.cc

Issue 10417010: Run Crankshaft on a separate thread. (Closed) Base URL: https://chromiumcodereview.appspot.com/10387157
Patch Set: Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/cctest/test-threads.cc
diff --git a/test/cctest/test-threads.cc b/test/cctest/test-threads.cc
index 713d1e842509f919781322e0b09cf8c8e727ecac..5fcecc64228b57279e204b5947f1badc3b0e6ff4 100644
--- a/test/cctest/test-threads.cc
+++ b/test/cctest/test-threads.cc
@@ -126,14 +126,18 @@ class ThreadB : public v8::internal::Thread {
TEST(JSFunctionResultCachesInTwoThreads) {
v8::V8::Initialize();
- ThreadA threadA;
- ThreadB threadB;
+ {
+ v8::Unlocker unlock;
- threadA.Start();
- threadB.Start();
+ ThreadA threadA;
+ ThreadB threadB;
- threadA.Join();
- threadB.Join();
+ threadA.Start();
+ threadB.Start();
+
+ threadA.Join();
+ threadB.Join();
+ }
CHECK_EQ(DONE, turn);
}

Powered by Google App Engine
This is Rietveld 408576698