Remove parsing support for SameSite=Extended.

The attribute value is still parsed for UMA metrics only.

Bug: 953995
Change-Id: If80a818c9722625a07fc25e6f35be6394c587d12
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1883291Reviewed-by: Lily Chen <chlily@chromium.org>
Commit-Queue: Lily Chen <chlily@chromium.org>
Commit-Queue: Caleb Raitto <caraitto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710015}

net/cookies/canonical_cookie_unittest.cc

@@ -143,7 +143,7 @@ TEST(CanonicalCookieTest, Create) {
   cookie = CanonicalCookie::Create(url, "A=2; SameSite=Extended", creation_time,
                                    server_time);
   ASSERT_TRUE(cookie.get());
-  EXPECT_EQ(CookieSameSite::EXTENDED_MODE, cookie->SameSite());
+  EXPECT_EQ(CookieSameSite::UNSPECIFIED, cookie->SameSite());
   cookie = CanonicalCookie::Create(url, "A=2; SameSite=None", creation_time,
                                    server_time);
   ASSERT_TRUE(cookie.get());

net/cookies/cookie_constants.cc

@@ -91,7 +91,7 @@ CookieSameSite StringToCookieSameSite(const std::string& same_site,
     samesite = CookieSameSite::STRICT_MODE;
     *samesite_string = CookieSameSiteString::kStrict;
   } else if (base::EqualsCaseInsensitiveASCII(same_site, kSameSiteExtended)) {
-    samesite = CookieSameSite::EXTENDED_MODE;
+    // Extended isn't supported anymore -- we just parse it for UMA stats.
     *samesite_string = CookieSameSiteString::kExtended;
   } else if (same_site == "") {
     *samesite_string = CookieSameSiteString::kEmptyString;

net/cookies/cookie_constants_unittest.cc

@@ -50,9 +50,9 @@ TEST(CookieConstantsTest, TestCookieSameSite) {
   EXPECT_EQ(CookieSameSite::STRICT_MODE, StringToCookieSameSite("Strict"));
   EXPECT_EQ(CookieSameSite::STRICT_MODE, StringToCookieSameSite("STRICT"));
   EXPECT_EQ(CookieSameSite::STRICT_MODE, StringToCookieSameSite("sTrIcT"));
-  EXPECT_EQ(CookieSameSite::EXTENDED_MODE, StringToCookieSameSite("extended"));
-  EXPECT_EQ(CookieSameSite::EXTENDED_MODE, StringToCookieSameSite("EXTENDED"));
-  EXPECT_EQ(CookieSameSite::EXTENDED_MODE, StringToCookieSameSite("ExtenDED"));
+  EXPECT_EQ(CookieSameSite::UNSPECIFIED, StringToCookieSameSite("extended"));
+  EXPECT_EQ(CookieSameSite::UNSPECIFIED, StringToCookieSameSite("EXTENDED"));
+  EXPECT_EQ(CookieSameSite::UNSPECIFIED, StringToCookieSameSite("ExtenDED"));
 
   // Unrecognized tokens are interpreted as UNSPECIFIED.
   const char* const bad_tokens[] = {"",          "foo",   "none ",

net/cookies/parsed_cookie_unittest.cc

@@ -526,7 +526,7 @@ TEST(ParsedCookieTest, CookieSameSiteStringEnum) {
   EXPECT_EQ(CookieSameSiteString::kNone, actual);
 
   pc.SetSameSite("Extended");
-  EXPECT_EQ(CookieSameSite::EXTENDED_MODE, pc.SameSite(&actual));
+  EXPECT_EQ(CookieSameSite::UNSPECIFIED, pc.SameSite(&actual));
   EXPECT_EQ(CookieSameSiteString::kExtended, actual);
 
   pc.SetSameSite("Bananas");