Don't measure CPU time in timing out GPU watchdog.

Don't measure CPU time in timing out GPU watchdog.

It's possible for the main thread to hang on a GPU event or waiting for the OS, so if CPU usage is required for timing out the watchdog then it's possible it'll never time out.

Just remove the code that checks for CPU time. Time lost due to suspend/resume events should be handled by the PowerMonitor, and this code seems to work on other platforms without issues.

BUG=588342
Committed: https://crrev.com/ae57b78041762268e1898c165e33b904477433bb
Cr-Commit-Position: refs/heads/master@{#376867}

[jbauman, 2016-02-22 21:24:19]

jbauman@chromium.org changed reviewers:
+ kbr@chromium.org

[Ken Russell (switch to Gerrit), 2016-02-22 22:09:39]

LGTM if this has been tested, in particular on a laptop with multiple
suspend/resume cycles.

[jbauman, 2016-02-22 22:51:10]

On 2016/02/22 22:09:39, Ken Russell wrote:
> LGTM if this has been tested, in particular on a laptop with multiple
> suspend/resume cycles.

Yeah, it works fine on my laptop at least.

[jbauman, 2016-02-22 22:51:24]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2016-02-22 22:53:00]

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

[commit-bot: I haz the power, 2016-02-23 00:28:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-23 00:29:44]

Description was changed from

==========
Don't measure CPU time in timing out GPU watchdog.

It's possible for the main thread to hang on a GPU event or waiting for the OS,
so if CPU usage is required for timing out the watchdog then it's possible it'll
never time out.

Just remove the code that checks for CPU time. Time lost due to suspend/resume
events should be handled by the PowerMonitor, and this code seems to work on
other platforms without issues.

BUG=588342
==========

to

==========
Don't measure CPU time in timing out GPU watchdog.

It's possible for the main thread to hang on a GPU event or waiting for the OS,
so if CPU usage is required for timing out the watchdog then it's possible it'll
never time out.

Just remove the code that checks for CPU time. Time lost due to suspend/resume
events should be handled by the PowerMonitor, and this code seems to work on
other platforms without issues.

BUG=588342

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

[commit-bot: I haz the power, 2016-02-23 00:29:45]

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

[jbauman, 2016-02-23 20:30:08]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1725983002/ by jbauman@chromium.org.

The reason for reverting is: Too many hangs are caused by slow startup: 
BUG=589175.