Try to reduce "unnecessary" children-changed event emission

NEW
Unassigned

Status

()

Core
Disability Access APIs
2 years ago
5 months ago

People

(Reporter: Joanmarie Diggs, 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

2 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

2 years ago
Created attachment 8724609 [details]
Output from performing steps in opening report

Updated

2 years ago
Blocks: 531850

Updated

2 years ago
Keywords: perf

Comment 2

7 months 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

Comment 3

6 months ago
(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

5 months 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.