Chrome side of passing on the service worker registration with geofencing API calls.

content/browser/geofencing/geofencing_dispatcher_host.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_GEOFENCING_GEOFENCING_DISPATCHER_HOST_H_
 #define CONTENT_BROWSER_GEOFENCING_GEOFENCING_DISPATCHER_HOST_H_
 
 #include "content/common/geofencing_status.h"
 #include "content/public/browser/browser_message_filter.h"
 
 namespace blink {
 struct WebCircularGeofencingRegion;
 }
 
 namespace content {
 
 class BrowserContext;
+class ServiceWorkerDispatcherHost;
 
 class GeofencingDispatcherHost : public BrowserMessageFilter {
  public:
-  explicit GeofencingDispatcherHost(BrowserContext* browser_context);
+  GeofencingDispatcherHost(
+      BrowserContext* browser_context,
+      scoped_refptr<ServiceWorkerDispatcherHost> service_worker_filter);
 
  private:
   virtual ~GeofencingDispatcherHost();
 
   // BrowserMessageFilter implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
   void OnRegisterRegion(int thread_id,
                         int request_id,
                         const std::string& region_id,
-                        const blink::WebCircularGeofencingRegion& region);
+                        const blink::WebCircularGeofencingRegion& region,
+                        int service_worker_registration_handle_id);
   void OnUnregisterRegion(int thread_id,
                           int request_id,
-                          const std::string& region_id);
-  void OnGetRegisteredRegions(int thread_id, int request_id);
+                          const std::string& region_id,
+                          int service_worker_registration_handle_id);
+  void OnGetRegisteredRegions(int thread_id,
+                              int request_id,
+                              int service_worker_registration_handle_id);
 
   void RegisterRegionCompleted(int thread_id,
                                int request_id,
                                GeofencingStatus result);
   void UnregisterRegionCompleted(int thread_id,
                                  int request_id,
                                  GeofencingStatus result);
 
   BrowserContext* browser_context_;
+  scoped_refptr<ServiceWorkerDispatcherHost> service_worker_filter_;

[Michael van Ouwerkerk, 2014/10/09 12:28:09]

It seems strange for one filter to have a pointer to another filter. Is there no
better API to interact with the service worker system?

[Marijn Kruisselbrink, 2014/10/09 22:28:58]

That's the question, yes.

   base::WeakPtrFactory<GeofencingDispatcherHost> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GeofencingDispatcherHost);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_GEOFENCING_GEOFENCING_DISPATCHER_HOST_H_