Closed
Bug 561491
Opened 15 years ago
Closed 3 years ago
Dromaeo DOM Events (Prototype) and DOM Event (jQuery) tests don't seem to trace well
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
platform-rel | --- | + |
People
(Reporter: smaug, Unassigned)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [platform-rel-jQuery])
While profiling Dromaeo, I noticed that the DOM Events tests don't
actually seem to test DOM event handling, because JS code doesn't run
fast there.
Shark shows rather deep recursion of js_fun_call/js_Interpret/js_fun_apply
and if I read that all correctly, tracing doesn't work there.
(I was told that nested js_Interpret calls pretty much disable tracing)
The actual event handling takes just few % in the test.
Comment 1•15 years ago
|
||
Shaver tells me that Jason has been working on a fix for the deep bail problem.
Comment 2•15 years ago
|
||
I assume this is a tracking bug? Bug 532696 and its dependencies cover the things I saw on the prototype tests last time I looked at them. We probably need to look at the abort logs and trace exit logs for jquery too. Smaug, were you going to do that?
Reporter | ||
Comment 3•15 years ago
|
||
I don't know how to look at abort logs or trace exits, but probably
time to learn that.
Comment 4•15 years ago
|
||
> I don't know how to look at abort logs or trace exits
Catch me on irc or e-mail and we can talk? Would love to have another DOM person looking at that sort of stuff. ;)
Comment 5•13 years ago
|
||
bz/smaug, is this bug still a valid non-TM bug or has DOM perf work moved on to other bugs at this point?
Comment 6•13 years ago
|
||
Would need to redo the profile from comment 0 to tell. Olli, can you do that, or should I?
Reporter | ||
Comment 7•13 years ago
|
||
I could re-profile.
Reporter | ||
Comment 8•11 years ago
|
||
Based on Bug 928318 the situation hasn't really improved.
Assignee | ||
Updated•10 years ago
|
Assignee: general → nobody
Updated•8 years ago
|
Whiteboard: [platform-rel-jQuery]
Updated•8 years ago
|
platform-rel: --- → ?
Updated•8 years ago
|
platform-rel: ? → +
Updated•8 years ago
|
Rank: 50
Comment 9•3 years ago
|
||
Old TM bug, no longer valid.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•