chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.cc - Issue 886103002: Make SetDisplayPower() take a callback to signal completion

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 886103002: Make SetDisplayPower() take a callback to signal completion (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fixed nits Created 5 years, 11 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

« no previous file with comments | « chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.h ('k') | chromeos/dbus/services/display_power_service_provider.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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 9fc1718809e6b5ab4207787c3367ea960c9da52c..57269d99437ac0d2ee259c40636ef0442c136663 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

@@ -19,7 +19,8 @@ ChromeDisplayPowerServiceProviderDelegate::

}

void ChromeDisplayPowerServiceProviderDelegate::SetDisplayPower(

- DisplayPowerState power_state) {

+ DisplayPowerState power_state,

+ const ResponseCallback& callback) {

// Turning displays off when the device becomes idle or on just before

// we suspend may trigger a mouse move, which would then be incorrectly

// reported as user activity. Let the UserActivityDetector

@@ -27,7 +28,7 @@ void ChromeDisplayPowerServiceProviderDelegate::SetDisplayPower(

ui::UserActivityDetector::Get()->OnDisplayPowerChanging();

ash::Shell::GetInstance()->display_configurator()->SetDisplayPower(

- power_state, ui::DisplayConfigurator::kSetDisplayPowerNoFlags);

+ power_state, ui::DisplayConfigurator::kSetDisplayPowerNoFlags, callback);

}

void ChromeDisplayPowerServiceProviderDelegate::SetDimming(bool dimmed) {