cros: remove code duplication between ArcProcess and ArcProcessTask

ArcProcessTask contains a subset of the members in ArcProcess. It
also contains similar logic with regards to determining which processes
are killable from the task manager. This change has ArcProcessTask
contain an ArcProcess instead of duplicating some of it's members, and
delegates the logic of whether a process is killable to the ArcProcess
instance. This causes a small change in which processes can be killed in
the in the task manager: processes protected by the function
ArcProcess::IsArcProtected can no longer be killed. This change is
considered an improvement by the author of the IsArcProtected function.

BUG=none
TEST=Ensure task manager can still kill right processes.

Change-Id: Id483b22e3a25f6fe8bbeeee8cddddcf0b5dbe298
Reviewed-on: https://chromium-review.googlesource.com/1068615
Commit-Queue: Christopher Morin <[email protected]>
Reviewed-by: Yusuke Sato <[email protected]>
Reviewed-by: Ahmed Fakhry <[email protected]>
Reviewed-by: Hidehiko Abe <[email protected]>
Cr-Commit-Position: refs/heads/master@{#560771}
5 files changed