Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1677)

Unified Diff: backlight_controller.cc

Issue 6715014: Display off is done by a callback into backlight controller (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/power_manager.git
Patch Set: Deleted unused lines, 2010->2011 Created 9 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: backlight_controller.cc
diff --git a/backlight_controller.cc b/backlight_controller.cc
index cfad847d72c8d8f98ab55244f34b494144ea4a26..fa1507440478601b134bb5e8bb389edb176f3a84 100644
--- a/backlight_controller.cc
+++ b/backlight_controller.cc
@@ -67,6 +67,7 @@ bool BacklightController::Init() {
if (backlight_->GetBrightness(&level, &max_)) {
ReadPrefs();
is_initialized_ = true;
+ backlight_->SetScreenOffFunc(TurnScreenOffThunk, this);
return true;
}
return false;
@@ -149,7 +150,6 @@ bool BacklightController::SetPowerState(PowerState state) {
} else {
CHECK(DPMSEnable(GDK_DISPLAY()));
if (state == BACKLIGHT_IDLE_OFF) {
- g_timeout_add(kDisplayOffDelayMS, DisplayOffWhenBacklightOffThunk, this);
SetBrightnessToZero();
} else if (state == BACKLIGHT_ACTIVE_ON) {
CHECK(DPMSForceLevel(GDK_DISPLAY(), DPMSModeOn));
@@ -299,17 +299,10 @@ void BacklightController::SetBrightnessToZero() {
WritePrefs();
}
-gboolean BacklightController::DisplayOffWhenBacklightOff() {
- int64 brightness = 0;
- CHECK(GetBrightness(&brightness));
- if (brightness == 0) {
+void BacklightController::TurnScreenOff() {
+ if (state_ == BACKLIGHT_IDLE_OFF && GDK_DISPLAY() != NULL)
CHECK(DPMSForceLevel(GDK_DISPLAY(), DPMSModeOff));
- return false;
- } else {
- // If the brightness has not been set all the way down, keep polling it
- // until it reaches zero.
- return true;
- }
}
+
} // namespace power_manager
« no previous file with comments | « backlight_controller.h ('k') | backlight_interface.h » ('j') | mock_backlight.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698