Remove extension group from DOMWrapperWorld.

third_party/WebKit/Source/bindings/core/v8/LocalWindowProxy.cpp

 /*
  * Copyright (C) 2008, 2009, 2011 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 99 matching lines @@
     context->SetErrorMessageForCodeGenerationFromStrings(
         v8String(isolate(), csp->evalDisabledErrorMessage()));
   } else {
     updateActivityLogger();
     origin = m_world->isolatedWorldSecurityOrigin();
     setSecurityToken(origin);
   }
 
   MainThreadDebugger::instance()->contextCreated(m_scriptState.get(), frame(),
                                                  origin);
-  frame()->loader().client()->didCreateScriptContext(
-      context, m_world->extensionGroup(), m_world->worldId());
+  frame()->loader().client()->didCreateScriptContext(context,
+                                                     m_world->worldId());
   // If conditional features for window have been queued before the V8 context
   // was ready, then inject them into the context now
   if (m_world->isMainWorld()) {
     installPendingConditionalFeaturesOnWindow(m_scriptState.get());
   }
 
   if (m_world->isMainWorld())
     frame()->loader().dispatchDidClearWindowObjectInMainWorld();
 }
 
 void LocalWindowProxy::createContext() {
   // Create a new v8::Context with the window object as the global object
   // (aka the inner global).  Reuse the global proxy object (aka the outer
   // global) if it already exists.  See the comments in
   // setupWindowPrototypeChain for the structure of the prototype chain of
   // the global object.
   v8::Local<v8::ObjectTemplate> globalTemplate =
       V8Window::domTemplate(isolate(), *m_world)->InstanceTemplate();
   CHECK(!globalTemplate.IsEmpty());
 
-  // FIXME: It's not clear what the right thing to do for remote frames is.
-  // The extensions registered don't generally seem to make sense for remote
-  // frames, so skip it for now.
   Vector<const char*> extensionNames;
   // Dynamically tell v8 about our extensions now.
-  const V8Extensions& extensions = ScriptController::registeredExtensions();
-  extensionNames.reserveInitialCapacity(extensions.size());
-  int extensionGroup = m_world->extensionGroup();
-  int worldId = m_world->worldId();
-  for (const auto* extension : extensions) {
-    if (!frame()->loader().client()->allowScriptExtension(
-            extension->name(), extensionGroup, worldId))
-      continue;
-
-    extensionNames.push_back(extension->name());
+  if (frame()->loader().client()->allowScriptExtensions()) {
+    const V8Extensions& extensions = ScriptController::registeredExtensions();
+    extensionNames.reserveInitialCapacity(extensions.size());
+    for (const auto* extension : extensions)
+      extensionNames.push_back(extension->name());
   }
   v8::ExtensionConfiguration extensionConfiguration(extensionNames.size(),
                                                     extensionNames.data());
 
   v8::Local<v8::Context> context;
   {
     V8PerIsolateData::UseCounterDisabledScope useCounterDisabled(
         V8PerIsolateData::from(isolate()));
     context =
         v8::Context::New(isolate(), &extensionConfiguration, globalTemplate,
@@ skipping 194 matching lines @@
 
   setSecurityToken(origin);
 }
 
 LocalWindowProxy::LocalWindowProxy(v8::Isolate* isolate,
                                    LocalFrame& frame,
                                    RefPtr<DOMWrapperWorld> world)
     : WindowProxy(isolate, frame, std::move(world)) {}
 
 }  // namespace blink