net: make CanonicalCookie's constructor private

chrome/browser/android/cookies/cookies_fetcher.cc

Index: chrome/browser/android/cookies/cookies_fetcher.cc
diff --git a/chrome/browser/android/cookies/cookies_fetcher.cc b/chrome/browser/android/cookies/cookies_fetcher.cc
index e21dabc5f2e61900e2e200d51983ca4fb93aee10..8e11791f42e4cffdd5c5de9a3e5bf631153380f4 100644
--- a/chrome/browser/android/cookies/cookies_fetcher.cc
+++ b/chrome/browser/android/cookies/cookies_fetcher.cc
@@ -157,16 +157,16 @@ static void RestoreCookies(JNIEnv* env,
   scoped_refptr<net::URLRequestContextGetter> getter(
       profile->GetRequestContext());
 
-  net::CanonicalCookie cookie(
-      GURL(), base::android::ConvertJavaStringToUTF8(env, name),
+  net::CanonicalCookie cookie(*net::CanonicalCookie::Create(
+      GURL("http://example.com"), base::android::ConvertJavaStringToUTF8(env, name),

[mmenke, 2016/07/21 15:34:57]

Should just remove this GURL.  Looks like no tests are running this code, or
they'd fail.  :(

[tfarina, 2016/07/21 19:05:41]

Done.

       base::android::ConvertJavaStringToUTF8(env, value),
       base::android::ConvertJavaStringToUTF8(env, domain),
       base::android::ConvertJavaStringToUTF8(env, path),
       base::Time::FromInternalValue(creation),
       base::Time::FromInternalValue(expiration),
-      base::Time::FromInternalValue(last_access), secure, httponly,
-      static_cast<net::CookieSameSite>(same_site),
-      static_cast<net::CookiePriority>(priority));
+      secure, httponly,
+      static_cast<net::CookieSameSite>(same_site), false,
+      static_cast<net::CookiePriority>(priority)));
 
   // The rest must be done from the IO thread.
   content::BrowserThread::PostTask(