Make ScriptController inherit LocalWindowProxyManager

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

 /*
  * Copyright (C) 2008, 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. 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
@@ skipping 59 matching lines @@
 #include "public/platform/Platform.h"
 #include "wtf/CurrentTime.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/StringExtras.h"
 #include "wtf/text/CString.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/text/TextPosition.h"
 
 namespace blink {
 
-ScriptController::ScriptController(LocalFrame* frame)
-    : m_windowProxyManager(LocalWindowProxyManager::create(*frame)) {}
-
-DEFINE_TRACE(ScriptController) {
-  visitor->trace(m_windowProxyManager);
-}
-
-void ScriptController::clearForClose() {
-  m_windowProxyManager->clearForClose();
-  MainThreadDebugger::instance()->didClearContextsForFrame(frame());
-}
-
-void ScriptController::updateSecurityOrigin(SecurityOrigin* securityOrigin) {
-  m_windowProxyManager->updateSecurityOrigin(securityOrigin);
-}
+ScriptController::ScriptController(LocalFrame& frame)
+    : LocalWindowProxyManager(frame) {}
 
 namespace {
 
 V8CacheOptions cacheOptions(const ScriptResource* resource,
                             const Settings* settings) {
   V8CacheOptions v8CacheOptions(V8CacheOptionsDefault);
   if (settings)
     v8CacheOptions = settings->getV8CacheOptions();
   if (resource && !resource->response().cacheStorageCacheName().isNull()) {
     switch (settings->getV8CacheStrategiesForCacheStorage()) {
@@ skipping 48 matching lines @@
       return result;
   }
 
   TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("devtools.timeline"),
                        "UpdateCounters", TRACE_EVENT_SCOPE_THREAD, "data",
                        InspectorUpdateCountersEvent::data());
 
   return result;
 }
 
-LocalWindowProxy* ScriptController::windowProxy(DOMWrapperWorld& world) {
-  LocalWindowProxy* windowProxy = m_windowProxyManager->windowProxy(world);
-  windowProxy->initializeIfNeeded();
-  return windowProxy;
-}
-
 bool ScriptController::shouldBypassMainWorldCSP() {
   v8::HandleScope handleScope(isolate());
   v8::Local<v8::Context> context = isolate()->GetCurrentContext();
   if (context.IsEmpty() || !toDOMWindow(context))
     return false;
   DOMWrapperWorld& world = DOMWrapperWorld::current(isolate());
   return world.isIsolatedWorld() ? world.isolatedWorldHasContentSecurityPolicy()
                                  : false;
 }
 
 TextPosition ScriptController::eventHandlerPosition() const {
   ScriptableDocumentParser* parser =
       frame()->document()->scriptableDocumentParser();
   if (parser)
     return parser->textPosition();
   return TextPosition::minimumPosition();
 }
 
+void ScriptController::namedItemAdded(HTMLDocument* document,
+                                      const AtomicString& name) {
+  initializeMainWorld();
+
+  mainWorldProxy()->namedItemAdded(document, name);
+}
+
+void ScriptController::namedItemRemoved(HTMLDocument* document,
+                                        const AtomicString& name) {
+  initializeMainWorld();
+
+  mainWorldProxy()->namedItemRemoved(document, name);
+}
+
 void ScriptController::enableEval() {
   v8::HandleScope handleScope(isolate());
-  v8::Local<v8::Context> v8Context =
-      m_windowProxyManager->mainWorldProxy()->contextIfInitialized();
+  v8::Local<v8::Context> v8Context = mainWorldProxy()->contextIfInitialized();
   if (v8Context.IsEmpty())
     return;
   v8Context->AllowCodeGenerationFromStrings(true);
 }
 
 void ScriptController::disableEval(const String& errorMessage) {
   v8::HandleScope handleScope(isolate());
-  v8::Local<v8::Context> v8Context =
-      m_windowProxyManager->mainWorldProxy()->contextIfInitialized();
+  v8::Local<v8::Context> v8Context = mainWorldProxy()->contextIfInitialized();
   if (v8Context.IsEmpty())
     return;
   v8Context->AllowCodeGenerationFromStrings(false);
   v8Context->SetErrorMessageForCodeGenerationFromStrings(
       v8String(isolate(), errorMessage));
 }
 
 PassRefPtr<SharedPersistent<v8::Object>> ScriptController::createPluginWrapper(
     Widget* widget) {
   ASSERT(widget);
 
   if (!widget->isPluginView())
     return nullptr;
 
   v8::HandleScope handleScope(isolate());
   v8::Local<v8::Object> scriptableObject =
       toPluginView(widget)->scriptableObject(isolate());
 
   if (scriptableObject.IsEmpty())
     return nullptr;
 
   return SharedPersistent<v8::Object>::create(scriptableObject, isolate());
 }
 
+void ScriptController::clearForClose() {
+  LocalWindowProxyManager::clearForClose();
+  MainThreadDebugger::instance()->didClearContextsForFrame(frame());
+}
+
+void ScriptController::clearForNavigation() {
+  // V8 binding expects ScriptController::clearForNavigation() only be called
+  // when a frame is loading a new page.
+  LocalWindowProxyManager::clearForNavigation();
+  MainThreadDebugger::instance()->didClearContextsForFrame(frame());
+}
+
+void ScriptController::updateDocument() {
+  mainWorldProxy()->updateDocument();
+}
+
+void ScriptController::updateSecurityOrigin(SecurityOrigin* securityOrigin) {
+  mainWorldProxy()->updateSecurityOrigin(securityOrigin);
+  for (auto& entry : isolatedWorlds()) {
+    // This is a little hacky, but a LocalWindowProxyManager can only hold
+    // LocalWindowProxy objects, so the downcast to LocalWindowProxy is safe.
+    auto* isolatedWindowProxy =
+        static_cast<LocalWindowProxy*>(entry.value.get());
+    SecurityOrigin* isolatedSecurityOrigin =
+        isolatedWindowProxy->world().isolatedWorldSecurityOrigin();
+    isolatedWindowProxy->updateSecurityOrigin(isolatedSecurityOrigin);
+  }
+}
+
+void ScriptController::initializeMainWorld() {
+  mainWorldProxy()->initializeIfNeeded();
+}
+
 V8Extensions& ScriptController::registeredExtensions() {
   DEFINE_STATIC_LOCAL(V8Extensions, extensions, ());
   return extensions;
 }
 
 void ScriptController::registerExtensionIfNeeded(v8::Extension* extension) {
   const V8Extensions& extensions = registeredExtensions();
   for (size_t i = 0; i < extensions.size(); ++i) {
     if (extensions[i] == extension)
       return;
   }
   v8::RegisterExtension(extension);
   registeredExtensions().push_back(extension);
 }
 
-void ScriptController::clearWindowProxy() {
-  // V8 binding expects ScriptController::clearWindowProxy only be called when a
-  // frame is loading a new page. This creates a new context for the new page.
-  m_windowProxyManager->clearForNavigation();
-  MainThreadDebugger::instance()->didClearContextsForFrame(frame());
-}
-
-void ScriptController::updateDocument() {
-  m_windowProxyManager->mainWorldProxy()->updateDocument();
-}
-
 bool ScriptController::canExecuteScripts(
     ReasonForCallingCanExecuteScripts reason) {
 
   if (frame()->document() && frame()->document()->isSandboxed(SandboxScripts)) {
     // FIXME: This message should be moved off the console once a solution to
     // https://bugs.webkit.org/show_bug.cgi?id=103274 exists.
     if (reason == AboutToExecuteScript)
       frame()->document()->addConsoleMessage(ConsoleMessage::create(
           SecurityMessageSource, ErrorMessageLevel,
           "Blocked script execution in '" +
@@ skipping 153 matching lines @@
     for (size_t i = 0; i < resultArray->Length(); ++i) {
       v8::Local<v8::Value> value;
       if (!resultArray->Get(scriptState->context(), i).ToLocal(&value))
         return;
       results->push_back(value);
     }
   }
 }
 
 }  // namespace blink