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

Unified Diff: net/quic/quic_stream_sequencer_buffer.cc

Issue 1983183002: Landing Recent QUIC changes until 5/14/2016 02:25:25 UTC (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: "first try to fix link error for win_clang build" Created 4 years, 7 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/quic/quic_stream_sequencer.cc ('k') | net/quic/quic_stream_sequencer_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_stream_sequencer_buffer.cc
diff --git a/net/quic/quic_stream_sequencer_buffer.cc b/net/quic/quic_stream_sequencer_buffer.cc
index 0f56bf96db4908edd20755cc36fcb4f5fef476f7..352db230dc6fffa32c78fe00bf5aee0e9ecab3ba 100644
--- a/net/quic/quic_stream_sequencer_buffer.cc
+++ b/net/quic/quic_stream_sequencer_buffer.cc
@@ -106,26 +106,21 @@ QuicErrorCode QuicStreamSequencerBuffer::OnStreamData(
*error_details =
string("Beginning of received data overlaps with buffered data.\n") +
"New frame range " + RangeDebugString(offset, offset + size) +
- "\n"
- "Currently received frames: " +
- ReceivedFramesDebugString() +
- "\n"
- "Current gaps: " +
- GapsDebugString() + "\n";
+ " with first 128 bytes: " +
+ string(data.data(), data.length() < 128 ? data.length() : 128) +
+ "\nCurrently received frames: " + ReceivedFramesDebugString() +
+ "\nCurrent gaps: " + GapsDebugString();
return QUIC_OVERLAPPING_STREAM_DATA;
}
if (offset + size > current_gap->end_offset) {
// End of new data overlaps with data after current gap.
*error_details =
- "End of received data overlaps with buffered data.\n"
- "New frame range " +
- RangeDebugString(offset, offset + size) +
- "\n"
- "Currently received frames: " +
- ReceivedFramesDebugString() +
- "\n"
- "Current gaps: " +
- GapsDebugString() + "\n";
+ string("End of received data overlaps with buffered data.\n") +
+ "New frame range " + RangeDebugString(offset, offset + size) +
+ " with first 128 bytes: " +
+ string(data.data(), data.length() < 128 ? data.length() : 128) +
+ "\nCurrently received frames: " + ReceivedFramesDebugString() +
+ "\nCurrent gaps: " + GapsDebugString();
return QUIC_OVERLAPPING_STREAM_DATA;
}
« no previous file with comments | « net/quic/quic_stream_sequencer.cc ('k') | net/quic/quic_stream_sequencer_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698