Move all geofencing code into its own module separate from geolocation.

Source/modules/geofencing/NavigatorGeofencing.h

Index: Source/modules/geofencing/NavigatorGeofencing.h
diff --git a/Source/modules/geofencing/NavigatorGeofencing.h b/Source/modules/geofencing/NavigatorGeofencing.h
new file mode 100644
index 0000000000000000000000000000000000000000..b1b149709f39d7d9f45a348531113c3e802943ac
--- /dev/null
+++ b/Source/modules/geofencing/NavigatorGeofencing.h
@@ -0,0 +1,39 @@
+// 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 NavigatorGeofencing_h
+#define NavigatorGeofencing_h
+
+#include "platform/Supplementable.h"
+#include "platform/heap/Handle.h"
+
+namespace blink {
+
+class Geofencing;
+class Navigator;
+
+class NavigatorGeofencing FINAL
+    : public NoBaseWillBeGarbageCollectedFinalized<NavigatorGeofencing>

[Peter Beverloo, 2014/07/25 09:22:20]

Since this would be a new module, please just default to using Oilpan rather
than the transitional types.

[Marijn Kruisselbrink, 2014/07/30 23:40:54]

Yeah, there I made the mistake again of assuming that modules that have been
converted to use Oilpan already are doing the right thing (geolocation,
push_messaging)... Hopefully I've changed it correctly since this can only be
partly garbage collectable with Navigator itself still being refcounted.

+    , public WillBeHeapSupplement<Navigator> {
+    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(NavigatorGeofencing);
+
+public:
+    virtual ~NavigatorGeofencing();
+    static NavigatorGeofencing& from(Navigator&);
+
+    static Geofencing* geofencing(Navigator&);

[Peter Beverloo, 2014/07/25 09:22:20]

Return a reference.

[Marijn Kruisselbrink, 2014/07/30 23:40:54]

Afaik the non-static version has to return a pointer since that's what the
generated code expects. Furthermore all other similar code I've looked at
returns a pointer in the static getter as well (again, geolocation and
push_messaging for two examples). So for consistency I think just keeping this a
pointer is the way to go.

+    Geofencing* geofencing();
+
+    virtual void trace(Visitor*) OVERRIDE;
+
+private:
+    NavigatorGeofencing();
+    static const char* supplementName();
+
+    PersistentWillBeMember<Geofencing> m_geofencing;
+};
+
+} // namespace blink
+
+#endif // NavigatorGeofencing_h