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

Unified Diff: chrome/browser/browsing_data_remover.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/browsing_data_remover.cc
===================================================================
--- chrome/browser/browsing_data_remover.cc (revision 126737)
+++ chrome/browser/browsing_data_remover.cc (working copy)
@@ -303,11 +303,8 @@
}
if (remove_mask & REMOVE_LOCAL_STORAGE) {
- DOMStorageContext* context = BrowserContext::GetDOMStorageContext(profile_);
- context->task_runner()->PostTask(
- FROM_HERE,
- base::Bind(&BrowsingDataRemover::ClearDOMStorageInSequencedTask,
- base::Unretained(this), make_scoped_refptr(context)));
+ BrowserContext::GetDOMStorageContext(profile_)->DeleteDataModifiedSince(
+ delete_begin_);
}
if (remove_mask & REMOVE_INDEXEDDB || remove_mask & REMOVE_WEBSQL ||
@@ -436,13 +433,6 @@
return delete_begin_time - diff;
}
-void BrowsingDataRemover::ClearDOMStorageInSequencedTask(
- DOMStorageContext* dom_storage_context) {
- // We assume the end time is now.
- DCHECK(dom_storage_context->task_runner()->RunsTasksOnCurrentThread());
- dom_storage_context->DeleteDataModifiedSince(delete_begin_);
-}
-
void BrowsingDataRemover::Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {

Powered by Google App Engine
This is Rietveld 408576698