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

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

Issue 10407105: Improve error messaging of webRequest API in case of conflicts (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merged with ToT Created 8 years, 3 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: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 730a126118407d539d177b038967f192331c4ee9..493498cf1e58e8b66d19dca7849aff2a28d08825 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -337,7 +337,6 @@ ExtensionService::ExtensionService(Profile* profile,
update_once_all_providers_are_ready_(false),
app_sync_bundle_(ALLOW_THIS_IN_INITIALIZER_LIST(this)),
extension_sync_bundle_(ALLOW_THIS_IN_INITIALIZER_LIST(this)),
- extension_warnings_(profile),
app_shortcut_manager_(profile) {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ -806,12 +805,6 @@ bool ExtensionService::UninstallExtension(
1, 2);
}
- // Uninstalling one extension might have solved the problems of others.
- // Therefore, we clear warnings of this type for all extensions.
- std::set<ExtensionWarningSet::WarningType> warnings;
- extension_warnings_.GetWarningsAffectingExtension(extension_id, &warnings);
- extension_warnings_.ClearWarnings(warnings);
-
return true;
}
@@ -906,12 +899,6 @@ void ExtensionService::DisableExtension(
}
SyncExtensionChangeIfNeeded(*extension);
-
- // Deactivating one extension might have solved the problems of others.
- // Therefore, we clear warnings of this type for all extensions.
- std::set<ExtensionWarningSet::WarningType> warnings;
- extension_warnings_.GetWarningsAffectingExtension(extension_id, &warnings);
- extension_warnings_.ClearWarnings(warnings);
}
void ExtensionService::GrantPermissionsAndEnableExtension(

Powered by Google App Engine
This is Rietveld 408576698