Closed
Bug 947495
Opened 11 years ago
Closed 11 years ago
Network usage stats for TCPSocket get undefined value
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: jdm, Assigned: lpy)
Details
(Whiteboard: [mentor=jdm][lang=js][good first bug])
Attachments
(1 file)
|
777 bytes,
patch
|
Details | Diff | Splinter Review |
From TCPSocket.js:
> this._txBytes += length;
> this._saveNetworkStats(false);
The length variable no longer exists; it should be using byteLength instead.
|
Reporter | |
Comment 1•11 years ago
|
||
This can cause incorrect network usage statistics, especially in regards to usage by the email app.
blocking-b2g: --- → koi?
|
Assignee | |
Comment 2•11 years ago
|
||
Well, I just saw |length| in http://mxr.mozilla.org/mozilla-central/source/dom/network/src/TCPSocket.js#670
Assignee: nobody → pylaurent1314
Attachment #8344168 -
Flags: review?(josh)
|
|
Reporter | |
Comment 3•11 years ago
|
||
Argh, I was looking at mozilla-central with the patch from bug 885982 applied :(
Status: NEW → RESOLVED
blocking-b2g: koi? → ---
Closed: 11 years ago
Resolution: --- → INVALID
|
|
Reporter | |
Updated•11 years ago
|
Attachment #8344168 -
Flags: review?(josh)
|
||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•