Update cookie value and attribute setting behavior to match other UAs

chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc

Index: chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc b/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
index d8c9decf6f073f28a31d02a25ff7fa9392df90a3..d399fd769e9f08a6d67a13f63b431849b4398f69 100644
--- a/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc
@@ -286,10 +286,9 @@ TEST_F(BrowsingDataCookieHelperTest, CannedDomainCookie) {
   helper->AddChangedCookie(origin, origin, "A=1; Domain=.www.google.com",
                            net::CookieOptions());
   // Try adding invalid cookies that will be ignored.
-  helper->AddChangedCookie(origin, origin, std::string(), net::CookieOptions());
-  helper->AddChangedCookie(origin,
-                           origin,
-                           "C=bad guy; Domain=wrongdomain.com",
+  helper->AddChangedCookie(origin, origin, "C=not http; HttpOnly",

[mmenke, 2016/08/17 16:23:17]

Why is this an invalid cookie?  I'm not seeing why these are treated as non-HTTP
cookies.

[jww, 2016/08/17 16:57:08]

To accept HTTP-only cookies, you have to set a special net::CookieOption, which
by default is set to reject HTTP-only cookies. So in this case, we didn't set
that option (see like 285), so we expect HttpOnly cookies to be rejected.

+                           net::CookieOptions());
+  helper->AddChangedCookie(origin, origin, "C=bad guy; Domain=wrongdomain.com",
                            net::CookieOptions());
 
   helper->StartFetching(