Make ExtensionService use ExtensionSet.

chrome/common/extensions/extension_set.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/common/extensions/extension_set.h"
 
 #include "base/logging.h"
 #include "chrome/common/url_constants.h"
 
 ExtensionSet::ExtensionSet() {
 }
 
 ExtensionSet::~ExtensionSet() {
 }
 
 size_t ExtensionSet::size() const {
   return extensions_.size();
 }
 
+bool ExtensionSet::empty() const {

[Aaron Boodman, 2011/12/01 08:27:12]

I prefer is_empty() to avoid confusion with clear(). Though I can understand the
desire for consistency with STL.

[Yoyo Zhou, 2011/12/05 19:34:09]

Done.

+  return extensions_.empty();
+}
+
 bool ExtensionSet::Contains(const std::string& extension_id) const {
   return extensions_.find(extension_id) != extensions_.end();
 }
 
 void ExtensionSet::Insert(const scoped_refptr<const Extension>& extension) {
   extensions_[extension->id()] = extension;
 }
 
 void ExtensionSet::Remove(const std::string& id) {
   extensions_.erase(id);
 }
 
-std::string ExtensionSet::GetIdByURL(const GURL& url) const {
+void ExtensionSet::Clear() {
+  extensions_.clear();
+}
+
+std::string ExtensionSet::GetIDByURL(const GURL& url) const {
   if (url.SchemeIs(chrome::kExtensionScheme))
     return url.host();
 
   const Extension* extension = GetByURL(url);
   if (!extension)
     return "";
 
   return extension->id();
 }
 
@@ skipping 29 matching lines @@
 
   ExtensionMap::const_iterator i = extensions_.begin();
   for (; i != extensions_.end(); ++i) {
     if (i->second->location() == Extension::COMPONENT &&
         i->second->web_extent().MatchesURL(url))
       return true;
   }
 
   return false;
 }