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

Unified Diff: content/browser/frame_host/render_widget_host_view_guest.cc

Issue 100473010: Adding RenderWidgetHostViewChildFrame for OOPIF view. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed broken unit test Created 7 years 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: content/browser/frame_host/render_widget_host_view_guest.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_guest.cc b/content/browser/frame_host/render_widget_host_view_guest.cc
similarity index 87%
rename from content/browser/renderer_host/render_widget_host_view_guest.cc
rename to content/browser/frame_host/render_widget_host_view_guest.cc
index 49562c8dad7e4a074555919d0bfdbbee6bea241d..add4c17991dd681eaa50f75880cb479aff21ecac 100644
--- a/content/browser/renderer_host/render_widget_host_view_guest.cc
+++ b/content/browser/frame_host/render_widget_host_view_guest.cc
@@ -7,8 +7,8 @@
#include "base/logging.h"
#include "base/message_loop/message_loop.h"
#include "content/browser/browser_plugin/browser_plugin_guest.h"
+#include "content/browser/frame_host/render_widget_host_view_guest.h"
#include "content/browser/renderer_host/render_view_host_impl.h"
-#include "content/browser/renderer_host/render_widget_host_view_guest.h"
#include "content/common/browser_plugin/browser_plugin_messages.h"
#include "content/common/gpu/gpu_messages.h"
#include "content/common/view_messages.h"
@@ -51,14 +51,13 @@ RenderWidgetHostViewGuest::RenderWidgetHostViewGuest(
RenderWidgetHost* widget_host,
BrowserPluginGuest* guest,
RenderWidgetHostView* platform_view)
- : host_(RenderWidgetHostImpl::From(widget_host)),
+ : RenderWidgetHostViewChildFrame(widget_host),
guest_(guest),
platform_view_(static_cast<RenderWidgetHostViewPort*>(platform_view)) {
#if defined(OS_WIN) || defined(USE_AURA)
gesture_recognizer_.reset(ui::GestureRecognizer::Create());
gesture_recognizer_->AddGestureEventHelper(this);
#endif // defined(OS_WIN) || defined(USE_AURA)
- host_->SetView(this);
}
RenderWidgetHostViewGuest::~RenderWidgetHostViewGuest() {
@@ -67,10 +66,6 @@ RenderWidgetHostViewGuest::~RenderWidgetHostViewGuest() {
#endif // defined(OS_WIN) || defined(USE_AURA)
}
-RenderWidgetHost* RenderWidgetHostViewGuest::GetRenderWidgetHost() const {
- return host_;
-}
-
void RenderWidgetHostViewGuest::WasShown() {
// If the WebContents associated with us showed an interstitial page in the
// beginning, the teardown path might call WasShown() while |host_| is in
@@ -96,13 +91,8 @@ void RenderWidgetHostViewGuest::SetSize(const gfx::Size& size) {
host_->WasResized();
}
-gfx::Rect RenderWidgetHostViewGuest::GetBoundsInRootWindow() {
- // We do not have any root window specific parts in this view.
- return GetViewBounds();
-}
-
-gfx::GLSurfaceHandle RenderWidgetHostViewGuest::GetCompositingSurface() {
- return gfx::GLSurfaceHandle(gfx::kNullPluginWindow, gfx::TEXTURE_TRANSPORT);
+void RenderWidgetHostViewGuest::SetBounds(const gfx::Rect& rect) {
+ SetSize(rect.size());
}
#if defined(OS_WIN) || defined(USE_AURA)
@@ -129,18 +119,6 @@ void RenderWidgetHostViewGuest::ProcessAckedTouchEvent(
}
#endif
-void RenderWidgetHostViewGuest::Show() {
- WasShown();
-}
-
-void RenderWidgetHostViewGuest::Hide() {
- WasHidden();
-}
-
-bool RenderWidgetHostViewGuest::IsShowing() {
- return !host_->is_hidden();
-}
-
gfx::Rect RenderWidgetHostViewGuest::GetViewBounds() const {
RenderWidgetHostViewPort* rwhv = static_cast<RenderWidgetHostViewPort*>(
guest_->GetEmbedderRenderWidgetHostView());
@@ -169,15 +147,15 @@ void RenderWidgetHostViewGuest::Destroy() {
platform_view_->Destroy();
}
+gfx::Size RenderWidgetHostViewGuest::GetPhysicalBackingSize() const {
+ return RenderWidgetHostViewBase::GetPhysicalBackingSize();
+}
+
void RenderWidgetHostViewGuest::SetTooltipText(
const base::string16& tooltip_text) {
platform_view_->SetTooltipText(tooltip_text);
}
-void RenderWidgetHostViewGuest::AcceleratedSurfaceInitialized(int host_id,
- int route_id) {
-}
-
void RenderWidgetHostViewGuest::AcceleratedSurfaceBuffersSwapped(
const GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params& params,
int gpu_host_id) {
@@ -229,10 +207,6 @@ void RenderWidgetHostViewGuest::OnSwapCompositorFrame(
host_->GetProcess()->GetID()));
}
-void RenderWidgetHostViewGuest::SetBounds(const gfx::Rect& rect) {
- SetSize(rect.size());
-}
-
bool RenderWidgetHostViewGuest::OnMessageReceived(const IPC::Message& msg) {
return platform_view_->OnMessageReceived(msg);
}
@@ -274,21 +248,6 @@ void RenderWidgetHostViewGuest::MovePluginWindows(
platform_view_->MovePluginWindows(scroll_offset, moves);
}
-void RenderWidgetHostViewGuest::Focus() {
-}
-
-void RenderWidgetHostViewGuest::Blur() {
-}
-
-bool RenderWidgetHostViewGuest::HasFocus() const {
- return false;
-}
-
-bool RenderWidgetHostViewGuest::IsSurfaceAvailableForCopy() const {
- NOTIMPLEMENTED();
- return false;
-}
-
void RenderWidgetHostViewGuest::UpdateCursor(const WebCursor& cursor) {
platform_view_->UpdateCursor(cursor);
}
@@ -336,55 +295,10 @@ void RenderWidgetHostViewGuest::SelectionBoundsChanged(
platform_view_->SelectionBoundsChanged(params);
}
-void RenderWidgetHostViewGuest::ScrollOffsetChanged() {
-}
-
-BackingStore* RenderWidgetHostViewGuest::AllocBackingStore(
- const gfx::Size& size) {
- NOTREACHED();
- return NULL;
-}
-
-void RenderWidgetHostViewGuest::CopyFromCompositingSurface(
- const gfx::Rect& src_subrect,
- const gfx::Size& /* dst_size */,
- const base::Callback<void(bool, const SkBitmap&)>& callback) {
- callback.Run(false, SkBitmap());
-}
-
-void RenderWidgetHostViewGuest::CopyFromCompositingSurfaceToVideoFrame(
- const gfx::Rect& src_subrect,
- const scoped_refptr<media::VideoFrame>& target,
- const base::Callback<void(bool)>& callback) {
- NOTIMPLEMENTED();
- callback.Run(false);
-}
-
-bool RenderWidgetHostViewGuest::CanCopyToVideoFrame() const {
- return false;
-}
-
-void RenderWidgetHostViewGuest::AcceleratedSurfaceSuspend() {
- NOTREACHED();
-}
-
-void RenderWidgetHostViewGuest::AcceleratedSurfaceRelease() {
-}
-
-bool RenderWidgetHostViewGuest::HasAcceleratedSurface(
- const gfx::Size& desired_size) {
- return false;
-}
-
void RenderWidgetHostViewGuest::SetBackground(const SkBitmap& background) {
platform_view_->SetBackground(background);
}
-#if defined(OS_WIN) && !defined(USE_AURA)
-void RenderWidgetHostViewGuest::SetClickthroughRegion(SkRegion* region) {
-}
-#endif
-
void RenderWidgetHostViewGuest::SetHasHorizontalScrollbar(
bool has_horizontal_scrollbar) {
platform_view_->SetHasHorizontalScrollbar(has_horizontal_scrollbar);
@@ -396,9 +310,6 @@ void RenderWidgetHostViewGuest::SetScrollOffsetPinning(
is_pinned_to_left, is_pinned_to_right);
}
-void RenderWidgetHostViewGuest::OnAcceleratedCompositingStateChange() {
-}
-
bool RenderWidgetHostViewGuest::LockMouse() {
return platform_view_->LockMouse();
}

Powered by Google App Engine
This is Rietveld 408576698