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

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

Issue 485483003: Revert of Omnibox: Make URLs of Bookmarks Searchable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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
OLDNEW
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
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
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
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
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
OLDNEW
« no previous file with comments | « chrome/test/base/testing_profile.cc ('k') | components/bookmarks/browser/bookmark_expanded_state_tracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698