Clean up PluginGroup and related code.

chrome/browser/plugin_exceptions_table_model.cc

Index: chrome/browser/plugin_exceptions_table_model.cc
diff --git a/chrome/browser/plugin_exceptions_table_model.cc b/chrome/browser/plugin_exceptions_table_model.cc
index 21ed5470b1b331cf0659a0de044911b1a6a9bb57..d7f438d53d12eab24fc61daffdee43d62c315610 100644
--- a/chrome/browser/plugin_exceptions_table_model.cc
+++ b/chrome/browser/plugin_exceptions_table_model.cc
@@ -129,18 +129,15 @@ void PluginExceptionsTableModel::ClearSettings() {
   resources_.clear();
 }
 
-void PluginExceptionsTableModel::GetPlugins(
-    NPAPI::PluginList::PluginMap* plugins) {
-  NPAPI::PluginList::Singleton()->GetPluginGroups(false, plugins);
+std::vector<PluginGroup> PluginExceptionsTableModel::GetPlugins() {
+  return NPAPI::PluginList::Singleton()->GetPluginGroups(false);
 }
 
 void PluginExceptionsTableModel::LoadSettings() {
   int group_id = 0;
-  NPAPI::PluginList::PluginMap plugins;
-  GetPlugins(&plugins);
-  for (NPAPI::PluginList::PluginMap::iterator it = plugins.begin();
-       it != plugins.end(); ++it) {
-    std::string plugin = it->first;
+  std::vector<PluginGroup> plugins(GetPlugins());

[Bernhard Bauer, 2010/12/03 16:13:28]

Are we sure that the compiler optimizes any unnecessary copying away (on all
platforms)?

[Jakob Kummerow, 2010/12/06 18:21:12]

Done. (Changed the interface.)

+  for (size_t i = 0; i < plugins.size(); ++i) {
+    std::string plugin = plugins[i].identifier();
     HostContentSettingsMap::SettingsForOneType settings;
     map_->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_PLUGINS,
                                 plugin,
@@ -151,7 +148,7 @@ void PluginExceptionsTableModel::LoadSettings() {
                                       plugin,
                                       &otr_settings);
     }
-    std::wstring title = UTF16ToWide(it->second->GetGroupName());
+    std::wstring title = UTF16ToWide(plugins[i].GetGroupName());
     for (HostContentSettingsMap::SettingsForOneType::iterator setting_it =
              settings.begin(); setting_it != settings.end(); ++setting_it) {
       SettingsEntry entry = {