Browser Plugin: New Implementation (Browser Side)

content/browser/renderer_host/render_widget_host_impl.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/renderer_host/render_widget_host_impl.h"
 
 #include <math.h>
 #include <utility>
 
 #include "base/auto_reset.h"
@@ skipping 216 matching lines @@
     return view_->GetCompositingSurface();
   return gfx::GLSurfaceHandle();
 }
 
 void RenderWidgetHostImpl::CompositingSurfaceUpdated() {
   GpuSurfaceTracker::Get()->SetSurfaceHandle(
       surface_id_, GetCompositingSurface());
   process_->SurfaceUpdated(surface_id_);
 }
 
+void RenderWidgetHostImpl::ResetFlags() {

[rjkroege, 2012/08/22 21:57:38]

must reset the GestureEventFilter too

[lazyboy, 2012/08/23 00:45:22]

reset gesture_event_filter to new GestureEventFilter?
Also the current code path when RendererExited() gets called doesn't reset it,
should we reset it only for BrowserPlugin or always?

+  // Must reset these to ensure that mouse move/wheel events work with a new
+  // renderer.
+  mouse_move_pending_ = false;
+  next_mouse_move_.reset();
+  mouse_wheel_pending_ = false;
+  coalesced_mouse_wheel_events_.clear();
+
+  // Must reset these to ensure that keyboard events work with a new renderer.
+  key_queue_.clear();
+  suppress_next_char_events_ = false;
+
+  // Reset some fields in preparation for recovering from a crash.
+  resize_ack_pending_ = false;
+  repaint_ack_pending_ = false;
+
+  in_flight_size_.SetSize(0, 0);
+  current_size_.SetSize(0, 0);
+  is_hidden_ = false;
+  is_accelerated_compositing_active_ = false;
+
+  // Reset this to ensure the hung renderer mechanism is working properly.
+  in_flight_event_count_ = 0;
+}
+
 void RenderWidgetHostImpl::Init() {
   DCHECK(process_->HasConnection());
 
   renderer_initialized_ = true;
 
   GpuSurfaceTracker::Get()->SetSurfaceHandle(
       surface_id_, GetCompositingSurface());
 
   // Send the ack along with the information on placement.
   Send(new ViewMsg_CreatingNew_ACK(routing_id_, GetNativeViewId()));
@@ skipping 822 matching lines @@
 void RenderWidgetHostImpl::SetDeviceScaleFactor(float scale) {
   Send(new ViewMsg_SetDeviceScaleFactor(GetRoutingID(), scale));
 }
 
 void RenderWidgetHostImpl::RendererExited(base::TerminationStatus status,
                                           int exit_code) {
   // Clearing this flag causes us to re-create the renderer when recovering
   // from a crashed renderer.
   renderer_initialized_ = false;
 
-  // Must reset these to ensure that mouse move/wheel events work with a new
-  // renderer.
-  mouse_move_pending_ = false;
-  next_mouse_move_.reset();
-  mouse_wheel_pending_ = false;
-  coalesced_mouse_wheel_events_.clear();
-
-  // Must reset these to ensure that gesture events work with a new renderer.
-  gesture_event_filter_->Reset();
-
-  // Must reset these to ensure that keyboard events work with a new renderer.
-  key_queue_.clear();
-  suppress_next_char_events_ = false;
-
-  // Reset some fields in preparation for recovering from a crash.
-  resize_ack_pending_ = false;
-  repaint_ack_pending_ = false;
-
-  in_flight_size_.SetSize(0, 0);
-  current_size_.SetSize(0, 0);
-  is_hidden_ = false;
-  is_accelerated_compositing_active_ = false;
-
-  // Reset this to ensure the hung renderer mechanism is working properly.
-  in_flight_event_count_ = 0;
+  ResetFlags();
 
   if (view_) {
     GpuSurfaceTracker::Get()->SetSurfaceHandle(surface_id_,
                                                gfx::GLSurfaceHandle());
     view_->RenderViewGone(status, exit_code);
     view_ = NULL;  // The View should be deleted by RenderViewGone.
   }
 
   BackingStoreManager::RemoveBackingStore(this);
 }
@@ skipping 871 matching lines @@
   // indicate that no callback is in progress (i.e. without this line
   // DelayedAutoResized will not get called again).
   new_auto_size_.SetSize(0, 0);
   if (!should_auto_resize_)
     return;
 
   OnRenderAutoResized(new_size);
 }
 
 }  // namespace content