Allow invalid patterns for comparison; Make the compare method a order relation.

chrome/browser/content_settings/content_settings_pattern_unittest.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/content_settings/content_settings_pattern.h"
 
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 355 matching lines @@
   ContentSettingsPattern pattern1 =
       Pattern("http://www.google.com");
   EXPECT_EQ(ContentSettingsPattern::IDENTITY, pattern1.Compare(pattern1));
   EXPECT_EQ(ContentSettingsPattern::IDENTITY,
             Pattern("http://www.google.com:80").Compare(
                 Pattern("http://www.google.com:80")));
   EXPECT_EQ(ContentSettingsPattern::IDENTITY,
             Pattern("*://[*.]google.com:*").Compare(
                 Pattern("*://[*.]google.com:*")));
 
-  // Test disjoint patterns.
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  ContentSettingsPattern invalid_pattern1 = ContentSettingsPattern();
+  ContentSettingsPattern invalid_pattern2 = Pattern("www.google.com*");
+
+  // Compare invalid patterns.
+  EXPECT_TRUE(!invalid_pattern1.IsValid());
+  EXPECT_TRUE(!invalid_pattern2.IsValid());
+  EXPECT_EQ(ContentSettingsPattern::IDENTITY,
+            invalid_pattern1.Compare(invalid_pattern2));
+  EXPECT_TRUE(invalid_pattern1 == invalid_pattern2);
+
+  // Compare a pattern with an IPv4 addresse to a pattern with a domain name.
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_POST,
+            Pattern("http://www.google.com").Compare(
+                Pattern("127.0.0.1")));
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
+            Pattern("127.0.0.1").Compare(
+                Pattern("http://www.google.com")));
+  EXPECT_TRUE(Pattern("127.0.0.1") > Pattern("http://www.google.com"));
+  EXPECT_TRUE(Pattern("http://www.google.com") < Pattern("127.0.0.1"));
+
+  // Compare a pattern with an IPv6 address to a patterns with a domain name.
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_POST,
+            Pattern("http://www.google.com").Compare(
+                Pattern("[::1]")));
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
+            Pattern("[::1]").Compare(
+                Pattern("http://www.google.com")));
+  EXPECT_TRUE(Pattern("[::1]") > Pattern("http://www.google.com"));
+  EXPECT_TRUE(Pattern("http://www.google.com") < Pattern("[::1]"));
+
+  // Compare a pattern with an IPv6 addresse to a pattern with an IPv4 addresse.
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
+            Pattern("127.0.0.1").Compare(
+                Pattern("[::1]")));
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_POST,
+            Pattern("[::1]").Compare(
+                Pattern("127.0.0.1")));
+  EXPECT_TRUE(Pattern("[::1]") < Pattern("127.0.0.1"));
+  EXPECT_TRUE(Pattern("127.0.0.1") > Pattern("[::1]"));
+
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("http://www.google.com").Compare(
                 Pattern("http://www.youtube.com")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("http://[*.]google.com").Compare(
                 Pattern("http://[*.]youtube.com")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_POST,
             Pattern("http://[*.]host.com").Compare(
                 Pattern("http://[*.]evilhost.com")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_POST,
             Pattern("*://www.google.com:80").Compare(
                 Pattern("*://www.google.com:8080")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("https://www.google.com:80").Compare(
                 Pattern("http://www.google.com:80")));
 
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("http://[*.]google.com:90").Compare(
                 Pattern("http://mail.google.com:80")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("https://[*.]google.com:80").Compare(
                 Pattern("http://mail.google.com:80")));
-  EXPECT_EQ(ContentSettingsPattern::DISJOINT,
+  EXPECT_EQ(ContentSettingsPattern::DISJOINT_ORDER_PRE,
             Pattern("https://mail.google.com:*").Compare(
                 Pattern("http://mail.google.com:80")));
 
   // Test patterns with different precedences.
   EXPECT_EQ(ContentSettingsPattern::PREDECESSOR,
             Pattern("mail.google.com").Compare(
                 Pattern("[*.]google.com")));
   EXPECT_EQ(ContentSettingsPattern::SUCCESSOR,
             Pattern("[*.]google.com").Compare(
                 Pattern("mail.google.com")));
@@ skipping 106 matching lines @@
   // file:/// normalization.
   EXPECT_STREQ("file:///tmp/test.html",
                Pattern("file:///tmp/bar/../test.html").ToString().c_str());
 
   // Invalid patterns.
   EXPECT_STREQ("", Pattern("*example.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("example.*").ToString().c_str());
   EXPECT_STREQ("", Pattern("*\xC4\x87ira.com").ToString().c_str());
   EXPECT_STREQ("", Pattern("\xC4\x87ira.*").ToString().c_str());
 }