diff options
| author | brettw <brettw@chromium.org> | 2015-07-21 14:37:38 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-07-21 21:38:12 +0000 |
| commit | c6f82b158f374c172a46326699f03d97777faa8e (patch) | |
| tree | c96472fa185ca20b22fad773cd195689e4097894 /chrome/browser/extensions/extension_apitest.cc | |
| parent | db484c3eff68c46ddf1530dd60441dba4614cb5c (diff) | |
| download | chromium_src-c6f82b158f374c172a46326699f03d97777faa8e.zip chromium_src-c6f82b158f374c172a46326699f03d97777faa8e.tar.gz chromium_src-c6f82b158f374c172a46326699f03d97777faa8e.tar.bz2 | |
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}
Diffstat (limited to 'chrome/browser/extensions/extension_apitest.cc')
| -rw-r--r-- | chrome/browser/extensions/extension_apitest.cc | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/chrome/browser/extensions/extension_apitest.cc b/chrome/browser/extensions/extension_apitest.cc index 48ddbaf..261ee75 100644 --- a/chrome/browser/extensions/extension_apitest.cc +++ b/chrome/browser/extensions/extension_apitest.cc @@ -94,11 +94,9 @@ scoped_ptr<net::test_server::HttpResponse> HandleSetCookieRequest( std::string cookie_value = request.relative_url.substr(query_string_pos + 1); - std::vector<std::string> cookies; - base::SplitString(cookie_value, '&', &cookies); - - for (size_t i = 0; i < cookies.size(); i++) - http_response->AddCustomHeader("Set-Cookie", cookies[i]); + for (const std::string& cookie : base::SplitString( + cookie_value, "&", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) + http_response->AddCustomHeader("Set-Cookie", cookie); return http_response.Pass(); } |