Fix a memory leak in ExtensionExtentUnittest by fixing a bad

chrome/common/extensions/url_pattern.h

 // Copyright (c) 2006-2009 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.
 #ifndef CHROME_COMMON_EXTENSIONS_URL_PATTERN_H_
 #define CHROME_COMMON_EXTENSIONS_URL_PATTERN_H_
 
 #include <string>
 
 #include "googleurl/src/gurl.h"
 
@@ skipping 55 matching lines @@
 // subdomain matching. We can pick up at least another 10% by supporting
 // subdomain matching. It is probably possible to coerce more of the existing
 // patterns to URLPattern, but the resulting pattern will be more restrictive
 // than the original glob, which is probably better than nothing.
 class URLPattern {
  public:
   // Returns true if the specified scheme can be used in URL patterns, and false
   // otherwise.
   static bool IsValidScheme(const std::string& scheme);
 
-  // Convenience to create a pattern from a string.
-  static URLPattern* CreateFromString(const std::string& pattern);
+  URLPattern();
 
-  URLPattern() : match_subdomains_(false) {}
+  // Convenience to construct a URLPattern from a string. The string is expected
+  // to be a valid pattern. If the string is not known ahead of time, use
+  // Parse() instead, which returns success or failure.
+  explicit URLPattern(const std::string& pattern);
 
   // Initializes this instance by parsing the provided string. On failure, the
   // instance will have some intermediate values and is in an invalid state.
   bool Parse(const std::string& pattern_str);
 
   // Returns true if this instance matches the specified URL.
   bool MatchesUrl(const GURL& url) const;
 
   // Returns true if |test| matches our host.
   bool MatchesHost(const std::string& host) const;
@@ skipping 41 matching lines @@
   // everything after the scheme in the case of file:// URLs.
   std::string path_;
 
   // The path with "?" and "\" characters escaped for use with the
   // MatchPatternASCII() function. This is populated lazily, the first time it
   // is needed.
   mutable std::string path_escaped_;
 };
 
 #endif  // CHROME_COMMON_EXTENSIONS_URL_PATTERN_H_