diff options
author | tapted <tapted@chromium.org> | 2015-12-02 20:43:05 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-12-03 04:43:42 +0000 |
commit | 012326b1c4342b0bf2dde88d326efbed06ebcf9b (patch) | |
tree | 4c234dfbaa0add8025edce0f5b310c87788b0e84 /components/component_updater | |
parent | 6adbf2e61e7a6a3dae0c72d5dcf946a6535c13df (diff) | |
download | chromium_src-012326b1c4342b0bf2dde88d326efbed06ebcf9b.zip chromium_src-012326b1c4342b0bf2dde88d326efbed06ebcf9b.tar.gz chromium_src-012326b1c4342b0bf2dde88d326efbed06ebcf9b.tar.bz2 |
Reland of Tests: Simplify SequencedWorkerPoolOwner, call Shutdown on destructor. (patchset #1 id:1 of https://codereview.chromium.org/1496493004/ )
Reason for revert:
Relanding, no recovery in
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/32878
Original issue's description:
> 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}
TBR=phajdan.jr@chromium.org,jam@chromium.org,brettw@chromium.org,tommycli@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450228
Review URL: https://codereview.chromium.org/1500503002
Cr-Commit-Position: refs/heads/master@{#362907}
Diffstat (limited to 'components/component_updater')
-rw-r--r-- | components/component_updater/component_updater_service_unittest.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/components/component_updater/component_updater_service_unittest.cc b/components/component_updater/component_updater_service_unittest.cc index 9d78eb8..a5a839d 100644 --- a/components/component_updater/component_updater_service_unittest.cc +++ b/components/component_updater/component_updater_service_unittest.cc @@ -174,7 +174,6 @@ ComponentUpdaterTest::ComponentUpdaterTest() ComponentUpdaterTest::~ComponentUpdaterTest() { EXPECT_CALL(update_client(), RemoveObserver(_)).Times(1); - worker_pool_->pool()->Shutdown(); component_updater_.reset(); } |