Devirtualize Frame::domWindow().

third_party/WebKit/Source/core/frame/RemoteFrame.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 "core/frame/RemoteFrame.h"
 
 #include "bindings/core/v8/WindowProxy.h"
 #include "bindings/core/v8/WindowProxyManager.h"
 #include "core/dom/RemoteSecurityContext.h"
 #include "core/frame/LocalFrame.h"
@@ skipping 12 matching lines @@
 #include "platform/weborigin/SecurityPolicy.h"
 #include "public/platform/WebLayer.h"
 
 namespace blink {
 
 inline RemoteFrame::RemoteFrame(RemoteFrameClient* client,
                                 FrameHost* host,
                                 FrameOwner* owner)
     : Frame(client, host, owner),
       m_securityContext(RemoteSecurityContext::create()),
-      m_domWindow(RemoteDOMWindow::create(*this)),
-      m_windowProxyManager(WindowProxyManager::create(*this)) {}
+      m_windowProxyManager(WindowProxyManager::create(*this)) {
+  m_domWindow = RemoteDOMWindow::create(*this);
+}
 
 RemoteFrame* RemoteFrame::create(RemoteFrameClient* client,
                                  FrameHost* host,
                                  FrameOwner* owner) {
   return new RemoteFrame(client, host, owner);
 }
 
 RemoteFrame::~RemoteFrame() {
   ASSERT(!m_view);
 }
 
 DEFINE_TRACE(RemoteFrame) {
   visitor->trace(m_view);
   visitor->trace(m_securityContext);
-  visitor->trace(m_domWindow);
   visitor->trace(m_windowProxyManager);
   Frame::trace(visitor);
 }
 
-DOMWindow* RemoteFrame::domWindow() const {
-  return m_domWindow.get();
-}
-
 WindowProxy* RemoteFrame::windowProxy(DOMWrapperWorld& world) {
   WindowProxy* windowProxy = m_windowProxyManager->windowProxy(world);
   ASSERT(windowProxy);
   windowProxy->initializeIfNeeded();
   return windowProxy;
 }
 
 void RemoteFrame::navigate(Document& originDocument,
                            const KURL& url,
                            bool replaceCurrentItem,
@@ skipping 80 matching lines @@
   // Oilpan: as RemoteFrameView performs no finalization actions,
   // no explicit dispose() of it needed here. (cf. FrameView::dispose().)
   m_view = view;
 
   // ... the RemoteDOMWindow will need to be informed of detachment,
   // as otherwise it will keep a strong reference back to this RemoteFrame.
   // That combined with wrappers (owned and kept alive by RemoteFrame) keeping
   // persistent strong references to RemoteDOMWindow will prevent the GCing
   // of all these objects. Break the cycle by notifying of detachment.
   if (!m_view)
-    m_domWindow->frameDetached();
+    toRemoteDOMWindow(m_domWindow)->frameDetached();
 }
 
 void RemoteFrame::createView() {
   // If the RemoteFrame does not have a LocalFrame parent, there's no need to
   // create a widget for it.
   if (!deprecatedLocalOwner())
     return;
 
   ASSERT(!deprecatedLocalOwner()->ownedWidget());
 
@@ skipping 27 matching lines @@
   FrameVector childrenToDetach;
   childrenToDetach.reserveCapacity(tree().childCount());
   for (Frame* child = tree().firstChild(); child;
        child = child->tree().nextSibling())
     childrenToDetach.append(child);
   for (const auto& child : childrenToDetach)
     child->detach(FrameDetachType::Remove);
 }
 
 }  // namespace blink