Remove FrameViewBase as base class of PluginView.

third_party/WebKit/Source/core/plugins/PluginView.h

 /*
  * Copyright (C) 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 11 matching lines @@
  * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
  * DAMAGE.
  */
 
 #ifndef PluginView_h
 #define PluginView_h
 
 #include "core/CoreExport.h"
-#include "platform/FrameViewBase.h"
+#include "core/frame/FrameOrPlugin.h"
+#include "platform/geometry/IntRect.h"
 #include "platform/scroll/ScrollTypes.h"
 #include "public/platform/WebFocusType.h"
 #include "v8/include/v8.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
+
+class Event;
+class FrameView;
+class ResourceResponse;
 class WebLayer;
-}
 
-namespace blink {
+class CORE_EXPORT PluginView : public FrameOrPlugin {
+ public:
+  virtual ~PluginView() {}
 
-class ResourceResponse;
-
-class CORE_EXPORT PluginView : public FrameViewBase {
- public:
-  bool IsPluginView() const final { return true; }
-  virtual void SetFocused(bool, WebFocusType) {}
+  virtual void SetParent(FrameView*) = 0;
+  virtual FrameView* Parent() const = 0;
+  virtual void SetParentVisible(bool) = 0;
+  virtual void SetFocused(bool, WebFocusType) = 0;
+  virtual void FrameRectsChanged() = 0;
+  virtual void GeometryMayHaveChanged() = 0;
+  virtual void HandleEvent(Event*) = 0;
+  virtual void EventListenersRemoved() = 0;
+  virtual bool IsPluginContainer() const { return false; }

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

We may want to consider removing this as well (there is only one subclass of
PluginView now, as far as I can tell): we basically have same situation as
WebViewImpl here now:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/web/WebViewImp...

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

+1.  There is only a single implementation, so I agree there is no need for this
abstraction.  I would then be inclined to rename web/WebPluginContainerImpl to
just web/WebPluginContainer (drop the Impl suffix), but I see that many classes
in web have Impl suffix.

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

OK, please add a TODO since we're not doing it in this CL.

[joelhockey, 2017/04/18 01:05:16]

Done

 
   virtual WebLayer* PlatformLayer() const { return 0; }
   virtual v8::Local<v8::Object> ScriptableObject(v8::Isolate*) {
     return v8::Local<v8::Object>();
   }
   virtual bool WantsWheelEvents() { return false; }
   virtual bool SupportsKeyboardFocus() const { return false; }
   virtual bool SupportsInputMethod() const { return false; }
   virtual bool CanProcessDrag() const { return false; }
 
   virtual void DidReceiveResponse(const ResourceResponse&) {}
   virtual void DidReceiveData(const char*, int) {}
 
   virtual void UpdateAllLifecyclePhases() {}
   virtual void InvalidatePaintIfNeeded() {}
-
- protected:
-  PluginView() : FrameViewBase() {}
 };
 
-DEFINE_TYPE_CASTS(PluginView,
-                  FrameViewBase,
-                  frameViewBase,
-                  frameViewBase->IsPluginView(),
-                  frameViewBase.IsPluginView());
-
 }  // namespace blink
 
 #endif  // PluginView_h