Use SequencedWorkerPoolOwner in MemoryDumpManagerTest.

Use SequencedWorkerPoolOwner in MemoryDumpManagerTest.

MemoryDumpManagerTest didn't join its thread. This made a write in
TearDown() racy with a read from a thread created by the test.

With this CL, threads created by MemoryDumpManagerTest are joined
before TearDown() runs.

BUG=658766
Committed: https://crrev.com/42bdd9c57551be778bfda66453cb6054d78dfdac
Cr-Commit-Position: refs/heads/master@{#427251}

[fdoray, 2016-10-24 20:06:29]

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

[fdoray, 2016-10-24 20:06:29]

PTAL

[danakj, 2016-10-24 20:29:12]

LGTM

[fdoray, 2016-10-24 20:30:44]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-24 20:31:26]

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-10-24 20:41:09]

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

[commit-bot: I haz the power, 2016-10-24 20:41:10]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[fdoray, 2016-10-24 20:41:36]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-24 20:42:14]

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-10-24 20:51:22]

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

[commit-bot: I haz the power, 2016-10-24 20:51:23]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Reilly Grant (use Gerrit), 2016-10-25 01:06:07]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-10-25 01:06:46]

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-10-25 02:46:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 02:49:58]

Description was changed from

==========
Use SequencedWorkerPoolOwner in MemoryDumpManagerTest.

MemoryDumpManagerTest didn't join its thread. This made a write in
TearDown() racy with a read from a thread created by the test.

With this CL, threads created by MemoryDumpManagerTest are joined
before TearDown() runs.

BUG=658766
==========

to

==========
Use SequencedWorkerPoolOwner in MemoryDumpManagerTest.

MemoryDumpManagerTest didn't join its thread. This made a write in
TearDown() racy with a read from a thread created by the test.

With this CL, threads created by MemoryDumpManagerTest are joined
before TearDown() runs.

BUG=658766

Committed: https://crrev.com/42bdd9c57551be778bfda66453cb6054d78dfdac
Cr-Commit-Position: refs/heads/master@{#427251}
==========

[commit-bot: I haz the power, 2016-10-25 02:49:59]

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