Implements incremental decode in HPACK.

Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc, and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484
Committed: https://crrev.com/8fc656b6fdadbb902bdb39d5de8538dbdb2f3751
Cr-Commit-Position: refs/heads/master@{#389857}

[Biren Roy, 2016-04-26 16:57:16]

Description was changed from

==========
This cl enables incremental decode in hpack. With incremental decode, headers
may contain incomplete opcode. The change is divided into two parts: the
HpackInputStream change in hpack_input_stream.h/cc, and the HpackDecoder change
in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong@yasong:decodeonline2:27:citc.

BUG=488484
==========

to

==========
Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is
divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc,
and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484
==========

[Biren Roy, 2016-04-26 17:17:09]

birenroy@chromium.org changed reviewers:
+ bnc@chromium.org, ckrasic@chromium.org

[Biren Roy, 2016-04-26 17:17:09]

birenroy@chromium.org changed required reviewers:
+ bnc@chromium.org

[Biren Roy, 2016-04-26 17:17:10]

This is my first merge to Chromium. Please be gentle. :)

[Bence, 2016-04-26 17:38:13]

LGTM.

[Buck, 2016-04-26 17:48:01]

lgtm

[Buck, 2016-04-26 17:48:04]

lgtm

[Biren Roy, 2016-04-26 18:03:01]

The CQ bit was checked by birenroy@chromium.org

[commit-bot: I haz the power, 2016-04-26 18:03:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1914193002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1914193002/1

[commit-bot: I haz the power, 2016-04-26 19:49:39]

Description was changed from

==========
Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is
divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc,
and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484
==========

to

==========
Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is
divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc,
and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484
==========

[commit-bot: I haz the power, 2016-04-26 19:49:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-26 19:52:05]

Description was changed from

==========
Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is
divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc,
and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484
==========

to

==========
Implements incremental decode in HPACK.

With incremental decode, headers may contain incomplete opcode. The change is
divided into two parts: the HpackInputStream change in hpack_input_stream.h/cc,
and the HpackDecoder change in Hpack_decoder.h/cc.

After new data are added to the buffer (by calling
HandleControlFrameHeadersData()), a new HpackInputStream is created
based on the current data in the buffer, and as many data as possible
are parsed. When an opcode is parsed successfully, remember the current
position in the header (by calling MarkCurrentPosition()), and move on
to the next opcode. If an opcode is incomplete, we return from the current
parsing with false, and mark need_more_data_ to be true.
The data have been successfully parsed will be removed
from the buffer. Since we incrementally decode headers as data arrive,
we remove the decode code in HandleControlFrameHeadersComplete().

This CL lands server change 116551169 by yasong.

BUG=488484

Committed: https://crrev.com/8fc656b6fdadbb902bdb39d5de8538dbdb2f3751
Cr-Commit-Position: refs/heads/master@{#389857}
==========

[commit-bot: I haz the power, 2016-04-26 19:52:06]

Patchset 1 (id:??) landed as
https://crrev.com/8fc656b6fdadbb902bdb39d5de8538dbdb2f3751
Cr-Commit-Position: refs/heads/master@{#389857}