Remove dangerous CanonicalCookie::Create method.

net/cookies/cookie_monster_store_test.cc

Index: net/cookies/cookie_monster_store_test.cc
diff --git a/net/cookies/cookie_monster_store_test.cc b/net/cookies/cookie_monster_store_test.cc
index 558d4bc02c3dd500a2c6b47037fb9bd2b04c2106..de1d898656c6244cd4efbb1c7100706c7a58c013 100644
--- a/net/cookies/cookie_monster_store_test.cc
+++ b/net/cookies/cookie_monster_store_test.cc
@@ -130,8 +130,8 @@ std::unique_ptr<CanonicalCookie> BuildCanonicalCookie(
                       : base::Time();
   std::string cookie_path = pc.Path();
 
-  return CanonicalCookie::Create(url, pc.Name(), pc.Value(), url.host(),
-                                 cookie_path, creation_time, cookie_expires,
+  return CanonicalCookie::Create(pc.Name(), pc.Value(), url.host(), cookie_path,

[mmenke, 2017/05/09 16:51:09]

This is a behavior change.  Should be "." + cookie_path.

[Randy Smith (Not in Mondays), 2017/05/09 23:52:35]

Presuming you meant "." + url.host().  Done.

+                                 creation_time, cookie_expires, base::Time(),
                                  pc.IsSecure(), pc.IsHttpOnly(), pc.SameSite(),
                                  pc.Priority());
 }
@@ -239,8 +239,8 @@ std::unique_ptr<CookieMonster> CreateMonsterFromStoreForGC(
     // strict secure cookies are enforced, the cookie will fail to be created if
     // |secure| is true but the URL is an insecure scheme.
     std::unique_ptr<CanonicalCookie> cc(CanonicalCookie::Create(
-        GURL(base::StringPrintf("https://h%05d.izzle/", i)), "a", "1",
-        std::string(), "/path", creation_time, expiration_time, secure, false,
+        "a", "1", base::StringPrintf("h%05d.izzle", i), "/path", creation_time,
+        expiration_time, base::Time(), secure, false,
         CookieSameSite::DEFAULT_MODE, COOKIE_PRIORITY_DEFAULT));
     cc->SetLastAccessDate(last_access_time);
     store->AddCookie(*cc);