Some clients of Geolocation directly call the
Gelocation::OverrideLocationForTesting() in their browser-test or Layout-test.
We should make them avoid calling OverrideLocationForTesting() since the Geolocation will be moved to //service/device.
The test case of GeolocationBrowserTestWithOutOverrider::UrlWithApiKey tries to connect the real geolocation implementation instead of the FakeGeolocation.
So we leave a TODO that to remove the class GeolocationBrowserTestWithOutOverrider and rewrite the test case of UrlWithApiKey as a services_unittest. Also remove the
ui_test_utils::OverrideGeolocation() then.
Comment 1 by ke...@intel.com
, Nov 24 2017