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

Unified Diff: ash/common/system/chromeos/power/dual_role_notification.cc

Issue 2377703003: Support SystemTrayDelegate::ShowPowerSettings() in mustash (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | ash/mus/system_tray_delegate_mus.cc » ('j') | ash/mus/system_tray_delegate_mus.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/system/chromeos/power/dual_role_notification.cc
diff --git a/ash/common/system/chromeos/power/dual_role_notification.cc b/ash/common/system/chromeos/power/dual_role_notification.cc
index 1b902784442302a8fe4deebe00e2ebc59723f1ca..400403b811fcc1de3e3c647f936c7ccfb1c32e84 100644
--- a/ash/common/system/chromeos/power/dual_role_notification.cc
+++ b/ash/common/system/chromeos/power/dual_role_notification.cc
@@ -37,7 +37,8 @@ class DualRoleNotificationDelegate
// Overridden from message_center::NotificationDelegate.
void Click() override {
- WmShell::Get()->system_tray_delegate()->ShowPowerSettings();
+ if (PowerStatus::Get()->IsBatteryPresent())
James Cook 2016/09/27 23:23:13 I'm unsure if we still need this check (I moved it
michaelpg 2016/09/28 23:09:15 We shouldn't show the *notification* if there isn'
James Cook 2016/09/29 15:28:29 Removed the if().
+ WmShell::Get()->system_tray_delegate()->ShowPowerSettings();
}
private:
« no previous file with comments | « no previous file | ash/mus/system_tray_delegate_mus.cc » ('j') | ash/mus/system_tray_delegate_mus.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698