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

Unified Diff: components/bookmarks/browser/bookmark_index_unittest.cc

Issue 1234973004: Update SplitString calls in components (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixes Created 5 years, 5 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 side-by-side diff with in-line comments
Download patch
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,
« no previous file with comments | « components/autofill/core/browser/credit_card.cc ('k') | components/bookmarks/browser/bookmark_model_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698