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

Unified Diff: chrome/browser/browsing_data_local_storage_helper.h

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_local_storage_helper.h
===================================================================
--- chrome/browser/browsing_data_local_storage_helper.h (revision 126737)
+++ chrome/browser/browsing_data_local_storage_helper.h (working copy)
@@ -84,7 +84,8 @@
// Notifies the completion callback in the UI thread.
void NotifyInUIThread();
- scoped_refptr<content::DOMStorageContext> dom_storage_context_;
+ // Owned by the profile
+ content::DOMStorageContext* dom_storage_context_;
// This only mutates on the UI thread.
base::Callback<void(const std::list<LocalStorageInfo>&)> completion_callback_;
@@ -99,10 +100,10 @@
std::list<LocalStorageInfo> local_storage_info_;
private:
- // Enumerates all local storage files in a sequenced task.
- void FetchLocalStorageInfoHelper();
- // Delete a single local storage file in a sequenced task.
- void DeleteLocalStorageFileHelper(const FilePath& file_path);
+ // Called back with the all the local storage files.
+ void GetAllStorageFilesCallback(const std::vector<FilePath>& files);
+ // Get the file info on the file thread.
+ void FetchLocalStorageInfo(const std::vector<FilePath>& files);
DISALLOW_COPY_AND_ASSIGN(BrowsingDataLocalStorageHelper);
};

Powered by Google App Engine
This is Rietveld 408576698