Try to reduce "unnecessary" children-changed event emission

NEW
Unassigned

Status

()

3 years ago
a year ago

People

(Reporter: jdiggs, Unassigned)

Tracking

(Blocks: 2 bugs, {perf})

Trunk
Unspecified
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox47 affected)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
Created attachment 8724608 [details]
pyatspi accessible-event listener

Steps to reproduce:
1. Load the attached pyatspi accessible-event listener in a terminal
2. Load/reload matrix.org

Desired results: Not 450+ object:children-changed:add:system events for images adding child images. :)

(I don't know if that's possible or not. But the act of examining these events in Orca just to rule them out as event spam still takes more time and a performance hit that is far from ideal. So if the spam could be detected and fewer events emitted, I'd be grateful.)
(Reporter)

Comment 1

3 years ago
Created attachment 8724609 [details]
Output from performing steps in opening report
Blocks: 531850

Updated

3 years ago
Keywords: perf

Comment 2

a year ago
Hello,

I tried to reproduce the issue with firefox 45.9.0, 52.3.0 and 55.0.3, and didn't get it with either version. I do get some events like this:

25 15:30:23 [image | ] object:children-changed:add:system [image | ]

but only a dozen of them, and only after scrolling down to the "How does it work?" part of matrix.org.

Are you still getting this precise issue?

Samuel
(In reply to Samuel Thibault from comment #2)
> Hello,
> 
> I tried to reproduce the issue with firefox 45.9.0, 52.3.0 and 55.0.3, and
> didn't get it with either version. I do get some events like this:
> 
> 25 15:30:23 [image | ] object:children-changed:add:system [image | ]
> 
> but only a dozen of them, and only after scrolling down to the "How does it
> work?" part of matrix.org.
> 
> Are you still getting this precise issue?
> 
> Samuel
Flags: needinfo?(jdiggs)
(Reporter)

Comment 4

a year ago
It looks like Matrix.org changed its site. If I had to guess, the problem may remain but a new test case would be needed. Since I'm not a JavaScript authoring type, I'll leave that to someone else. Clearing the NEEDINFO.
Flags: needinfo?(jdiggs)
You need to log in before you can comment on or make changes to this bug.