Closed Bug 454202 Opened 16 years ago Closed 5 years ago

[meta] mac performance problems

Categories

(Core :: Disability Access APIs, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: surkov, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

Attachments

(1 obsolete file)

Attached patch cache parent (obsolete) (deleted) — Splinter Review
caching parent makes firefox faster in 3 times :) but it still incredibly slow. Running browser with http://www.mozilla.org/projects/minefield/ and focusing "search mozilla: " textbox takes 30 secundes before and 10 secundes after. Opera takes nothing.
What if the DOM changes?
Another (related) problem is that you're keeping a weak reference to the parent, so if the object goes away and you later assume is there - boom! If you retain it (add to the reference count) you will create a cyclical reference (because parents retain their children, and now those children would also retain the parent) so then it would be very important that you break that cycle at some point to avoid memory leaks.
Yes, that's just an example. I think we can use weak reference here but make sure we invalidate it in invalidateChildren method. Though actually parent caching doesn't bring a win always. Sometimes only.
Depends on: 455442
Depends on: 455443
Summary: mac performance problems → [meta] mac performance problems
I recently did some more thorough testing, and found that the most performance problems happen when VoiceOver hits plain text. As long as I navigate through a series of link accessibles, buttons or text fields, like on www.google.com, performance is not as snappy as with Safari, but response comes back in under 1 second, too. Only if I then continue and get to search results that have plain text nodes, the performance gets bad. As soon as I am past the text on another non-text element, I'm better again. Does this belong here with the other discussion, or should I file a separate bug for this?
Depends on: 606085
Depends on: 705404
Comment on attachment 337439 [details] [diff] [review] cache parent obsoleted by bug 455443
Attachment #337439 - Attachment is obsolete: true
Assignee: nobody → hub
As is is now, the performance does not seem to be so bad anymore. There are a few areas where Firefox appear to be "Busy" for a bit too long.
Target Milestone: --- → mozilla11
Target Milestone: mozilla11 → ---
(In reply to Hub Figuiere [:hub] from comment #6) > As is is now, the performance does not seem to be so bad anymore. great to hear > There are a few areas where Firefox appear to be "Busy" for a bit too long. let's keep the bug open, that shouldn't happen (I bet your machine is not slow).
Depends on: 772060
Assignee: hub → nobody
Keywords: meta

I'm going to close this meta bug since all the blockers have been closed. We still need to focus on performance, but maybe a 12 year old bug is a bit less relevant! Thanks y'all for your work on this.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: