Remove usage of SequencedWorkerPool::GetNamedSequenceToken from ChromeDownloadManagerDelegate.

Remove usage of SequencedWorkerPool::GetNamedSequenceToken from ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=hirono@chromium.org, kinuko@chromium.org,

Review-Url: https://codereview.chromium.org/2887933002
Cr-Commit-Position: refs/heads/master@{#473894}
Committed: https://chromium.googlesource.com/chromium/src/+/53943c16ba00d6b9771a5a3e14852b516207139a

[fdoray, 2017-05-16 19:27:00]

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

[commit-bot: I haz the power, 2017-05-16 19:28:12]

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-05-17 02:44:33]

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

[commit-bot: I haz the power, 2017-05-17 02:44:34]

Dry run: This issue passed the CQ dry run.

[fdoray, 2017-05-17 14:43:30]

fdoray@chromium.org changed reviewers:
+ dtrainor@chromium.org

[fdoray, 2017-05-17 14:43:31]

Please take a look.

In particular, please make sure that appropriate traits are used to post tasks
in chrome/browser/download/chrome_download_manager_delegate.cc

https://cs.chromium.org/chromium/src/base/task_scheduler/task_traits.h

[fdoray, 2017-05-19 20:07:45]

On 2017/05/17 14:43:31, fdoray wrote:
> Please take a look.
> 
> In particular, please make sure that appropriate traits are used to post tasks
> in chrome/browser/download/chrome_download_manager_delegate.cc
> 
> https://cs.chromium.org/chromium/src/base/task_scheduler/task_traits.h

ping

[David Trainor- moved to gerrit, 2017-05-19 20:59:20]

lgtm

[fdoray, 2017-05-21 15:11:02]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-05-21 15:11:13]

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-05-21 15:17:43]

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

[commit-bot: I haz the power, 2017-05-21 15:17:44]

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...)

[fdoray, 2017-05-23 13:56:46]

Description was changed from

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
==========

to

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=clamy@chromium.org,satorux@chromium.org
==========

[fdoray, 2017-05-23 13:56:46]

fdoray@chromium.org changed reviewers:
+ clamy@chromium.org, satorux@chromium.org

[fdoray, 2017-05-23 13:58:52]

Description was changed from

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=clamy@chromium.org,satorux@chromium.org
==========

to

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=hirono@chromium.org, kinuko@chromium.org,
==========

[fdoray, 2017-05-23 13:58:52]

fdoray@chromium.org changed reviewers:
+ hirono@chromium.org, kinuko@chromium.org
- clamy@chromium.org, satorux@chromium.org

[fdoray, 2017-05-23 13:59:21]

TBR hirono@chromium.org for changes to chrome/browser/chromeos/
TBR kinuko@chromium.org for changes to content/

[fdoray, 2017-05-23 13:59:25]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-05-23 13:59:41]

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-05-23 15:00:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495547965464080, "parent_rev":
"3abe1699211d5ff8cc44383a01e395ee13d7820c", "commit_rev":
"53943c16ba00d6b9771a5a3e14852b516207139a"}

[commit-bot: I haz the power, 2017-05-23 15:00:31]

Description was changed from

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=hirono@chromium.org, kinuko@chromium.org,
==========

to

==========
Remove usage of SequencedWorkerPool::GetNamedSequenceToken from
ChromeDownloadManagerDelegate.

SequencedWorkerPool is being deprecated in favor of TaskScheduler.

BUG=667892
TBR=hirono@chromium.org, kinuko@chromium.org,

Review-Url: https://codereview.chromium.org/2887933002
Cr-Commit-Position: refs/heads/master@{#473894}
Committed:
https://chromium.googlesource.com/chromium/src/+/53943c16ba00d6b9771a5a3e1485...
==========

[commit-bot: I haz the power, 2017-05-23 15:00:32]

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

[kinuko, 2017-05-24 00:16:49]

lgtm

[hirono, 2017-05-24 01:01:50]

lgtm