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

Unified Diff: ash/system/update/tray_update.h

Issue 2882933002: Add update available icon in system tray (Closed)
Patch Set: 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/update/tray_update.h
diff --git a/ash/system/update/tray_update.h b/ash/system/update/tray_update.h
index 6d3a9c001ed08bafebe8d7d6e15af165e262e8c6..5a5c84a62ddbd30797a5c86d9bebbb0b442c6a24 100644
--- a/ash/system/update/tray_update.h
+++ b/ash/system/update/tray_update.h
@@ -38,6 +38,11 @@ class ASH_EXPORT TrayUpdate : public TrayImageItem {
bool factory_reset_required,
mojom::UpdateType update_type);
+ // Shows an icon in the system tray indicating that a software update is
+ // available but user's agreement is required as current connection is
+ // cellular. Once shown the icon persists until reboot.
+ void ShowUpdateOverCellularAvailableIcon();
+
private:
FRIEND_TEST_ALL_PREFIXES(TrayUpdateTest, VisibilityAfterUpdate);
FRIEND_TEST_ALL_PREFIXES(TrayUpdateTest, VisibilityAfterFlashUpdate);
@@ -56,10 +61,11 @@ class ASH_EXPORT TrayUpdate : public TrayImageItem {
// If an external monitor is connected then the system tray may be created
// after the update data is sent from chrome, so share the update info between
// all instances.
- static bool update_required_;
+ static bool upgrade_required_;
weidongg 2017/05/15 04:54:13 I refactored here to distinguish 'update' with 'up
static mojom::UpdateSeverity severity_;
static bool factory_reset_required_;
static mojom::UpdateType update_type_;
+ static bool update_over_cellular_available_;
DISALLOW_COPY_AND_ASSIGN(TrayUpdate);
};

Powered by Google App Engine
This is Rietveld 408576698