Move securityCheck out of V8WrapperInstantiationScope

third_party/WebKit/Source/core/CoreInitializer.cpp

Index: third_party/WebKit/Source/core/CoreInitializer.cpp
diff --git a/third_party/WebKit/Source/core/CoreInitializer.cpp b/third_party/WebKit/Source/core/CoreInitializer.cpp
index a094386fa1e30ff7dde0292e88237e9161e14bfb..a8a6a59810522da67ee48decc33f89d17793907c 100644
--- a/third_party/WebKit/Source/core/CoreInitializer.cpp
+++ b/third_party/WebKit/Source/core/CoreInitializer.cpp
@@ -30,6 +30,7 @@
 
 #include "core/CoreInitializer.h"
 
+#include "bindings/core/v8/BindingSecurity.h"
 #include "bindings/core/v8/ScriptStreamerThread.h"
 #include "core/EventNames.h"
 #include "core/EventTargetNames.h"
@@ -142,6 +143,8 @@ void CoreInitializer::initialize() {
   StringImpl::freezeStaticStrings();
 
   ScriptStreamerThread::init();
+
+  BindingSecurity::initWrapperCreationSecurityCheck();

[haraken, 2017/04/07 04:23:36]

I'd prefer moving this to V8Initializer.

[adithyas, 2017/04/07 18:42:53]

Would V8Initializer::initializeMainThread() work? I can't put it in
V8Initializer::initializeCommon, because it could get run more than once (main
thread + worker threads).

[haraken, 2017/04/09 15:28:59]

Yes, V8Initializer::initializeMainThread sounds nice.
 

[adithyas, 2017/04/10 18:02:00]

Done.

 }
 
 }  // namespace blink