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

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

Issue 102713002: Support folders in bookmark search (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Handle single node restoration and only update once per batch delete Created 7 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
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_utils.h" 5 #include "chrome/browser/bookmarks/bookmark_utils.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 15 matching lines...) Expand all
26 } 26 }
27 27
28 private: 28 private:
29 // Clipboard requires a message loop. 29 // Clipboard requires a message loop.
30 base::MessageLoopForUI loop; 30 base::MessageLoopForUI loop;
31 }; 31 };
32 32
33 TEST_F(BookmarkUtilsTest, GetBookmarksContainingText) { 33 TEST_F(BookmarkUtilsTest, GetBookmarksContainingText) {
34 BookmarkModel model(NULL); 34 BookmarkModel model(NULL);
35 const BookmarkNode* node1 = model.AddURL(model.other_node(), 35 const BookmarkNode* node1 = model.AddURL(model.other_node(),
36 0, 36 0,
37 ASCIIToUTF16("foo bar"), 37 ASCIIToUTF16("foo bar"),
38 GURL("http://www.google.com")); 38 GURL("http://www.google.com"));
39 const BookmarkNode* node2 = model.AddURL(model.other_node(), 39 const BookmarkNode* node2 = model.AddURL(model.other_node(),
40 0, 40 0,
41 ASCIIToUTF16("baz buz"), 41 ASCIIToUTF16("baz buz"),
42 GURL("http://www.cnn.com")); 42 GURL("http://www.cnn.com"));
43 const BookmarkNode* folder1 = model.AddFolder(model.other_node(),
44 0,
45 ASCIIToUTF16("foo"));
46 std::vector<const BookmarkNode*> nodes;
43 47
44 const BookmarkNode* folder1 = model.AddFolder(model.other_node(), 48 // Ensure search matches node "foo bar" and folder "foo"
45 0,
46 ASCIIToUTF16("foo"));
47
48 std::vector<const BookmarkNode*> nodes;
49 GetBookmarksContainingText( 49 GetBookmarksContainingText(
50 &model, ASCIIToUTF16("foo"), 100, string(), &nodes); 50 &model, ASCIIToUTF16("foo"), 100, string(), &nodes);
51 ASSERT_EQ(2U, nodes.size()); 51 ASSERT_EQ(2U, nodes.size());
52 EXPECT_TRUE(nodes[0] == folder1); 52 EXPECT_TRUE(nodes[0] == folder1);
53 EXPECT_TRUE(nodes[1] == node1); 53 EXPECT_TRUE(nodes[1] == node1);
54 nodes.clear(); 54 nodes.clear();
55 55
56 GetBookmarksContainingText( 56 GetBookmarksContainingText(
57 &model, ASCIIToUTF16("cnn"), 100, string(), &nodes); 57 &model, ASCIIToUTF16("cnn"), 100, string(), &nodes);
58 ASSERT_EQ(1U, nodes.size()); 58 ASSERT_EQ(1U, nodes.size());
59 EXPECT_TRUE(nodes[0] == node2); 59 EXPECT_TRUE(nodes[0] == node2);
60 nodes.clear(); 60 nodes.clear();
61 61
62 GetBookmarksContainingText( 62 GetBookmarksContainingText(
63 &model, ASCIIToUTF16("foo bar"), 100, string(), &nodes); 63 &model, ASCIIToUTF16("foo bar"), 100, string(), &nodes);
64 ASSERT_EQ(1U, nodes.size()); 64 ASSERT_EQ(1U, nodes.size());
65 EXPECT_TRUE(nodes[0] == node1); 65 EXPECT_TRUE(nodes[0] == node1);
66 nodes.clear(); 66 nodes.clear();
67
68 // Ensure Bookmark Bar and Other Bookmarks do not return in search results
sky 2013/12/17 22:46:33 'do not return' -> 'are not returned'. And end sen
69 GetBookmarksContainingText(
70 &model, ASCIIToUTF16("Bookmark"), 100, string(), &nodes);
71 ASSERT_EQ(0U, nodes.size());
72 nodes.clear();
67 } 73 }
68 74
69 TEST_F(BookmarkUtilsTest, CopyPaste) { 75 TEST_F(BookmarkUtilsTest, CopyPaste) {
70 BookmarkModel model(NULL); 76 BookmarkModel model(NULL);
71 const BookmarkNode* node = model.AddURL(model.other_node(), 77 const BookmarkNode* node = model.AddURL(model.other_node(),
72 0, 78 0,
73 ASCIIToUTF16("foo bar"), 79 ASCIIToUTF16("foo bar"),
74 GURL("http://www.google.com")); 80 GURL("http://www.google.com"));
75 81
76 // Copy a node to the clipboard. 82 // Copy a node to the clipboard.
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 EXPECT_EQ(2u, clone->GetMetaInfoMap()->size()); 165 EXPECT_EQ(2u, clone->GetMetaInfoMap()->size());
160 std::string value; 166 std::string value;
161 EXPECT_TRUE(clone->GetMetaInfo("somekey", &value)); 167 EXPECT_TRUE(clone->GetMetaInfo("somekey", &value));
162 EXPECT_EQ("somevalue", value); 168 EXPECT_EQ("somevalue", value);
163 EXPECT_TRUE(clone->GetMetaInfo("someotherkey", &value)); 169 EXPECT_TRUE(clone->GetMetaInfo("someotherkey", &value));
164 EXPECT_EQ("someothervalue", value); 170 EXPECT_EQ("someothervalue", value);
165 } 171 }
166 172
167 } // namespace 173 } // namespace
168 } // namespace bookmark_utils 174 } // namespace bookmark_utils
OLDNEW
« no previous file with comments | « chrome/browser/bookmarks/bookmark_utils.cc ('k') | chrome/browser/resources/bookmark_manager/js/main.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698