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

Unified Diff: net/quic/test_tools/quic_stream_sequencer_peer.cc

Issue 1414573004: Introduces QuicStreamSequencerBufferInterface with one implementation. Refactor, no behavior change. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixing the build file (previously missing a comma) Created 5 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
Index: net/quic/test_tools/quic_stream_sequencer_peer.cc
diff --git a/net/quic/test_tools/quic_stream_sequencer_peer.cc b/net/quic/test_tools/quic_stream_sequencer_peer.cc
index 0d2c61b11abdc3412985bc46bc7bfc81a7599d10..fea4579cbcb21498206e47042ee54a05de7bd409 100644
--- a/net/quic/test_tools/quic_stream_sequencer_peer.cc
+++ b/net/quic/test_tools/quic_stream_sequencer_peer.cc
@@ -13,20 +13,19 @@ namespace net {
namespace test {
// static
-size_t QuicStreamSequencerPeer::GetNumBufferedFrames(
+size_t QuicStreamSequencerPeer::GetNumBufferedBytes(
QuicStreamSequencer* sequencer) {
- return sequencer->buffered_frames_.size();
+ return sequencer->buffered_frames_->BytesBuffered();
}
// static
bool QuicStreamSequencerPeer::FrameOverlapsBufferedData(
- QuicStreamSequencer* sequencer,
+ QuicFrameList* buffer,
const QuicStreamFrame& frame) {
list<QuicFrameList::FrameData>::iterator it =
- sequencer->buffered_frames_.FindInsertionPoint(frame.offset,
- frame.data.length());
- return sequencer->buffered_frames_.FrameOverlapsBufferedData(
- frame.offset, frame.data.length(), it);
+ buffer->FindInsertionPoint(frame.offset, frame.data.length());
+ return buffer->FrameOverlapsBufferedData(frame.offset, frame.data.length(),
+ it);
}
// static
« net/quic/quic_stream_sequencer.h ('K') | « net/quic/test_tools/quic_stream_sequencer_peer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698