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 GeofencingManager;
+class ServiceWorkerContextWrapper;
 
 class GeofencingDispatcherHost : public BrowserMessageFilter {
  public:
   explicit GeofencingDispatcherHost(GeofencingManager* geofencing_manager);
 
  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,
+                        int64 service_worker_registration_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,

[palmer, 2014/10/16 01:22:34]

What is the lexical/syntactic structure of these region IDs? Can/do we assert
the correct structure somewhere in the browser process?

[Marijn Kruisselbrink, 2014/10/16 17:00:19]

There is no lexical/syntactic structure, these are arbitrary website provided
strings. At your suggestion these methods already do a basic sanity check to
make sure the size of these strings is < 200.

+                          int64 service_worker_registration_id);
+  void OnGetRegisteredRegions(int thread_id,
+                              int request_id,
+                              int64 service_worker_registration_id);
 
   void RegisterRegionCompleted(int thread_id,
                                int request_id,
                                GeofencingStatus result);
   void UnregisterRegionCompleted(int thread_id,
                                  int request_id,
                                  GeofencingStatus result);
 
   scoped_refptr<GeofencingManager> manager_;
   base::WeakPtrFactory<GeofencingDispatcherHost> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GeofencingDispatcherHost);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_GEOFENCING_GEOFENCING_DISPATCHER_HOST_H_