Partially convert geolocation IPC to Mojo.

content/renderer/geolocation_dispatcher.h

Index: content/renderer/geolocation_dispatcher.h
diff --git a/content/renderer/geolocation_dispatcher.h b/content/renderer/geolocation_dispatcher.h
index f4f91635057371bfe15cee74c2f7cd89f11c5a7c..1908eecea8e4b7b30f066732cb7a42bbd250c0b1 100644
--- a/content/renderer/geolocation_dispatcher.h
+++ b/content/renderer/geolocation_dispatcher.h
@@ -6,6 +6,7 @@
 #define CONTENT_RENDERER_GEOLOCATION_DISPATCHER_H_
 
 #include "base/memory/scoped_ptr.h"
+#include "content/common/geolocation_service.mojom.h"
 #include "content/public/renderer/render_frame_observer.h"
 #include "third_party/WebKit/public/web/WebGeolocationClient.h"
 #include "third_party/WebKit/public/web/WebGeolocationController.h"
@@ -23,8 +24,10 @@ struct Geoposition;
 // GeolocationDispatcher is a delegate for Geolocation messages used by
 // WebKit.
 // It's the complement of GeolocationDispatcherHost.
-class GeolocationDispatcher : public RenderFrameObserver,
-                              public blink::WebGeolocationClient {
+class GeolocationDispatcher
+    : public RenderFrameObserver,
+      public blink::WebGeolocationClient,
+      public mojo::InterfaceImpl<GeolocationServiceClient> {
  public:
   explicit GeolocationDispatcher(RenderFrame* render_frame);
   virtual ~GeolocationDispatcher();
@@ -44,18 +47,18 @@ class GeolocationDispatcher : public RenderFrameObserver,
   virtual void cancelPermissionRequest(
       const blink::WebGeolocationPermissionRequest& permissionRequest);
 
+  // GeolocationServiceClient
+  virtual void OnLocationUpdate(MojoGeopositionPtr geoposition) OVERRIDE;

[Michael van Ouwerkerk, 2014/10/09 13:11:23]

What are the characteristics of these mojo pointers?

[blundell, 2014/10/21 12:27:50]

Could you be more precise as to the question?

[Michael van Ouwerkerk, 2014/10/21 12:47:21]

Are they similar to one of the other smart pointers e.g. scoped_ptr or ref_ptr?
Where is this behavior documented?

+
   // Permission for using geolocation has been set.
   void OnPermissionSet(int bridge_id, bool is_allowed);
 
-  // We have an updated geolocation position or error code.
-  void OnPositionUpdated(const content::Geoposition& geoposition);
-
   scoped_ptr<blink::WebGeolocationController> controller_;
 
   scoped_ptr<blink::WebGeolocationPermissionRequestManager>
       pending_permissions_;
+  GeolocationServicePtr geolocation_service_;
   bool enable_high_accuracy_;
-  bool updating_;
 };
 
 }  // namespace content