Allow a unique origin to compare equal to itself

url/origin_unittest.cc

Index: url/origin_unittest.cc
diff --git a/url/origin_unittest.cc b/url/origin_unittest.cc
index a5c30426f346214213f94c2b46c7da5924e48b89..34c289b484b7806eca8348d0df61984dd64e2d49 100644
--- a/url/origin_unittest.cc
+++ b/url/origin_unittest.cc
@@ -41,7 +41,12 @@ TEST(OriginTest, UniqueOriginComparison) {
   EXPECT_EQ("", unique_origin.host());
   EXPECT_EQ(0, unique_origin.port());
   EXPECT_TRUE(unique_origin.unique());
-  EXPECT_FALSE(unique_origin.IsSameOriginWith(unique_origin));
+  // A unique origin is same origin with itself
+  EXPECT_TRUE(unique_origin.IsSameOriginWith(unique_origin));
+  // A unique origin is not same origin with any other unique origin
+  url::Origin second_unique_origin;
+  EXPECT_FALSE(unique_origin.IsSameOriginWith(second_unique_origin));
+  EXPECT_FALSE(second_unique_origin.IsSameOriginWith(unique_origin));
 
   const char* const urls[] = {"data:text/html,Hello!",
                               "javascript:alert(1)",
@@ -57,10 +62,25 @@ TEST(OriginTest, UniqueOriginComparison) {
     EXPECT_EQ("", origin.host());
     EXPECT_EQ(0, origin.port());
     EXPECT_TRUE(origin.unique());
-    EXPECT_FALSE(origin.IsSameOriginWith(origin));
+
+    // A unique origin is same origin with itself
+    EXPECT_TRUE(origin.IsSameOriginWith(origin));
+
+    // A unique origin is not same origin with any other unique origin
     EXPECT_FALSE(unique_origin.IsSameOriginWith(origin));
     EXPECT_FALSE(origin.IsSameOriginWith(unique_origin));
 
+    // A second unique origin from the same source string is not same origin
+    // with the first.
+    url::Origin duplicated_origin(url);
+    EXPECT_FALSE(origin.IsSameOriginWith(duplicated_origin));
+    EXPECT_FALSE(duplicated_origin.IsSameOriginWith(origin));
+
+    // A copy of a unique origin is not same origin with the original.
+    url::Origin copied_origin(origin);
+    EXPECT_FALSE(origin.IsSameOriginWith(copied_origin));
+    EXPECT_FALSE(copied_origin.IsSameOriginWith(origin));
+
     ExpectParsedUrlsEqual(GURL(origin.Serialize()), origin.GetURL());
   }
 }
@@ -405,7 +425,17 @@ TEST(OriginTest, UnsafelyCreateUniqueOnInvalidInput) {
     EXPECT_EQ("", origin.host());
     EXPECT_EQ(0, origin.port());
     EXPECT_TRUE(origin.unique());
-    EXPECT_FALSE(origin.IsSameOriginWith(origin));
+
+    // A unique origin is same origin with itself
+    EXPECT_TRUE(origin.IsSameOriginWith(origin));
+
+    // A second unique origin from the same URL components is not same origin
+    // with the first.
+    url::Origin duplicated_origin =
+        url::Origin::UnsafelyCreateOriginWithoutNormalization(
+            test.scheme, test.host, test.port);
+    EXPECT_FALSE(origin.IsSameOriginWith(duplicated_origin));
+    EXPECT_FALSE(duplicated_origin.IsSameOriginWith(origin));
 
     ExpectParsedUrlsEqual(GURL(origin.Serialize()), origin.GetURL());
   }
@@ -435,7 +465,18 @@ TEST(OriginTest, UnsafelyCreateUniqueViaEmbeddedNulls) {
     EXPECT_EQ("", origin.host());
     EXPECT_EQ(0, origin.port());
     EXPECT_TRUE(origin.unique());
-    EXPECT_FALSE(origin.IsSameOriginWith(origin));
+
+    // A unique origin is same origin with itself
+    EXPECT_TRUE(origin.IsSameOriginWith(origin));
+
+    // A second unique origin from the same URL components is not same origin
+    // with the first.
+    url::Origin duplicated_origin =
+        url::Origin::UnsafelyCreateOriginWithoutNormalization(
+            std::string(test.scheme, test.scheme_length),
+            std::string(test.host, test.host_length), test.port);
+    EXPECT_FALSE(origin.IsSameOriginWith(duplicated_origin));
+    EXPECT_FALSE(duplicated_origin.IsSameOriginWith(origin));
 
     ExpectParsedUrlsEqual(GURL(origin.Serialize()), origin.GetURL());
   }