Move plugins to be stored in HTMLPlugInElement.

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

Index: third_party/WebKit/Source/core/html/HTMLPlugInElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLPlugInElement.h b/third_party/WebKit/Source/core/html/HTMLPlugInElement.h
index 7840ace642fded995f3f75cb31762f6a471f0fb6..a659619511d7c1e895a18a6c5185ef429c817fdb 100644
--- a/third_party/WebKit/Source/core/html/HTMLPlugInElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLPlugInElement.h
@@ -50,7 +50,8 @@ class CORE_EXPORT HTMLPlugInElement : public HTMLFrameOwnerElement {
   // TODO(dcheng): Consider removing this, since HTMLEmbedElementLegacyCall
   // and HTMLObjectElementLegacyCall usage is extremely low.
   SharedPersistent<v8::Object>* pluginWrapper();
-  FrameViewBase* pluginWidget() const;
+  PluginView* pluginWidget() const;
+  PluginView* plugin() const;

[dcheng, 2017/04/04 07:20:12]

Please add a TODO to clean this up (and note that pluginWidget() triggers sync
creation of the plugin)

Edit: I see the comment is in the .cpp file, let's move it to the header, since
that's where people will probably see it.

[joelhockey, 2017/04/04 23:39:05]

Done.

   bool canProcessDrag() const;
   const String& url() const { return m_url; }
 
@@ -159,7 +160,6 @@ class CORE_EXPORT HTMLPlugInElement : public HTMLFrameOwnerElement {
 
   void setPlugin(PluginView*);
   PluginView* releasePlugin();
-  PluginView* ownedPlugin() const;
   void setPersistedPlugin(PluginView*);
 
   bool requestObjectInternal(const String& url,
@@ -175,6 +175,7 @@ class CORE_EXPORT HTMLPlugInElement : public HTMLFrameOwnerElement {
   // avoid accessing |layoutObject()| in layoutObjectIsFocusable().
   bool m_pluginIsAvailable = false;
 
+  Member<PluginView> m_plugin;
   // Normally the plugin is stored in HTMLFrameOwnerElement::m_widget.
   // However, plugins can persist even when not rendered. In order to
   // prevent confusing code which may assume that ownedWidget() != null