Remove calls to IsRunningSequenceOnCurrentThread() from memory_dump_manager_unittest.cc

Remove calls to IsRunningSequenceOnCurrentThread() from memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400
Committed: https://crrev.com/ee7a59389604bc254617c71ef53aaf0eaf9c7237
Cr-Commit-Position: refs/heads/master@{#416994}

[fdoray, 2016-09-07 13:43:45]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 13:43:52]

Dry run: 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-09-07 14:17:04]

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

[commit-bot: I haz the power, 2016-09-07 14:17:05]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-09-07 14:34:08]

Description was changed from

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

Developers should use SequenceChecker to verify that tasks run
sequentially. Unlike
SequencedWorkerPool::IsRunningSequenceOnCurrentThread(),
SequenceChecker works everywhere in Chrome (MessageLoop,
SequencedWorkerPool, base/task_scheduler...).

BUG=622400
==========

to

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400
==========

[fdoray, 2016-09-07 14:36:15]

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

[fdoray, 2016-09-07 14:36:15]

PTAL

https://codereview.chromium.org/2320643002/diff/1/base/trace_event/memory_dum...
File base/trace_event/memory_dump_manager_unittest.cc (right):

https://codereview.chromium.org/2320643002/diff/1/base/trace_event/memory_dum...
base/trace_event/memory_dump_manager_unittest.cc:190: bool
RunsTasksOnCurrentThread() const override {
RunsTasksOnCurrentThread() is supposed to "Returns true if the current thread is
a thread on which a task may be run". It makes sense to implement it in terms on
SequencedWorkerPool::RunsTasksOnCurrentThread() rather than
SequencedWorkerPool::IsRunningSequenceOnCurrentThread().
https://cs.chromium.org/chromium/src/base/task_runner.h?sq=package:chromium&d...

[Primiano Tucci (use gerrit), 2016-09-07 17:43:38]

lgtm

[fdoray, 2016-09-07 17:58:29]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-09-07 17:58:55]

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-09-07 18:03:27]

Description was changed from

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400
==========

to

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400
==========

[commit-bot: I haz the power, 2016-09-07 18:03:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-07 18:04:38]

Description was changed from

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400
==========

to

==========
Remove calls to IsRunningSequenceOnCurrentThread() from
memory_dump_manager_unittest.cc

SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.

BUG=622400

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

[commit-bot: I haz the power, 2016-09-07 18:04:39]

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