Introduces QuicStreamSequencerBufferInterface with one implementation. Refactor, no behavior change.

Introduces QuicStreamSequencerBufferInterface with one implementation. Refactor, no behavior change.

Write an abstract class for QuicFrameList to prepare changes of underlying data
structure used to buffer received StreamFrame's in sequencer. And changed some
method names for readability.

Merge internal change: 105311433

R=rch@chromium.org
BUG=

[Ryan Hamilton, 2015-10-21 16:44:52]

lgtm, modulo the unique_ptr. Feel free to fix in the "Final" CL if you wish.

https://codereview.chromium.org/1414573004/diff/20001/net/quic/quic_stream_se...
File net/quic/quic_stream_sequencer.h (right):

https://codereview.chromium.org/1414573004/diff/20001/net/quic/quic_stream_se...
net/quic/quic_stream_sequencer.h:115:
std::unique_ptr<QuicStreamSequencerBufferInterface> buffered_frames_;
use scoped_ptr instead of std::unique_ptr or this won't build on OS X.

We should improve the merge scripts to do s/std::unique_ptr/scoped_ptr/

[alyssar1, 2015-11-02 18:22:44]

Description was changed from

==========
Introduces QuicStreamSequencerBufferInterface with one implementation. Refactor,
no behavior change.

Write an abstract class for QuicFrameList to prepare changes of underlying data
structure used to buffer received StreamFrame's in sequencer. And changed some
method names for readability.

Merge internal change: 105311433

R=rch@chromium.org
BUG=
==========

to

==========
Introduces QuicStreamSequencerBufferInterface with one implementation. Refactor,
no behavior change.

Write an abstract class for QuicFrameList to prepare changes of underlying data
structure used to buffer received StreamFrame's in sequencer. And changed some
method names for readability.

Merge internal change: 105311433

R=rch@chromium.org
BUG=
==========