Do not typedef ExtensionId/ID, since it's already available in extension_id.h

extensions/browser/api/declarative/declarative_rule.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // DeclarativeRule<>, DeclarativeConditionSet<>, and DeclarativeActionSet<>
 // templates usable with multiple different declarativeFoo systems.  These are
 // templated on the Condition and Action types that define the behavior of a
 // particular declarative event.
 
 #ifndef EXTENSIONS_BROWSER_API_DECLARATIVE_DECLARATIVE_RULE_H__
 #define EXTENSIONS_BROWSER_API_DECLARATIVE_DECLARATIVE_RULE_H__
 
 #include <limits>
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/ptr_util.h"
 #include "base/stl_util.h"
 #include "base/time/time.h"
 #include "components/url_matcher/url_matcher.h"
 #include "extensions/common/api/events.h"
 #include "extensions/common/extension.h"

[Devlin, 2016/11/11 23:10:06]

#include extension_id

[lazyboy, 2016/11/12 00:13:16]

Done.

 
 namespace base {
 class Time;
 class Value;
 }
 
 namespace content {
 class BrowserContext;
 }
 
@@ skipping 154 matching lines @@
 // Representation of a rule of a declarative API:
 // https://developer.chrome.com/beta/extensions/events.html#declarative.
 // Generally a RulesRegistry will hold a collection of Rules for a given
 // declarative API and contain the logic for matching and applying them.
 //
 // See DeclarativeConditionSet and DeclarativeActionSet for the requirements on
 // ConditionT and ActionT.
 template<typename ConditionT, typename ActionT>
 class DeclarativeRule {
  public:
-  typedef std::string ExtensionId;
   typedef std::string RuleId;
   typedef std::pair<ExtensionId, RuleId> GlobalRuleId;
   typedef int Priority;
   typedef DeclarativeConditionSet<ConditionT> ConditionSet;
   typedef DeclarativeActionSet<ActionT> ActionSet;
   typedef extensions::api::events::Rule JsonRule;
   typedef std::vector<std::string> Tags;
 
   // Checks whether the set of |conditions| and |actions| are consistent.
   // Returns true in case of consistency and MUST set |error| otherwise.
@@ skipping 285 matching lines @@
 }
 
 template<typename ConditionT, typename ActionT>
 int DeclarativeRule<ConditionT, ActionT>::GetMinimumPriority() const {
   return actions_->GetMinimumPriority();
 }
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_DECLARATIVE_DECLARATIVE_RULE_H__