Clean up max_frame_size related constants, move storage of setting to the framer.

Clean up max_frame_size related constants, move storage of setting to the framer.

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

BUG=488484
Committed: https://crrev.com/164d046389e2ccf68bda622362427dc6c7bf94fb
Cr-Commit-Position: refs/heads/master@{#404585}

[yinjie, 2016-07-08 00:36:34]

The CQ bit was checked by yinjie@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-08 00:37:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 01:54:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-08 01:54:13]

Dry run: This issue passed the CQ dry run.

[yinjie, 2016-07-08 16:27:25]

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

[Bence, 2016-07-08 20:19:01]

Please change CL title to fix on one line.  (Second sentence can be the second
paragraph on its own.)

CL title seems to contain the same chunck of text twice, please fix that too.

Thank you.

https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h
File net/spdy/spdy_protocol.h (right):

https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h#ne...
net/spdy/spdy_protocol.h:53: // The maxmium possible frame payload size allowed
by the spec.
Please fix this typo: this is part of change 126437339.

[yinjie, 2016-07-08 22:34:28]

Description was changed from

==========
Clean up max_frame_size related constants, move storage of setting to the
framer. No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

BUG=488484

Clean up max_frame_size related constants, move storage of setting to the
framer. No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

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

to

==========
Clean up max_frame_size related constants, move storage of setting to the
framer. 

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

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

[yinjie, 2016-07-08 22:41:25]

The CQ bit was checked by yinjie@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-08 22:41:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yinjie, 2016-07-08 22:41:44]

On 2016/07/08 20:19:01, Bence wrote:
> Please change CL title to fix on one line.  (Second sentence can be the second
> paragraph on its own.)
> 
> CL title seems to contain the same chunck of text twice, please fix that too.
> 
> Thank you.
> 
> https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h
> File net/spdy/spdy_protocol.h (right):
> 
>
https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h#ne...
> net/spdy/spdy_protocol.h:53: // The maxmium possible frame payload size
allowed
> by the spec.
> Please fix this typo: this is part of change 126437339.

Thanks for the review. Please take a look!

[commit-bot: I haz the power, 2016-07-08 23:57:29]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-08 23:57:29]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-07-09 14:05:07]

On 2016/07/08 22:41:44, yinjie wrote:
> On 2016/07/08 20:19:01, Bence wrote:
> > Please change CL title to fix on one line.  (Second sentence can be the
second
> > paragraph on its own.)
> > 
> > CL title seems to contain the same chunck of text twice, please fix that
too.
> > 
> > Thank you.
> > 
> > https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h
> > File net/spdy/spdy_protocol.h (right):
> > 
> >
>
https://codereview.chromium.org/2130153002/diff/1/net/spdy/spdy_protocol.h#ne...
> > net/spdy/spdy_protocol.h:53: // The maxmium possible frame payload size
> allowed
> > by the spec.
> > Please fix this typo: this is part of change 126437339.
> 
> Thanks for the review. Please take a look!

Thank you.  LGTM.

[yinjie1, 2016-07-10 21:19:11]

The CQ bit was checked by yinjie@google.com

[commit-bot: I haz the power, 2016-07-10 21:19:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-11 01:13:42]

Description was changed from

==========
Clean up max_frame_size related constants, move storage of setting to the
framer. 

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

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

to

==========
Clean up max_frame_size related constants, move storage of setting to the
framer. 

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

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

[commit-bot: I haz the power, 2016-07-11 01:13:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-11 01:16:39]

Description was changed from

==========
Clean up max_frame_size related constants, move storage of setting to the
framer. 

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

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

to

==========
Clean up max_frame_size related constants, move storage of setting to the
framer.

No functional change, not flag-protected.

This CL lands server change 126437339 by dahollings.

BUG=488484

Committed: https://crrev.com/164d046389e2ccf68bda622362427dc6c7bf94fb
Cr-Commit-Position: refs/heads/master@{#404585}
==========

[commit-bot: I haz the power, 2016-07-11 01:16:40]

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