[Merge to M54] arc: Add a global 5 minute timeout for ARC opt-in

[Merge to M54] arc: Add a global 5 minute timeout for ARC opt-in

This should be a safety net for *all* boot failures we have (or might
have). This change replaces the flag |waiting_for_reply_| with a
base::OneShotTimeout that will log the situation and report a timeout
error to the user.

BUG=645233
TEST=Intentionally make ARC not send the reply, see the UI, ARC is still
     running.

Review-Url: https://codereview.chromium.org/2323043003
Cr-Commit-Position: refs/heads/master@{#417770}
(cherry picked from commit 3d3317a6e0dde5de7effb4406c4b60bb7aa91bb5)

R=holte@chromium.org, yusukes@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/bb3ec13c84a856dfc36d80b51fc6455626b1ad4e

[Luis Héctor Chávez, 2016-09-12 20:38:28]

lhchavez@chromium.org changed reviewers:
+ holte@chromium.org, yusukes@chromium.org

[Luis Héctor Chávez, 2016-09-12 20:38:29]

PTAL

yusukes: c/b/c/arc
holte: tools/metrics/histograms/histograms.xml

N.B. this was not a clean merge: the pre-submit hook requested me to remove an
empty whitespace line in histograms.xml.

[Yusuke Sato, 2016-09-12 20:44:05]

arc/ lgtm

[Luis Héctor Chávez, 2016-09-13 23:50:12]

also ping?

[Steven Holte, 2016-09-13 23:53:40]

On 2016/09/13 23:50:12, Luis Héctor Chávez wrote:
> also ping?

histograms lgtm

[Luis Héctor Chávez, 2016-09-14 00:02:43]

Description was changed from

==========
[Merge to M54] arc: Add a global 5 minute timeout for ARC opt-in

This should be a safety net for *all* boot failures we have (or might
have). This change replaces the flag |waiting_for_reply_| with a
base::OneShotTimeout that will log the situation and report a timeout
error to the user.

BUG=645233
TEST=Intentionally make ARC not send the reply, see the UI, ARC is still
     running.

Review-Url: https://codereview.chromium.org/2323043003
Cr-Commit-Position: refs/heads/master@{#417770}
(cherry picked from commit 3d3317a6e0dde5de7effb4406c4b60bb7aa91bb5)
==========

to

==========
[Merge to M54] arc: Add a global 5 minute timeout for ARC opt-in

This should be a safety net for *all* boot failures we have (or might
have). This change replaces the flag |waiting_for_reply_| with a
base::OneShotTimeout that will log the situation and report a timeout
error to the user.

BUG=645233
TEST=Intentionally make ARC not send the reply, see the UI, ARC is still
     running.

Review-Url: https://codereview.chromium.org/2323043003
Cr-Commit-Position: refs/heads/master@{#417770}
(cherry picked from commit 3d3317a6e0dde5de7effb4406c4b60bb7aa91bb5)

R=holte@chromium.org, yusukes@chromium.org

Committed:
https://chromium.googlesource.com/chromium/src/+/bb3ec13c84a856dfc36d80b51fc6...
==========

[Luis Héctor Chávez, 2016-09-14 00:02:44]

Committed patchset #2 (id:20001) manually as
bb3ec13c84a856dfc36d80b51fc6455626b1ad4e (presubmit successful).