Unwire the clear on exit preference from the storage systems.

chrome/browser/net/sqlite_persistent_cookie_store.h

 // 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.
 
 // A sqlite implementation of a cookie monster persistent store.
 
 #ifndef CHROME_BROWSER_NET_SQLITE_PERSISTENT_COOKIE_STORE_H_
 #define CHROME_BROWSER_NET_SQLITE_PERSISTENT_COOKIE_STORE_H_
 #pragma once
 
@@ skipping 27 matching lines @@
   // net::CookieMonster::PersistentCookieStore:
   virtual void Load(const LoadedCallback& loaded_callback) OVERRIDE;
   virtual void LoadCookiesForKey(const std::string& key,
       const LoadedCallback& callback) OVERRIDE;
   virtual void AddCookie(
       const net::CookieMonster::CanonicalCookie& cc) OVERRIDE;
   virtual void UpdateCookieAccessTime(
       const net::CookieMonster::CanonicalCookie& cc) OVERRIDE;
   virtual void DeleteCookie(
       const net::CookieMonster::CanonicalCookie& cc) OVERRIDE;
-  virtual void SetClearLocalStateOnExit(bool clear_local_state) OVERRIDE;
+  virtual void SetKeepSessionCookies() OVERRIDE;

[marja, 2012/05/31 15:30:32]

Isn't this the same as SaveSessionState in other storage systems? Could it also
be named SaveSessionState?

[jochen (gone - plz use gerrit), 2012/05/31 15:54:41]

sure. Erik, wdyt?

[erikwright (departed), 2012/05/31 18:02:02]

I think I prefer it having the 'Set' in the name.

After all, it's not saving _anything_ now, just setting some state for later.

[jochen (gone - plz use gerrit), 2012/06/01 12:15:19]

I called it SetForceKeepSessionState

   virtual void Flush(const base::Closure& callback) OVERRIDE;
 
  protected:
    virtual ~SQLitePersistentCookieStore();
 
  private:
   class Backend;
 
   scoped_refptr<Backend> backend_;
 
   DISALLOW_COPY_AND_ASSIGN(SQLitePersistentCookieStore);
 };
 
 #endif  // CHROME_BROWSER_NET_SQLITE_PERSISTENT_COOKIE_STORE_H_