GitLab

Tests shouldn't call the OverrideLocationForTesting() since the Geolocation
core will be moved to //service/device.

In this CL a common ScopedGeolocationOverrider implementation is moved to
//device/geolocation/public/cpp, so it can be reused by multiple clients.

The ui_test_utils::OverrideGeolocation() is not removed in this CL because
it is still needed by UrlWithApiKey in GeolocationBrowserTest. We'll rewrite
it as a Service-Unittest when moving geolocation-core into //service/device
folder, and remove the ui_test_utils::OverrideGeolocation() then.

BUG=788298

Change-Id: I43c029bb5e82058673e738138cdadf139a076ff2
Reviewed-on: https://chromium-review.googlesource.com/778488
Commit-Queue: Ke He <ke.he@intel.com>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524312}