-
pkotwicz authored
The geolocation tests did not work because: - They referred to incorrect test pages. Correct test pages: https://chrome-internal-review.googlesource.com/#/c/27084/3/test/data/android/geolocation/geolocation_on_load.html https://chrome-internal-review.googlesource.com/#/c/27084/3/test/data/android/geolocation/geolocation_watch_on_load.html - A race condition was causing LocationProviderAdapter.newLocationAvailable() to be often called prior to LocationProviderFactory.LocationProvider.start(). In addition to fixing the above problems, this CL makes org.chromium.chrome.browser.GeolocationTest more similar to - org.chromium.android_webview.test.GeolocationTest - org.chromium.content.browser.ContentViewLocationTest In particular, all of the tests now: - Use the same test page - Use MockLocationProvider BUG=141518 Test=Less flakiness Review URL: https://codereview.chromium.org/1599113002 Cr-Commit-Position: refs/heads/master@{#370807}5228d5af
