Adds GeolocationFeature for Blimp Geolocation project.

device/geolocation/fake_location_provider.cc

 // Copyright (c) 2012 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.
 
-// This file implements a mock location provider and the factory functions for
+// This file implements a fake location provider and the factory functions for
 // various ways of creating it.
+// TODO(lethalantidote): Convert location_arbitrator_impl to use actual mock
+// instead of FakeLocationProvider.
 
-#include "device/geolocation/mock_location_provider.h"
+#include "device/geolocation/fake_location_provider.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/thread_task_runner_handle.h"
 
 namespace device {
 
-MockLocationProvider::MockLocationProvider()
+FakeLocationProvider::FakeLocationProvider()
     : state_(STOPPED),
       is_permission_granted_(false),
       provider_task_runner_(base::ThreadTaskRunnerHandle::Get()) {}
 
-MockLocationProvider::~MockLocationProvider() {}
+FakeLocationProvider::~FakeLocationProvider() {}
 
-bool MockLocationProvider::IsProviderStarted() const {
+bool FakeLocationProvider::IsProviderStarted() const {
   return state_ != STOPPED;
 }
 
-void MockLocationProvider::HandlePositionChanged(const Geoposition& position) {
+void FakeLocationProvider::HandlePositionChanged(const Geoposition& position) {
   if (provider_task_runner_->BelongsToCurrentThread()) {
     // The location arbitrator unit tests rely on this method running
     // synchronously.
     position_ = position;
     NotifyCallback(position_);
   } else {
     provider_task_runner_->PostTask(
-        FROM_HERE, base::Bind(&MockLocationProvider::HandlePositionChanged,
+        FROM_HERE, base::Bind(&FakeLocationProvider::HandlePositionChanged,
                               base::Unretained(this), position));
   }
 }
 
-bool MockLocationProvider::StartProvider(bool high_accuracy) {
+bool FakeLocationProvider::StartProvider(bool high_accuracy) {
   state_ = high_accuracy ? HIGH_ACCURACY : LOW_ACCURACY;
   return true;
 }
 
-void MockLocationProvider::StopProvider() {
+void FakeLocationProvider::StopProvider() {
   state_ = STOPPED;
 }
 
-const Geoposition& MockLocationProvider::GetPosition() {
+const Geoposition& FakeLocationProvider::GetPosition() {
   return position_;
 }
 
-void MockLocationProvider::OnPermissionGranted() {
+void FakeLocationProvider::OnPermissionGranted() {
   is_permission_granted_ = true;
 }
 
-// Mock location provider that automatically calls back its client at most
+// Fake location provider that automatically calls back its client at most
 // once, when StartProvider or OnPermissionGranted is called. Use
 // |requires_permission_to_start| to select which event triggers the callback.
-class AutoMockLocationProvider : public MockLocationProvider {
+class AutoFakeLocationProvider : public FakeLocationProvider {

[Wez, 2016/08/19 18:50:25]

If you've removed the New*() methods below, surely there is no way to create
this class any more?

[CJ, 2016/08/19 22:22:34]

I dont think this class is used anywhere. I'm removing it.

[Wez, 2016/08/20 01:42:11]

My point was that it _can't_ be used from anywhere - it's declared in this .cc
file, so no other code could possibly refer to it, and you've removed the uses
of it in the .cc file itself already - so you can be 100% sure it's dead code :)

[CJ, 2016/08/22 17:56:39]

No I understood. Sorry poor phrasing by me I guess.

  public:
-  AutoMockLocationProvider(bool has_valid_location,
+  AutoFakeLocationProvider(bool has_valid_location,
                            bool requires_permission_to_start)
       : requires_permission_to_start_(requires_permission_to_start),
         listeners_updated_(false) {
     if (has_valid_location) {
       position_.accuracy = 3;
       position_.latitude = 4.3;
       position_.longitude = -7.8;
       // Webkit compares the timestamp to wall clock time, so we need it to be
       // contemporary.
       position_.timestamp = base::Time::Now();
     } else {
       position_.error_code = Geoposition::ERROR_CODE_POSITION_UNAVAILABLE;
     }
   }
   bool StartProvider(bool high_accuracy) override {
-    MockLocationProvider::StartProvider(high_accuracy);
+    FakeLocationProvider::StartProvider(high_accuracy);
     if (!requires_permission_to_start_) {
       UpdateListenersIfNeeded();
     }
     return true;
   }
 
   void OnPermissionGranted() override {
-    MockLocationProvider::OnPermissionGranted();
+    FakeLocationProvider::OnPermissionGranted();
     if (requires_permission_to_start_) {
       UpdateListenersIfNeeded();
     }
   }
 
   void UpdateListenersIfNeeded() {
     if (!listeners_updated_) {
       listeners_updated_ = true;
       base::ThreadTaskRunnerHandle::Get()->PostTask(
-          FROM_HERE, base::Bind(&MockLocationProvider::HandlePositionChanged,
+          FROM_HERE, base::Bind(&FakeLocationProvider::HandlePositionChanged,
                                 base::Unretained(this), position_));
     }
   }
 
  private:
   const bool requires_permission_to_start_;
   bool listeners_updated_;
 
-  DISALLOW_COPY_AND_ASSIGN(AutoMockLocationProvider);
+  DISALLOW_COPY_AND_ASSIGN(AutoFakeLocationProvider);
 };
 
-LocationProvider* NewMockLocationProvider() {

[Wez, 2016/08/19 18:50:25]

Looks like you've removed these from the .cc but renamed them in the .h?

[CJ, 2016/08/19 22:22:33]

Ah. Sorry, another merge failure on my part. Gone.

-  return new MockLocationProvider;
-}
-
-LocationProvider* NewAutoSuccessMockLocationProvider() {
-  return new AutoMockLocationProvider(true, false);
-}
-
-LocationProvider* NewAutoFailMockLocationProvider() {
-  return new AutoMockLocationProvider(false, false);
-}
-
-LocationProvider* NewAutoSuccessMockNetworkLocationProvider() {
-  return new AutoMockLocationProvider(true, true);
-}
-
 }  // namespace device