Test that QuicTextUtilsTest:Split preserves heading and trailing whitespace.
Chrome's implementation did not use to preserve whitespace until
https://crrev.com/c/1739311, where it was necessary to change it in order to
make QuicFramerTest.TestExtendedErrorCodeParser (introduced at cr/261676212)
pass.
I locally verified that the test passes after merging this change to Chrome.
gfe-relnote: n/a, test-only change.
PiperOrigin-RevId: 262536209
Change-Id: Ibfcb0d6b74e54f00820c7ea8a94f017228019a80
diff --git a/quic/platform/api/quic_text_utils_test.cc b/quic/platform/api/quic_text_utils_test.cc
index 9b156d0..af4e81d 100644
--- a/quic/platform/api/quic_text_utils_test.cc
+++ b/quic/platform/api/quic_text_utils_test.cc
@@ -201,6 +201,11 @@
QuicTextUtils::Split("a:b:c", ':'));
EXPECT_EQ(std::vector<QuicStringPiece>({"a:b:c"}),
QuicTextUtils::Split("a:b:c", ','));
+ // Leading and trailing whitespace is preserved.
+ EXPECT_EQ(std::vector<QuicStringPiece>({"a", "b", "c"}),
+ QuicTextUtils::Split("a,b,c", ','));
+ EXPECT_EQ(std::vector<QuicStringPiece>({" a", "b ", " c "}),
+ QuicTextUtils::Split(" a:b : c ", ':'));
}
} // namespace test
