Add default intents methods for web data service.

chrome/common/extensions/url_pattern.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/url_pattern.h"
 
 #include "base/string_number_conversions.h"
 #include "base/string_piece.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
@@ skipping 112 matching lines @@
 bool URLPattern::operator<(const URLPattern& other) const {
   return GetAsString() < other.GetAsString();
 }
 
 bool URLPattern::operator==(const URLPattern& other) const {
   return GetAsString() == other.GetAsString();
 }
 
 URLPattern::ParseResult URLPattern::Parse(const std::string& pattern) {
   spec_.clear();
+  SetMatchAllURLs(false);
+  SetMatchSubdomains(false);
+  SetPort("*");

[groby-ooo-7-16, 2012/02/14 00:14:30]

Why are we setting those here? Isn't that covered by the constructor already? 

[Greg Billock, 2012/02/14 01:21:46]

This is actually in another CL that has gone in already. It'll merge out of this
change. (Basically, the class wasn't resetting properly if it was ever set to
all-urls.)

 
   // Special case pattern to match every valid URL.
   if (pattern == kAllUrlsPattern) {
     SetMatchAllURLs(true);
     return PARSE_SUCCESS;
   }
 
   // Parse out the scheme.
   size_t scheme_end_pos = pattern.find(chrome::kStandardSchemeSeparator);
   bool has_standard_scheme_separator = true;
@@ skipping 346 matching lines @@
   }
 
   return result;
 }
 
 // static
 const char* URLPattern::GetParseResultString(
     URLPattern::ParseResult parse_result) {
   return kParseResultMessages[parse_result];
 }