Hook PushMessagingMessageFilter up to GCMDriver

content/public/browser/browser_context.h

Index: content/public/browser/browser_context.h
diff --git a/content/public/browser/browser_context.h b/content/public/browser/browser_context.h
index 27a5569dba49ec295b6ec6cefccbebe0c778b8df..a34fd8dc80e2a24bb8d906bd4b76c39344b978c6 100644
--- a/content/public/browser/browser_context.h
+++ b/content/public/browser/browser_context.h
@@ -37,6 +37,7 @@ class DownloadManager;
 class DownloadManagerDelegate;
 class GeolocationPermissionContext;
 class IndexedDBContext;
+class PushMessagingService;
 class ResourceContext;
 class SiteInstance;
 class StoragePartition;
@@ -183,6 +184,11 @@ class CONTENT_EXPORT BrowserContext : public base::SupportsUserData {
 
   // Returns a special storage policy implementation, or NULL.
   virtual quota::SpecialStoragePolicy* GetSpecialStoragePolicy() = 0;
+
+  // Returns a push messaging service. The embedder owns the service, and is
+  // responsible for ensuring that it outlives RenderProcessHost. It's valid to
+  // return NULL.
+  virtual PushMessagingService* GetPushMessagingService() = 0;

[Michael van Ouwerkerk, 2014/06/06 13:42:46]

It seems we could provide a default implementation that returns NULL. Is that
against pattern?

[johnme, 2014/06/06 14:17:54]

None of the others do this. Perhaps it's intentional to force the embedder to
notice any new methods and think about them?

 };
 
 }  // namespace content