Move securityCheck out of V8WrapperInstantiationScope

third_party/WebKit/Source/core/CoreInitializer.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 12 matching lines @@
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/CoreInitializer.h"
 
+#include "bindings/core/v8/BindingSecurity.h"
 #include "bindings/core/v8/ScriptStreamerThread.h"
 #include "core/EventNames.h"
 #include "core/EventTargetNames.h"
 #include "core/EventTypeNames.h"
 #include "core/HTMLNames.h"
 #include "core/HTMLTokenizerNames.h"
 #include "core/InputModeNames.h"
 #include "core/InputTypeNames.h"
 #include "core/MathMLNames.h"
 #include "core/MediaFeatureNames.h"
@@ skipping 92 matching lines @@
 
   KURL::initialize();
   SchemeRegistry::initialize();
   SecurityPolicy::init();
 
   registerEventFactory();
 
   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