New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 850357 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove dependencies from geolocation public java library to implementation.

Project Member Reported by ke...@intel.com, Jun 7 2018

Issue description

There are several clients(android_webview, chrome and content) directly set a MockLocationProvider to LocationProviderFactory.java for testing purpose. Which leads to the geolocation/public/java:geolocation_java_test_support unavoidably depend on geolocation:geolocation_java.

It is ok given that the dependency is just for tests, but ideally we should remove this dependency in the long term.

There are more discussions in CL: https://chromium-review.googlesource.com/c/chromium/src/+/1079857 

 

Comment 1 Deleted

Comment 2 Deleted

Comment 3 Deleted

Comment 4 Deleted

Comment 5 Deleted

Comment 6 Deleted

Comment 7 Deleted

Comment 8 Deleted

Comment 9 Deleted

Comment 10 Deleted

Comment 11 Deleted

Comment 12 Deleted

Comment 13 Deleted

Sign in to add a comment