Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.

Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor. (patchset #4 id:60001 of https://codereview.chromium.org/1417353006/ )

Reason for revert:
Suspected for persistent failures on ios_net_unittests

- RoundTripTestCookieStore/CookieStoreTest/0.TestNonDottedAndTLD
- CookieStoreIOS/CookieStoreTest/0.TestNonDottedAndTLD

Failing since https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/32876

Errors like

../../net/cookies/cookie_store_unittest.h:536: Failure
Value of: this->SetCookie(cs.get(), url, "a=1; domain=com")
  Actual: false
Expected: true

(It's a tricky one! Sorry if this is not it)

Original issue's description:
> Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
>
> Also ports remaining tests using raw SWPs to use SWPOwner.
>
> BUG=450228
>
> Committed: https://crrev.com/fc939726c283e34112d9bc845a39460410fe9cd9
> Cr-Commit-Position: refs/heads/master@{#362805}

TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228
Committed: https://crrev.com/d7365d9aaa55e88d9ecca49eb2158b62d1190f09
Cr-Commit-Position: refs/heads/master@{#362890}

[tapted, 2015-12-03 03:37:13]

Created Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on
destructor.

[commit-bot: I haz the power, 2015-12-03 03:37:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1496493004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1496493004/1

[commit-bot: I haz the power, 2015-12-03 03:39:10]

Description was changed from

==========
Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
(patchset #4 id:60001 of https://codereview.chromium.org/1417353006/ )

Reason for revert:
Suspected for persistent failures on ios_net_unittests

- RoundTripTestCookieStore/CookieStoreTest/0.TestNonDottedAndTLD
- CookieStoreIOS/CookieStoreTest/0.TestNonDottedAndTLD

Failing since
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/bu...

Errors like 

../../net/cookies/cookie_store_unittest.h:536: Failure
Value of: this->SetCookie(cs.get(), url, "a=1; domain=com")
  Actual: false
Expected: true

(It's a tricky one! Sorry if this is not it)

Original issue's description:
> Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
> 
> Also ports remaining tests using raw SWPs to use SWPOwner.
> 
> BUG=450228
> 
> Committed: https://crrev.com/fc939726c283e34112d9bc845a39460410fe9cd9
> Cr-Commit-Position: refs/heads/master@{#362805}

TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chr...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228
==========

to

==========
Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
(patchset #4 id:60001 of https://codereview.chromium.org/1417353006/ )

Reason for revert:
Suspected for persistent failures on ios_net_unittests

- RoundTripTestCookieStore/CookieStoreTest/0.TestNonDottedAndTLD
- CookieStoreIOS/CookieStoreTest/0.TestNonDottedAndTLD

Failing since
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/bu...

Errors like 

../../net/cookies/cookie_store_unittest.h:536: Failure
Value of: this->SetCookie(cs.get(), url, "a=1; domain=com")
  Actual: false
Expected: true

(It's a tricky one! Sorry if this is not it)

Original issue's description:
> Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
> 
> Also ports remaining tests using raw SWPs to use SWPOwner.
> 
> BUG=450228
> 
> Committed: https://crrev.com/fc939726c283e34112d9bc845a39460410fe9cd9
> Cr-Commit-Position: refs/heads/master@{#362805}

TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chr...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228
==========

[commit-bot: I haz the power, 2015-12-03 03:39:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-03 03:39:49]

Description was changed from

==========
Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
(patchset #4 id:60001 of https://codereview.chromium.org/1417353006/ )

Reason for revert:
Suspected for persistent failures on ios_net_unittests

- RoundTripTestCookieStore/CookieStoreTest/0.TestNonDottedAndTLD
- CookieStoreIOS/CookieStoreTest/0.TestNonDottedAndTLD

Failing since
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/bu...

Errors like 

../../net/cookies/cookie_store_unittest.h:536: Failure
Value of: this->SetCookie(cs.get(), url, "a=1; domain=com")
  Actual: false
Expected: true

(It's a tricky one! Sorry if this is not it)

Original issue's description:
> Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
> 
> Also ports remaining tests using raw SWPs to use SWPOwner.
> 
> BUG=450228
> 
> Committed: https://crrev.com/fc939726c283e34112d9bc845a39460410fe9cd9
> Cr-Commit-Position: refs/heads/master@{#362805}

TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chr...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228
==========

to

==========
Revert of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
(patchset #4 id:60001 of https://codereview.chromium.org/1417353006/ )

Reason for revert:
Suspected for persistent failures on ios_net_unittests

- RoundTripTestCookieStore/CookieStoreTest/0.TestNonDottedAndTLD
- CookieStoreIOS/CookieStoreTest/0.TestNonDottedAndTLD

Failing since
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/bu...

Errors like

../../net/cookies/cookie_store_unittest.h:536: Failure
Value of: this->SetCookie(cs.get(), url, "a=1; domain=com")
  Actual: false
Expected: true

(It's a tricky one! Sorry if this is not it)

Original issue's description:
> Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor.
>
> Also ports remaining tests using raw SWPs to use SWPOwner.
>
> BUG=450228
>
> Committed: https://crrev.com/fc939726c283e34112d9bc845a39460410fe9cd9
> Cr-Commit-Position: refs/heads/master@{#362805}

TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chr...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228

Committed: https://crrev.com/d7365d9aaa55e88d9ecca49eb2158b62d1190f09
Cr-Commit-Position: refs/heads/master@{#362890}
==========

[commit-bot: I haz the power, 2015-12-03 03:39:50]

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

[tapted, 2015-12-03 04:40:38]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1500503002/ by tapted@chromium.org.

The reason for reverting is: Relanding, no recovery in

https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/bu....