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

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

Issue 10824344: Renamed ExtensionIdSet to ExtensionIds (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: undid files where I went too far Created 8 years, 4 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_toolbar_model.cc
diff --git a/chrome/browser/extensions/extension_toolbar_model.cc b/chrome/browser/extensions/extension_toolbar_model.cc
index dc1e4a4ea098a1fab9dad05c366bd6ddd4725f17..13c5e3ff144d83b52f9269326fb36711811a0260 100644
--- a/chrome/browser/extensions/extension_toolbar_model.cc
+++ b/chrome/browser/extensions/extension_toolbar_model.cc
@@ -271,9 +271,9 @@ void ExtensionToolbarModel::InitializeExtensionLists() {
}
void ExtensionToolbarModel::PopulateForActionBoxMode() {
- const extensions::ExtensionPrefs::ExtensionIdSet toolbar_order =
+ const extensions::ExtensionPrefs::ExtensionIds toolbar_order =
service_->extension_prefs()->GetToolbarOrder();
- extensions::ExtensionPrefs::ExtensionIdSet action_box_order =
+ extensions::ExtensionPrefs::ExtensionIds action_box_order =
service_->extension_prefs()->GetActionBoxOrder();
// Add all browser actions not already in the toolbar or action box
@@ -300,7 +300,7 @@ void ExtensionToolbarModel::PopulateForActionBoxMode() {
}
void ExtensionToolbarModel::PopulateForNonActionBoxMode() {
- const extensions::ExtensionPrefs::ExtensionIdSet pref_order =
+ const extensions::ExtensionPrefs::ExtensionIds pref_order =
service_->extension_prefs()->GetToolbarOrder();
// Items that have a pref for their position.
ExtensionList sorted;
@@ -317,7 +317,7 @@ void ExtensionToolbarModel::PopulateForNonActionBoxMode() {
if (!service_->extension_prefs()->GetBrowserActionVisibility(extension))
continue;
- extensions::ExtensionPrefs::ExtensionIdSet::const_iterator pos =
+ extensions::ExtensionPrefs::ExtensionIds::const_iterator pos =
std::find(pref_order.begin(), pref_order.end(), extension->id());
if (pos != pref_order.end())
sorted[pos - pref_order.begin()] = extension;
@@ -350,7 +350,7 @@ void ExtensionToolbarModel::PopulateForNonActionBoxMode() {
}
void ExtensionToolbarModel::FillExtensionList(
- const extensions::ExtensionPrefs::ExtensionIdSet& order,
+ const extensions::ExtensionPrefs::ExtensionIds& order,
ExtensionList* list) {
list->clear();
list->reserve(order.size());
@@ -366,13 +366,13 @@ void ExtensionToolbarModel::UpdatePrefs() {
if (!service_->extension_prefs())
return;
- extensions::ExtensionPrefs::ExtensionIdSet toolbar_ids;
+ extensions::ExtensionPrefs::ExtensionIds toolbar_ids;
toolbar_ids.reserve(toolbar_items_.size());
for (size_t i = 0; i < toolbar_items_.size(); ++i)
toolbar_ids.push_back(toolbar_items_[i]->id());
service_->extension_prefs()->SetToolbarOrder(toolbar_ids);
- extensions::ExtensionPrefs::ExtensionIdSet action_box_ids;
+ extensions::ExtensionPrefs::ExtensionIds action_box_ids;
action_box_ids.reserve(action_box_menu_items_.size());
for (size_t i = 0; i < action_box_menu_items_.size(); ++i)
action_box_ids.push_back(action_box_menu_items_[i]->id());
« no previous file with comments | « chrome/browser/extensions/extension_toolbar_model.h ('k') | chrome/browser/protector/protected_prefs_watcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698