Persist sessionStorage on disk.

content/browser/browser_context.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 "content/public/browser/browser_context.h"
 
 #include "content/browser/appcache/chrome_appcache_service.h"
 #include "content/browser/dom_storage/dom_storage_context_impl.h"
 #include "content/browser/download/download_file_manager.h"
 #include "content/browser/download/download_manager_impl.h"
@@ skipping 64 matching lines @@
 
   scoped_refptr<DatabaseTracker> db_tracker = new DatabaseTracker(
       context->GetPath(), context->IsOffTheRecord(),
       context->GetSpecialStoragePolicy(), quota_manager->proxy(),
       BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE));
   context->SetUserData(kDatabaseTrackerKeyName,
                        new UserDataAdapter<DatabaseTracker>(db_tracker));
 
   FilePath path = context->IsOffTheRecord() ? FilePath() : context->GetPath();
   scoped_refptr<DOMStorageContextImpl> dom_storage_context =
-      new DOMStorageContextImpl(path, context->GetSpecialStoragePolicy());
+      new DOMStorageContextImpl(path, context->GetSpecialStoragePolicy(),
+                                context->save_session_storage_on_disk());

[michaeln, 2012/07/10 19:04:13]

All of the other methods in the content::BrowsingContext class are virtual. This
getter maybe should be as well for consistency with all else? Maybe the content
interface for this should be virtual getter with a default impl that returns
'false', and a chrome specific derivation can override the method and check the
cmdline flag.

You're going to need 'content' owners approval for these changes to the
BrowsingContext class and interface. Maybe jam can give us some direction about
how he'd like to see this exposed in that interface?

[marja, 2012/07/11 12:41:31]

Added virtual ShouldSaveSessionStorageOnDisk which ProfileImpl overrides.

   context->SetUserData(
       kDOMStorageContextKeyName,
       new UserDataAdapter<DOMStorageContextImpl>(dom_storage_context));
 
   scoped_refptr<IndexedDBContext> indexed_db_context = new IndexedDBContextImpl(
       path, context->GetSpecialStoragePolicy(), quota_manager->proxy(),
       BrowserThread::GetMessageLoopProxyForThread(
           BrowserThread::WEBKIT_DEPRECATED));
   context->SetUserData(
       kIndexedDBContextKeyName,
@@ skipping 143 matching lines @@
   if (BrowserThread::IsMessageLoopValid(BrowserThread::IO)) {
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(&PurgeMemoryOnIOThread,
                    browser_context->GetResourceContext()));
   }
 
   GetDOMStorageContextImpl(browser_context)->PurgeMemory();
 }
 
+BrowserContext::BrowserContext()
+    : save_session_storage_on_disk_(false) {
+}
+
 BrowserContext::~BrowserContext() {
   // These message loop checks are just to avoid leaks in unittests.
   if (GetUserData(kDatabaseTrackerKeyName) &&
       BrowserThread::IsMessageLoopValid(BrowserThread::FILE)) {
     BrowserThread::PostTask(
         BrowserThread::FILE, FROM_HERE,
         base::Bind(&webkit_database::DatabaseTracker::Shutdown,
                    GetDatabaseTracker(this)));
   }
 
   if (GetUserData(kDOMStorageContextKeyName))
     GetDOMStorageContextImpl(this)->Shutdown();
 
   if (GetUserData(kDownloadManagerKeyName))
     GetDownloadManager(this)->Shutdown();
 }
 
+void BrowserContext::SetSaveSessionStorageOnDisk(
+    bool save_session_storage_on_disk) {
+  DCHECK(!GetUserData(kQuotaManagerKeyName));
+  save_session_storage_on_disk_ = save_session_storage_on_disk;
+}
+
 }  // namespace content