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

Unified Diff: chrome/browser/webdata/web_database_service.h

Issue 13191007: Pass app_locale directly to AutofillTable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Chrome frame tests fix Created 7 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/webdata/web_database_service.h
diff --git a/chrome/browser/webdata/web_database_service.h b/chrome/browser/webdata/web_database_service.h
index d2d70e83b115599a32edb134ae5ab98b2749ce3a..c15d12b86fe00753df6d2110828f2d0337db4668 100644
--- a/chrome/browser/webdata/web_database_service.h
+++ b/chrome/browser/webdata/web_database_service.h
@@ -50,10 +50,8 @@ class WebDatabaseService
typedef base::Callback<WebDatabase::State(WebDatabase*)> WriteTask;
typedef base::Callback<void(sql::InitStatus)> InitCallback;
- // Takes the path to the WebDatabase file, and the locale (for migration
- // purposes).
- explicit WebDatabaseService(
- const base::FilePath& path, const std::string app_locale);
+ // Takes the path to the WebDatabase file.
+ explicit WebDatabaseService(const base::FilePath& path);
// Adds |table| as a WebDatabaseTable that will participate in
// managing the database, transferring ownership. All calls to this
@@ -100,11 +98,6 @@ class WebDatabaseService
base::FilePath path_;
- // The application locale. The locale is needed for some database migrations,
- // and must be read on the UI thread. It's cached here so that we can pass it
- // to the migration code on the DB thread.
- std::string app_locale_;
-
// The primary owner is |WebDatabaseService| but is refcounted because
// PostTask on DB thread may outlive us.
scoped_refptr<WebDataServiceBackend> wds_backend_;
« no previous file with comments | « chrome/browser/webdata/web_database_migration_unittest.cc ('k') | chrome/browser/webdata/web_database_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698