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

Unified Diff: extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc

Issue 2494113003: Do not typedef ExtensionId/ID, since it's already available in extension_id.h (Closed)
Patch Set: fix includes from comments Created 4 years, 1 month 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: extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc
diff --git a/extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc b/extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc
index 0907ebd7887aa88c82605053550be7b7b16eee5d..a9f370a3c40cc52fedfbe854be3a92384890a3a2 100644
--- a/extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc
+++ b/extensions/browser/api/declarative_webrequest/webrequest_rules_registry.cc
@@ -99,7 +99,6 @@ std::list<LinkedPtrEventResponseDelta> WebRequestRulesRegistry::CreateDeltas(
// for rules of that extension. Initially, this priority is -infinite and
// will be increased when the rules are processed and raise the bar via
// WebRequestIgnoreRulesActions.
- typedef std::string ExtensionId;
typedef std::map<ExtensionId, WebRequestRule::Priority> MinPriorities;
typedef std::map<ExtensionId, std::set<std::string> > IgnoreTags;
MinPriorities min_priorities;
@@ -289,8 +288,8 @@ bool WebRequestRulesRegistry::IsEmpty() const {
return false;
// Now all the registered rules for each extensions.
- for (const std::pair<WebRequestRule::ExtensionId, RulesMap>&
- extension_id_rules_map_pair : webrequest_rules_) {
+ for (const std::pair<ExtensionId, RulesMap>& extension_id_rules_map_pair :
+ webrequest_rules_) {
if (!extension_id_rules_map_pair.second.empty())
return false;
}

Powered by Google App Engine
This is Rietveld 408576698