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

Side by Side Diff: chrome/browser/webdata/web_database_migration_unittest.cc

Issue 8687002: Cleanup: Convert ASCIIToUTF16("") to string16(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/ui/webui/flash_ui.cc ('k') | net/base/cert_database_nss_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/message_loop.h" 8 #include "base/message_loop.h"
9 #include "base/scoped_temp_dir.h" 9 #include "base/scoped_temp_dir.h"
10 #include "base/stl_util.h" 10 #include "base/stl_util.h"
(...skipping 1026 matching lines...) Expand 10 before | Expand all | Expand 10 after
1037 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6)); 1037 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6));
1038 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7)); 1038 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7));
1039 EXPECT_EQ(1297882100L, s1.ColumnInt64(8)); 1039 EXPECT_EQ(1297882100L, s1.ColumnInt64(8));
1040 1040
1041 // John P. Doe. 1041 // John P. Doe.
1042 // Gets merged during migration from 35 to 37 due to multi-valued fields. 1042 // Gets merged during migration from 35 to 37 due to multi-valued fields.
1043 1043
1044 // Dave Smith. 1044 // Dave Smith.
1045 ASSERT_TRUE(s1.Step()); 1045 ASSERT_TRUE(s1.Step());
1046 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s1.ColumnString(0)); 1046 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s1.ColumnString(0));
1047 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(1)); 1047 EXPECT_EQ(string16(), s1.ColumnString16(1));
1048 EXPECT_EQ(ASCIIToUTF16("2 Main Street"), s1.ColumnString16(2)); 1048 EXPECT_EQ(ASCIIToUTF16("2 Main Street"), s1.ColumnString16(2));
1049 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(3)); 1049 EXPECT_EQ(string16(), s1.ColumnString16(3));
1050 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4)); 1050 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4));
1051 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5)); 1051 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5));
1052 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6)); 1052 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6));
1053 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7)); 1053 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7));
1054 EXPECT_EQ(1297882100L, s1.ColumnInt64(8)); 1054 EXPECT_EQ(1297882100L, s1.ColumnInt64(8));
1055 1055
1056 // Dave Smith (Part 2). 1056 // Dave Smith (Part 2).
1057 ASSERT_TRUE(s1.Step()); 1057 ASSERT_TRUE(s1.Step());
1058 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s1.ColumnString(0)); 1058 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s1.ColumnString(0));
1059 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(1)); 1059 EXPECT_EQ(string16(), s1.ColumnString16(1));
1060 EXPECT_EQ(ASCIIToUTF16("2 Main St"), s1.ColumnString16(2)); 1060 EXPECT_EQ(ASCIIToUTF16("2 Main St"), s1.ColumnString16(2));
1061 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(3)); 1061 EXPECT_EQ(string16(), s1.ColumnString16(3));
1062 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4)); 1062 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4));
1063 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5)); 1063 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5));
1064 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6)); 1064 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6));
1065 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7)); 1065 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7));
1066 EXPECT_EQ(1297882100L, s1.ColumnInt64(8)); 1066 EXPECT_EQ(1297882100L, s1.ColumnInt64(8));
1067 1067
1068 // Alfred E Newman. 1068 // Alfred E Newman.
1069 // Gets culled during migration from 35 to 36 due to incomplete address. 1069 // Gets culled during migration from 35 to 36 due to incomplete address.
1070 1070
1071 // 3 Main St. 1071 // 3 Main St.
1072 ASSERT_TRUE(s1.Step()); 1072 ASSERT_TRUE(s1.Step());
1073 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s1.ColumnString(0)); 1073 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s1.ColumnString(0));
1074 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(1)); 1074 EXPECT_EQ(string16(), s1.ColumnString16(1));
1075 EXPECT_EQ(ASCIIToUTF16("3 Main St"), s1.ColumnString16(2)); 1075 EXPECT_EQ(ASCIIToUTF16("3 Main St"), s1.ColumnString16(2));
1076 EXPECT_EQ(ASCIIToUTF16(""), s1.ColumnString16(3)); 1076 EXPECT_EQ(string16(), s1.ColumnString16(3));
1077 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4)); 1077 EXPECT_EQ(ASCIIToUTF16("Los Altos"), s1.ColumnString16(4));
1078 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5)); 1078 EXPECT_EQ(ASCIIToUTF16("CA"), s1.ColumnString16(5));
1079 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6)); 1079 EXPECT_EQ(ASCIIToUTF16("94022"), s1.ColumnString16(6));
1080 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7)); 1080 EXPECT_EQ(ASCIIToUTF16("United States"), s1.ColumnString16(7));
1081 EXPECT_EQ(1297882100L, s1.ColumnInt64(8)); 1081 EXPECT_EQ(1297882100L, s1.ColumnInt64(8));
1082 1082
1083 // That should be all. 1083 // That should be all.
1084 EXPECT_FALSE(s1.Step()); 1084 EXPECT_FALSE(s1.Step());
1085 1085
1086 sql::Statement s2( 1086 sql::Statement s2(
1087 connection.GetUniqueStatement( 1087 connection.GetUniqueStatement(
1088 "SELECT guid, first_name, middle_name, last_name " 1088 "SELECT guid, first_name, middle_name, last_name "
1089 "FROM autofill_profile_names")); 1089 "FROM autofill_profile_names"));
1090 1090
1091 // John Doe. 1091 // John Doe.
1092 ASSERT_TRUE(s2.Step()); 1092 ASSERT_TRUE(s2.Step());
1093 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s2.ColumnString(0)); 1093 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s2.ColumnString(0));
1094 EXPECT_EQ(ASCIIToUTF16("John"), s2.ColumnString16(1)); 1094 EXPECT_EQ(ASCIIToUTF16("John"), s2.ColumnString16(1));
1095 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(2)); 1095 EXPECT_EQ(string16(), s2.ColumnString16(2));
1096 EXPECT_EQ(ASCIIToUTF16("Doe"), s2.ColumnString16(3)); 1096 EXPECT_EQ(ASCIIToUTF16("Doe"), s2.ColumnString16(3));
1097 1097
1098 // John P. Doe. Note same guid as above due to merging of multi-valued 1098 // John P. Doe. Note same guid as above due to merging of multi-valued
1099 // fields. 1099 // fields.
1100 ASSERT_TRUE(s2.Step()); 1100 ASSERT_TRUE(s2.Step());
1101 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s2.ColumnString(0)); 1101 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s2.ColumnString(0));
1102 EXPECT_EQ(ASCIIToUTF16("John"), s2.ColumnString16(1)); 1102 EXPECT_EQ(ASCIIToUTF16("John"), s2.ColumnString16(1));
1103 EXPECT_EQ(ASCIIToUTF16("P."), s2.ColumnString16(2)); 1103 EXPECT_EQ(ASCIIToUTF16("P."), s2.ColumnString16(2));
1104 EXPECT_EQ(ASCIIToUTF16("Doe"), s2.ColumnString16(3)); 1104 EXPECT_EQ(ASCIIToUTF16("Doe"), s2.ColumnString16(3));
1105 1105
1106 // Dave Smith. 1106 // Dave Smith.
1107 ASSERT_TRUE(s2.Step()); 1107 ASSERT_TRUE(s2.Step());
1108 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s2.ColumnString(0)); 1108 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s2.ColumnString(0));
1109 EXPECT_EQ(ASCIIToUTF16("Dave"), s2.ColumnString16(1)); 1109 EXPECT_EQ(ASCIIToUTF16("Dave"), s2.ColumnString16(1));
1110 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(2)); 1110 EXPECT_EQ(string16(), s2.ColumnString16(2));
1111 EXPECT_EQ(ASCIIToUTF16("Smith"), s2.ColumnString16(3)); 1111 EXPECT_EQ(ASCIIToUTF16("Smith"), s2.ColumnString16(3));
1112 1112
1113 // Dave Smith (Part 2). 1113 // Dave Smith (Part 2).
1114 ASSERT_TRUE(s2.Step()); 1114 ASSERT_TRUE(s2.Step());
1115 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s2.ColumnString(0)); 1115 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s2.ColumnString(0));
1116 EXPECT_EQ(ASCIIToUTF16("Dave"), s2.ColumnString16(1)); 1116 EXPECT_EQ(ASCIIToUTF16("Dave"), s2.ColumnString16(1));
1117 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(2)); 1117 EXPECT_EQ(string16(), s2.ColumnString16(2));
1118 EXPECT_EQ(ASCIIToUTF16("Smith"), s2.ColumnString16(3)); 1118 EXPECT_EQ(ASCIIToUTF16("Smith"), s2.ColumnString16(3));
1119 1119
1120 // Alfred E Newman. 1120 // Alfred E Newman.
1121 // Gets culled during migration from 35 to 36 due to incomplete address. 1121 // Gets culled during migration from 35 to 36 due to incomplete address.
1122 1122
1123 // 3 Main St. 1123 // 3 Main St.
1124 ASSERT_TRUE(s2.Step()); 1124 ASSERT_TRUE(s2.Step());
1125 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s2.ColumnString(0)); 1125 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s2.ColumnString(0));
1126 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(1)); 1126 EXPECT_EQ(string16(), s2.ColumnString16(1));
1127 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(2)); 1127 EXPECT_EQ(string16(), s2.ColumnString16(2));
1128 EXPECT_EQ(ASCIIToUTF16(""), s2.ColumnString16(3)); 1128 EXPECT_EQ(string16(), s2.ColumnString16(3));
1129 1129
1130 // Should be all. 1130 // Should be all.
1131 EXPECT_FALSE(s2.Step()); 1131 EXPECT_FALSE(s2.Step());
1132 1132
1133 sql::Statement s3( 1133 sql::Statement s3(
1134 connection.GetUniqueStatement( 1134 connection.GetUniqueStatement(
1135 "SELECT guid, email " 1135 "SELECT guid, email "
1136 "FROM autofill_profile_emails")); 1136 "FROM autofill_profile_emails"));
1137 1137
1138 // John Doe. 1138 // John Doe.
1139 ASSERT_TRUE(s3.Step()); 1139 ASSERT_TRUE(s3.Step());
1140 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s3.ColumnString(0)); 1140 EXPECT_EQ("00580526-FF81-EE2A-0546-1AC593A32E2F", s3.ColumnString(0));
1141 EXPECT_EQ(ASCIIToUTF16("john@doe.com"), s3.ColumnString16(1)); 1141 EXPECT_EQ(ASCIIToUTF16("john@doe.com"), s3.ColumnString16(1));
1142 1142
1143 // John P. Doe. 1143 // John P. Doe.
1144 // Gets culled during migration from 35 to 37 due to merging of John Doe and 1144 // Gets culled during migration from 35 to 37 due to merging of John Doe and
1145 // John P. Doe addresses. 1145 // John P. Doe addresses.
1146 1146
1147 // 2 Main Street. 1147 // 2 Main Street.
1148 ASSERT_TRUE(s3.Step()); 1148 ASSERT_TRUE(s3.Step());
1149 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s3.ColumnString(0)); 1149 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s3.ColumnString(0));
1150 EXPECT_EQ(ASCIIToUTF16(""), s3.ColumnString16(1)); 1150 EXPECT_EQ(string16(), s3.ColumnString16(1));
1151 1151
1152 // 2 Main St. 1152 // 2 Main St.
1153 ASSERT_TRUE(s3.Step()); 1153 ASSERT_TRUE(s3.Step());
1154 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s3.ColumnString(0)); 1154 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s3.ColumnString(0));
1155 EXPECT_EQ(ASCIIToUTF16(""), s3.ColumnString16(1)); 1155 EXPECT_EQ(string16(), s3.ColumnString16(1));
1156 1156
1157 // Alfred E Newman. 1157 // Alfred E Newman.
1158 // Gets culled during migration from 35 to 36 due to incomplete address. 1158 // Gets culled during migration from 35 to 36 due to incomplete address.
1159 1159
1160 // 3 Main St. 1160 // 3 Main St.
1161 ASSERT_TRUE(s3.Step()); 1161 ASSERT_TRUE(s3.Step());
1162 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s3.ColumnString(0)); 1162 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s3.ColumnString(0));
1163 EXPECT_EQ(ASCIIToUTF16(""), s3.ColumnString16(1)); 1163 EXPECT_EQ(string16(), s3.ColumnString16(1));
1164 1164
1165 // Should be all. 1165 // Should be all.
1166 EXPECT_FALSE(s3.Step()); 1166 EXPECT_FALSE(s3.Step());
1167 1167
1168 sql::Statement s4( 1168 sql::Statement s4(
1169 connection.GetUniqueStatement( 1169 connection.GetUniqueStatement(
1170 "SELECT guid, type, number " 1170 "SELECT guid, type, number "
1171 "FROM autofill_profile_phones")); 1171 "FROM autofill_profile_phones"));
1172 1172
1173 // John Doe phone. 1173 // John Doe phone.
(...skipping 10 matching lines...) Expand all
1184 // John P. Doe addresses. 1184 // John P. Doe addresses.
1185 1185
1186 // John P. Doe fax. 1186 // John P. Doe fax.
1187 // Gets culled during migration from 35 to 37 due to merging of John Doe and 1187 // Gets culled during migration from 35 to 37 due to merging of John Doe and
1188 // John P. Doe addresses. 1188 // John P. Doe addresses.
1189 1189
1190 // 2 Main Street phone. 1190 // 2 Main Street phone.
1191 ASSERT_TRUE(s4.Step()); 1191 ASSERT_TRUE(s4.Step());
1192 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s4.ColumnString(0)); 1192 EXPECT_EQ("4C74A9D8-7EEE-423E-F9C2-E7FA70ED1396", s4.ColumnString(0));
1193 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone. 1193 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone.
1194 EXPECT_EQ(ASCIIToUTF16(""), s4.ColumnString16(2)); 1194 EXPECT_EQ(string16(), s4.ColumnString16(2));
1195 1195
1196 // 2 Main Street fax. 1196 // 2 Main Street fax.
1197 // Gets culled after fax type removed. 1197 // Gets culled after fax type removed.
1198 1198
1199 // 2 Main St phone. 1199 // 2 Main St phone.
1200 ASSERT_TRUE(s4.Step()); 1200 ASSERT_TRUE(s4.Step());
1201 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s4.ColumnString(0)); 1201 EXPECT_EQ("722DF5C4-F74A-294A-46F0-31FFDED0D635", s4.ColumnString(0));
1202 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone. 1202 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone.
1203 EXPECT_EQ(ASCIIToUTF16(""), s4.ColumnString16(2)); 1203 EXPECT_EQ(string16(), s4.ColumnString16(2));
1204 1204
1205 // 2 Main St fax. 1205 // 2 Main St fax.
1206 // Gets culled after fax type removed. 1206 // Gets culled after fax type removed.
1207 1207
1208 // Note no phone or fax for Alfred E Newman. 1208 // Note no phone or fax for Alfred E Newman.
1209 1209
1210 // 3 Main St phone. 1210 // 3 Main St phone.
1211 ASSERT_TRUE(s4.Step()); 1211 ASSERT_TRUE(s4.Step());
1212 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s4.ColumnString(0)); 1212 EXPECT_EQ("9E5FE298-62C7-83DF-6293-381BC589183F", s4.ColumnString(0));
1213 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone. 1213 EXPECT_EQ(0, s4.ColumnInt(1)); // 0 means phone.
1214 EXPECT_EQ(ASCIIToUTF16(""), s4.ColumnString16(2)); 1214 EXPECT_EQ(string16(), s4.ColumnString16(2));
1215 1215
1216 // 2 Main St fax. 1216 // 2 Main St fax.
1217 // Gets culled after fax type removed. 1217 // Gets culled after fax type removed.
1218 1218
1219 // Should be all. 1219 // Should be all.
1220 EXPECT_FALSE(s4.Step()); 1220 EXPECT_FALSE(s4.Step());
1221 } 1221 }
1222 } 1222 }
1223 1223
1224 // Adds a column for the autofill profile's country code. 1224 // Adds a column for the autofill profile's country code.
(...skipping 423 matching lines...) Expand 10 before | Expand all | Expand 10 after
1648 EXPECT_EQ(default_search_provider_id, default_search_provider_id_backup); 1648 EXPECT_EQ(default_search_provider_id, default_search_provider_id_backup);
1649 1649
1650 std::string default_search_provider_id_backup_signature; 1650 std::string default_search_provider_id_backup_signature;
1651 EXPECT_TRUE(meta_table.GetValue( 1651 EXPECT_TRUE(meta_table.GetValue(
1652 "Default Search Provider ID Backup Signature", 1652 "Default Search Provider ID Backup Signature",
1653 &default_search_provider_id_backup_signature)); 1653 &default_search_provider_id_backup_signature));
1654 EXPECT_FALSE(default_search_provider_id_backup_signature.empty()); 1654 EXPECT_FALSE(default_search_provider_id_backup_signature.empty());
1655 } 1655 }
1656 } 1656 }
1657 1657
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/flash_ui.cc ('k') | net/base/cert_database_nss_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698