Update SplitString calls in components

This converts calls from the old form to the new form. Some calls that iterated over the results were changed to a range-based for loop with an inline call to SplitString. Some places were changed to use StringPieces when it was safe to do so.

Review URL: https://codereview.chromium.org/1234973004

Cr-Commit-Position: refs/heads/master@{#340209}

1 files changed, 6 insertions, 6 deletions

diff --git a/components/history/core/browser/top_sites_database.cc b/components/history/core/browser/top_sites_database.cc
index d1f024d..00c4559 100644
--- a/components/history/core/browser/top_sites_database.cc
+++ b/components/history/core/browser/top_sites_database.cc
@@ -90,12 +90,12 @@ std::string GetRedirects(const MostVisitedURL& url) {
 
 // Decodes redirects from a string and sets them for the url.
 void SetRedirects(const std::string& redirects, MostVisitedURL* url) {
-  std::vector<std::string> redirects_vector;
-  base::SplitStringAlongWhitespace(redirects, &redirects_vector);
-  for (size_t i = 0; i < redirects_vector.size(); ++i) {
-    GURL redirects_url(redirects_vector[i]);
-    if (redirects_url.is_valid())
-      url->redirects.push_back(redirects_url);
+  for (const std::string& redirect : base::SplitString(
+           redirects, base::kWhitespaceASCII,
+           base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY)) {
+    GURL redirect_url(redirect);
+    if (redirect_url.is_valid())
+      url->redirects.push_back(redirect_url);
   }
 }