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

Unified Diff: components/autofill/core/browser/webdata/autofill_profile_syncable_service.h

Issue 212873003: Store the language code for the address in autofill profile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments. Created 6 years, 8 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/core/browser/webdata/autofill_profile_syncable_service.h
diff --git a/components/autofill/core/browser/webdata/autofill_profile_syncable_service.h b/components/autofill/core/browser/webdata/autofill_profile_syncable_service.h
index a29ff4fd701894dc7826c611f92e5f47d329fd44..715b40c8316d52542f78d769d2acc1e89f8a4a35 100644
--- a/components/autofill/core/browser/webdata/autofill_profile_syncable_service.h
+++ b/components/autofill/core/browser/webdata/autofill_profile_syncable_service.h
@@ -176,8 +176,7 @@ class AutofillProfileSyncableService
// Calls merge_into->OverwriteWithOrAddTo() and then checks if the
// |merge_into| has extra data. Returns |true| if |merge_into| posseses some
- // multi-valued field values that are not in |merge_from| or if the origins
- // of the two profiles differ, false otherwise.
+ // multi-valued field values that are not in |merge_from|.
// TODO(isherman): Seems like this should return |true| if |merge_into| was
// modified at all: http://crbug.com/248440
static bool MergeProfile(const AutofillProfile& merge_from,

Powered by Google App Engine
This is Rietveld 408576698