Ignore cookies with invalid 'SameSite' attribute values.

chrome/browser/extensions/api/cookies/cookies_helpers.cc

Index: chrome/browser/extensions/api/cookies/cookies_helpers.cc
diff --git a/chrome/browser/extensions/api/cookies/cookies_helpers.cc b/chrome/browser/extensions/api/cookies/cookies_helpers.cc
index 350a5b94a1dd2ee08bf9400125ef1a0a064f4b7f..c18669ee4b061072be13187c8d0ef1b833dedc79 100644
--- a/chrome/browser/extensions/api/cookies/cookies_helpers.cc
+++ b/chrome/browser/extensions/api/cookies/cookies_helpers.cc
@@ -91,6 +91,10 @@ Cookie CreateCookie(const net::CanonicalCookie& canonical_cookie,
   case net::CookieSameSite::STRICT_MODE:
     cookie.same_site = api::cookies::SAME_SITE_STATUS_STRICT;
     break;
+  case net::CookieSameSite::INVALID_MODE:

[mmenke, 2016/04/06 15:16:30]

It seems unfortunate that we need to expose this new invalid SameSite state as
part of the API.  I don't suppose we could simple reflect this error as part of
the ParsedCookie.IsValid, and not add a value in the enum?

+    NOTREACHED();
+    cookie.same_site = api::cookies::SAME_SITE_STATUS_STRICT;
+    break;
   }
 
   cookie.session = !canonical_cookie.IsPersistent();