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

Unified Diff: chrome/browser/extensions/extension_service.cc

Issue 8785016: Add menu option for disabling app notifications (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: another fix from review comments Created 9 years 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: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 0a8196db35f0baac222c48ba97c4e6eef29a7219..95f3ad5efba7cf6c5cbd112e02c6591bf463c1de 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -1444,6 +1444,11 @@ void ExtensionService::ProcessExtensionSyncData(
bundle.pending_sync_data[extension_sync_data.id()] = extension_sync_data;
CheckForUpdatesSoon();
}
+ if (extension_sync_data.notifications_disabled() !=
+ extension_prefs_->IsAppNotificationDisabled(id)) {
+ extension_prefs_->SetAppNotificationDisabled(
+ id, extension_sync_data.notifications_disabled());
+ }
} else {
// TODO(akalin): Replace silent update with a list of enabled
// permissions.

Powered by Google App Engine
This is Rietveld 408576698