Browser Plugin: New Implementation (Browser Side)

content/browser/browser_plugin/browser_plugin_guest_helper.cc

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/browser/browser_plugin/browser_plugin_guest_helper.h"
+
+#include "base/time.h"
+#include "content/browser/browser_plugin/browser_plugin_guest.h"
+#include "content/browser/renderer_host/render_view_host_impl.h"
+#include "content/browser/renderer_host/render_widget_host_impl.h"
+#include "content/browser/web_contents/web_contents_impl.h"
+#include "content/common/browser_plugin_messages.h"
+#include "content/common/view_messages.h"
+#include "content/public/browser/notification_details.h"
+#include "content/public/browser/notification_service.h"
+#include "content/public/browser/notification_source.h"
+#include "content/public/browser/notification_types.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/render_widget_host_view.h"
+#include "ui/gfx/size.h"
+
+namespace content {
+
+BrowserPluginGuestHelper::BrowserPluginGuestHelper(
+    BrowserPluginGuest* guest,
+    RenderViewHost* render_view_host)
+    : RenderViewHostObserver(render_view_host),
+      guest_(guest) {
+}
+
+BrowserPluginGuestHelper::~BrowserPluginGuestHelper() {
+}
+
+bool BrowserPluginGuestHelper::OnMessageReceived(
+    const IPC::Message& message) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP(BrowserPluginGuestHelper, message)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_UpdateRect, OnUpdateRect)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_HandleInputEvent_ACK, OnHandleInputEventAck)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_TakeFocus, OnTakeFocus)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_ShowWidget, OnShowWidget)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_SetCursor, OnSetCursor)
+    IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  return handled;
+}
+
+void BrowserPluginGuestHelper::OnUpdateRect(
+    const ViewHostMsg_UpdateRect_Params& params) {
+  RenderWidgetHostImpl* render_widget_host =
+      RenderWidgetHostImpl::From(render_view_host());
+  render_widget_host->ResetSizeAndRepaintPendingFlags();

[awong, 2012/09/09 18:08:09]

If this whole object is really meant to be a passthrough that you can
instantiate on the IO thread, should the call to
ResetSizeAndRepaintPendingFlags() be inside guest_->UpdateRect()?

[lazyboy, 2012/09/10 16:30:17]

Done.

+  guest_->UpdateRect(render_view_host(), params);
+}
+
+void BrowserPluginGuestHelper::OnHandleInputEventAck(
+    WebKit::WebInputEvent::Type event_type,
+    bool processed) {
+  guest_->HandleInputEventAck(render_view_host(), processed);
+}
+
+void BrowserPluginGuestHelper::OnTakeFocus(bool reverse) {
+  guest_->ViewTakeFocus(reverse);
+}
+
+void BrowserPluginGuestHelper::OnShowWidget(int route_id,
+                                            const gfx::Rect& initial_pos) {
+  guest_->ShowWidget(render_view_host(), route_id, initial_pos);
+}
+
+void BrowserPluginGuestHelper::OnSetCursor(const WebCursor& cursor) {
+  guest_->SetCursor(cursor);
+}
+
+}  // namespace content