Update cookie expiry policy to attempt to avoid deleting "in-use" cookies. S...

chrome/common/net/cookie_monster_sqlite.cc

 // Copyright (c) 2006-2008 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 "chrome/common/net/cookie_monster_sqlite.h"
 
 #include <list>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
@@ skipping 288 matching lines @@
   // so we want those people to get it. Ignore errors, since it may exist.
   sqlite3_exec(db, "CREATE INDEX cookie_times ON cookies (creation_utc)",
                NULL, NULL, NULL);
 
   return true;
 }
 
 }  // namespace
 
 bool SQLitePersistentCookieStore::Load(
-    std::vector<net::CookieMonster::KeyedCanonicalCookie>* cookies) {
+    std::vector<net::CookieMonster::KeyedCanonicalCookie>* cookies,
+    base::Time* least_recent_access) {
   DCHECK(!path_.empty());
   sqlite3* db;
   if (sqlite3_open(WideToUTF8(path_).c_str(), &db) != SQLITE_OK) {
     NOTREACHED() << "Unable to open cookie DB.";
     return false;
   }
 
   if (!EnsureDatabaseVersion(db) || !InitTable(db)) {
     NOTREACHED() << "Unable to initialize cookie DB.";
     sqlite3_close(db);
@@ skipping 22 matching lines @@
             smt.column_int(6) != 0,                          // secure
             smt.column_int(7) != 0,                          // httponly
             Time::FromInternalValue(smt.column_int64(0)),    // creation_utc
             Time::FromInternalValue(smt.column_int64(8)),    // last_access_utc
             true,                                            // has_expires
             Time::FromInternalValue(smt.column_int64(5))));  // expires_utc
     // Memory allocation failed.
     if (!cc.get())
       break;
 
+    const base::Time last_access = cc->LastAccessDate();
+    if (least_recent_access->is_null() || (last_access < *least_recent_access))
+      *least_recent_access = last_access;
+
     DLOG_IF(WARNING,
             cc->CreationDate() > Time::Now()) << L"CreationDate too recent";
     cookies->push_back(
         net::CookieMonster::KeyedCanonicalCookie(smt.column_string(1),
                                                  cc.release()));
   }
 
   // Create the backend, this will take ownership of the db pointer.
   backend_ = new Backend(db, background_loop_);
 
@@ skipping 52 matching lines @@
     const net::CookieMonster::CanonicalCookie& cc) {
   if (backend_.get())
     backend_->UpdateCookieAccessTime(cc);
 }
 
 void SQLitePersistentCookieStore::DeleteCookie(
     const net::CookieMonster::CanonicalCookie& cc) {
   if (backend_.get())
     backend_->DeleteCookie(cc);
 }