Added Aura notification that relaunch and powerwash is required in case of downgrade.

chrome/browser/ui/ash/system_tray_delegate_chromeos.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/ash/system_tray_delegate_chromeos.h"
 
 #include <algorithm>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 164 matching lines @@
   content::RecordAction(base::UserMetricsAction("OpenInternetOptionsDialog"));
   ShowSettingsSubPageForActiveUser(page);
 }
 
 void OnAcceptMultiprofilesIntro(bool no_show_again) {
   PrefService* prefs = ProfileManager::GetActiveUserProfile()->GetPrefs();
   prefs->SetBoolean(prefs::kMultiProfileNeverShowIntro, no_show_again);
   UserAddingScreen::Get()->Start();
 }
 
+void GetUpdateInfo(const UpgradeDetector* detector, ash::UpdateInfo* info) {

[sky, 2014/10/02 17:23:06]

Isn't this code the same as the _win and _linux code?

[ygorshenin1, 2014/10/03 14:55:36]

Yes, this code is the same for all platforms. I've moved it to
system_tray_delegate_utils.{cc|h}, thank you for pointing on this!

On 2014/10/02 17:23:06, sky wrote:

+  if (!detector || !info)

[sky, 2014/10/02 17:23:06]

Do we really allow either of these to be NULL? Should this be a DCHECK?

[ygorshenin1, 2014/10/03 14:55:37]

Done.

+    return;
+  ash::UpdateInfo::UpdateSeverity severity = ash::UpdateInfo::UPDATE_NORMAL;

[sky, 2014/10/02 17:23:06]

Why bother with the local and not set info->severity directly?

[ygorshenin1, 2014/10/03 14:55:36]

Done.

+  switch (detector->upgrade_notification_stage()) {
+    case UpgradeDetector::UPGRADE_ANNOYANCE_SEVERE:
+      severity = ash::UpdateInfo::UPDATE_SEVERE_RED;
+      break;
+
+    case UpgradeDetector::UPGRADE_ANNOYANCE_HIGH:
+      severity = ash::UpdateInfo::UPDATE_HIGH_ORANGE;
+      break;
+
+    case UpgradeDetector::UPGRADE_ANNOYANCE_ELEVATED:
+      severity = ash::UpdateInfo::UPDATE_LOW_GREEN;
+      break;
+
+    case UpgradeDetector::UPGRADE_ANNOYANCE_LOW:
+    default:
+      severity = ash::UpdateInfo::UPDATE_NORMAL;

[sky, 2014/10/02 17:23:06]

Isn't this covered by 188? I recommend nuking 188 and 207 entirely. Also, no
default.

[ygorshenin1, 2014/10/03 14:55:36]

Done.

+      break;
+  }
+  info->severity = severity;
+  info->update_required = detector->notify_upgrade();
+  info->factory_reset_required = detector->is_factory_reset_required();
+}
+
 }  // namespace
 
 SystemTrayDelegateChromeOS::SystemTrayDelegateChromeOS()
     : user_profile_(NULL),
       clock_type_(base::GetHourClockType()),
       search_key_mapped_to_(input_method::kSearchKey),
       screen_locked_(false),
       have_session_start_time_(false),
       have_session_length_limit_(false),
       should_run_bluetooth_discovery_(false),
@@ skipping 204 matching lines @@
   return l10n_util::GetStringFUTF16(
       IDS_USER_IS_SUPERVISED_BY_NOTICE,
       base::UTF8ToUTF16(GetSupervisedUserManager()));
 }
 
 bool SystemTrayDelegateChromeOS::IsUserSupervised() const {
   user_manager::User* user = user_manager::UserManager::Get()->GetActiveUser();
   return user && user->IsSupervised();
 }
 
-bool SystemTrayDelegateChromeOS::SystemShouldUpgrade() const {
-  return UpgradeDetector::GetInstance()->notify_upgrade();
+void SystemTrayDelegateChromeOS::GetSystemUpdateInfo(
+    ash::UpdateInfo* info) const {
+  GetUpdateInfo(UpgradeDetector::GetInstance(), info);
 }
 
 base::HourClockType SystemTrayDelegateChromeOS::GetHourClockType() const {
   return clock_type_;
 }
 
 void SystemTrayDelegateChromeOS::ShowSettings() {
   ShowSettingsSubPageForActiveUser("");
 }
 
@@ skipping 659 matching lines @@
   return GetNativeWindowByStatus(GetUserLoginStatus(), session_started);
 }
 
 // content::NotificationObserver implementation.
 void SystemTrayDelegateChromeOS::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case chrome::NOTIFICATION_UPGRADE_RECOMMENDED: {
-      UpgradeDetector* detector =
-          content::Source<UpgradeDetector>(source).ptr();
-      ash::UpdateObserver::UpdateSeverity severity =
-          ash::UpdateObserver::UPDATE_NORMAL;
-      switch (detector->upgrade_notification_stage()) {
-        case UpgradeDetector::UPGRADE_ANNOYANCE_SEVERE:
-          severity = ash::UpdateObserver::UPDATE_SEVERE_RED;
-          break;
-
-        case UpgradeDetector::UPGRADE_ANNOYANCE_HIGH:
-          severity = ash::UpdateObserver::UPDATE_HIGH_ORANGE;
-          break;
-
-        case UpgradeDetector::UPGRADE_ANNOYANCE_ELEVATED:
-          severity = ash::UpdateObserver::UPDATE_LOW_GREEN;
-          break;
-
-        case UpgradeDetector::UPGRADE_ANNOYANCE_LOW:
-        default:
-          severity = ash::UpdateObserver::UPDATE_NORMAL;
-          break;
-      }
-      GetSystemTrayNotifier()->NotifyUpdateRecommended(severity);
+      ash::UpdateInfo info;
+      GetUpdateInfo(content::Source<UpgradeDetector>(source).ptr(), &info);
+      GetSystemTrayNotifier()->NotifyUpdateRecommended(info);
       break;
     }
     case chrome::NOTIFICATION_LOGIN_USER_IMAGE_CHANGED: {
       // This notification is also sent on login screen when user avatar
       // is loaded from file.
       if (GetUserLoginStatus() != ash::user::LOGGED_IN_NONE) {
         GetSystemTrayNotifier()->NotifyUserUpdate();
       }
       break;
     }
@@ skipping 172 matching lines @@
     accessibility_subscription_.reset();
   else
     OnAccessibilityModeChanged(details.notify);
 }
 
 ash::SystemTrayDelegate* CreateSystemTrayDelegate() {
   return new SystemTrayDelegateChromeOS();
 }
 
 }  // namespace chromeos