Move window.close implementation to DOMWindow

Source/core/frame/DOMWindow.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/frame/DOMWindow.h"
 
 #include "bindings/core/v8/ScriptCallStackFactory.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/SecurityContext.h"
 #include "core/events/MessageEvent.h"
 #include "core/frame/Frame.h"
 #include "core/frame/FrameClient.h"
+#include "core/frame/FrameConsole.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/Location.h"
+#include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
+#include "core/inspector/ConsoleMessageStorage.h"
 #include "core/inspector/InspectorInstrumentation.h"
 #include "core/inspector/ScriptCallStack.h"
+#include "core/loader/FrameLoaderClient.h"
 #include "core/loader/MixedContentChecker.h"
+#include "core/page/ChromeClient.h"
+#include "core/page/Page.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/SecurityOrigin.h"
 
 namespace blink {
 
 DOMWindow::DOMWindow()
     : m_windowIsClosing(false)
 {
 }
 
@@ skipping 240 matching lines @@
         return message + "The frame requesting access set \"document.domain\" to \"" + activeOrigin->domain() + "\", the frame being accessed set it to \"" + targetOrigin->domain() + "\". Both must set \"document.domain\" to the same value to allow access.";
     if (activeOrigin->domainWasSetInDOM())
         return message + "The frame requesting access set \"document.domain\" to \"" + activeOrigin->domain() + "\", but the frame being accessed did not. Both must set \"document.domain\" to the same value to allow access.";
     if (targetOrigin->domainWasSetInDOM())
         return message + "The frame being accessed set \"document.domain\" to \"" + targetOrigin->domain() + "\", but the frame requesting access did not. Both must set \"document.domain\" to the same value to allow access.";
 
     // Default.
     return message + "Protocols, domains, and ports must match.";
 }
 
+void DOMWindow::close(ExecutionContext* context)
+{
+    if (!frame() || !frame()->isMainFrame())
+        return;
+
+    Page* page = frame()->page();
+    if (!page)
+        return;
+
+    Document* activeDocument = nullptr;
+    if (context) {
+        ASSERT(isMainThread());
+        activeDocument = toDocument(context);
+        if (!activeDocument)
+            return;
+
+        if (!activeDocument->frame() || !activeDocument->frame()->canNavigate(*frame()))
+            return;
+    }
+
+    Settings* settings = frame()->settings();
+    bool allowScriptsToCloseWindows = settings && settings->allowScriptsToCloseWindows();
+
+    // FIXME: The session history length should be implemented for RemoteFrame and

[dcheng, 2015/06/16 20:08:23]

TODO(japhet) for this one? (if you're OK with this Nate: I have the vague
impression that we kind of need this for other stuff in one of the other layers)

Also, add a link to the crbug.

[nasko, 2015/06/16 22:58:54]

Done.

+    // this check should be made unconditionally.
+    bool hasNavigated = true;
+    if (frame()->isLocalFrame())
+        hasNavigated = toLocalFrame(frame())->loader().client()->backForwardLength() > 1;

[dcheng, 2015/06/16 20:08:23]

Maybe let's move this to FrameClient and leave the TODO to implement in
RemoteFrameClientImpl?

[nasko, 2015/06/16 22:58:54]

Done.

+
+    if (!page->openedByDOM() && hasNavigated && !allowScriptsToCloseWindows) {
+        if (activeDocument) {
+            activeDocument->domWindow()->frameConsole()->addMessage(ConsoleMessage::create(JSMessageSource, WarningMessageLevel, "Scripts may close only the windows that were opened by it."));
+        }
+        return;
+    }
+
+    // FIXME: The shouldClose functionality should be implemented for RemoteFrame

[dcheng, 2015/06/16 20:08:23]

TODO(username) is the style now. Ditto on linking to crbug.

[nasko, 2015/06/16 22:58:54]

Done.

+    // and the check should be made unconditionally.
+    if (frame()->isLocalFrame() && !toLocalFrame(frame())->loader().shouldClose())

[dcheng, 2015/06/16 20:08:23]

Same idea: maybe make this a virtual on Frame and leave the TODO to implement in
RemoteFrame?

[nasko, 2015/06/16 22:58:54]

Done.

+        return;
+
+    InspectorInstrumentation::willCloseWindow(context);
+
+    page->chromeClient().closeWindowSoon();
+
+    // So as to make window.closed return the expected result
+    // after window.close(), separately record the to-be-closed
+    // state of this window. Scripts may access window.closed
+    // before the deferred close operation has gone ahead.
+    m_windowIsClosing = true;
+}
+
 DEFINE_TRACE(DOMWindow)
 {
     visitor->trace(m_location);
     EventTargetWithInlineData::trace(visitor);
 }
 
 } // namespace blink