chromeos: Notify power manager about user activity.

chromeos: Notify power manager about user activity.

This makes Chrome call powerd's HandleUserActivity D-Bus
method in response to input events (but no more often than
once every five seconds).

BUG=chromium-os:31351
TEST=manual: tailed powerd's log and saw that it's receiving HandleUserActivity calls every five seconds as i type or move the mouse


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140849

[Daniel Erat, 2012-06-05 19:39:43]

Steven: chromeos/ and chrome/browser/chromeos/
Scott: ash/

[stevenjb, 2012-06-05 20:04:36]

lgtm

[sky, 2012-06-05 20:24:21]

LGTM

https://chromiumcodereview.appspot.com/10544011/diff/1/ash/wm/user_activity_d...
File ash/wm/user_activity_detector.h (right):

https://chromiumcodereview.appspot.com/10544011/diff/1/ash/wm/user_activity_d...
ash/wm/user_activity_detector.h:18: class ASH_EXPORT UserActivityObserver {
nit: move into its own header.

https://chromiumcodereview.appspot.com/10544011/diff/1/ash/wm/user_activity_d...
ash/wm/user_activity_detector.h:43: aura::Window* target, aura::KeyEvent* event)
OVERRIDE;
nit: its my understanding once you wrap, then each param should be on its own
line.

[Daniel Erat, 2012-06-05 20:56:43]

http://codereview.chromium.org/10544011/diff/1/ash/wm/user_activity_detector.h
File ash/wm/user_activity_detector.h (right):

http://codereview.chromium.org/10544011/diff/1/ash/wm/user_activity_detector....
ash/wm/user_activity_detector.h:18: class ASH_EXPORT UserActivityObserver {
On 2012/06/05 20:24:21, sky wrote:
> nit: move into its own header.

Done.

http://codereview.chromium.org/10544011/diff/1/ash/wm/user_activity_detector....
ash/wm/user_activity_detector.h:43: aura::Window* target, aura::KeyEvent* event)
OVERRIDE;
On 2012/06/05 20:24:21, sky wrote:
> nit: its my understanding once you wrap, then each param should be on its own
> line.

Wasn't aware of this.  Done.

[commit-bot: I haz the power, 2012-06-06 19:26:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/10544011/16

[commit-bot: I haz the power, 2012-06-06 21:51:19]

Change committed as 140849