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

Unified Diff: ash/system/tray/system_tray_controller.cc

Issue 2882933002: Add update available icon in system tray (Closed)
Patch Set: Delay RequestUpdateCheck until the aboutChromeOS page is loaded Created 3 years, 7 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: ash/system/tray/system_tray_controller.cc
diff --git a/ash/system/tray/system_tray_controller.cc b/ash/system/tray/system_tray_controller.cc
index 5f3d07a08c30ca5d03089df12ab6e0671ea72e11..d9154dae9a5b19638374b9b8c7589c0c8f6cbf96 100644
--- a/ash/system/tray/system_tray_controller.cc
+++ b/ash/system/tray/system_tray_controller.cc
@@ -45,6 +45,11 @@ void SystemTrayController::ShowDateSettings() {
system_tray_client_->ShowDateSettings();
}
+void SystemTrayController::ShowAboutChromeOS() {
+ if (system_tray_client_)
+ system_tray_client_->ShowAboutChromeOS();
+}
+
void SystemTrayController::ShowSetTimeDialog() {
if (system_tray_client_)
system_tray_client_->ShowSetTimeDialog();
@@ -141,6 +146,11 @@ void SystemTrayController::RequestRestartForUpdate() {
system_tray_client_->RequestRestartForUpdate();
}
+void SystemTrayController::RequestUpdateCheck() {
+ if (system_tray_client_)
+ system_tray_client_->ScheduleRequestUpdateCheck();
+}
+
void SystemTrayController::BindRequest(mojom::SystemTrayRequest request) {
bindings_.AddBinding(this, std::move(request));
}
@@ -210,4 +220,15 @@ void SystemTrayController::ShowUpdateIcon(mojom::UpdateSeverity severity,
}
}
+void SystemTrayController::ShowUpdateOverCellularAvailableIcon() {
+ // Show the icon on all displays.
+ for (WmWindow* root : ShellPort::Get()->GetAllRootWindows()) {
+ ash::SystemTray* tray = root->GetRootWindowController()->GetSystemTray();
+ // External monitors might not have a tray yet.
+ if (!tray)
+ continue;
+ tray->tray_update()->ShowUpdateOverCellularAvailableIcon();
+ }
+}
+
} // namespace ash

Powered by Google App Engine
This is Rietveld 408576698