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

Unified Diff: components/autofill/core/browser/autofill_profile_unittest.cc

Issue 2164143002: Use the max use count on autofill profile merge. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: better documentation Created 4 years, 5 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/autofill_profile_unittest.cc
diff --git a/components/autofill/core/browser/autofill_profile_unittest.cc b/components/autofill/core/browser/autofill_profile_unittest.cc
index b3c1c8cc8f3b857a5ee5ef7491964ab67423c9ba..f367f87413b4c29a5f49827a414fbb3bca46f630 100644
--- a/components/autofill/core/browser/autofill_profile_unittest.cc
+++ b/components/autofill/core/browser/autofill_profile_unittest.cc
@@ -880,7 +880,7 @@ TEST(AutofillProfileTest, MergeDataFrom_SameProfile) {
AutofillProfile b = a;
b.set_guid(base::GenerateGUID());
EXPECT_TRUE(a.MergeDataFrom(b, "en-US"));
- EXPECT_EQ(2u, a.use_count());
+ EXPECT_EQ(1u, a.use_count());
// Now the profile is fully populated. Merging it again has no effect (except
// for usage statistics).
@@ -888,7 +888,7 @@ TEST(AutofillProfileTest, MergeDataFrom_SameProfile) {
c.set_guid(base::GenerateGUID());
c.set_use_count(3);
EXPECT_FALSE(a.MergeDataFrom(c, "en-US"));
- EXPECT_EQ(5u, a.use_count());
+ EXPECT_EQ(3u, a.use_count());
}
TEST(AutofillProfileTest, OverwriteName_AddNameFull) {

Powered by Google App Engine
This is Rietveld 408576698