Move URLPattern::ParseOption into a field.

chrome/browser/extensions/extension_content_settings_helpers.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/browser/extensions/extension_content_settings_helpers.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/common/extensions/url_pattern.h"
@@ skipping 39 matching lines @@
   NOTREACHED();
   return "";
 }
 
 }  // namespace
 
 namespace extension_content_settings_helpers {
 
 ContentSettingsPattern ParseExtensionPattern(const std::string& pattern_str,
                                              std::string* error) {
-  URLPattern url_pattern(URLPattern::SCHEME_HTTP |
-                         URLPattern::SCHEME_HTTPS |
-                         URLPattern::SCHEME_FILE);
-  URLPattern::ParseResult result =
-      url_pattern.Parse(pattern_str, URLPattern::USE_PORTS);
+  const int kAllowedSchemes =
+      URLPattern::SCHEME_HTTP | URLPattern::SCHEME_HTTPS |
+      URLPattern::SCHEME_FILE;
+  URLPattern url_pattern(URLPattern::USE_PORTS, kAllowedSchemes);
+  URLPattern::ParseResult result = url_pattern.Parse(pattern_str);
   if (result != URLPattern::PARSE_SUCCESS) {
     *error = URLPattern::GetParseResultString(result);
     return ContentSettingsPattern();
   } else {
     scoped_ptr<ContentSettingsPattern::BuilderInterface> builder(
         ContentSettingsPattern::CreateBuilder(false));
     builder->WithHost(url_pattern.host());
     if (url_pattern.match_subdomains())
       builder->WithDomainWildcard();
 
@@ skipping 63 matching lines @@
   return false;
 }
 
 const char* ContentSettingToString(ContentSetting setting) {
   size_t index = static_cast<size_t>(setting);
   DCHECK_LT(index, arraysize(kContentSettingNames));
   return kContentSettingNames[index];
 }
 
 }  // namespace extension_content_settings_helpers