| OLD | NEW |
| 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 Loading... |
| 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 Loading... |
| 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 Loading... |
| 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 Loading... |
| 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 } |
| OLD | NEW |