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

Unified Diff: chrome/browser/extensions/api/content_settings/content_settings_store.h

Issue 2436023002: Remove stl_util's deletion function use from chrome/browser/extensions/api/content_settings. (Closed)
Patch Set: all cases Created 4 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
« no previous file with comments | « no previous file | chrome/browser/extensions/api/content_settings/content_settings_store.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/content_settings/content_settings_store.h
diff --git a/chrome/browser/extensions/api/content_settings/content_settings_store.h b/chrome/browser/extensions/api/content_settings/content_settings_store.h
index f0f9e39bee280dc660aa552aa432f5960840d6f9..0567613a1728f3bfe6c0eaa3c7333b6b6fd64ad3 100644
--- a/chrome/browser/extensions/api/content_settings/content_settings_store.h
+++ b/chrome/browser/extensions/api/content_settings/content_settings_store.h
@@ -6,7 +6,9 @@
#define CHROME_BROWSER_EXTENSIONS_API_CONTENT_SETTINGS_CONTENT_SETTINGS_STORE_H_
#include <map>
+#include <memory>
#include <string>
+#include <vector>
#include "base/macros.h"
#include "base/memory/ref_counted.h"
@@ -112,7 +114,9 @@ class ContentSettingsStore
struct ExtensionEntry;
- typedef std::multimap<base::Time, ExtensionEntry*> ExtensionEntryMap;
+ // A list of the entries, maintained in reverse-chronological order (most-
+ // recently installed items first) to facilitate search.
+ using ExtensionEntries = std::vector<std::unique_ptr<ExtensionEntry>>;
virtual ~ContentSettingsStore();
@@ -129,10 +133,11 @@ class ContentSettingsStore
bool OnCorrectThread();
- ExtensionEntryMap::iterator FindEntry(const std::string& ext_id);
- ExtensionEntryMap::const_iterator FindEntry(const std::string& ext_id) const;
+ ExtensionEntry* FindEntry(const std::string& ext_id) const;
+ ExtensionEntries::iterator FindIterator(const std::string& ext_id);
- ExtensionEntryMap entries_;
+ // The entries.
+ ExtensionEntries entries_;
base::ObserverList<Observer, false> observers_;
« no previous file with comments | « no previous file | chrome/browser/extensions/api/content_settings/content_settings_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698