Use Signal Callback in Ambient Light Sensor

Use Signal Callback in Ambient Light Sensor

Use the signal callback macro to clean up the ambient
light sensor polling g_timeout callback.

Change-Id: Ie2c4a5aaa0b8f57ec8be55939cfbcc6041005a62
Signed-off-by: Benson Leung <bleung@chromium.org>;
Signed-off-by: Simon Que <sque@chromium.org>;

BUG=chromium-os:7934
TEST=Built power manager. Tested that light sensor works as before.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=d44134b

[Simon Que, 2011-01-21 20:51:14]

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc
File ambient_light_sensor.cc (right):

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc#newcode1
ambient_light_sensor.cc:1: // Copyright (c) 2010 The Chromium OS Authors. All
rights reserved.
2011

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc#newcode75
ambient_light_sensor.cc:75: g_timeout_add(kSensorPollPeriodMs,
AmbientLightSensor::ReadAlsThunk, this);
Is "AmbientLightSensor::" necessary?

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.h
File ambient_light_sensor.h (right):

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.h#newcode1
ambient_light_sensor.h:1: // Copyright (c) 2010 The Chromium OS Authors. All
rights reserved.
2011

[Benson Leung, 2011-01-21 20:55:57]

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc
File ambient_light_sensor.cc (right):

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc#newcode1
ambient_light_sensor.cc:1: // Copyright (c) 2010 The Chromium OS Authors. All
rights reserved.
On 2011/01/21 20:51:14, Simon Que wrote:
> 2011

Done.

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.cc#newcode75
ambient_light_sensor.cc:75: g_timeout_add(kSensorPollPeriodMs,
AmbientLightSensor::ReadAlsThunk, this);
On 2011/01/21 20:51:14, Simon Que wrote:
> Is "AmbientLightSensor::" necessary?

Nope. Removed.

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.h
File ambient_light_sensor.h (right):

http://codereview.chromium.org/6307010/diff/1/ambient_light_sensor.h#newcode1
ambient_light_sensor.h:1: // Copyright (c) 2010 The Chromium OS Authors. All
rights reserved.
On 2011/01/21 20:51:14, Simon Que wrote:
> 2011

Done.

[Simon Que, 2011-01-21 22:52:45]

LGTM.