Gamepad API: add support for connection events

content/renderer/render_thread_impl.h

Index: content/renderer/render_thread_impl.h
diff --git a/content/renderer/render_thread_impl.h b/content/renderer/render_thread_impl.h
index 65883c773befd420d5061f6969339921291f429f..d1bd26a0e601100d2f71f9629efb3599cac1d9a7 100644
--- a/content/renderer/render_thread_impl.h
+++ b/content/renderer/render_thread_impl.h
@@ -33,6 +33,7 @@ struct WorkerProcessMsg_CreateWorker_Params;
 
 namespace blink {
 class WebGamepads;
+class WebGamepadListener;
 class WebGraphicsContext3D;
 class WebMediaStreamCenter;
 class WebMediaStreamCenterClient;
@@ -366,6 +367,10 @@ class CONTENT_EXPORT RenderThreadImpl : public RenderThread,
   // Retrieve current gamepad data.
   void SampleGamepads(blink::WebGamepads* data);
 
+  // Set a listener for gamepad connected/disconnected events.
+  // A non-null listener must be set first before calling SampleGamepads.
+  void SetGamepadListener(blink::WebGamepadListener* listener);

[jam, 2014/05/05 17:20:32]

nit: how about we remove this method, and instead expose a getter for
gamepad_shared_memory_reader_? RenderThreadImpl shouldn't have to know about
every feature, other than being a central place that owns process-wide objects
in the renderer

+
   // Called by a RenderWidget when it is created or destroyed. This
   // allows the process to know when there are no visible widgets.
   void WidgetCreated();
@@ -530,7 +535,7 @@ class CONTENT_EXPORT RenderThreadImpl : public RenderThread,
 
   scoped_ptr<WebRTCIdentityService> webrtc_identity_service_;
 
-  scoped_ptr<GamepadSharedMemoryReader> gamepad_shared_memory_reader_;
+  scoped_refptr<GamepadSharedMemoryReader> gamepad_shared_memory_reader_;
 
   // TODO(reveman): Allow AllocateGpuMemoryBuffer to be called from
   // multiple threads. Current allocation mechanism for IOSurface