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

content/browser/geofencing/geofencing_dispatcher_host.cc

 // 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.
 
 #include "content/browser/geofencing/geofencing_dispatcher_host.h"
 
 #include "content/browser/geofencing/geofencing_manager.h"
+#include "content/browser/service_worker/service_worker_context_core.h"
+#include "content/browser/service_worker/service_worker_context_wrapper.h"
+#include "content/browser/service_worker/service_worker_registration.h"
 #include "content/common/geofencing_messages.h"
 #include "third_party/WebKit/public/platform/WebCircularGeofencingRegion.h"
 #include "url/gurl.h"
 
 namespace content {
 
 static const int kMaxRegionIdLength = 200;
 
 GeofencingDispatcherHost::GeofencingDispatcherHost(
-    BrowserContext* browser_context)
+    BrowserContext* browser_context,
+    scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
     : BrowserMessageFilter(GeofencingMsgStart),
       browser_context_(browser_context),
+      service_worker_context_(service_worker_context),
       weak_factory_(this) {
 }
 
 GeofencingDispatcherHost::~GeofencingDispatcherHost() {
 }
 
 bool GeofencingDispatcherHost::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(GeofencingDispatcherHost, message)
   IPC_MESSAGE_HANDLER(GeofencingHostMsg_RegisterRegion, OnRegisterRegion)
   IPC_MESSAGE_HANDLER(GeofencingHostMsg_UnregisterRegion, OnUnregisterRegion)
   IPC_MESSAGE_HANDLER(GeofencingHostMsg_GetRegisteredRegions,
                       OnGetRegisteredRegions)
   IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void GeofencingDispatcherHost::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) {
   // Sanity check on region_id
   if (region_id.length() > kMaxRegionIdLength) {
     Send(new GeofencingMsg_RegisterRegionComplete(
         thread_id, request_id, GeofencingStatus::GEOFENCING_STATUS_ERROR));
     return;
   }
-  // TODO(mek): Actually pass service worker information to manager.
+
+  // Look up service worker.
+  ServiceWorkerRegistration* service_worker_registration =
+      service_worker_context_->context()->GetLiveRegistration(
+          service_worker_registration_id);
+
+  if (!service_worker_registration) {
+    Send(new GeofencingMsg_RegisterRegionComplete(
+        thread_id,
+        request_id,
+        GeofencingStatus::
+            GEOFENCING_STATUS_OPERATION_FAILED_NO_SERVICE_WORKER));
+    return;
+  }
+
+  GURL service_worker_origin =
+      service_worker_registration

[michaeln, 2014/10/14 01:03:52]

don't need to check for null here

[Marijn Kruisselbrink, 2014/10/14 18:51:22]

Done.

+          ? service_worker_registration->pattern().GetOrigin()
+          : GURL();
+
   GeofencingManager::GetInstance()->RegisterRegion(
       browser_context_,
-      0,      /* service_worker_registration_id */
-      GURL(), /* service_worker_origin */
+      service_worker_registration_id,
+      service_worker_origin,
       region_id,
       region,
       base::Bind(&GeofencingDispatcherHost::RegisterRegionCompleted,
                  weak_factory_.GetWeakPtr(),
                  thread_id,
                  request_id));
 }
 
 void GeofencingDispatcherHost::OnUnregisterRegion(
     int thread_id,
     int request_id,
-    const std::string& region_id) {
+    const std::string& region_id,
+    int64 service_worker_registration_id) {
   // Sanity check on region_id
   if (region_id.length() > kMaxRegionIdLength) {
     Send(new GeofencingMsg_UnregisterRegionComplete(
         thread_id, request_id, GeofencingStatus::GEOFENCING_STATUS_ERROR));
     return;
   }
-  // TODO(mek): Actually pass service worker information to manager.
+
+  // Look up service worker.
+  ServiceWorkerRegistration* service_worker_registration =
+      service_worker_context_->context()->GetLiveRegistration(
+          service_worker_registration_id);
+
+  if (!service_worker_registration) {
+    Send(new GeofencingMsg_UnregisterRegionComplete(
+        thread_id,
+        request_id,
+        GeofencingStatus::
+            GEOFENCING_STATUS_OPERATION_FAILED_NO_SERVICE_WORKER));
+    return;
+  }
+
+  GURL service_worker_origin =
+      service_worker_registration
+          ? service_worker_registration->pattern().GetOrigin()

[michaeln, 2014/10/14 01:03:52]

ditto, null check not needed

[Marijn Kruisselbrink, 2014/10/14 18:51:22]

Done.

+          : GURL();
+
   GeofencingManager::GetInstance()->UnregisterRegion(
       browser_context_,
-      0,      /* service_worker_registration_id */
-      GURL(), /* service_worker_origin */
+      service_worker_registration_id,
+      service_worker_origin,
       region_id,
       base::Bind(&GeofencingDispatcherHost::UnregisterRegionCompleted,
                  weak_factory_.GetWeakPtr(),
                  thread_id,
                  request_id));
 }
 
-void GeofencingDispatcherHost::OnGetRegisteredRegions(int thread_id,
-                                                      int request_id) {
+void GeofencingDispatcherHost::OnGetRegisteredRegions(
+    int thread_id,
+    int request_id,
+    int64 service_worker_registration_id) {
   GeofencingRegistrations result;
-  // TODO(mek): Actually pass service worker information to manager.
+
+  // Look up service worker.
+  ServiceWorkerRegistration* service_worker_registration =
+      service_worker_context_->context()->GetLiveRegistration(
+          service_worker_registration_id);
+
+  if (!service_worker_registration) {
+    Send(new GeofencingMsg_GetRegisteredRegionsComplete(
+        thread_id,
+        request_id,
+        GeofencingStatus::GEOFENCING_STATUS_OPERATION_FAILED_NO_SERVICE_WORKER,
+        result));
+    return;
+  }
+
+  GURL service_worker_origin =
+      service_worker_registration

[michaeln, 2014/10/14 01:03:52]

here too

[Marijn Kruisselbrink, 2014/10/14 18:51:22]

Done.

+          ? service_worker_registration->pattern().GetOrigin()
+          : GURL();
+
   GeofencingStatus status =
       GeofencingManager::GetInstance()->GetRegisteredRegions(
           browser_context_,
-          0,      /* service_worker_registration_id */
-          GURL(), /* service_worker_origin */
+          service_worker_registration_id,
+          service_worker_origin,
           &result);
   Send(new GeofencingMsg_GetRegisteredRegionsComplete(
       thread_id, request_id, status, result));
 }
 
 void GeofencingDispatcherHost::RegisterRegionCompleted(
     int thread_id,
     int request_id,
     GeofencingStatus status) {
   Send(new GeofencingMsg_RegisterRegionComplete(thread_id, request_id, status));
 }
 
 void GeofencingDispatcherHost::UnregisterRegionCompleted(
     int thread_id,
     int request_id,
     GeofencingStatus status) {
   Send(new GeofencingMsg_UnregisterRegionComplete(
       thread_id, request_id, status));
 }
 
 }  // namespace content