Remove most of Widget (FrameViewBase) class from HTMLPlugInElement.

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 18 matching lines @@
 #include "platform/scroll/ScrollTypes.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "wtf/HashCountedSet.h"
 
 namespace blink {
 
 class ExceptionState;
 class Frame;
 class FrameViewBase;
 class LayoutPart;
+class PluginView;
 
 class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
                                           public FrameOwner {
   USING_GARBAGE_COLLECTED_MIXIN(HTMLFrameOwnerElement);
 
  public:
   ~HTMLFrameOwnerElement() override;
 
   DOMWindow* contentWindow() const;
   Document* contentDocument() const;
 
   virtual void disconnectContentFrame();
 
   // Most subclasses use LayoutPart (either LayoutEmbeddedObject or
   // LayoutIFrame) except for HTMLObjectElement and HTMLEmbedElement which may
   // return any LayoutObject when using fallback content.
   LayoutPart* layoutPart() const;
 
   Document* getSVGDocument(ExceptionState&) const;
 
   virtual bool loadedNonEmptyDocument() const { return false; }
   virtual void didLoadNonEmptyDocument() {}
 
   void setWidget(FrameViewBase*);

[slangley, 2017/03/20 03:05:42]

Do we need a TODO here to remove these in followup CL?

[haraken, 2017/03/20 03:06:35]

Add:

// TODO(joelhockey): Remove these methods once PluginView stops inheriting from
FrameViewBase.

[joelhockey, 2017/03/20 03:54:29]

This method is and will still be called for FrameView.  It will likely change
name to setFrameView or setFrame.  I don't think there is a need for TODO right
now.  If I was going to start putting them in, they would be in a lot of places.

   FrameViewBase* releaseWidget();
   FrameViewBase* ownedWidget() const;
+  void setPlugin(PluginView*);
+  PluginView* releasePlugin();
+  PluginView* ownedPlugin() const;
 
   class UpdateSuspendScope {
     STACK_ALLOCATED();
 
    public:
     UpdateSuspendScope();
     ~UpdateSuspendScope();
 
    private:
     void performDeferredWidgetTreeOperations();
@@ skipping 90 matching lines @@
 
 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement,
                   FrameOwner,
                   owner,
                   owner->isLocal(),
                   owner.isLocal());
 
 }  // namespace blink
 
 #endif  // HTMLFrameOwnerElement_h