Refactor to make BlimpLocationProvider accessible to content layer.

content/browser/geolocation/mock_location_provider.h

Index: content/browser/geolocation/mock_location_provider.h
diff --git a/content/browser/geolocation/mock_location_provider.h b/content/browser/geolocation/mock_location_provider.h
index ec1a9777313ee8e881e75ff4a6515c94054f57e6..7c3372e29190d115fdb3ecdb52765edf2948ce31 100644
--- a/content/browser/geolocation/mock_location_provider.h
+++ b/content/browser/geolocation/mock_location_provider.h
@@ -19,9 +19,9 @@ namespace content {
 // Mock implementation of a location provider for testing.
 class MockLocationProvider : public LocationProviderBase {
  public:
-  // Will update |*self_ref| to point to |this| on construction, and to NULL
-  // on destruction.
-  explicit MockLocationProvider(MockLocationProvider** self_ref);
+  enum State { STOPPED, LOW_ACCURACY, HIGH_ACCURACY } state_;
+
+  MockLocationProvider();

[Wez, 2016/06/10 23:40:35]

As noted elsewhere, this should be FakeLocationProvider, but we can fix that
later. ;)

[CJ, 2016/06/13 23:45:20]

Acknowledged.

   ~MockLocationProvider() override;
 
   // Updates listeners with the new position.
@@ -33,16 +33,11 @@ class MockLocationProvider : public LocationProviderBase {
   void GetPosition(Geoposition* position) override;
   void OnPermissionGranted() override;
 
-  Geoposition position_;
-  enum State { STOPPED, LOW_ACCURACY, HIGH_ACCURACY } state_;
   bool is_permission_granted_;
-  MockLocationProvider** self_ref_;
-
+  Geoposition position_;
   scoped_refptr<base::SingleThreadTaskRunner> provider_task_runner_;
 
-  // Set when an instance of the mock is created via a factory function.
-  static MockLocationProvider* instance_;
-
+ private:
   DISALLOW_COPY_AND_ASSIGN(MockLocationProvider);
 };