Refactor by modifying base::SplitStringIntoKeyValuePairs |
||
Issue descriptionA slight change to the above mentioned SplitString function will do the job of both ParseHeader && ParseHeaders in chromium:1144316. Add similar overloaded function that allows for strings/StringPiece's as delimiters.
,
Oct 31
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/de04887a2fb977f433502e9485176d94e8f918d3 commit de04887a2fb977f433502e9485176d94e8f918d3 Author: Luum Habtemariam <luum@chromium.org> Date: Wed Oct 31 08:28:23 2018 Re-add deleted base::StringSplitIntoKeyValuePairs unittest Re-adds the MissingKeyValueDelimiter unittest, accidentally removed in commit 9268d9a11d1d967f5a090f5411f7ed041192b561. Bug: chromium:894274 Test: Tests still pass. Change-Id: I37228e812eaf71b1d51fb9982aedea70570311d2 Reviewed-on: https://chromium-review.googlesource.com/c/1306657 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#604185} [modify] https://crrev.com/de04887a2fb977f433502e9485176d94e8f918d3/base/strings/string_split_unittest.cc
,
Oct 31
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Oct 29