Index: components/bookmarks/browser/bookmark_index_unittest.cc |
diff --git a/components/bookmarks/browser/bookmark_index_unittest.cc b/components/bookmarks/browser/bookmark_index_unittest.cc |
index cab18e56987f86637307686fa8d73c3d1cc0b2e8..c7e8f53b69b5abdda58fd0d7fb04fdc9760f531d 100644 |
--- a/components/bookmarks/browser/bookmark_index_unittest.cc |
+++ b/components/bookmarks/browser/bookmark_index_unittest.cc |
@@ -109,11 +109,11 @@ class BookmarkIndexTest : public testing::Test { |
void ExtractMatchPositions(const std::string& string, |
BookmarkMatch::MatchPositions* matches) { |
- std::vector<std::string> match_strings; |
- base::SplitString(string, ':', &match_strings); |
- for (size_t i = 0; i < match_strings.size(); ++i) { |
- std::vector<std::string> chunks; |
- base::SplitString(match_strings[i], ',', &chunks); |
+ for (const base::StringPiece& match : |
+ base::SplitStringPiece(string, ":", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
+ std::vector<base::StringPiece> chunks = base::SplitStringPiece( |
+ match, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
ASSERT_EQ(2U, chunks.size()); |
matches->push_back(BookmarkMatch::MatchPosition()); |
int chunks0, chunks1; |
@@ -192,18 +192,20 @@ TEST_F(BookmarkIndexTest, GetBookmarksMatching) { |
{ "abc def", "abc d", "" }, |
}; |
for (size_t i = 0; i < arraysize(data); ++i) { |
- std::vector<std::string> titles; |
- base::SplitString(data[i].titles, ';', &titles); |
std::vector<TitleAndURL> bookmarks; |
- for (size_t j = 0; j < titles.size(); ++j) { |
- TitleAndURL bookmark(titles[j], kAboutBlankURL); |
+ for (const std::string& title : base::SplitString( |
+ data[i].titles, ";", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
+ TitleAndURL bookmark(title, kAboutBlankURL); |
bookmarks.push_back(bookmark); |
} |
AddBookmarks(bookmarks); |
std::vector<std::string> expected; |
- if (!data[i].expected.empty()) |
- base::SplitString(data[i].expected, ';', &expected); |
+ if (!data[i].expected.empty()) { |
+ expected = base::SplitString(data[i].expected, ";", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
+ } |
ExpectMatches(data[i].query, query_parser::MatchingAlgorithm::DEFAULT, |
expected); |
@@ -251,18 +253,20 @@ TEST_F(BookmarkIndexTest, GetBookmarksMatchingAlwaysPrefixSearch) { |
{ "ab cdef;abcd;abcd cdefg", "ab cdef", "ab cdef;abcd cdefg" }, |
}; |
for (size_t i = 0; i < arraysize(data); ++i) { |
- std::vector<std::string> titles; |
- base::SplitString(data[i].titles, ';', &titles); |
std::vector<TitleAndURL> bookmarks; |
- for (size_t j = 0; j < titles.size(); ++j) { |
- TitleAndURL bookmark(titles[j], kAboutBlankURL); |
+ for (const std::string& title : base::SplitString( |
+ data[i].titles, ";", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
+ TitleAndURL bookmark(title, kAboutBlankURL); |
bookmarks.push_back(bookmark); |
} |
AddBookmarks(bookmarks); |
std::vector<std::string> expected; |
- if (!data[i].expected.empty()) |
- base::SplitString(data[i].expected, ';', &expected); |
+ if (!data[i].expected.empty()) { |
+ expected = base::SplitString(data[i].expected, ";", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
+ } |
ExpectMatches(data[i].query, |
query_parser::MatchingAlgorithm::ALWAYS_PREFIX_SEARCH, |