SameSite: Teach devtools about the new 'samesite' syntax.

content/browser/devtools/protocol/network_handler.cc

Index: content/browser/devtools/protocol/network_handler.cc
diff --git a/content/browser/devtools/protocol/network_handler.cc b/content/browser/devtools/protocol/network_handler.cc
index d18d87a6b96f36a677e387e4d8c65886c4b75ca3..2e1935f4340bfddec7aa3fd5363996ecad90a4d9 100644
--- a/content/browser/devtools/protocol/network_handler.cc
+++ b/content/browser/devtools/protocol/network_handler.cc
@@ -233,6 +233,18 @@ void NetworkHandler::SendGetCookiesResponse(
   std::vector<scoped_refptr<Cookie>> cookies;
   for (size_t i = 0; i < cookie_list.size(); ++i) {
     const net::CanonicalCookie& cookie = cookie_list[i];
+    std::string same_site;
+    switch (cookie.SameSite()) {
+      case net::CookieSameSite::NO_RESTRICTION:
+        same_site = "NoRestriction";
+        break;
+      case net::CookieSameSite::STRICT_MODE:
+        same_site = "Strict";
+        break;
+      case net::CookieSameSite::LAX_MODE:
+        same_site = "Lax";
+        break;
+    }
     cookies.push_back(Cookie::Create()
          ->set_name(cookie.Name())
          ->set_value(cookie.Value())
@@ -242,7 +254,8 @@ void NetworkHandler::SendGetCookiesResponse(
          ->set_size(cookie.Name().length() + cookie.Value().length())
          ->set_http_only(cookie.IsHttpOnly())
          ->set_secure(cookie.IsSecure())
-         ->set_session(!cookie.IsPersistent()));
+         ->set_session(!cookie.IsPersistent())
+         ->set_same_site(same_site));

[pfeldman, 2016/04/01 22:04:08]

What is the value in case SameSite is not specified? Can we make this protocol
parameter optional?

[Mike West, 2016/04/05 08:07:08]

Done.

   }
   client_->SendGetCookiesResponse(command_id,
       GetCookiesResponse::Create()->set_cookies(cookies));