Update SplitString calls in chrome.

In many places that iterated over the results, the code was changed to use a range-based for loop over the result of the SplitStirng call.

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

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

1 files changed, 8 insertions, 8 deletions

diff --git a/chrome/browser/media/webrtc_log_uploader_unittest.cc b/chrome/browser/media/webrtc_log_uploader_unittest.cc
index c3bf518..f36e002 100644
--- a/chrome/browser/media/webrtc_log_uploader_unittest.cc
+++ b/chrome/browser/media/webrtc_log_uploader_unittest.cc
@@ -35,8 +35,8 @@ class WebRtcLogUploaderTest : public testing::Test {
     std::string last_line = GetLastLineFromListFile();
     if (last_line.empty())
       return false;
-    std::vector<std::string> line_parts;
-    base::SplitString(last_line, ',', &line_parts);
+    std::vector<std::string> line_parts = base::SplitString(
+        last_line, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
     EXPECT_EQ(3u, line_parts.size());
     if (3u != line_parts.size())
       return false;
@@ -52,8 +52,8 @@ class WebRtcLogUploaderTest : public testing::Test {
     std::string last_line = GetLastLineFromListFile();
     if (last_line.empty())
       return false;
-    std::vector<std::string> line_parts;
-    base::SplitString(last_line, ',', &line_parts);
+    std::vector<std::string> line_parts = base::SplitString(
+        last_line, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
     EXPECT_EQ(3u, line_parts.size());
     if (3u != line_parts.size())
       return false;
@@ -67,8 +67,8 @@ class WebRtcLogUploaderTest : public testing::Test {
     std::string last_line = GetLastLineFromListFile();
     if (last_line.empty())
       return false;
-    std::vector<std::string> line_parts;
-    base::SplitString(last_line, ',', &line_parts);
+    std::vector<std::string> line_parts = base::SplitString(
+        last_line, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
     EXPECT_EQ(3u, line_parts.size());
     if (3u != line_parts.size())
       return false;
@@ -111,8 +111,8 @@ class WebRtcLogUploaderTest : public testing::Test {
     // Since every line should end with '\n', the last line should be empty. So
     // we expect at least two lines including the final empty. Remove the empty
     // line before returning.
-    std::vector<std::string> lines;
-    base::SplitString(contents, '\n', &lines);
+    std::vector<std::string> lines = base::SplitString(
+        contents, "\n", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
     EXPECT_GT(lines.size(), 1u);
     if (lines.size() < 2)
       return std::vector<std::string>();