ContentAPI change - Post DomStorage tasks via a SequencedTaskRunner instead of directly to WEBKIT_DE

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/fileapi/browser_file_system_helper.h"
 #include "content/browser/in_process_webkit/dom_storage_context_impl.h"
 #include "content/browser/in_process_webkit/indexed_db_context_impl.h"
@@ skipping 230 matching lines @@
 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) &&
-      BrowserThread::IsMessageLoopValid(BrowserThread::WEBKIT_DEPRECATED)) {
-    DOMStorageContext* dom_storage_context =
+  if (GetUserData(kDOMStorageContextKeyName)) {
+    DOMStorageContext* context =
         (static_cast<UserDataAdapter<DOMStorageContext>*>(
             GetUserData(kDOMStorageContextKeyName)))->release();
-    BrowserThread::ReleaseSoon(
-        BrowserThread::WEBKIT_DEPRECATED, FROM_HERE, dom_storage_context);
+    context->task_runner()->ReleaseSoon(FROM_HERE, context);

[michaeln, 2012/03/14 05:17:26]

to make the browser/ui/unittest gods happy, may have to call Release() if
ReleaseSoon returns false?

[michaeln, 2012/03/14 05:18:59]

or maybe all relevant tests have been marked flaky-->disabled so it's all swept
under the rugs already

   }
 }
 
 }  // namespace content