Banish boilerplate. Profile-keyed API factory for extension API classes.

chrome/browser/extensions/api/tabs/tabs_windows_api.h

Index: chrome/browser/extensions/api/tabs/tabs_windows_api.h
diff --git a/chrome/browser/extensions/api/tabs/tabs_windows_api.h b/chrome/browser/extensions/api/tabs/tabs_windows_api.h
index ae24d5c8b34a4dd7e8907810a69af5b6d90cede2..3e66b5f869e0e8bd0c7c6b8a3b164a8956ae55d1 100644
--- a/chrome/browser/extensions/api/tabs/tabs_windows_api.h
+++ b/chrome/browser/extensions/api/tabs/tabs_windows_api.h
@@ -6,13 +6,14 @@
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_WINDOWS_API_H_
 
 #include "base/memory/scoped_ptr.h"
+#include "chrome/browser/extensions/api/profile_keyed_api_factory.h"
 #include "chrome/browser/extensions/event_router.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 
 namespace extensions {
 class WindowsEventRouter;
 
-class TabsWindowsAPI : public ProfileKeyedService,
+class TabsWindowsAPI : public ProfileKeyedAPI,
                        public extensions::EventRouter::Observer {
  public:
   explicit TabsWindowsAPI(Profile* profile);
@@ -31,8 +32,16 @@ class TabsWindowsAPI : public ProfileKeyedService,
       OVERRIDE;
 
  private:
+  friend class ProfileKeyedAPIFactory<TabsWindowsAPI>;
+
   Profile* profile_;
 
+  static const char* service_name() {

[Jeffrey Yasskin, 2012/12/22 00:16:04]

Possibly comment this as "ProfileKeyedAPI implementation" to parallel what we do
for virtual functions, since these are basically the template version of virtual
functions.

[Yoyo Zhou, 2012/12/26 17:46:33]

Done.

+    return "TabsWindowsAPI";
+  }
+
+  static const bool kServiceIsNULLWhileTesting = true;
+
   scoped_ptr<WindowsEventRouter> windows_event_router_;
 };