Chromium Code Reviews| Index: components/dom_distiller/standalone/content_extractor_browsertest.cc |
| diff --git a/components/dom_distiller/standalone/content_extractor_browsertest.cc b/components/dom_distiller/standalone/content_extractor_browsertest.cc |
| index be5c0bc333dd38f729830c4f2d9df992e61e0776..15f09fd779509d2bfbee55e08f7032aa7c6e64ee 100644 |
| --- a/components/dom_distiller/standalone/content_extractor_browsertest.cc |
| +++ b/components/dom_distiller/standalone/content_extractor_browsertest.cc |
| @@ -248,8 +248,8 @@ class ContentExtractionRequest : public ViewRequestDelegate { |
| } |
| } else if (command_line.HasSwitch(kUrlsSwitch)) { |
| std::string urls_string = command_line.GetSwitchValueASCII(kUrlsSwitch); |
| - std::vector<std::string> urls; |
| - base::SplitString(urls_string, ' ', &urls); |
| + std::vector<std::string> urls = base::SplitString( |
| + urls_string, " ", base::KEEP_WHITESPACE, base::SPLIT_WANT_ALL); |
|
yzshen1
2015/07/22 22:50:34
TRIM_WHITESPACE
|
| // Check for original-urls switch, which must exactly pair up with |
| // |kUrlsSwitch| i.e. number of original urls must be same as that of |
| // urls. |
| @@ -257,8 +257,11 @@ class ContentExtractionRequest : public ViewRequestDelegate { |
| if (command_line.HasSwitch(kOriginalUrls)) { |
| std::string original_urls_string = |
| command_line.GetSwitchValueASCII(kOriginalUrls); |
| - base::SplitString(original_urls_string, ' ', &original_urls); |
| - if (original_urls.size() != urls.size()) original_urls.clear(); |
| + original_urls = base::SplitString( |
| + original_urls_string, " ", |
| + base::KEEP_WHITESPACE, base::SPLIT_WANT_ALL); |
|
yzshen1
2015/07/22 22:50:34
TRIM_WHITESPACE
|
| + if (original_urls.size() != urls.size()) |
| + original_urls.clear(); |
| } |
| for (size_t i = 0; i < urls.size(); ++i) { |
| GURL url(urls[i]); |