Detect invalid content settings pattern that were not detected yet.

chrome/common/content_settings_pattern_parser.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/content_settings_pattern_parser.h"
 
 #include "base/string_util.h"
 #include "chrome/common/url_constants.h"
 #include "net/base/net_util.h"
 #include "googleurl/src/gurl.h"
@@ skipping 122 matching lines @@
     builder->WithSchemeWildcard();
   }
 
   if (host_component.IsNonEmpty()) {
     std::string host = pattern_spec.substr(host_component.start,
                                            host_component.len);
     if (host == kHostWildcard) {
       builder->WithDomainWildcard();
     } else if (StartsWithASCII(host, kDomainWildcard, true)) {
       host = host.substr(kDomainWildcardLength);
-      // If the host still contains a wildcard symbol then it is invalid.
-      if (host.find(kHostWildcard) != std::string::npos) {
+      // If the host contains a domain wildcard but no domain then it is

[Bernhard Bauer, 2011/11/24 15:50:32]

This would be a pattern that's just "[*.]"? I think we should allow it (but then
stringify it to "*" so that users know what's the preferred syntax).

[markusheintz_, 2011/11/24 16:37:51]

True. Done

+      // invalid.
+      if (host.empty()) {
         builder->Invalid();
         return;
       } else {
         builder->WithDomainWildcard();
         builder->WithHost(host);
       }
     } else {
       // If the host contains a wildcard symbol then it is invalid.
       if (host.find(kHostWildcard) != std::string::npos) {
         builder->Invalid();
@@ skipping 62 matching lines @@
   }
 
   if (!parts.is_port_wildcard) {
     str += std::string(kUrlPortSeparator) + parts.port;
   }
 
   return str;
 }
 
 }  // namespace content_settings