Add a function that calculates the size of WebSocket frame header.

Add a function that calculates the size of WebSocket frame header.

This function will be used to determine the size of buffer needed for
writing WebSocket frame header.

BUG=116624
R=mmenke@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152518

[Yuta Kitamura, 2012-08-20 09:02:43]

Matt, could you review this?

Feel free to add willchan as a reviewer if you think this patch needs to be also
reviewed by him.

[mmenke, 2012-08-20 16:33:46]

LGTM.  Seems like a good idea.

http://codereview.chromium.org/10825444/diff/1/net/websockets/websocket_frame.cc
File net/websockets/websocket_frame.cc (right):

http://codereview.chromium.org/10825444/diff/1/net/websockets/websocket_frame...
net/websockets/websocket_frame.cc:54: int header_size =
optional:  May want to just return this, rather than assigning it to a variable
and then returning it on the next line.

[Yuta Kitamura, 2012-08-21 04:22:21]

Thanks!

http://codereview.chromium.org/10825444/diff/1/net/websockets/websocket_frame.cc
File net/websockets/websocket_frame.cc (right):

http://codereview.chromium.org/10825444/diff/1/net/websockets/websocket_frame...
net/websockets/websocket_frame.cc:54: int header_size =
On 2012/08/20 16:33:46, Matt Menke wrote:
> optional:  May want to just return this, rather than assigning it to a
variable
> and then returning it on the next line.

Done.

[commit-bot: I haz the power, 2012-08-21 04:23:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yutak@chromium.org/10825444/4

[commit-bot: I haz the power, 2012-08-21 06:41:32]

Change committed as 152518