Fix a bug where an infinite stream of BeginFrame messages are sent to extensions.

content/browser/renderer_host/render_widget_host_view_mac.mm

Index: content/browser/renderer_host/render_widget_host_view_mac.mm
diff --git a/content/browser/renderer_host/render_widget_host_view_mac.mm b/content/browser/renderer_host/render_widget_host_view_mac.mm
index 5bef4d03ad0ff4c71539729b9cb150b3702aecd1..d7c05d394688908451d1dc31bd77d54eb49d3238 100644
--- a/content/browser/renderer_host/render_widget_host_view_mac.mm
+++ b/content/browser/renderer_host/render_widget_host_view_mac.mm
@@ -484,10 +484,13 @@ RenderWidgetHostViewMac::RenderWidgetHostViewMac(RenderWidgetHost* widget,
   }
 
   RenderViewHost* rvh = RenderViewHost::From(render_widget_host_);
+  bool needs_begin_frames = true;
+
   if (rvh) {
     // TODO(mostynb): actually use prefs.  Landing this as a separate CL
     // first to rebaseline some unreliable layout tests.
     ignore_result(rvh->GetWebkitPreferences());
+    needs_begin_frames = !rvh->GetDelegate()->IsNeverVisible();
   }
 
   if (GetTextInputManager())
@@ -497,10 +500,12 @@ RenderWidgetHostViewMac::RenderWidgetHostViewMac(RenderWidgetHost* widget,
   // in RenderMessageFilter::OnMessageReceived), SetNeedsBeginFrame
   // messages are not delayed on Mac.  This leads to creation-time
   // raciness where renderer sends a SetNeedsBeginFrame(true) before
-  // the renderer host is created to recieve it.  In general, all
-  // renderers want begin frames initially anyway, so start this value
-  // at true here to avoid startup raciness and decrease latency.
-  needs_begin_frames_ = true;
+  // the renderer host is created to recieve it.
+  //
+  // Any renderer that will produce frames needs to have begin frames sent to
+  // it. So unless it is never visible, start this value at true here to avoid
+  // startup raciness and decrease latency.
+  needs_begin_frames_ = needs_begin_frames;
   UpdateNeedsBeginFramesInternal();
 }