Revert 92219 - Update URLPatternSet to contain a std::set instead of std::vector.

This updates URLPatternSet to contain a std::set instead of a std::vector, making it easier to implement the set operations in ExtensionPermissionSet.

BUG=84507
TEST=unit_tests


Review URL: http://codereview.chromium.org/7347011

TBR=jstritar@chromium.org
Review URL: http://codereview.chromium.org/7346019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92224 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 6 deletions

diff --git a/chrome/browser/extensions/crx_installer.cc b/chrome/browser/extensions/crx_installer.cc
index a1e3863..3581571 100644
--- a/chrome/browser/extensions/crx_installer.cc
+++ b/chrome/browser/extensions/crx_installer.cc
@@ -283,13 +283,12 @@ bool CrxInstaller::AllowInstall(const Extension* extension,
       // host (or a subdomain of the host) the download happened from.  There's
       // no way for us to verify that the app controls any other hosts.
       URLPattern pattern(UserScript::kValidUserScriptSchemes);
-      pattern.SetHost(original_url_.host());
-      pattern.SetMatchSubdomains(true);
+      pattern.set_host(original_url_.host());
+      pattern.set_match_subdomains(true);
 
-      URLPatternSet patterns = extension_->web_extent();
-      for (URLPatternSet::const_iterator i = patterns.begin();
-           i != patterns.end(); ++i) {
-        if (!pattern.MatchesHost(i->host())) {
+      URLPatternList patterns = extension_->web_extent().patterns();
+      for (size_t i = 0; i < patterns.size(); ++i) {
+        if (!pattern.MatchesHost(patterns[i].host())) {
           *error = base::StringPrintf(
               "Apps must be served from the host that they affect.");
           return false;