Remove FrameViewBase as base class of PluginView.

third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h

 /*
  * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 16 matching lines @@
 #include "core/html/HTMLElement.h"
 #include "platform/heap/Handle.h"
 #include "platform/scroll/ScrollTypes.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "platform/wtf/HashCountedSet.h"
 
 namespace blink {
 
 class ExceptionState;
 class Frame;
+class FrameOrPlugin;
 class FrameViewBase;
 class LayoutPart;
 
 class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
                                           public FrameOwner {
   USING_GARBAGE_COLLECTED_MIXIN(HTMLFrameOwnerElement);
 
  public:
   ~HTMLFrameOwnerElement() override;
 
@@ skipping 51 matching lines @@
 
  protected:
   HTMLFrameOwnerElement(const QualifiedName& tag_name, Document&);
   void SetSandboxFlags(SandboxFlags);
 
   bool LoadOrRedirectSubframe(const KURL&,
                               const AtomicString& frame_name,
                               bool replace_current_item);
   bool IsKeyboardFocusable() const override;
 
-  void DisposeWidgetSoon(FrameViewBase*);
+  void DisposeSoon(FrameOrPlugin*);

[dcheng, 2017/04/17 21:09:04]

Let's fix the naming for this method, since we're changing it in this CL anyway.
I don't have a better suggestion than haraken's suggestion of LayoutController,
so let's just go with that.

[joelhockey, 2017/04/18 00:42:41]

I have renamed the method to DisposeFrameOrPluginSoon.  Haraken's suggestion was
to rename a few different classes.  If we all agree that we want to do that, I
can follow up with that in another CL, and I will also rename this method to the
appropriate new name such as DisposeLayoutControllerSoon.

[dcheng, 2017/04/18 00:56:31]

I think we should implement haraken's suggested names, as it provides a better
hint of what this class hierarchy is used for. Separate CL is fine then I guess.

   void FrameOwnerPropertiesChanged();
 
  private:
   // Intentionally private to prevent redundant checks when the type is
   // already HTMLFrameOwnerElement.
   bool IsLocal() const final { return true; }
   bool IsRemote() const final { return false; }
 
   bool IsFrameOwnerElement() const final { return true; }
 
@@ skipping 48 matching lines @@
 
 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement,
                   FrameOwner,
                   owner,
                   owner->IsLocal(),
                   owner.IsLocal());
 
 }  // namespace blink
 
 #endif  // HTMLFrameOwnerElement_h