Treat exact domain match cookies on public suffixes as host cookies.

net/cookies/cookie_store_unittest.h

Index: net/cookies/cookie_store_unittest.h
diff --git a/net/cookies/cookie_store_unittest.h b/net/cookies/cookie_store_unittest.h
index f62f7ac2f67078d9642d3eb8e4e36ac76fdfa29b..f4f473906142316f87bea341ade4c9a7c14bb2eb 100644
--- a/net/cookies/cookie_store_unittest.h
+++ b/net/cookies/cookie_store_unittest.h
@@ -515,7 +515,26 @@ TYPED_TEST_P(CookieStoreTest, TestNonDottedAndTLD) {
     // Allow setting on "com", (but only as a host cookie).
     EXPECT_TRUE(this->SetCookie(cs.get(), url, "a=1"));
     EXPECT_FALSE(this->SetCookie(cs.get(), url, "b=2; domain=.com"));
-    EXPECT_FALSE(this->SetCookie(cs.get(), url, "c=3; domain=com"));
+
+    this->MatchCookieLines("a=1", this->GetCookies(cs.get(), url));
+    // Make sure it doesn't show up for a normal .com, it should be a host
+    // not a domain cookie.
+    this->MatchCookieLines(
+        std::string(),
+        this->GetCookies(cs.get(), GURL("http://hopefully-no-cookies.com/")));
+    if (TypeParam::supports_non_dotted_domains) {
+      this->MatchCookieLines(std::string(),
+                             this->GetCookies(cs.get(), GURL("http://.com/")));
+    }
+  }
+
+  {
+    // Exact matches between the domain attribute and the host are treated as
+    // host cookies, not domain cookies.
+    scoped_refptr<CookieStore> cs(this->GetCookieStore());
+    GURL url("http://com/");
+    EXPECT_TRUE(this->SetCookie(cs.get(), url, "a=1; domain=com"));
+
     this->MatchCookieLines("a=1", this->GetCookies(cs.get(), url));
     // Make sure it doesn't show up for a normal .com, it should be a host
     // not a domain cookie.
@@ -575,9 +594,27 @@ TYPED_TEST_P(CookieStoreTest, TestNonDottedAndTLD) {
     GURL url("http://b");
     EXPECT_TRUE(this->SetCookie(cs.get(), url, "a=1"));
     EXPECT_FALSE(this->SetCookie(cs.get(), url, "b=2; domain=.b"));
-    EXPECT_FALSE(this->SetCookie(cs.get(), url, "c=3; domain=b"));
     this->MatchCookieLines("a=1", this->GetCookies(cs.get(), url));
   }
+
+  {
+    // Exact matches between the domain attribute and an intranet host are
+    // treated as host cookies, not domain cookies.
+    scoped_refptr<CookieStore> cs(this->GetCookieStore());
+    GURL url("http://b/");
+    EXPECT_TRUE(this->SetCookie(cs.get(), url, "a=1; domain=b"));
+
+    this->MatchCookieLines("a=1", this->GetCookies(cs.get(), url));
+    // Make sure it doesn't show up for an intranet subdomain, it should be a
+    // host not a domain cookie.
+    this->MatchCookieLines(
+        std::string(),
+        this->GetCookies(cs.get(), GURL("http://hopefully-no-cookies.b/")));
+    if (TypeParam::supports_non_dotted_domains) {
+      this->MatchCookieLines(std::string(),
+                             this->GetCookies(cs.get(), GURL("http://.b/")));
+    }
+  }
 }
 
 // Test reading/writing cookies when the domain ends with a period,