Move IndexedDB from WEBKIT_DEPRECATED to dedicated thread

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"
 
 #if !defined(OS_IOS)
 #include "content/browser/appcache/chrome_appcache_service.h"
 #include "content/browser/dom_storage/dom_storage_context_impl.h"
 #include "content/browser/download/download_manager_impl.h"
@@ skipping 64 matching lines @@
     const scoped_refptr<net::URLRequestContextGetter>& context_getter,
     appcache::AppCacheService* appcache_service) {
   net::URLRequestContext* context = context_getter->GetURLRequestContext();
   context->cookie_store()->GetCookieMonster()->
       SetForceKeepSessionState();
   context->server_bound_cert_service()->GetCertStore()->
       SetForceKeepSessionState();
   appcache_service->set_force_keep_session_state();
 }
 
-void SaveSessionStateOnWebkitThread(
+void SaveSessionStateOnIndexedDBThread(
     scoped_refptr<IndexedDBContextImpl> indexed_db_context) {
   indexed_db_context->SetForceKeepSessionState();
 }
 
 void PurgeMemoryOnIOThread(appcache::AppCacheService* appcache_service) {
   appcache_service->PurgeMemory();
 }
 
 }  // namespace
 
@@ skipping 135 matching lines @@
             &SaveSessionStateOnIOThread,
             make_scoped_refptr(browser_context->GetRequestContext()),
             storage_partition->GetAppCacheService()));
   }
 
   DOMStorageContextImpl* dom_storage_context_impl =
       static_cast<DOMStorageContextImpl*>(
           storage_partition->GetDOMStorageContext());
   dom_storage_context_impl->SetForceKeepSessionState();
 
-  if (BrowserThread::IsMessageLoopValid(BrowserThread::WEBKIT_DEPRECATED)) {
-    IndexedDBContextImpl* indexed_db = static_cast<IndexedDBContextImpl*>(
+  IndexedDBContextImpl* indexed_db_context_impl =
+      static_cast<IndexedDBContextImpl*>(
         storage_partition->GetIndexedDBContext());
-    BrowserThread::PostTask(
-        BrowserThread::WEBKIT_DEPRECATED, FROM_HERE,
-        base::Bind(&SaveSessionStateOnWebkitThread,
-                   make_scoped_refptr(indexed_db)));
+  // No task runner in unit tests.

[jam, 2013/06/21 19:41:59]

same question as before, i thought the tests set this? or is it not all tests?

+  if (indexed_db_context_impl->TaskRunner()) {
+    indexed_db_context_impl->TaskRunner()->PostTask(
+        FROM_HERE,
+        base::Bind(&SaveSessionStateOnIndexedDBThread,
+                   make_scoped_refptr(indexed_db_context_impl)));
   }
 }
 
 void BrowserContext::PurgeMemory(BrowserContext* browser_context) {
   if (BrowserThread::IsMessageLoopValid(BrowserThread::IO)) {
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(
             &PurgeMemoryOnIOThread,
             BrowserContext::GetDefaultStoragePartition(browser_context)->
                 GetAppCacheService()));
   }
 
   ForEachStoragePartition(browser_context,
                           base::Bind(&PurgeDOMStorageContextInPartition));
 }
 
 #endif  // !OS_IOS
 
 BrowserContext::~BrowserContext() {
 #if !defined(OS_IOS)
   if (GetUserData(kDownloadManagerKeyName))
     GetDownloadManager(this)->Shutdown();
 #endif
 }
 
 }  // namespace content