Upstream Context.BIND_EXTERNAL_SERVICE

Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
Committed: https://crrev.com/47314b9e8fb4caf7a3bc6d91c6a0aad9c0a04109
Cr-Commit-Position: refs/heads/master@{#429642}

[michaelbai, 2016-11-02 22:22:07]

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

[commit-bot: I haz the power, 2016-11-02 22:22:58]

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, 2016-11-02 23:21:34]

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

[commit-bot: I haz the power, 2016-11-02 23:21:35]

Dry run: This issue passed the CQ dry run.

[michaelbai, 2016-11-02 23:41:58]

Description was changed from

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
==========

to

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
==========

[michaelbai, 2016-11-02 23:41:58]

michaelbai@chromium.org changed reviewers:
+ boliu@chromium.org

[boliu, 2016-11-03 04:18:32]

lgtm, but I don't think the sdk has rolled yet:

https://bugs.chromium.org/p/chromium/issues/detail?id=623989#c78

[michaelbai, 2016-11-03 04:29:44]

On 2016/11/03 04:18:32, boliu wrote:
> lgtm, but I don't think the sdk has rolled yet:
> 
> https://bugs.chromium.org/p/chromium/issues/detail?id=623989#c78

Well, it has been rolled.

[boliu, 2016-11-03 04:36:55]

On 2016/11/03 04:29:44, michaelbai wrote:
> On 2016/11/03 04:18:32, boliu wrote:
> > lgtm, but I don't think the sdk has rolled yet:
> > 
> > https://bugs.chromium.org/p/chromium/issues/detail?id=623989#c78
> 
> Well, it has been rolled.

oh, bots are green

go for it then

[michaelbai, 2016-11-03 15:38:34]

The CQ bit was checked by michaelbai@chromium.org

[commit-bot: I haz the power, 2016-11-03 15:38:58]

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, 2016-11-03 15:46:36]

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

[commit-bot: I haz the power, 2016-11-03 15:46:38]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[michaelbai, 2016-11-03 15:54:28]

michaelbai@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[michaelbai, 2016-11-03 15:54:29]

dfalcantara@ for WebApkActivity.java

[gone, 2016-11-03 17:16:18]

lgtm

[michaelbai, 2016-11-03 17:49:09]

The CQ bit was checked by michaelbai@chromium.org

[commit-bot: I haz the power, 2016-11-03 17:49:37]

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, 2016-11-03 17:55:30]

Description was changed from

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
==========

to

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
==========

[commit-bot: I haz the power, 2016-11-03 17:55:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-03 18:14:24]

Description was changed from

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722
==========

to

==========
Upstream Context.BIND_EXTERNAL_SERVICE

BUG=592722

Committed: https://crrev.com/47314b9e8fb4caf7a3bc6d91c6a0aad9c0a04109
Cr-Commit-Position: refs/heads/master@{#429642}
==========

[commit-bot: I haz the power, 2016-11-03 18:14:25]

Patchset 1 (id:??) landed as
https://crrev.com/47314b9e8fb4caf7a3bc6d91c6a0aad9c0a04109
Cr-Commit-Position: refs/heads/master@{#429642}