Test for BrowserPlugin-based WebView Focus

content/browser/frame_host/render_widget_host_view_child_frame.cc

Index: content/browser/frame_host/render_widget_host_view_child_frame.cc
diff --git a/content/browser/frame_host/render_widget_host_view_child_frame.cc b/content/browser/frame_host/render_widget_host_view_child_frame.cc
index e3a036176bbdbf690e3546a543b0b2c005d3fd63..a1af9c70645f7f89108059364b6d7f053524e35d 100644
--- a/content/browser/frame_host/render_widget_host_view_child_frame.cc
+++ b/content/browser/frame_host/render_widget_host_view_child_frame.cc
@@ -45,6 +45,7 @@ RenderWidgetHostViewChildFrame::RenderWidgetHostViewChildFrame(
       last_output_surface_id_(0),
       current_surface_scale_factor_(1.f),
       ack_pending_count_(0),
+      frame_count_(0),
       frame_connector_(nullptr),
       begin_frame_source_(nullptr),
       observing_begin_frame_source_(false),
@@ -352,6 +353,9 @@ void RenderWidgetHostViewChildFrame::SurfaceDrawn(uint32_t output_surface_id,
                                                   cc::SurfaceDrawStatus drawn) {
   cc::CompositorFrameAck ack;
   DCHECK_GT(ack_pending_count_, 0U);
+
+  frame_count_++;

[kenrb, 2016/07/05 14:30:56]

Should this be reset to zero when a new surface is created? I think this could
use a comment in the header describing what it is, and it doesn't affect your
current tests in any way, but it feels like clearing it for each surface would
be more intuitive for potential future uses of this count.

[wjmaclean, 2016/07/05 15:04:22]

Done.

+
   if (!surface_returned_resources_.empty())
     ack.resources.swap(surface_returned_resources_);
   if (host_) {