net: make CanonicalCookie's constructor private

ios/net/cookies/cookie_cache_unittest.cc

Index: ios/net/cookies/cookie_cache_unittest.cc
diff --git a/ios/net/cookies/cookie_cache_unittest.cc b/ios/net/cookies/cookie_cache_unittest.cc
index 188d0bcb5a82b10cdd7331bebd3542998fad3cea..5395ff097b0956a87e81a25af5a0f86343b8b1d4 100644
--- a/ios/net/cookies/cookie_cache_unittest.cc
+++ b/ios/net/cookies/cookie_cache_unittest.cc
@@ -17,10 +17,10 @@ namespace {
 CanonicalCookie MakeCookie(const GURL& url,
                            const std::string& name,
                            const std::string& value) {
-  return CanonicalCookie(url, name, value, url.host(), url.path(), base::Time(),
-                         base::Time(), base::Time(), false, false,
-                         net::CookieSameSite::DEFAULT_MODE,
-                         net::COOKIE_PRIORITY_DEFAULT);
+  return *CanonicalCookie::Create(url, name, value, url.host(), url.path(),

[mmenke, 2016/07/20 20:50:11]

url.host() should be std::string(), to maintain behavior.

[tfarina, 2016/07/20 21:58:47]

Done.

+                                  base::Time(), base::Time(), false, false,
+                                  net::CookieSameSite::DEFAULT_MODE, false,
+                                  net::COOKIE_PRIORITY_DEFAULT);
 }
 
 }  // namespace