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

Unified Diff: net/quic/core/spdy_utils.h

Issue 2385323004: relnote: change ExtractContentLengthFromHeaders to return false if content-length is missing. If th… (Closed)
Patch Set: fix test Created 4 years, 2 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 | « no previous file | net/quic/core/spdy_utils.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/spdy_utils.h
diff --git a/net/quic/core/spdy_utils.h b/net/quic/core/spdy_utils.h
index d1f14ad27786a4f043ef7aee1e7305c360073270..f69b4474d197f69160efcd5fe82c2cf2c05f2b95 100644
--- a/net/quic/core/spdy_utils.h
+++ b/net/quic/core/spdy_utils.h
@@ -34,9 +34,9 @@ class NET_EXPORT_PRIVATE SpdyUtils {
int64_t* content_length,
SpdyHeaderBlock* headers);
- // Populate |content length| with the value of the content-length header if
- // one or more are present in the headers. Returns true on success, false if
- // parsing fails.
+ // Populate |content length| with the value of the content-length header.
+ // Returns true on success, false if parsing fails or content-length header is
+ // missing.
static bool ExtractContentLengthFromHeaders(int64_t* content_length,
SpdyHeaderBlock* headers);
« no previous file with comments | « no previous file | net/quic/core/spdy_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698