Leak detector posts notifications to task runner

Leak detector posts notifications to task runner

Instead of posting to the browser process' UI thread, pass in a generic
TaskRunner and post the notifications to it. On the browser process, pass
in the TaskRunner for the browser process' UI thread. On other processes,
a different TaskRunner can be passed in.

BUG=chromium:615223
TEST=unit tests pass
R=asvitkine@chromium.org, primiano@chromium.org

Committed: https://crrev.com/685b914a3f55fe47863648086e3f4bedd4188814
Cr-Commit-Position: refs/heads/master@{#396560}

[Alexei Svitkine (slow), 2016-05-27 13:58:22]

lgtm

https://codereview.chromium.org/2012393002/diff/20001/components/metrics/leak...
File components/metrics/leak_detector/leak_detector_unittest.cc (right):

https://codereview.chromium.org/2012393002/diff/20001/components/metrics/leak...
components/metrics/leak_detector/leak_detector_unittest.cc:79:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
Nit: Since this is a test, use EXPECT_TRUE()

[Primiano Tucci (use gerrit), 2016-05-27 16:05:41]

LGTM % asvitkine@ comment

[Simon Que, 2016-05-27 18:26:55]

https://codereview.chromium.org/2012393002/diff/20001/components/metrics/leak...
File components/metrics/leak_detector/leak_detector_unittest.cc (right):

https://codereview.chromium.org/2012393002/diff/20001/components/metrics/leak...
components/metrics/leak_detector/leak_detector_unittest.cc:79:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
On 2016/05/27 13:58:22, Alexei Svitkine (slow) wrote:
> Nit: Since this is a test, use EXPECT_TRUE()

Done.

[Simon Que, 2016-05-27 18:28:41]

The CQ bit was checked by sque@chromium.org

[Simon Que, 2016-05-27 18:28:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org, asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/2012393002/#ps40001
(title: "CHECK -> EXPECT_TRUE")

[commit-bot: I haz the power, 2016-05-27 18:29:05]

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

[commit-bot: I haz the power, 2016-05-27 21:13:44]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-27 21:15:18]

Description was changed from

==========
Leak detector posts notifications to task runner

Instead of posting to the browser process' UI thread, pass in a generic
TaskRunner and post the notifications to it. On the browser process, pass
in the TaskRunner for the browser process' UI thread. On other processes,
a different TaskRunner can be passed in.

BUG=chromium:615223
TEST=unit tests pass
R=asvitkine@chromium.org, primiano@chromium.org
==========

to

==========
Leak detector posts notifications to task runner

Instead of posting to the browser process' UI thread, pass in a generic
TaskRunner and post the notifications to it. On the browser process, pass
in the TaskRunner for the browser process' UI thread. On other processes,
a different TaskRunner can be passed in.

BUG=chromium:615223
TEST=unit tests pass
R=asvitkine@chromium.org, primiano@chromium.org

Committed: https://crrev.com/685b914a3f55fe47863648086e3f4bedd4188814
Cr-Commit-Position: refs/heads/master@{#396560}
==========

[commit-bot: I haz the power, 2016-05-27 21:15:19]

Patchset 3 (id:??) landed as
https://crrev.com/685b914a3f55fe47863648086e3f4bedd4188814
Cr-Commit-Position: refs/heads/master@{#396560}