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

Unified Diff: chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc

Issue 2332893002: Changes ownership of ScreenDimmer (Closed)
Patch Set: merge to tot Created 4 years, 3 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: chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc
diff --git a/chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc b/chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc
index e5badf7c30ea6a0e4b8d64d82330409e86933fb1..1dcf74411703e927f5abed40c0b66b4348ec2752 100644
--- a/chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc
+++ b/chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc
@@ -6,6 +6,7 @@
#include "ash/shell.h"
#include "ash/wm/screen_dimmer.h"
+#include "base/memory/ptr_util.h"
#include "ui/base/user_activity/user_activity_detector.h"
#include "ui/display/chromeos/display_configurator.h"
@@ -33,7 +34,11 @@ void ChromeDisplayPowerServiceProviderDelegate::SetDisplayPower(
}
void ChromeDisplayPowerServiceProviderDelegate::SetDimming(bool dimmed) {
- ash::ScreenDimmer::GetForRoot()->SetDimming(dimmed);
+ if (!screen_dimmer_) {
+ screen_dimmer_ =
+ base::MakeUnique<ash::ScreenDimmer>(ash::ScreenDimmer::Container::ROOT);
+ }
+ screen_dimmer_->SetDimming(dimmed);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698