947495 - Network usage stats for TCPSocket get undefined value

Status: RESOLVED INVALID

(Core :: DOM: Core & HTML, defect)

Description

[Josh Matthews [:jdm]]

From TCPSocket.js:

>    this._txBytes += length;
>    this._saveNetworkStats(false);

The length variable no longer exists; it should be using byteLength instead.

Comment 1

[Josh Matthews [:jdm]]

This can cause incorrect network usage statistics, especially in regards to usage by the email app.

Comment 2

[Peiyong Lin[:lpy](UTC+8)]

Attachment: bug947495.patch

Well, I just saw |length| in http://mxr.mozilla.org/mozilla-central/source/dom/network/src/TCPSocket.js#670

Comment 3

[Josh Matthews [:jdm]]

Argh, I was looking at mozilla-central with the patch from bug 885982 applied :(