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

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

Issue 8789018: Revert 113047 - Make ExtensionService use ExtensionSet. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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/installed_loader.cc
===================================================================
--- chrome/browser/extensions/installed_loader.cc (revision 113069)
+++ chrome/browser/extensions/installed_loader.cc (working copy)
@@ -181,8 +181,8 @@
int theme_count = 0;
int page_action_count = 0;
int browser_action_count = 0;
- const ExtensionSet* extensions = extension_service_->extensions();
- ExtensionSet::const_iterator ex;
+ const ExtensionList* extensions = extension_service_->extensions();
+ ExtensionList::const_iterator ex;
for (ex = extensions->begin(); ex != extensions->end(); ++ex) {
Extension::Location location = (*ex)->location();
Extension::Type type = (*ex)->GetType();
@@ -246,7 +246,7 @@
++browser_action_count;
extension_service_->RecordPermissionMessagesHistogram(
- *ex, "Extensions.Permissions_Load");
+ ex->get(), "Extensions.Permissions_Load");
}
UMA_HISTOGRAM_COUNTS_100("Extensions.LoadApp",
app_user_count + app_external_count);
Property changes on: chrome/browser/extensions/installed_loader.cc
___________________________________________________________________
Added: svn:mergeinfo
« no previous file with comments | « chrome/browser/extensions/extension_webrequest_api.cc ('k') | chrome/browser/extensions/network_delay_listener_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698