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

Unified Diff: chrome/browser/autofill/autofill_profile_unittest.cc

Issue 6650014: Autofill extend profiles to include multi-valued fields, part 2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nit Created 9 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
« no previous file with comments | « chrome/browser/autofill/autofill_profile.cc ('k') | chrome/browser/autofill/autofill_type.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/autofill_profile_unittest.cc
diff --git a/chrome/browser/autofill/autofill_profile_unittest.cc b/chrome/browser/autofill/autofill_profile_unittest.cc
index 982bcb740770db04d3cb817bf60a7e8aed59e9c0..a84453bd741c184a3fed5cf8bb4f88488cdcff6d 100644
--- a/chrome/browser/autofill/autofill_profile_unittest.cc
+++ b/chrome/browser/autofill/autofill_profile_unittest.cc
@@ -603,7 +603,7 @@ TEST(AutoFillProfileTest, AssignmentOperator){
EXPECT_TRUE(a == b);
}
-TEST(AutoFillProfileTest, Clone) {
+TEST(AutoFillProfileTest, Copy) {
AutoFillProfile a;
// Clone should be logically equal to the original.
@@ -611,8 +611,8 @@ TEST(AutoFillProfileTest, Clone) {
"marion@me.xyz", "Fox", "123 Zoo St.", "unit 5",
"Hollywood", "CA", "91601", "US", "12345678910",
"01987654321");
- scoped_ptr<AutoFillProfile> b(static_cast<AutoFillProfile*>(a.Clone()));
- EXPECT_TRUE(a == *b);
+ AutoFillProfile b(a);
+ EXPECT_TRUE(a == b);
}
TEST(AutoFillProfileTest, Compare) {
« no previous file with comments | « chrome/browser/autofill/autofill_profile.cc ('k') | chrome/browser/autofill/autofill_type.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698