d8: Fix some TSAN bugs

src/d8.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_D8_H_
 #define V8_D8_H_
 
 #ifndef V8_SHARED
 #include "src/allocation.h"
 #include "src/hashmap.h"
@@ skipping 225 matching lines @@
 
 class Worker {
  public:
   Worker();
   ~Worker();
 
   void StartExecuteInThread(Isolate* isolate, const char* script);
   void PostMessage(SerializationData* data);
   SerializationData* GetMessage();
   void Terminate();
+  void WaitForThread();
 
  private:
   class WorkerThread : public base::Thread {
    public:
     explicit WorkerThread(Worker* worker)
         : base::Thread(base::Thread::Options("WorkerThread")),
           worker_(worker) {}
 
     virtual void Run() { worker_->ExecuteInThread(); }
 
    private:
     Worker* worker_;
   };
 
-  enum State { IDLE, RUNNING, TERMINATED };
+  enum State { IDLE, RUNNING, TERMINATING, STOPPED, JOINING, JOINED };

[Jarin, 2015/07/10 10:43:17]

Could we have some descriptions of what the states mean?

[binji, 2015/07/10 17:40:46]

Done.

 
   void ExecuteInThread();
   void Cleanup();
   static void PostMessageOut(const v8::FunctionCallbackInfo<v8::Value>& args);
 
   base::Semaphore in_semaphore_;
   base::Semaphore out_semaphore_;
   SerializationDataQueue in_queue_;
   SerializationDataQueue out_queue_;
   base::Thread* thread_;
@@ skipping 207 matching lines @@
   static void RunShell(Isolate* isolate);
   static bool SetOptions(int argc, char* argv[]);
   static Handle<ObjectTemplate> CreateGlobalTemplate(Isolate* isolate);
 };
 
 
 }  // namespace v8
 
 
 #endif  // V8_D8_H_