| OLD | NEW |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 "components/bookmarks/browser/bookmark_codec.h" | 5 #include "components/bookmarks/browser/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 67 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 78 ASSERT_NO_FATAL_FAILURE( | 78 ASSERT_NO_FATAL_FAILURE( |
| 79 AssertNodesEqual(expected->mobile_node(), actual->mobile_node())); | 79 AssertNodesEqual(expected->mobile_node(), actual->mobile_node())); |
| 80 } | 80 } |
| 81 | 81 |
| 82 } // namespace | 82 } // namespace |
| 83 | 83 |
| 84 class BookmarkCodecTest : public testing::Test { | 84 class BookmarkCodecTest : public testing::Test { |
| 85 protected: | 85 protected: |
| 86 // Helpers to create bookmark models with different data. | 86 // Helpers to create bookmark models with different data. |
| 87 BookmarkModel* CreateTestModel1() { | 87 BookmarkModel* CreateTestModel1() { |
| 88 scoped_ptr<BookmarkModel> model(client_.CreateModel()); | 88 scoped_ptr<BookmarkModel> model(client_.CreateModel(false)); |
| 89 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); | 89 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); |
| 90 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); | 90 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); |
| 91 return model.release(); | 91 return model.release(); |
| 92 } | 92 } |
| 93 BookmarkModel* CreateTestModel2() { | 93 BookmarkModel* CreateTestModel2() { |
| 94 scoped_ptr<BookmarkModel> model(client_.CreateModel()); | 94 scoped_ptr<BookmarkModel> model(client_.CreateModel(false)); |
| 95 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); | 95 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); |
| 96 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); | 96 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); |
| 97 model->AddURL(bookmark_bar, 1, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); | 97 model->AddURL(bookmark_bar, 1, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); |
| 98 return model.release(); | 98 return model.release(); |
| 99 } | 99 } |
| 100 BookmarkModel* CreateTestModel3() { | 100 BookmarkModel* CreateTestModel3() { |
| 101 scoped_ptr<BookmarkModel> model(client_.CreateModel()); | 101 scoped_ptr<BookmarkModel> model(client_.CreateModel(false)); |
| 102 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); | 102 const BookmarkNode* bookmark_bar = model->bookmark_bar_node(); |
| 103 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); | 103 model->AddURL(bookmark_bar, 0, ASCIIToUTF16(kUrl1Title), GURL(kUrl1Url)); |
| 104 const BookmarkNode* folder1 = | 104 const BookmarkNode* folder1 = |
| 105 model->AddFolder(bookmark_bar, 1, ASCIIToUTF16(kFolder1Title)); | 105 model->AddFolder(bookmark_bar, 1, ASCIIToUTF16(kFolder1Title)); |
| 106 model->AddURL(folder1, 0, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); | 106 model->AddURL(folder1, 0, ASCIIToUTF16(kUrl2Title), GURL(kUrl2Url)); |
| 107 return model.release(); | 107 return model.release(); |
| 108 } | 108 } |
| 109 | 109 |
| 110 void GetBookmarksBarChildValue(base::Value* value, | 110 void GetBookmarksBarChildValue(base::Value* value, |
| 111 size_t index, | 111 size_t index, |
| (...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 178 | 178 |
| 179 BookmarkModel* DecodeHelper(const base::Value& value, | 179 BookmarkModel* DecodeHelper(const base::Value& value, |
| 180 const std::string& expected_stored_checksum, | 180 const std::string& expected_stored_checksum, |
| 181 std::string* computed_checksum, | 181 std::string* computed_checksum, |
| 182 bool expected_changes) { | 182 bool expected_changes) { |
| 183 BookmarkCodec decoder; | 183 BookmarkCodec decoder; |
| 184 // Computed and stored checksums should be empty. | 184 // Computed and stored checksums should be empty. |
| 185 EXPECT_EQ("", decoder.computed_checksum()); | 185 EXPECT_EQ("", decoder.computed_checksum()); |
| 186 EXPECT_EQ("", decoder.stored_checksum()); | 186 EXPECT_EQ("", decoder.stored_checksum()); |
| 187 | 187 |
| 188 scoped_ptr<BookmarkModel> model(client_.CreateModel()); | 188 scoped_ptr<BookmarkModel> model(client_.CreateModel(false)); |
| 189 EXPECT_TRUE(Decode(&decoder, model.get(), value)); | 189 EXPECT_TRUE(Decode(&decoder, model.get(), value)); |
| 190 | 190 |
| 191 *computed_checksum = decoder.computed_checksum(); | 191 *computed_checksum = decoder.computed_checksum(); |
| 192 const std::string& stored_checksum = decoder.stored_checksum(); | 192 const std::string& stored_checksum = decoder.stored_checksum(); |
| 193 | 193 |
| 194 // Computed and stored checksums should not be empty. | 194 // Computed and stored checksums should not be empty. |
| 195 EXPECT_FALSE(computed_checksum->empty()); | 195 EXPECT_FALSE(computed_checksum->empty()); |
| 196 EXPECT_FALSE(stored_checksum.empty()); | 196 EXPECT_FALSE(stored_checksum.empty()); |
| 197 | 197 |
| 198 // Stored checksum should be as expected. | 198 // Stored checksum should be as expected. |
| (...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 318 bb_node, 0, ASCIIToUTF16("new url2"), GURL("http://newurl2.com")); | 318 bb_node, 0, ASCIIToUTF16("new url2"), GURL("http://newurl2.com")); |
| 319 | 319 |
| 320 ExpectIDsUnique(decoded_model.get()); | 320 ExpectIDsUnique(decoded_model.get()); |
| 321 } | 321 } |
| 322 | 322 |
| 323 TEST_F(BookmarkCodecTest, PersistIDsTest) { | 323 TEST_F(BookmarkCodecTest, PersistIDsTest) { |
| 324 scoped_ptr<BookmarkModel> model_to_encode(CreateTestModel3()); | 324 scoped_ptr<BookmarkModel> model_to_encode(CreateTestModel3()); |
| 325 BookmarkCodec encoder; | 325 BookmarkCodec encoder; |
| 326 scoped_ptr<base::Value> model_value(encoder.Encode(model_to_encode.get())); | 326 scoped_ptr<base::Value> model_value(encoder.Encode(model_to_encode.get())); |
| 327 | 327 |
| 328 scoped_ptr<BookmarkModel> decoded_model(client_.CreateModel()); | 328 scoped_ptr<BookmarkModel> decoded_model(client_.CreateModel(false)); |
| 329 BookmarkCodec decoder; | 329 BookmarkCodec decoder; |
| 330 ASSERT_TRUE(Decode(&decoder, decoded_model.get(), *model_value.get())); | 330 ASSERT_TRUE(Decode(&decoder, decoded_model.get(), *model_value.get())); |
| 331 ASSERT_NO_FATAL_FAILURE( | 331 ASSERT_NO_FATAL_FAILURE( |
| 332 AssertModelsEqual(model_to_encode.get(), decoded_model.get())); | 332 AssertModelsEqual(model_to_encode.get(), decoded_model.get())); |
| 333 | 333 |
| 334 // Add a couple of more items to the decoded bookmark model and make sure | 334 // Add a couple of more items to the decoded bookmark model and make sure |
| 335 // ID persistence is working properly. | 335 // ID persistence is working properly. |
| 336 const BookmarkNode* bookmark_bar = decoded_model->bookmark_bar_node(); | 336 const BookmarkNode* bookmark_bar = decoded_model->bookmark_bar_node(); |
| 337 decoded_model->AddURL(bookmark_bar, | 337 decoded_model->AddURL(bookmark_bar, |
| 338 bookmark_bar->child_count(), | 338 bookmark_bar->child_count(), |
| 339 ASCIIToUTF16(kUrl3Title), | 339 ASCIIToUTF16(kUrl3Title), |
| 340 GURL(kUrl3Url)); | 340 GURL(kUrl3Url)); |
| 341 const BookmarkNode* folder2_node = decoded_model->AddFolder( | 341 const BookmarkNode* folder2_node = decoded_model->AddFolder( |
| 342 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kFolder2Title)); | 342 bookmark_bar, bookmark_bar->child_count(), ASCIIToUTF16(kFolder2Title)); |
| 343 decoded_model->AddURL( | 343 decoded_model->AddURL( |
| 344 folder2_node, 0, ASCIIToUTF16(kUrl4Title), GURL(kUrl4Url)); | 344 folder2_node, 0, ASCIIToUTF16(kUrl4Title), GURL(kUrl4Url)); |
| 345 | 345 |
| 346 BookmarkCodec encoder2; | 346 BookmarkCodec encoder2; |
| 347 scoped_ptr<base::Value> model_value2(encoder2.Encode(decoded_model.get())); | 347 scoped_ptr<base::Value> model_value2(encoder2.Encode(decoded_model.get())); |
| 348 | 348 |
| 349 scoped_ptr<BookmarkModel> decoded_model2(client_.CreateModel()); | 349 scoped_ptr<BookmarkModel> decoded_model2(client_.CreateModel(false)); |
| 350 BookmarkCodec decoder2; | 350 BookmarkCodec decoder2; |
| 351 ASSERT_TRUE(Decode(&decoder2, decoded_model2.get(), *model_value2.get())); | 351 ASSERT_TRUE(Decode(&decoder2, decoded_model2.get(), *model_value2.get())); |
| 352 ASSERT_NO_FATAL_FAILURE( | 352 ASSERT_NO_FATAL_FAILURE( |
| 353 AssertModelsEqual(decoded_model.get(), decoded_model2.get())); | 353 AssertModelsEqual(decoded_model.get(), decoded_model2.get())); |
| 354 } | 354 } |
| 355 | 355 |
| 356 TEST_F(BookmarkCodecTest, CanDecodeModelWithoutMobileBookmarks) { | 356 TEST_F(BookmarkCodecTest, CanDecodeModelWithoutMobileBookmarks) { |
| 357 base::FilePath test_data_directory; | 357 base::FilePath test_data_directory; |
| 358 base::FilePath test_file = | 358 base::FilePath test_file = |
| 359 GetTestDataDir().AppendASCII("bookmarks/model_without_sync.json"); | 359 GetTestDataDir().AppendASCII("bookmarks/model_without_sync.json"); |
| 360 ASSERT_TRUE(base::PathExists(test_file)); | 360 ASSERT_TRUE(base::PathExists(test_file)); |
| 361 | 361 |
| 362 JSONFileValueSerializer serializer(test_file); | 362 JSONFileValueSerializer serializer(test_file); |
| 363 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); | 363 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); |
| 364 | 364 |
| 365 scoped_ptr<BookmarkModel> decoded_model(client_.CreateModel()); | 365 scoped_ptr<BookmarkModel> decoded_model(client_.CreateModel(false)); |
| 366 BookmarkCodec decoder; | 366 BookmarkCodec decoder; |
| 367 ASSERT_TRUE(Decode(&decoder, decoded_model.get(), *root.get())); | 367 ASSERT_TRUE(Decode(&decoder, decoded_model.get(), *root.get())); |
| 368 ExpectIDsUnique(decoded_model.get()); | 368 ExpectIDsUnique(decoded_model.get()); |
| 369 | 369 |
| 370 const BookmarkNode* bbn = decoded_model->bookmark_bar_node(); | 370 const BookmarkNode* bbn = decoded_model->bookmark_bar_node(); |
| 371 ASSERT_EQ(1, bbn->child_count()); | 371 ASSERT_EQ(1, bbn->child_count()); |
| 372 | 372 |
| 373 const BookmarkNode* child = bbn->GetChild(0); | 373 const BookmarkNode* child = bbn->GetChild(0); |
| 374 EXPECT_EQ(BookmarkNode::FOLDER, child->type()); | 374 EXPECT_EQ(BookmarkNode::FOLDER, child->type()); |
| 375 EXPECT_EQ(ASCIIToUTF16("Folder A"), child->GetTitle()); | 375 EXPECT_EQ(ASCIIToUTF16("Folder A"), child->GetTitle()); |
| (...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 442 // way meta info is stored. | 442 // way meta info is stored. |
| 443 TEST_F(BookmarkCodecTest, CanDecodeMetaInfoAsString) { | 443 TEST_F(BookmarkCodecTest, CanDecodeMetaInfoAsString) { |
| 444 base::FilePath test_data_directory; | 444 base::FilePath test_data_directory; |
| 445 base::FilePath test_file = | 445 base::FilePath test_file = |
| 446 GetTestDataDir().AppendASCII("bookmarks/meta_info_as_string.json"); | 446 GetTestDataDir().AppendASCII("bookmarks/meta_info_as_string.json"); |
| 447 ASSERT_TRUE(base::PathExists(test_file)); | 447 ASSERT_TRUE(base::PathExists(test_file)); |
| 448 | 448 |
| 449 JSONFileValueSerializer serializer(test_file); | 449 JSONFileValueSerializer serializer(test_file); |
| 450 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); | 450 scoped_ptr<base::Value> root(serializer.Deserialize(NULL, NULL)); |
| 451 | 451 |
| 452 scoped_ptr<BookmarkModel> model(client_.CreateModel()); | 452 scoped_ptr<BookmarkModel> model(client_.CreateModel(false)); |
| 453 BookmarkCodec decoder; | 453 BookmarkCodec decoder; |
| 454 ASSERT_TRUE(Decode(&decoder, model.get(), *root.get())); | 454 ASSERT_TRUE(Decode(&decoder, model.get(), *root.get())); |
| 455 | 455 |
| 456 EXPECT_EQ(1, model->root_node()->sync_transaction_version()); | 456 EXPECT_EQ(1, model->root_node()->sync_transaction_version()); |
| 457 const BookmarkNode* bbn = model->bookmark_bar_node(); | 457 const BookmarkNode* bbn = model->bookmark_bar_node(); |
| 458 EXPECT_EQ(BookmarkNode::kInvalidSyncTransactionVersion, | 458 EXPECT_EQ(BookmarkNode::kInvalidSyncTransactionVersion, |
| 459 bbn->GetChild(0)->sync_transaction_version()); | 459 bbn->GetChild(0)->sync_transaction_version()); |
| 460 EXPECT_EQ(42, bbn->GetChild(1)->sync_transaction_version()); | 460 EXPECT_EQ(42, bbn->GetChild(1)->sync_transaction_version()); |
| 461 | 461 |
| 462 const char kSyncTransactionVersionKey[] = "sync.transaction_version"; | 462 const char kSyncTransactionVersionKey[] = "sync.transaction_version"; |
| 463 const char kNormalKey[] = "key"; | 463 const char kNormalKey[] = "key"; |
| 464 const char kNestedKey[] = "nested.key"; | 464 const char kNestedKey[] = "nested.key"; |
| 465 std::string meta_value; | 465 std::string meta_value; |
| 466 EXPECT_FALSE( | 466 EXPECT_FALSE( |
| 467 model->root_node()->GetMetaInfo(kSyncTransactionVersionKey, &meta_value)); | 467 model->root_node()->GetMetaInfo(kSyncTransactionVersionKey, &meta_value)); |
| 468 EXPECT_FALSE( | 468 EXPECT_FALSE( |
| 469 bbn->GetChild(1)->GetMetaInfo(kSyncTransactionVersionKey, &meta_value)); | 469 bbn->GetChild(1)->GetMetaInfo(kSyncTransactionVersionKey, &meta_value)); |
| 470 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNormalKey, &meta_value)); | 470 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNormalKey, &meta_value)); |
| 471 EXPECT_EQ("value", meta_value); | 471 EXPECT_EQ("value", meta_value); |
| 472 EXPECT_TRUE(bbn->GetChild(1)->GetMetaInfo(kNormalKey, &meta_value)); | 472 EXPECT_TRUE(bbn->GetChild(1)->GetMetaInfo(kNormalKey, &meta_value)); |
| 473 EXPECT_EQ("value2", meta_value); | 473 EXPECT_EQ("value2", meta_value); |
| 474 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNestedKey, &meta_value)); | 474 EXPECT_TRUE(bbn->GetChild(0)->GetMetaInfo(kNestedKey, &meta_value)); |
| 475 EXPECT_EQ("value3", meta_value); | 475 EXPECT_EQ("value3", meta_value); |
| 476 } | 476 } |
| 477 | 477 |
| 478 } // namespace bookmarks | 478 } // namespace bookmarks |
| OLD | NEW |