Geolocation Android: split LocationProviderAndroid out of LocationProviderFactory

Geolocation Android: split LocationProviderAndroid out of LocationProviderFactory

LocationProviderImpl is a private inner subclass of LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

This CL intends no new code so the current
javatests, content_browsertests and 
browser_tests should cover this.

BUG=706014
Review-Url: https://codereview.chromium.org/2797903002
Cr-Commit-Position: refs/heads/master@{#461985}
Committed: https://chromium.googlesource.com/chromium/src/+/7b9777d7a227fff0fdde91437960ba5812207f7e

[mcasas, 2017-04-05 02:05:57]

Patchset #1 (id:1) has been deleted

[mcasas, 2017-04-05 02:06:03]

Patchset #1 (id:20001) has been deleted

[mcasas, 2017-04-05 02:21:59]

Description was changed from

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

This change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

BUG=706014
TEST= no new code, current javatests and content_browsertests/browser_tests
should
cover this.
==========

to

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

BUG=706014
TEST= no new code, current javatests and content_browsertests/browser_tests
should
cover this.
==========

[mcasas, 2017-04-05 02:38:02]

mcasas@chromium.org changed reviewers:
+ scheib@chromium.org

[mcasas, 2017-04-05 02:38:03]

scheib@ ptal

[mcasas, 2017-04-05 04:03:11]

The CQ bit was checked by mcasas@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-05 04:03:26]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-05 05:02:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-05 05:02:57]

Dry run: This issue passed the CQ dry run.

[scheib, 2017-04-05 05:03:48]

lgtm

FYI, the TEST= line is intended to flag instructions needed for manual testers.
https://www.chromium.org/developers/contributing-code#TOC-Writing-change-list...

It can be omitted from changes such as this. Communicating automated test
coverage changes (or lack of the need) can be done with normal prose.

[mcasas, 2017-04-05 05:15:32]

Description was changed from

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

BUG=706014
TEST= no new code, current javatests and content_browsertests/browser_tests
should
cover this.
==========

to

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

This CL intends no new code so the current
javatests, content_browsertests and 
browser_tests should cover this.

BUG=706014
==========

[mcasas, 2017-04-05 05:15:42]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2017-04-05 05:15:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-05 05:19:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491369342149970,
"parent_rev": "9f5f6e0d4af19a7925b4f0b04a900781c066b018", "commit_rev":
"7b9777d7a227fff0fdde91437960ba5812207f7e"}

[commit-bot: I haz the power, 2017-04-05 05:21:46]

Description was changed from

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

This CL intends no new code so the current
javatests, content_browsertests and 
browser_tests should cover this.

BUG=706014
==========

to

==========
Geolocation Android: split LocationProviderAndroid out of
LocationProviderFactory

LocationProviderImpl is a private inner subclass of
LocationProviderFactory.java;
this CL renames it to LocationProviderAndroid.java and extracts it out of the
factory class.

Such change is needed to support the incoming LocationProviderGMS (or similar)
which provides location based on Google Play Services / GMSCore.

This CL intends no new code so the current
javatests, content_browsertests and 
browser_tests should cover this.

BUG=706014

Review-Url: https://codereview.chromium.org/2797903002
Cr-Commit-Position: refs/heads/master@{#461985}
Committed:
https://chromium.googlesource.com/chromium/src/+/7b9777d7a227fff0fdde91437960...
==========

[commit-bot: I haz the power, 2017-04-05 05:21:47]

Committed patchset #1 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/7b9777d7a227fff0fdde91437960...