Move window.close implementation to DOMWindow

Source/web/ChromeClientImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  *
  * 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.
@@ skipping 382 matching lines @@
         return webframe->client()->runModalBeforeUnloadDialog(isReload, message);
     return false;
 }
 
 void ChromeClientImpl::closeWindowSoon()
 {
     // Make sure this Page can no longer be found by JS.
     Page::ordinaryPages().remove(m_webView->page());
 
     // Make sure that all loading is stopped.  Ensures that JS stops executing!
-    m_webView->mainFrame()->stopLoading();
+    if (m_webView->mainFrame()->isWebLocalFrame())

[dcheng, 2015/06/17 23:39:50]

TODO this?

[nasko, 2015/06/19 13:23:22]

Actually, the method is available on WebRemoteFrameImpl, but had a not reached
statement. I put a comment there and made this unconditional.

+        m_webView->mainFrame()->stopLoading();
 
     if (m_webView->client())
         m_webView->client()->closeWidgetSoon();
 }
 
 // Although a LocalFrame is passed in, we don't actually use it, since we
 // already know our own m_webView.
 bool ChromeClientImpl::openJavaScriptAlertDelegate(LocalFrame* frame, const String& message)
 {
     notifyPopupOpeningObservers();
@@ skipping 538 matching lines @@
 }
 
 void ChromeClientImpl::notifyPopupOpeningObservers() const
 {
     const Vector<PopupOpeningObserver*> observers(m_popupOpeningObservers);
     for (const auto& observer : observers)
         observer->willOpenPopup();
 }
 
 } // namespace blink