arc: Stop waiting for crash reporter on UI thread

arc: Stop waiting for crash reporter on UI thread

We shouldn't block the UI thread waiting for crash_reporter which
performs disk IO.

BUG=None

Committed: https://crrev.com/29688ee2a715d78d1f90e30356e32f3542142c4e
Cr-Commit-Position: refs/heads/master@{#421141}

[hashimoto, 2016-09-26 10:00:50]

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

[commit-bot: I haz the power, 2016-09-26 10:01:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hashimoto, 2016-09-26 10:01:36]

hashimoto@chromium.org changed reviewers:
+ hidehiko@chromium.org

[commit-bot: I haz the power, 2016-09-26 10:23:15]

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

[commit-bot: I haz the power, 2016-09-26 10:23:16]

Dry run: This issue passed the CQ dry run.

[hidehiko, 2016-09-26 14:17:22]

lgtm.

https://codereview.chromium.org/2367333002/diff/1/components/arc/crash_collec...
File components/arc/crash_collector/arc_crash_collector_bridge.cc (right):

https://codereview.chromium.org/2367333002/diff/1/components/arc/crash_collec...
components/arc/crash_collector/arc_crash_collector_bridge.cc:70:
blocking_task_runner_->PostTask(FROM_HERE,
Optional: How about moving everything, incl. LaunchProcess, to blocking task
runner? It looks ok to do, according to the client side code.

[hashimoto, 2016-09-27 06:45:04]

Patchset #2 (id:20001) has been deleted

[hashimoto, 2016-09-27 06:47:11]

Patchset #2 (id:40001) has been deleted

[hashimoto, 2016-09-27 06:48:16]

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

[commit-bot: I haz the power, 2016-09-27 06:48:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hashimoto, 2016-09-27 06:48:26]

https://codereview.chromium.org/2367333002/diff/1/components/arc/crash_collec...
File components/arc/crash_collector/arc_crash_collector_bridge.cc (right):

https://codereview.chromium.org/2367333002/diff/1/components/arc/crash_collec...
components/arc/crash_collector/arc_crash_collector_bridge.cc:70:
blocking_task_runner_->PostTask(FROM_HERE,
On 2016/09/26 14:17:22, hidehiko wrote:
> Optional: How about moving everything, incl. LaunchProcess, to blocking task
> runner? It looks ok to do, according to the client side code.

Done.

[hidehiko, 2016-09-27 06:49:42]

lgtm

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

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

[commit-bot: I haz the power, 2016-09-27 07:13:08]

Dry run: This issue passed the CQ dry run.

[hashimoto, 2016-09-27 07:46:11]

The CQ bit was checked by hashimoto@chromium.org

[commit-bot: I haz the power, 2016-09-27 07:46:23]

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-27 07:51:26]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-09-27 07:53:50]

Description was changed from

==========
arc: Stop waiting for crash reporter on UI thread

We shouldn't block the UI thread waiting for crash_reporter which
performs disk IO.

BUG=None
==========

to

==========
arc: Stop waiting for crash reporter on UI thread

We shouldn't block the UI thread waiting for crash_reporter which
performs disk IO.

BUG=None

Committed: https://crrev.com/29688ee2a715d78d1f90e30356e32f3542142c4e
Cr-Commit-Position: refs/heads/master@{#421141}
==========

[commit-bot: I haz the power, 2016-09-27 07:53:51]

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