Partially convert geolocation IPC to Mojo.

content/browser/geolocation/geolocation_service_impl.h

Index: content/browser/geolocation/geolocation_service_impl.h
diff --git a/content/browser/geolocation/geolocation_service_impl.h b/content/browser/geolocation/geolocation_service_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..0304aa2bb4c62ae60741365a8ef73e9345fe289e
--- /dev/null
+++ b/content/browser/geolocation/geolocation_service_impl.h
@@ -0,0 +1,54 @@
+// 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 "base/memory/scoped_ptr.h"
+#include "content/browser/geolocation/geolocation_provider_impl.h"
+#include "content/common/geolocation_service.mojom.h"
+
+#ifndef CONTENT_BROWSER_GEOLOCATION_GEOLOCATION_SERVICE_IMPL_H_
+#define CONTENT_BROWSER_GEOLOCATION_GEOLOCATION_SERVICE_IMPL_H_
+
+namespace content {
+
+class GeolocationProvider;
+class GeolocationServiceContext;
+
+// Implements the GeolocationService Mojo interface.
+class GeolocationServiceImpl : public mojo::InterfaceImpl<GeolocationService> {
+ public:
+  // |context| must outlive this object. |update_callback| will be
+  // called when location updates are sent.
+  GeolocationServiceImpl(GeolocationServiceContext* context,
+                         const base::Closure& update_callback);
+  virtual ~GeolocationServiceImpl();
+
+  // Pauses and resumes sending updates to the client of this instance.
+  void PauseUpdates();
+  void ResumeUpdates();
+
+ private:
+  // GeolocationService:
+  virtual void StartUpdating(bool high_accuracy) OVERRIDE;

[qsr, 2014/10/09 08:58:33]

If this is part of the interface of the object, is there any reason to make this
private?

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

I made it private because it should only be called from generated code that
knows about an InterfaceImpl<GeolocationService>. I don't feel strongly about it
though.

+
+  // mojo::InterfaceImpl:
+  virtual void OnConnectionError() OVERRIDE;
+
+  void StartListeningForUpdates();
+  void OnLocationUpdate(const Geoposition& position);
+
+  // Owns this object.
+  GeolocationServiceContext* context_;
+  scoped_ptr<GeolocationProvider::Subscription> geolocation_subscription_;
+  base::Closure update_callback_;
+
+  // Whether this instance is currently observing location updates with high
+  // accuracy.
+  bool high_accuracy_;
+
+  DISALLOW_COPY_AND_ASSIGN(GeolocationServiceImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_GEOLOCATION_GEOLOCATION_SERVICE_IMPL_H_