Closed
Bug 134149
Opened 23 years ago
Closed 23 years ago
CPU usage 100% when visiting http://www.omroep.nl/nos/nieuws/index.html
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 123273
People
(Reporter: ericdere, Assigned: jst)
References
()
Details
(Keywords: perf, qawanted)
CPU usage jumps to and remains at 100% when visiting
http://www.omroep.nl/nos/nieuws/index.html
Closing the tab/window reduces CPU load immediately.
Reporter | ||
Comment 1•23 years ago
|
||
Build 2002031104 (0.99 milestone) Windows NT
Confirming on linux 2002032808. Disabling JavaScript makes the CPU usage drop a
lot. I'd say the news ticker is a bit CPU greedy.
Assignee: asa → jst
Status: UNCONFIRMED → NEW
Component: Browser-General → DOM Level 0
Ever confirmed: true
OS: Windows NT → All
QA Contact: doronr → desale
Comment 3•23 years ago
|
||
We need a profile here...
Tried to profile it JS Debugger but I keep crashing or hanging -> bug 134377.
Changing URL, it's definitly the news-ticker.
Is there a step by step recipe on how to make a JS-Profile?
.
The right URL for real this time, sorry for the spam.
Comment 7•23 years ago
|
||
The news ticker is indeed the problem. The JavaScript on that page boils down to:
function doSomething() { /* do stuff */ setTimeout('doSomething()', 0); }
doSomething();
doSomething() is called, which uses setTimeout() to call doSomething() again in
0 milliseconds. Then that doSomething() calls doSomething() in 0 milliseconds.
And on and on.
Mozilla is working like I would expect it to -- the JavaScript interpreter tries
calling doSomething() as often as possible, causing it to eat all available CPU
time. If this behavior is undesirable, the JavaScript interpreter should impose
a minimum value on the 2nd argument to setTimeout(). In my testing, changing
the above code to use a delay of 1 cut CPU usage in half.
Comment 8•23 years ago
|
||
This is a duplicate of bug 123273. (Sorry for the spam -- didn't realize it was
a dupe until well after I added my previous comment.)
Comment 9•23 years ago
|
||
So it is. :)
*** This bug has been marked as a duplicate of 123273 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•