diff options
| author | brettw <brettw@chromium.org> | 2015-08-07 17:28:47 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-08 00:29:27 +0000 |
| commit | 26dab8f049217794bd73a230ea567021adc09672 (patch) | |
| tree | 226dec9ffde922a61c6d05f0d8ee549f0a9dd02d /base/debug | |
| parent | dbae293445fa5a6c8ba5cea0c7290f436d1034ab (diff) | |
| download | chromium_src-26dab8f049217794bd73a230ea567021adc09672.zip chromium_src-26dab8f049217794bd73a230ea567021adc09672.tar.gz chromium_src-26dab8f049217794bd73a230ea567021adc09672.tar.bz2 | |
Update SplitString calls to new form
Uses the new form for most (but not quite all) of the remaining users of the old form.
Reland of https://codereview.chromium.org/1272823003 but with no changes to the way media parses codec lists. The previous landing attempted to simplify some handling but there are layout tests that expect the old behavior, and I'm not qualified to tell if it's OK to change.
TBR=sky
Review URL: https://codereview.chromium.org/1274123003
Cr-Commit-Position: refs/heads/master@{#342489}
Diffstat (limited to 'base/debug')
| -rw-r--r-- | base/debug/proc_maps_linux.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/base/debug/proc_maps_linux.cc b/base/debug/proc_maps_linux.cc index 4c1aedf..8c8965b 100644 --- a/base/debug/proc_maps_linux.cc +++ b/base/debug/proc_maps_linux.cc @@ -96,8 +96,8 @@ bool ParseProcMaps(const std::string& input, // This isn't async safe nor terribly efficient, but it doesn't need to be at // this point in time. - std::vector<std::string> lines; - SplitString(input, '\n', &lines); + std::vector<std::string> lines = SplitString( + input, "\n", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); for (size_t i = 0; i < lines.size(); ++i) { // Due to splitting on '\n' the last line should be empty. |