Update HTTP/2 DATA frame padding to 0x00.

Update HTTP/2 DATA frame padding to 0x00.

Use 0x00 rather than 0x30 ('0') to pad the DATA frame to be comply with the
HTTP/2 spec.

This lands server change 77448497 by raullenchai.

BUG=345769
Committed: https://crrev.com/8f97e51df97787225c1952e3d1bd094b01cae740
Cr-Commit-Position: refs/heads/master@{#300234}

[Bence, 2014-10-17 19:55:18]

bnc@chromium.org changed reviewers:
+ rch@chromium.org

[Bence, 2014-10-17 19:55:18]

Ryan: PTAL.  Thanks.

[Ryan Hamilton, 2014-10-17 20:08:36]

lgtm

https://codereview.chromium.org/654253004/diff/1/net/spdy/spdy_framer.cc
File net/spdy/spdy_framer.cc (right):

https://codereview.chromium.org/654253004/diff/1/net/spdy/spdy_framer.cc#newc...
net/spdy/spdy_framer.cc:2318: padding = string(data_ir.padding_payload_len(),
0);
It seems like this would be better written as:

string padding(data_ir.padding_payload_len(), 0);

Wouldn't it?

[Bence, 2014-10-19 19:04:34]

The CQ bit was checked by bnc@chromium.org

[Bence, 2014-10-19 19:05:07]

Sorry, I overlooked that one.  Thank you.

https://codereview.chromium.org/654253004/diff/1/net/spdy/spdy_framer.cc
File net/spdy/spdy_framer.cc (right):

https://codereview.chromium.org/654253004/diff/1/net/spdy/spdy_framer.cc#newc...
net/spdy/spdy_framer.cc:2318: padding = string(data_ir.padding_payload_len(),
0);
On 2014/10/17 20:08:36, Ryan Hamilton wrote:
> It seems like this would be better written as:
> 
> string padding(data_ir.padding_payload_len(), 0);
> 
> Wouldn't it?

Done.

[commit-bot: I haz the power, 2014-10-19 19:06:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/654253004/20001

[commit-bot: I haz the power, 2014-10-20 09:33:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-20 09:35:11]

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