Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1796)

Unified Diff: net/tools/balsa/balsa_frame.cc

Issue 1215933004: New new versions of Starts/EndsWith and SplitString in net (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@starts_with
Patch Set: Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/test/spawned_test_server/remote_test_server.cc ('k') | net/tools/balsa/balsa_headers_token_utils.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/balsa/balsa_frame.cc
diff --git a/net/tools/balsa/balsa_frame.cc b/net/tools/balsa/balsa_frame.cc
index 1105c80889d22642214e857fcb350d0f328a6e07..cbd11baeab098255f7b207c2dc321f42c505f50b 100644
--- a/net/tools/balsa/balsa_frame.cc
+++ b/net/tools/balsa/balsa_frame.cc
@@ -22,6 +22,7 @@
#include "base/logging.h"
#include "base/strings/string_piece.h"
+#include "base/strings/string_util.h"
#include "net/tools/balsa/balsa_enums.h"
#include "net/tools/balsa/balsa_headers.h"
#include "net/tools/balsa/balsa_visitor_interface.h"
@@ -724,8 +725,8 @@ bool SplitStringPiece(base::StringPiece original, char delim,
*after = base::StringPiece(p + 1, end - (p + 1));
else
*after = base::StringPiece("");
- StringPieceUtils::RemoveWhitespaceContext(before);
- StringPieceUtils::RemoveWhitespaceContext(after);
+ *before = base::TrimWhitespaceASCII(*before, base::TRIM_ALL);
+ *after = base::TrimWhitespaceASCII(*after, base::TRIM_ALL);
return true;
}
}
@@ -742,7 +743,7 @@ void ProcessChunkExtensionsManual(base::StringPiece all_extensions,
BalsaHeaders* extensions) {
base::StringPiece extension;
base::StringPiece remaining;
- StringPieceUtils::RemoveWhitespaceContext(&all_extensions);
+ all_extensions = base::TrimWhitespaceASCII(all_extensions, base::TRIM_ALL);
SplitStringPiece(all_extensions, ';', &extension, &remaining);
while (!extension.empty()) {
base::StringPiece key;
@@ -758,7 +759,7 @@ void ProcessChunkExtensionsManual(base::StringPiece all_extensions,
extensions->AppendHeader(key, value);
- StringPieceUtils::RemoveWhitespaceContext(&remaining);
+ remaining = base::TrimWhitespaceASCII(remaining, base::TRIM_ALL);
SplitStringPiece(remaining, ';', &extension, &remaining);
}
}
« no previous file with comments | « net/test/spawned_test_server/remote_test_server.cc ('k') | net/tools/balsa/balsa_headers_token_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698