Fix calculation of length in websocket.cc.

Fix calculation of length in websocket.cc.

The compiler warned about this. See
http://tools.ietf.org/html/draft-hixie-thewebsocketprotocol-49#section-4.2
for how to calculate the length.

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

[Craig, 2009-10-21 19:03:47]

Please review. Thank you.

[ukai, 2009-10-22 01:19:36]

LGTM

[darin (slow to review), 2009-10-22 06:40:56]

Shouldn't there have been a unit test for this?

[ukai, 2009-10-22 06:42:54]

On 2009/10/22 06:40:56, darin wrote:
> Shouldn't there have been a unit test for this?

Sure. I'll add unit tests.

[darin (slow to review), 2009-10-22 06:57:15]

Every code change that impacts the behavior of the code, I mean :)

On Wed, Oct 21, 2009 at 11:56 PM, Darin Fisher <darin@chromium.org> wrote:

> Thanks!  (Every code change should have a test.)
>
>
> On Wed, Oct 21, 2009 at 11:42 PM, <ukai@chromium.org> wrote:
>
>> On 2009/10/22 06:40:56, darin wrote:
>>
>>> Shouldn't there have been a unit test for this?
>>>
>>
>> Sure. I'll add unit tests.
>>
>>
>> http://codereview.chromium.org/293055
>>
>
>

[darin (slow to review), 2009-10-22 06:59:38]

Thanks!  (Every code change should have a test.)

On Wed, Oct 21, 2009 at 11:42 PM, <ukai@chromium.org> wrote:

> On 2009/10/22 06:40:56, darin wrote:
>
>> Shouldn't there have been a unit test for this?
>>
>
> Sure. I'll add unit tests.
>
>
> http://codereview.chromium.org/293055
>