Refactor WindowProxy into Local and Remote subclasses.

third_party/WebKit/Source/core/frame/LocalFrame.h

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999-2001 Lars Knoll <knoll@kde.org>
  *                     1999-2001 Antti Koivisto <koivisto@kde.org>
  *                     2000-2001 Simon Hausmann <hausmann@kde.org>
  *                     2000-2001 Dirk Mueller <mueller@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 218 matching lines @@
  private:
   friend class FrameNavigationDisabler;
 
   LocalFrame(FrameLoaderClient*,
              FrameHost*,
              FrameOwner*,
              InterfaceProvider*,
              InterfaceRegistry*);
 
   // Internal Frame helper overrides:
-  WindowProxyManager* getWindowProxyManager() const override;
+  WindowProxyManagerBase* getWindowProxyManager() const override;

[Yuki, 2017/01/11 10:09:24]

nit: Why not LocalWindowProxyManager?

[dcheng, 2017/01/11 10:35:29]

This is an override. I can't include WindowProxyManager.h here, due to include
cycles. Since it doesn't need to be covariant, I just return the base type for
simplicity.

   // Intentionally private to prevent redundant checks when the type is
   // already LocalFrame.
   bool isLocalFrame() const override { return true; }
   bool isRemoteFrame() const override { return false; }
 
   void enableNavigation() { --m_navigationDisableCount; }
   void disableNavigation() { ++m_navigationDisableCount; }
 
   std::unique_ptr<WebFrameScheduler> m_frameScheduler;
 
@@ skipping 128 matching lines @@
   explicit ScopedFrameBlamer(LocalFrame*);
   ~ScopedFrameBlamer();
 
  private:
   Member<LocalFrame> m_frame;
 };
 
 }  // namespace blink
 
 #endif  // LocalFrame_h