WebView: prepare to depend on individual GMS services

WebView: prepare to depend on individual GMS services

Remove GMS from upstream WebView (it's only needed downstream) and
create a google_play_services_package variable with the upstream GMS
location, so it can be overridden downstream.

BUG=662166
Review-Url: https://codereview.chromium.org/2697543004
Cr-Commit-Position: refs/heads/master@{#450780}
Committed: https://chromium.googlesource.com/chromium/src/+/635a45202f8cdaef30cfa4c5d4b602660e81395e

[paulmiller, 2017-02-14 02:02:05]

Description was changed from

==========
WebView: prepare to depend on individual GMS services

Remove GMS from upstream WebView (it's only needed downstream) and
create a google_play_services_package variable with the upstream GMS
location, so it can be overridden downstream.

BUG=662166
==========

to

==========
WebView: prepare to depend on individual GMS services

Remove GMS from upstream WebView (it's only needed downstream) and
create a google_play_services_package variable with the upstream GMS
location, so it can be overridden downstream.

BUG=662166
==========

[paulmiller, 2017-02-14 02:02:06]

paulmiller@chromium.org changed reviewers:
+ agrieve@chromium.org, michaelbai@chromium.org

[paulmiller, 2017-02-14 02:06:50]

On 2017/02/14 02:02:06, paulmiller wrote:
> mailto:paulmiller@chromium.org changed reviewers:
> + mailto:agrieve@chromium.org, mailto:michaelbai@chromium.org

PTAL, Michael for android_webview/ & Andrew for build/config/android/.

The context here is that we want stand-alone WebView (and eventually Chrome and
Monochrome) to depend on individual parts of GMS rather than all of GMS.

This deviates slightly from my stated plan since I've discovered we're not
actually putting any of our GMS features upstream right now. So WebView only
needs GMS downstream.

[agrieve, 2017-02-14 02:11:40]

On 2017/02/14 02:06:50, paulmiller wrote:
> On 2017/02/14 02:02:06, paulmiller wrote:
> > mailto:paulmiller@chromium.org changed reviewers:
> > + mailto:agrieve@chromium.org, mailto:michaelbai@chromium.org
> 
> PTAL, Michael for android_webview/ & Andrew for build/config/android/.
> 
> The context here is that we want stand-alone WebView (and eventually Chrome
and
> Monochrome) to depend on individual parts of GMS rather than all of GMS.
> 
> This deviates slightly from my stated plan since I've discovered we're not
> actually putting any of our GMS features upstream right now. So WebView only
> needs GMS downstream.

build lgtm.

[michaelbai, 2017-02-15 02:56:24]

lgtm

[paulmiller, 2017-02-15 18:11:44]

The CQ bit was checked by paulmiller@chromium.org

[commit-bot: I haz the power, 2017-02-15 18:12:40]

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-02-15 19:50:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487182304776330, "parent_rev":
"2721a1e8771f3880b23ed94c787d9a1d3dd4cefd", "commit_rev":
"635a45202f8cdaef30cfa4c5d4b602660e81395e"}

[commit-bot: I haz the power, 2017-02-15 19:51:25]

Description was changed from

==========
WebView: prepare to depend on individual GMS services

Remove GMS from upstream WebView (it's only needed downstream) and
create a google_play_services_package variable with the upstream GMS
location, so it can be overridden downstream.

BUG=662166
==========

to

==========
WebView: prepare to depend on individual GMS services

Remove GMS from upstream WebView (it's only needed downstream) and
create a google_play_services_package variable with the upstream GMS
location, so it can be overridden downstream.

BUG=662166

Review-Url: https://codereview.chromium.org/2697543004
Cr-Commit-Position: refs/heads/master@{#450780}
Committed:
https://chromium.googlesource.com/chromium/src/+/635a45202f8cdaef30cfa4c5d4b6...
==========

[commit-bot: I haz the power, 2017-02-15 19:51:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/635a45202f8cdaef30cfa4c5d4b6...