Blink side of exposing the service worker registration associated with geofencing API calls.

Source/modules/geofencing/NavigatorGeofencing.cpp

 // 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 "config.h"
 #include "modules/geofencing/NavigatorGeofencing.h"
 
 #include "core/frame/Navigator.h"
 #include "modules/geofencing/Geofencing.h"
 
 namespace blink {
 
 NavigatorGeofencing::NavigatorGeofencing()

[Michael van Ouwerkerk, 2014/10/09 10:56:27]

Shouldn't this whole class be deleted now?

[Marijn Kruisselbrink, 2014/10/09 18:10:54]

Quoting the CL description "This still leaves the old codepaths for the
geofencing API in place, those will be cleaned up in a followup CL."
But if you want I can of course do more of the deletion/cleanup in this CL.

[michaeln, 2014/10/12 23:51:02]

From these review comments, looks like the existing navigator.geofencing surface
area is being replaced by registration.geofencing. Is that right?

It might be good to delete the old and commit the new at the same time.
Otherwise, it's hard to know what api your actually working towards. Its not so
clear in from the comments in the CL description, or from crbug/383125, or from
the code itself what the "old codepaths" are. If you want to leave old in place
for now, maybe put FIXMEs to delete the old stuff.

[Marijn Kruisselbrink, 2014/10/13 19:09:59]

I removed NavigatorGeofencing, and added FIXMEs to WorkerNavigatorGeofencing to
delete that one once ServiceWorkerRegistration is exposed in service workers.
I'd like to not completely lose the ability to call this API from a worker, just
to make sure nothing blows up if it is called that way. But obviously the API
won't actually work yet when called from a service worker since there isn't a
way yet to associate API calls with the service worker they are called from...

 {
 }
 
 NavigatorGeofencing::~NavigatorGeofencing()
 {
 }
 
 const char* NavigatorGeofencing::supplementName()
 {
     return "NavigatorGeofencing";
@@ skipping 10 matching lines @@
 }
 
 Geofencing* NavigatorGeofencing::geofencing(Navigator& navigator)
 {
     return NavigatorGeofencing::from(navigator).geofencing();
 }
 
 Geofencing* NavigatorGeofencing::geofencing()
 {
     if (!m_geofencing)
-        m_geofencing = Geofencing::create();
+        m_geofencing = Geofencing::create(nullptr);
     return m_geofencing.get();
 }
 
 void NavigatorGeofencing::trace(Visitor* visitor)
 {
     visitor->trace(m_geofencing);
     WillBeHeapSupplement<Navigator>::trace(visitor);
 }
 
 } // namespace blink