arc: Clear the ARC task list when ARC get disabled.

arc: Clear the ARC task list when ARC get disabled.

BUG=b:29154797
TEST=trybots
TEST=Manually tested

Committed: https://crrev.com/cc39f41cd320bf1ce4f1eec30d278b44afc01847
Cr-Commit-Position: refs/heads/master@{#398780}

[Shuhei Takahashi, 2016-06-08 09:03:42]

nya@chromium.org changed reviewers:
+ afakhry@chromium.org, cylee@chromium.org

[Shuhei Takahashi, 2016-06-08 09:03:42]

afakhry, cylee: PTAL

[cylee, 2016-06-08 09:58:00]

cylee@google.com changed reviewers:
+ cylee@google.com

[cylee, 2016-06-08 09:58:01]

lgtm

https://codereview.chromium.org/2043293002/diff/1/chrome/browser/task_managem...
File chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
(right):

https://codereview.chromium.org/2043293002/diff/1/chrome/browser/task_managem...
chrome/browser/task_management/providers/arc/arc_process_task_provider.cc:101:
callback.Run(std::vector<ArcProcess>());
Could you add a comment to make it clear the case happens either when ARC is not
ready or ARC has been shutdown/disabled?)

[Shuhei Takahashi, 2016-06-08 10:50:21]

https://codereview.chromium.org/2043293002/diff/1/chrome/browser/task_managem...
File chrome/browser/task_management/providers/arc/arc_process_task_provider.cc
(right):

https://codereview.chromium.org/2043293002/diff/1/chrome/browser/task_managem...
chrome/browser/task_management/providers/arc/arc_process_task_provider.cc:101:
callback.Run(std::vector<ArcProcess>());
On 2016/06/08 09:58:01, cylee wrote:
> Could you add a comment to make it clear the case happens either when ARC is
not
> ready or ARC has been shutdown/disabled?) 

Done.

[afakhry, 2016-06-08 13:15:53]

lgtm

[Shuhei Takahashi, 2016-06-08 13:19:41]

Thanks for reviews!

Please allow me to submit this change a bit later (1-2 days) since this change
is not very high priority.

[Shuhei Takahashi, 2016-06-09 03:44:17]

The CQ bit was checked by nya@chromium.org

[Shuhei Takahashi, 2016-06-09 03:44:18]

The patchset sent to the CQ was uploaded after l-g-t-m from cylee@google.com
Link to the patchset: https://codereview.chromium.org/2043293002/#ps20001
(title: "Update comment.")

[commit-bot: I haz the power, 2016-06-09 03:44:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2043293002/20001

[commit-bot: I haz the power, 2016-06-09 04:18:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-09 04:20:09]

Description was changed from

==========
arc: Clear the ARC task list when ARC get disabled.

BUG=b:29154797
TEST=trybots
TEST=Manually tested
==========

to

==========
arc: Clear the ARC task list when ARC get disabled.

BUG=b:29154797
TEST=trybots
TEST=Manually tested

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

[commit-bot: I haz the power, 2016-06-09 04:20:10]

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