net: migrate CreateMonsterFromStoreForGC() to CanonicalCookies::Create method - WIP

net/cookies/cookie_monster_store_test.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/cookies/cookie_monster_store_test.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/memory/ptr_util.h"
 #include "base/single_thread_task_runner.h"
@@ skipping 229 matching lines @@
       num_old_cookies = num_old_secure_cookies;
       secure = true;
     } else {
       base = num_secure_cookies;
       num_old_cookies = num_old_non_secure_cookies;
       secure = false;
     }
     base::Time creation_time =
         past_creation + base::TimeDelta::FromMicroseconds(i);
     base::Time expiration_time = current + base::TimeDelta::FromDays(30);
-    base::Time last_access_time =
-        ((i - base) < num_old_cookies)
-            ? current - base::TimeDelta::FromDays(days_old)
-            : current;
 
-    CanonicalCookie cc(GURL(), "a", "1", base::StringPrintf("h%05d.izzle", i),
-                       "/path", creation_time, expiration_time,
-                       last_access_time, secure, false,
-                       CookieSameSite::DEFAULT_MODE, COOKIE_PRIORITY_DEFAULT);
-    store->AddCookie(cc);
+    std::unique_ptr<CanonicalCookie> cc(CanonicalCookie::Create(
+        GURL(base::StringPrintf("http://h%05d.izzle/", i)), "a", "1",
+        std::string(), "/path", creation_time, expiration_time, secure, false,
+        CookieSameSite::DEFAULT_MODE, false, COOKIE_PRIORITY_DEFAULT));

[mmenke, 2016/06/03 12:00:06]

Think you still need to set the last access date with cc->SetLastAccessDate.

[tfarina, 2016/06/03 16:30:13]

Awesome. Thanks Matt. This fixed it.

Done.

+    store->AddCookie(*cc);
   }
 
   return base::WrapUnique(new CookieMonster(store.get(), nullptr));
 }
 
 MockSimplePersistentCookieStore::~MockSimplePersistentCookieStore() {
 }
 
 }  // namespace net