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

Unified Diff: components/autofill/browser/personal_data_manager.h

Issue 13488009: Remove application locale cache in autofill code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: 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: components/autofill/browser/personal_data_manager.h
===================================================================
--- components/autofill/browser/personal_data_manager.h (revision 192175)
+++ components/autofill/browser/personal_data_manager.h (working copy)
@@ -42,7 +42,7 @@
// specific data variant.
typedef std::pair<std::string, size_t> GUIDPair;
- PersonalDataManager();
+ explicit PersonalDataManager(const std::string& app_locale);
virtual ~PersonalDataManager();
// Kicks off asynchronous loading of profiles and credit cards.
@@ -152,6 +152,8 @@
// Also see SetProfile for more details.
virtual void Refresh();
+ const std::string& app_locale() const { return app_locale_; }
+
// Checks suitability of |profile| for adding to the user's set of profiles.
static bool IsValidLearnableProfile(const AutofillProfile& profile);
@@ -161,6 +163,7 @@
static bool MergeProfile(
const AutofillProfile& profile,
const std::vector<AutofillProfile*>& existing_profiles,
+ const std::string& app_locale,
std::vector<AutofillProfile>* merged_profiles);
protected:
@@ -261,6 +264,7 @@
ObserverList<PersonalDataManagerObserver> observers_;
private:
+ std::string app_locale_;
// For logging UMA metrics. Overridden by metrics tests.
scoped_ptr<const AutofillMetrics> metric_logger_;

Powered by Google App Engine
This is Rietveld 408576698