Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Unified Diff: third_party/WebKit/Source/web/WebPluginContainerImpl.h

Issue 2814643003: Remove FrameViewBase as base class of PluginView. (Closed)
Patch Set: fix unused var Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/WebPluginContainerImpl.h
diff --git a/third_party/WebKit/Source/web/WebPluginContainerImpl.h b/third_party/WebKit/Source/web/WebPluginContainerImpl.h
index 1e01c54c94769cab5c7f756176a766d2449485bb..9a49e6c7b515f8d32522c9afc6966e3b6ecdab8b 100644
--- a/third_party/WebKit/Source/web/WebPluginContainerImpl.h
+++ b/third_party/WebKit/Source/web/WebPluginContainerImpl.h
@@ -72,6 +72,8 @@ class WEB_EXPORT WebPluginContainerImpl final
}
// PluginView methods
+ void SetParent(FrameView*) override;
+ FrameView* Parent() const override;
WebLayer* PlatformLayer() const override;
v8::Local<v8::Object> ScriptableObject(v8::Isolate*) override;
bool SupportsKeyboardFocus() const override;
@@ -80,21 +82,23 @@ class WEB_EXPORT WebPluginContainerImpl final
bool WantsWheelEvents() override;
void UpdateAllLifecyclePhases() override;
void InvalidatePaintIfNeeded() override { IssuePaintInvalidations(); }
-
- // FrameViewBase methods
- void SetFrameRect(const IntRect&) override;
- void Paint(GraphicsContext&, const CullRect&) const override;
- void InvalidateRect(const IntRect&) override;
+ void InvalidateRect(const IntRect&);
void SetFocused(bool, WebFocusType) override;
- void Show() override;
- void Hide() override;
void HandleEvent(Event*) override;
void FrameRectsChanged() override;
- void SetParentVisible(bool) override;
void GeometryMayHaveChanged() override;
bool IsPluginContainer() const override { return true; }
void EventListenersRemoved() override;
+ // FrameOrPlugin methods
+ void SetFrameRect(const IntRect& frame_rect) override {
+ frame_rect_ = frame_rect;
+ }
+ const IntRect& FrameRect() const override { return frame_rect_; }
+ void Paint(GraphicsContext&, const CullRect&) const override;
+ void Show() override;
+ void Hide() override;
+
// WebPluginContainer methods
WebElement GetElement() override;
WebDocument GetDocument() override;
@@ -206,24 +210,23 @@ class WEB_EXPORT WebPluginContainerImpl final
friend class WebPluginContainerTest;
+ Member<FrameView> parent_;
Member<HTMLPlugInElement> element_;
WebPlugin* web_plugin_;
-
WebLayer* web_layer_;
-
+ IntRect frame_rect_;
IntRect pending_invalidation_rect_;
-
TouchEventRequestType touch_event_request_type_;
bool wants_wheel_events_;
-
+ bool visible_;
bool is_disposed_;
};
DEFINE_TYPE_CASTS(WebPluginContainerImpl,
- FrameViewBase,
- frameViewBase,
- frameViewBase->IsPluginContainer(),
- frameViewBase.IsPluginContainer());
+ PluginView,
+ plugin,
+ plugin->IsPluginContainer(),
+ plugin.IsPluginContainer());
// Unlike FrameViewBase, we need not worry about object type for container.
// WebPluginContainerImpl is the only subclass of WebPluginContainer.
DEFINE_TYPE_CASTS(WebPluginContainerImpl,

Powered by Google App Engine
This is Rietveld 408576698