Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139
Committed: https://crrev.com/6e23c37e5f9bf776010908c2d19a0d842d2c70f1
Cr-Commit-Position: refs/heads/master@{#407577}

[gab, 2016-07-21 19:24:45]

Description was changed from

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

BUG=629139
==========

to

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139
==========

[gab, 2016-07-21 19:25:57]

gab@chromium.org changed reviewers:
+ phajdan.jr@chromium.org, primiano@chromium.org

[gab, 2016-07-21 19:25:58]

Primiano PTAL for MemoryDumpManagerTest
Phajdan PTAL for base/test updates (see
https://codereview.chromium.org/2145463002/ for reasoning)

[Primiano Tucci (use gerrit), 2016-07-22 16:55:46]

LGTM

[gab, 2016-07-22 16:59:48]

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

[gab, 2016-07-22 16:59:48]

Thanks, @dana for base/ since Pawel appears to have just left on vacation (this
is related to https://codereview.chromium.org/2145463002/ -- the only culprit
for Stop() AFAICT, will attempt to re-enable check in a follow-up CL).

[gab, 2016-07-25 14:54:23]

On 2016/07/22 16:59:48, gab wrote:
> Thanks, @dana for base/ since Pawel appears to have just left on vacation
(this
> is related to https://codereview.chromium.org/2145463002/ -- the only culprit
> for Stop() AFAICT, will attempt to re-enable check in a follow-up CL).

ping danakj, thanks :-)

[danakj, 2016-07-25 18:50:24]

LGTM

[gab, 2016-07-25 20:11:29]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-07-25 20:12:03]

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-07-25 21:21:55]

Description was changed from

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139
==========

to

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139
==========

[commit-bot: I haz the power, 2016-07-25 21:21:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-25 21:24:31]

Description was changed from

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139
==========

to

==========
Fix MemoryDumpManagerTest's thread unsafe usage of the base::TestIOThread API

Also move TestIOThread::PostTaskAndWait to those tests as they were the
only user.

BUG=629139

Committed: https://crrev.com/6e23c37e5f9bf776010908c2d19a0d842d2c70f1
Cr-Commit-Position: refs/heads/master@{#407577}
==========

[commit-bot: I haz the power, 2016-07-25 21:24:32]

Patchset 2 (id:??) landed as
https://crrev.com/6e23c37e5f9bf776010908c2d19a0d842d2c70f1
Cr-Commit-Position: refs/heads/master@{#407577}