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

Side by Side Diff: chrome/browser/bookmarks/bookmark_codec_unittest.cc

Issue 184663002: Omnibox: Make URLs of Bookmarks Searchable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix ALL_MATCHES (in response to recent changes) 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/bookmarks/DEPS ('k') | chrome/browser/bookmarks/bookmark_index.h » ('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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/bookmarks/bookmark_codec.h" 5 #include "chrome/browser/bookmarks/bookmark_codec.h"
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/json/json_file_value_serializer.h" 9 #include "base/json/json_file_value_serializer.h"
10 #include "base/json/json_string_value_serializer.h" 10 #include "base/json/json_string_value_serializer.h"
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 ASSERT_NO_FATAL_FAILURE( 66 ASSERT_NO_FATAL_FAILURE(
67 AssertNodesEqual(expected->mobile_node(), actual->mobile_node())); 67 AssertNodesEqual(expected->mobile_node(), actual->mobile_node()));
68 } 68 }
69 69
70 } // namespace 70 } // namespace
71 71
72 class BookmarkCodecTest : public testing::Test { 72 class BookmarkCodecTest : public testing::Test {
73 protected: 73 protected:
74 // Helpers to create bookmark models with different data. 74 // Helpers to create bookmark models with different data.
75 BookmarkModel* CreateTestModel1() { 75 BookmarkModel* CreateTestModel1() {
76 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL)); 76 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL, false));
77 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); 77 const BookmarkNode* bookmark_bar = model->bookmark_bar_node();
78 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); 78 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url));
79 return model.release(); 79 return model.release();
80 } 80 }
81 BookmarkModel* CreateTestModel2() { 81 BookmarkModel* CreateTestModel2() {
82 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL)); 82 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL, false));
83 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); 83 const BookmarkNode* bookmark_bar = model->bookmark_bar_node();
84 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); 84 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url));
85 model->AddURL(bookmark_bar, 1, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); 85 model->AddURL(bookmark_bar, 1, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url));
86 return model.release(); 86 return model.release();
87 } 87 }
88 BookmarkModel* CreateTestModel3() { 88 BookmarkModel* CreateTestModel3() {
89 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL)); 89 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL, false));
90 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); 90 const BookmarkNode* bookmark_bar = model->bookmark_bar_node();
91 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); 91 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url));
92 const BookmarkNode* folder1 = model->AddFolder(bookmark_bar, 1, 92 const BookmarkNode* folder1 = model->AddFolder(bookmark_bar, 1,
93 ASCIIToUTF16(kFolder1Title)); 93 ASCIIToUTF16(kFolder1Title));
94 model->AddURL(folder1, 0, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); 94 model->AddURL(folder1, 0, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url));
95 return model.release(); 95 return model.release();
96 } 96 }
97 97
98 void GetBookmarksBarChildValue(base::Value* value, 98 void GetBookmarksBarChildValue(base::Value* value,
99 size_t index, 99 size_t index,
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 166
167 BookmarkModel* DecodeHelper(const base::Value& value, 167 BookmarkModel* DecodeHelper(const base::Value& value,
168 const std::string& expected_stored_checksum, 168 const std::string& expected_stored_checksum,
169 std::string* computed_checksum, 169 std::string* computed_checksum,
170 bool expected_changes) { 170 bool expected_changes) {
171 BookmarkCodec decoder; 171 BookmarkCodec decoder;
172 // Computed and stored checksums should be empty. 172 // Computed and stored checksums should be empty.
173 EXPECT_EQ("", decoder.computed_checksum()); 173 EXPECT_EQ("", decoder.computed_checksum());
174 EXPECT_EQ("", decoder.stored_checksum()); 174 EXPECT_EQ("", decoder.stored_checksum());
175 175
176 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL)); 176 scoped_ptr<BookmarkModel> model(new BookmarkModel(NULL, false));
177 EXPECT_TRUE(Decode(&decoder, model.get(), value)); 177 EXPECT_TRUE(Decode(&decoder, model.get(), value));
178 178
179 *computed_checksum = decoder.computed_checksum(); 179 *computed_checksum = decoder.computed_checksum();
180 const std::string& stored_checksum = decoder.stored_checksum(); 180 const std::string& stored_checksum = decoder.stored_checksum();
181 181
182 // Computed and stored checksums should not be empty. 182 // Computed and stored checksums should not be empty.
183 EXPECT_FALSE(computed_checksum->empty()); 183 EXPECT_FALSE(computed_checksum->empty());
184 EXPECT_FALSE(stored_checksum.empty()); 184 EXPECT_FALSE(stored_checksum.empty());
185 185
186 // Stored checksum should be as expected. 186 // Stored checksum should be as expected.
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
304 GURL("http://newurl2.com")); 304 GURL("http://newurl2.com"));
305 305
306 ExpectIDsUnique(decoded_model.get()); 306 ExpectIDsUnique(decoded_model.get());
307 } 307 }
308 308
309 TEST_F(BookmarkCodecTest, PersistIDsTest) { 309 TEST_F(BookmarkCodecTest, PersistIDsTest) {
310 scoped_ptr<BookmarkModel> model_to_encode(CreateTestModel3()); 310 scoped_ptr<BookmarkModel> model_to_encode(CreateTestModel3());
311 BookmarkCodec encoder; 311 BookmarkCodec encoder;
312 scoped_ptr<base::Value> model_value(encoder.Encode(model_to_encode.get())); 312 scoped_ptr<base::Value> model_value(encoder.Encode(model_to_encode.get()));
313 313
314 BookmarkModel decoded_model(NULL); 314 BookmarkModel decoded_model(NULL, false);
315 BookmarkCodec decoder; 315 BookmarkCodec decoder;
316 ASSERT_TRUE(Decode(&decoder, &decoded_model, *model_value.get())); 316 ASSERT_TRUE(Decode(&decoder, &decoded_model, *model_value.get()));
317 ASSERT_NO_FATAL_FAILURE( 317 ASSERT_NO_FATAL_FAILURE(
318 AssertModelsEqual(model_to_encode.get(), &decoded_model)); 318 AssertModelsEqual(model_to_encode.get(), &decoded_model));
319 319
320 // Add a couple of more items to the decoded bookmark model and make sure 320 // Add a couple of more items to the decoded bookmark model and make sure
321 // ID persistence is working properly. 321 // ID persistence is working properly.
322 const BookmarkNode* bookmark_bar = decoded_model.bookmark_bar_node(); 322 const BookmarkNode* bookmark_bar = decoded_model.bookmark_bar_node();
323 decoded_model.AddURL( 323 decoded_model.AddURL(
324 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kUrl3Title), 324 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kUrl3Title),
325 GURL(kUrl3Url)); 325 GURL(kUrl3Url));
326 const BookmarkNode* folder2_node = decoded_model.AddFolder( 326 const BookmarkNode* folder2_node = decoded_model.AddFolder(
327 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kFolder2Title)); 327 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kFolder2Title));
328 decoded_model.AddURL(folder2_node, 0, ASCIIToUTF16(kUrl4Title), 328 decoded_model.AddURL(folder2_node, 0, ASCIIToUTF16(kUrl4Title),
329 GURL(kUrl4Url)); 329 GURL(kUrl4Url));
330 330
331 BookmarkCodec encoder2; 331 BookmarkCodec encoder2;
332 scoped_ptr<base::Value> model_value2(encoder2.Encode(&decoded_model)); 332 scoped_ptr<base::Value> model_value2(encoder2.Encode(&decoded_model));
333 333
334 BookmarkModel decoded_model2(NULL); 334 BookmarkModel decoded_model2(NULL, false);
335 BookmarkCodec decoder2; 335 BookmarkCodec decoder2;
336 ASSERT_TRUE(Decode(&decoder2, &decoded_model2, *model_value2.get())); 336 ASSERT_TRUE(Decode(&decoder2, &decoded_model2, *model_value2.get()));
337 ASSERT_NO_FATAL_FAILURE(AssertModelsEqual(&decoded_model, &decoded_model2)); 337 ASSERT_NO_FATAL_FAILURE(AssertModelsEqual(&decoded_model, &decoded_model2));
338 } 338 }
339 339
340 TEST_F(BookmarkCodecTest, CanDecodeModelWithoutMobileBookmarks) { 340 TEST_F(BookmarkCodecTest, CanDecodeModelWithoutMobileBookmarks) {
341 base::FilePath test_data_directory; 341 base::FilePath test_data_directory;
342 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_directory)); 342 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_directory));
343 base::FilePath test_file = test_data_directory.AppendASCII( 343 base::FilePath test_file = test_data_directory.AppendASCII(
344 "bookmarks/model_without_sync.json"); 344 "bookmarks/model_without_sync.json");
345 ASSERT_TRUE(base::PathExists(test_file)); 345 ASSERT_TRUE(base::PathExists(test_file));
346 346
347 JSONFileValueSerializer serializer(test_file); 347 JSONFileValueSerializer serializer(test_file);
348 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); 348 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL));
349 349
350 BookmarkModel decoded_model(NULL); 350 BookmarkModel decoded_model(NULL, false);
351 BookmarkCodec decoder; 351 BookmarkCodec decoder;
352 ASSERT_TRUE(Decode(&decoder, &decoded_model, *root.get())); 352 ASSERT_TRUE(Decode(&decoder, &decoded_model, *root.get()));
353 ExpectIDsUnique(&decoded_model); 353 ExpectIDsUnique(&decoded_model);
354 354
355 const BookmarkNode* bbn = decoded_model.bookmark_bar_node(); 355 const BookmarkNode* bbn = decoded_model.bookmark_bar_node();
356 ASSERT_EQ(1, bbn->child_count()); 356 ASSERT_EQ(1, bbn->child_count());
357 357
358 const BookmarkNode* child = bbn->GetChild(0); 358 const BookmarkNode* child = bbn->GetChild(0);
359 EXPECT_EQ(BookmarkNode::FOLDER, child->type()); 359 EXPECT_EQ(BookmarkNode::FOLDER, child->type());
360 EXPECT_EQ(ASCIIToUTF16("Folder A"), child->GetTitle()); 360 EXPECT_EQ(ASCIIToUTF16("Folder A"), child->GetTitle());
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
428 TEST_F(BookmarkCodecTest, CanDecodeMetaInfoAsString) { 428 TEST_F(BookmarkCodecTest, CanDecodeMetaInfoAsString) {
429 base::FilePath test_data_directory; 429 base::FilePath test_data_directory;
430 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_directory)); 430 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_directory));
431 base::FilePath test_file = test_data_directory.AppendASCII( 431 base::FilePath test_file = test_data_directory.AppendASCII(
432 "bookmarks/meta_info_as_string.json"); 432 "bookmarks/meta_info_as_string.json");
433 ASSERT_TRUE(base::PathExists(test_file)); 433 ASSERT_TRUE(base::PathExists(test_file));
434 434
435 JSONFileValueSerializer serializer(test_file); 435 JSONFileValueSerializer serializer(test_file);
436 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); 436 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL));
437 437
438 BookmarkModel model(NULL); 438 BookmarkModel model(NULL, false);
439 BookmarkCodec decoder; 439 BookmarkCodec decoder;
440 ASSERT_TRUE(Decode(&decoder, &model, *root.get())); 440 ASSERT_TRUE(Decode(&decoder, &model, *root.get()));
441 441
442 EXPECT_EQ(1, model.root_node()->sync_transaction_version()); 442 EXPECT_EQ(1, model.root_node()->sync_transaction_version());
443 const BookmarkNode* bbn = model.bookmark_bar_node(); 443 const BookmarkNode* bbn = model.bookmark_bar_node();
444 EXPECT_EQ(BookmarkNode::kInvalidSyncTransactionVersion, 444 EXPECT_EQ(BookmarkNode::kInvalidSyncTransactionVersion,
445 bbn->GetChild(0)->sync_transaction_version()); 445 bbn->GetChild(0)->sync_transaction_version());
446 EXPECT_EQ(42, bbn->GetChild(1)->sync_transaction_version()); 446 EXPECT_EQ(42, bbn->GetChild(1)->sync_transaction_version());
447 447
448 const char kSyncTransactionVersionKey[] = "sync.transaction_version"; 448 const char kSyncTransactionVersionKey[] = "sync.transaction_version";
449 const char kNormalKey[] = "key"; 449 const char kNormalKey[] = "key";
450 const char kNestedKey[] = "nested.key"; 450 const char kNestedKey[] = "nested.key";
451 std::string meta_value; 451 std::string meta_value;
452 EXPECT_FALSE(model.root_node()->GetMetaInfo(kSyncTransactionVersionKey, 452 EXPECT_FALSE(model.root_node()->GetMetaInfo(kSyncTransactionVersionKey,
453 &meta_value)); 453 &meta_value));
454 EXPECT_FALSE(bbn->GetChild(1)->GetMetaInfo(kSyncTransactionVersionKey, 454 EXPECT_FALSE(bbn->GetChild(1)->GetMetaInfo(kSyncTransactionVersionKey,
455 &meta_value)); 455 &meta_value));
456 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNormalKey, &meta_value)); 456 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNormalKey, &meta_value));
457 EXPECT_EQ("value", meta_value); 457 EXPECT_EQ("value", meta_value);
458 EXPECT_TRUE(bbn->GetChild(1)->GetMetaInfo(kNormalKey, &meta_value)); 458 EXPECT_TRUE(bbn->GetChild(1)->GetMetaInfo(kNormalKey, &meta_value));
459 EXPECT_EQ("value2", meta_value); 459 EXPECT_EQ("value2", meta_value);
460 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNestedKey, &meta_value)); 460 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNestedKey, &meta_value));
461 EXPECT_EQ("value3", meta_value); 461 EXPECT_EQ("value3", meta_value);
462 } 462 }
OLDNEW
« no previous file with comments | « chrome/browser/bookmarks/DEPS ('k') | chrome/browser/bookmarks/bookmark_index.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698