Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in TestBrowserThreadBundle.

Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in TestBrowserThreadBundle. (patchset #3 id:40001 of https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in TestBrowserThreadBundle.
> 
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
> 
> BUG=667892
> 
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed: https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a07112d44d6cc018

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892, 727994
Review-Url: https://codereview.chromium.org/2909393003
Cr-Commit-Position: refs/heads/master@{#475816}
Committed: https://chromium.googlesource.com/chromium/src/+/2a1abaac62e57773e4abe594f1de2b19dbb83e58

[falken, 2017-05-31 04:46:16]

The CQ bit was checked by falken@chromium.org

[falken, 2017-05-31 04:46:17]

Created Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle.

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

CQ is trying da patch.

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

[falken, 2017-05-31 04:46:45]

Description was changed from

==========
Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle. (patchset #3 id:40001 of
https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to
crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle.
> 
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
> 
> BUG=667892
> 
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a071...

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892
==========

to

==========
Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle. (patchset #3 id:40001 of
https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to
crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle.
> 
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
> 
> BUG=667892
> 
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a071...

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892,727994
==========

[commit-bot: I haz the power, 2017-05-31 05:48:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496205976827620, "parent_rev":
"6d317c3c73d4eadc4651c5a188c40b0296229e08", "commit_rev":
"2a1abaac62e57773e4abe594f1de2b19dbb83e58"}

[commit-bot: I haz the power, 2017-05-31 05:48:43]

Description was changed from

==========
Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle. (patchset #3 id:40001 of
https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to
crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle.
> 
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
> 
> BUG=667892
> 
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a071...

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892,727994
==========

to

==========
Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle. (patchset #3 id:40001 of
https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to
crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in
TestBrowserThreadBundle.
> 
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
> 
> BUG=667892
> 
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a071...

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892,727994

Review-Url: https://codereview.chromium.org/2909393003
Cr-Commit-Position: refs/heads/master@{#475816}
Committed:
https://chromium.googlesource.com/chromium/src/+/2a1abaac62e57773e4abe594f1de...
==========

[commit-bot: I haz the power, 2017-05-31 05:48:44]

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