Closed
Bug 938167
Opened 12 years ago
Closed 12 years ago
[NetworkStats API] Error when multiple queries to getSamples are received
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: albert, Assigned: albert)
References
Details
Attachments
(1 file)
|
1.12 KB,
patch
|
airpingu
:
review+
|
Details | Diff | Splinter Review |
When costcontrol launch more than one query to getsamples at the same time, networkstats launch the following error:
E/GeckoConsole( 530): [JavaScript Error: "item is undefined" {file: "resource://gre/modules/NetworkStatsService.jsm" line: 388}]
|
Assignee | |
Comment 1•12 years ago
|
||
There is a problem when queries are inserted into updateQueue to avoid processing multiple times the same thing. Now, if there is a request similar than the incoming one, the callback is inserted and the processQueue function is called, what is wrong. The correct flow should be insert the callback and return, because the query is being processed.
|
|
Assignee | |
Comment 2•12 years ago
|
||
Attachment #831582 -
Flags: review?(gene.lian)
|
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → acperez
|
|
||
Comment 3•12 years ago
|
||
Comment on attachment 831582 [details] [diff] [review]
Patch
Thank you!
Attachment #831582 -
Flags: review?(gene.lian) → review+
|
|
Assignee | |
Updated•12 years ago
|
Keywords: checkin-needed
|
|
||
Comment 4•12 years ago
|
||
Keywords: checkin-needed
|
||
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•