Reland "Android: Switch to thread pool executor"

Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053, 674293
Committed: https://crrev.com/80528b6a6acce7e1cc5829aaae876fb608aabb18
Cr-Commit-Position: refs/heads/master@{#439511}

[Peter Wen, 2016-12-15 13:51:00]

Description was changed from

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

BUG=601053
==========

to

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

BUG=601053,674293
==========

[Peter Wen, 2016-12-15 21:57:52]

Description was changed from

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

BUG=601053,674293
==========

to

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053,674293
==========

[Peter Wen, 2016-12-15 21:58:51]

wnwen@chromium.org changed reviewers:
+ iclelland@chromium.org

[Peter Wen, 2016-12-15 21:58:51]

Hi Ian,

Can you take a look at this fix? Just adding a wait function to the creation as
well since it kicks off an async task too.

Thanks,

Peter

[Peter Wen, 2016-12-19 14:51:14]

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

[commit-bot: I haz the power, 2016-12-19 14:51: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, 2016-12-19 15:28:10]

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

[commit-bot: I haz the power, 2016-12-19 15:28:11]

Dry run: This issue passed the CQ dry run.

[Peter Wen, 2016-12-19 17:58:07]

Friendly ping.

[iclelland, 2016-12-19 18:26:54]

On 2016/12/19 17:58:07, Peter Wen wrote:
> Friendly ping.

LGTM, sorry for the wait.

[Peter Wen, 2016-12-19 18:41:49]

On 2016/12/19 18:26:54, iclelland wrote:
> On 2016/12/19 17:58:07, Peter Wen wrote:
> > Friendly ping.
> 
> LGTM, sorry for the wait.

np, thanks for the review!

[Peter Wen, 2016-12-19 18:41:52]

The CQ bit was checked by wnwen@chromium.org

[commit-bot: I haz the power, 2016-12-19 18:42:19]

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-12-19 18:49:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482172912805520,
"parent_rev": "1c60d232d494bc6e2bbb3633022f13659a99c8d3", "commit_rev":
"152ded848dfb8a30e658129560f6391dcc5b87a9"}

[commit-bot: I haz the power, 2016-12-19 18:49:38]

Description was changed from

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053,674293
==========

to

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053,674293

Review-Url: https://codereview.chromium.org/2580523002
==========

[commit-bot: I haz the power, 2016-12-19 18:49:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-19 18:52:15]

Description was changed from

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053,674293

Review-Url: https://codereview.chromium.org/2580523002
==========

to

==========
Reland "Android: Switch to thread pool executor"

Original CL: http://crrev.com/2562643004

Fixes BackgroundSyncLauncherTest#testNewLauncherDisablesNextOnline.

TBR=mariakhomenko@chromium.org
BUG=601053,674293

Committed: https://crrev.com/80528b6a6acce7e1cc5829aaae876fb608aabb18
Cr-Commit-Position: refs/heads/master@{#439511}
==========

[commit-bot: I haz the power, 2016-12-19 18:52:16]

Patchset 2 (id:??) landed as
https://crrev.com/80528b6a6acce7e1cc5829aaae876fb608aabb18
Cr-Commit-Position: refs/heads/master@{#439511}