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

Unified Diff: chrome/browser/upgrade_detector.cc

Issue 325433002: Elevated install of recovery component (UI part). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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/upgrade_detector.h ('k') | chrome/browser/upgrade_detector_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/upgrade_detector.cc
diff --git a/chrome/browser/upgrade_detector.cc b/chrome/browser/upgrade_detector.cc
index 79e062351535b0f08ad751cf2ca7e08fee547c47..2023676618e2a3d1c828e24af3d56c89105d517f 100644
--- a/chrome/browser/upgrade_detector.cc
+++ b/chrome/browser/upgrade_detector.cc
@@ -79,7 +79,8 @@ void UpgradeDetector::NotifyUpgradeDetected() {
}
void UpgradeDetector::NotifyUpgradeRecommended() {
- notify_upgrade_ = true;
+ notify_upgrade_ = (upgrade_available_ != UPGRADE_AVAILABLE_NONE &&
+ upgrade_notification_stage() != UPGRADE_ANNOYANCE_NONE);
content::NotificationService::current()->Notify(
chrome::NOTIFICATION_UPGRADE_RECOMMENDED,
@@ -110,6 +111,13 @@ void UpgradeDetector::NotifyUpgradeRecommended() {
this, &UpgradeDetector::CheckIdle);
break;
}
+ case UPGRADE_NEEDS_ELEVATION: {
+ content::NotificationService::current()->Notify(
+ chrome::NOTIFICATION_UPGRADE_NEEDS_ELEVATION,
+ content::Source<UpgradeDetector>(this),
+ content::NotificationService::NoDetails());
+ break;
+ }
default:
break;
}
« no previous file with comments | « chrome/browser/upgrade_detector.h ('k') | chrome/browser/upgrade_detector_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698