Adding field to tell if gin::IsolateHolder needs lock to be accessed.

gin/public/isolate_holder.h

Index: gin/public/isolate_holder.h
diff --git a/gin/public/isolate_holder.h b/gin/public/isolate_holder.h
index ba75cd8f11654f653806776be1c750acd78e75a5..97719f82f45163c80626ecfdc6a77e8ea25508d8 100644
--- a/gin/public/isolate_holder.h
+++ b/gin/public/isolate_holder.h
@@ -27,7 +27,14 @@ class GIN_EXPORT IsolateHolder {
     kStrictMode
   };
 
+  // Stores wheather the client uses v8::Locker to access the isolate.

[rmcilroy, 2015/04/23 15:52:51]

/s/wheather/whether

[ssid, 2015/04/23 16:24:56]

Done.

+  enum LockMode {
+    kLockToAccess,

[Primiano Tucci (use gerrit), 2015/04/23 15:52:40]

I'd probably do:

enum AccessMode {
  kSingleThread,
  kUseLocker
}

[ssid, 2015/04/23 16:24:56]

Done.

+    kDirectAccess
+  };
+
   IsolateHolder();
+  IsolateHolder(LockMode lock_mode);
   ~IsolateHolder();
 
   // Should be invoked once before creating IsolateHolder instances to
@@ -52,10 +59,14 @@ class GIN_EXPORT IsolateHolder {
   // thread.
   void RemoveRunMicrotasksObserver();
 
+  // This method returns if v8::Locker is needed to access isolate.
+  bool UseLock();

[Primiano Tucci (use gerrit), 2015/04/23 15:52:40]

UseLock is a bit confusing as a name as it is not clear whether you are telling
the isolate to use a lock, or you are asking that.
What about: DoesUseV8Locker()?

Actually I'd defer to jochen for the name.

[ssid, 2015/04/23 16:24:56]

Done.

+
  private:
   v8::Isolate* isolate_;
   scoped_ptr<PerIsolateData> isolate_data_;
   scoped_ptr<RunMicrotasksObserver> task_observer_;
+  LockMode lock_mode_;
 
   DISALLOW_COPY_AND_ASSIGN(IsolateHolder);
 };