Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Unified Diff: chrome/browser/extensions/extension_data_deleter.cc

Issue 9704048: Make the content::DOMStorageContext methods callable on the main thread and hide the threading deta… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_data_deleter.cc
===================================================================
--- chrome/browser/extensions/extension_data_deleter.cc (revision 126737)
+++ chrome/browser/extensions/extension_data_deleter.cc (working copy)
@@ -48,13 +48,8 @@
base::Bind(
&ExtensionDataDeleter::DeleteCookiesOnIOThread, deleter));
- scoped_refptr<DOMStorageContext> dom_storage_context =
- BrowserContext::GetDOMStorageContext(profile);
- dom_storage_context->task_runner()->PostTask(
- FROM_HERE,
- base::Bind(
- &ExtensionDataDeleter::DeleteLocalStorageInSequencedTask, deleter,
- dom_storage_context));
+ BrowserContext::GetDOMStorageContext(profile)->DeleteForOrigin(
+ deleter->origin_id_);
BrowserThread::PostTask(
BrowserThread::WEBKIT_DEPRECATED, FROM_HERE,
@@ -127,12 +122,6 @@
DCHECK(rv == net::OK || rv == net::ERR_IO_PENDING);
}
-void ExtensionDataDeleter::DeleteLocalStorageInSequencedTask(
- DOMStorageContext* dom_storage_context) {
- DCHECK(dom_storage_context->task_runner()->RunsTasksOnCurrentThread());
- dom_storage_context->DeleteForOrigin(origin_id_);
-}
-
void ExtensionDataDeleter::DeleteIndexedDBOnWebkitThread(
scoped_refptr<IndexedDBContext> indexed_db_context) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::WEBKIT_DEPRECATED));

Powered by Google App Engine
This is Rietveld 408576698