Add GamepadService, owns 1 gamepad bg thread

content/browser/renderer_host/gamepad_browser_message_filter.h

Index: content/browser/renderer_host/gamepad_browser_message_filter.h
diff --git a/content/browser/renderer_host/gamepad_browser_message_filter.h b/content/browser/renderer_host/gamepad_browser_message_filter.h
index dc991e0198a5888d99400646ba8d0e0f5b37bc9c..a1451cf17a189a521d3287731c37257fe640f8a3 100644
--- a/content/browser/renderer_host/gamepad_browser_message_filter.h
+++ b/content/browser/renderer_host/gamepad_browser_message_filter.h
@@ -7,11 +7,18 @@
 #pragma once
 
 #include "base/compiler_specific.h"
+#include "base/shared_memory.h"

[jam, 2011/12/01 20:07:31]

nit: forward declare instead

[scottmg, 2011/12/01 22:05:32]

Unfortunately SharedMemoryHandle is a typedef.

 #include "content/browser/browser_message_filter.h"
-#include "content/browser/gamepad/gamepad_provider.h"
+
+namespace content {
+
+class GamepadService;
+class RenderProcessHost;
 
 class GamepadBrowserMessageFilter : public BrowserMessageFilter {
  public:
+  GamepadBrowserMessageFilter(RenderProcessHost* rph);

[jam, 2011/12/01 20:07:31]

nit: explicit

[scottmg, 2011/12/01 22:05:32]

Done.

+
   // BrowserMessageFilter implementation.
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok) OVERRIDE;
@@ -23,9 +30,12 @@ class GamepadBrowserMessageFilter : public BrowserMessageFilter {
   void OnGamepadStartPolling(base::SharedMemoryHandle* renderer_handle);
   void OnGamepadStopPolling();
 
-  scoped_refptr<content::GamepadProvider> provider_;
+  GamepadService* gamepad_service_;

[jam, 2011/12/01 20:07:31]

nit: no need to keep a pointer to a singleton. just use GetInstance every time
you need it

[scottmg, 2011/12/01 22:05:32]

Done.

+  RenderProcessHost* render_process_host_;
 
   DISALLOW_COPY_AND_ASSIGN(GamepadBrowserMessageFilter);
 };
 
+} // namespace content
+
 #endif  // CONTENT_BROWSER_RENDERER_HOST_GAMEPAD_BROWSER_MESSAGE_FILTER_H_