Remove most of Widget (FrameViewBase) class from HTMLPlugInElement.

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

Index: third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h b/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
index a80c0d701a2fdd2344f12c6544c567e789ac3725..7c34fcd6ad447b82105e21124d1cec227d093ea0 100644
--- a/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLFrameOwnerElement.h
@@ -36,6 +36,7 @@ class ExceptionState;
 class Frame;
 class FrameViewBase;
 class LayoutPart;
+class PluginView;
 
 class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
                                           public FrameOwner {
@@ -62,6 +63,9 @@ class CORE_EXPORT HTMLFrameOwnerElement : public HTMLElement,
   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();