1828373 - [CtW] Bounds incorrect when an element becomes scrollable later without reflow

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[James Teh [:Jamie]]

Spun off bug 1809082. Causes problems in LinkedIn and Google Bard.

From bug 1809082 comment 38:

data:text/html,<div style="height:100px; overflow:scroll;"><div id="text"></div></div><button onClick="let html=''; for (let i = 1; i <= 20; ++i) { html += i; html += '<br>'; } document.getElementById('text').innerHTML = html;">Add Text</button>

Click the "Add Text" button, then note the bounds of the "text" id. Scroll the scrollable region to the bottom. Note that the bounds of the text id have not changed accordingly. At this point, hit testing the scrolled text will report results as if it's unscrolled.

The contenteditable bit doesn't matter, but it does seem important that the text be entered after page load.

From bug 1809082 comment 39:

1. What's happening here is that the overflow div isn't getting an Accessible, so we can't cache scroll data on it.
2. Normally, when something is scrollable, it gets an Accessible because its scrollability makes it focusable and focusable things always get Accessibles.
3. In this case, the div wasn't scrollable initially, so we didn't create an Accessible for it.
4. Often, when something becomes scrollable, it causes reflow, in which case we would re-evaluate the div and create an Accessible for it.
5. In this case, I guess it doesn't get reflowed, perhaps because the size of the div itself doesn't change. Thus, we don't create an Accessible for it.

We can fix this by always creating an Accessible for elements which have overflow: scroll or overflow: auto. To do that, we should be able to extend Morgan's creation patch over in bug 1825611.

Comment 1

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1828373: Create accs for elements with overflow:scroll and overflow:auto styling r?nlapre,Jamie

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51f495933300
Create accs for elements with overflow:scroll and overflow:auto styling r=nlapre,Jamie

Comment 3

[Natalia Csoregi [:nataliaCs]]

Backed out with Bug 1825611 and Bug 1825411 for accessible failures on on /browser_general.js

• backout: https://hg.mozilla.org/integration/autoland/rev/8d8d99e7581851de698e27339dffd60ee8dc8e32

Comment 4

[Norisz Fay [:noriszfay]]

Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/8d8d99e75818

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1d00ca0cca3b
Create accs for elements with overflow:scroll and overflow:auto styling r=nlapre,Jamie

Comment 6

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/1d00ca0cca3b

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:morgan, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox114 to wontfix.

For more information, please visit BugBot documentation.

Comment 8

[Morgan Reschenberg [:morgan]]

I think we will want to uplift this, but not yet -- it's a pretty big patch stack and it could use some more time to bake. I'll leave this NI and revisit

Comment 9

[Morgan Reschenberg [:morgan]]

We've deemed this too risky to upload at this point in the cycle, marking 114 as wontfix

Comment 10

[Dianna Smith [:diannaS]]

backed out in 115 beta (115.0rc1) for causing bug 1840200 (backout patch is in bug 1840200)

Comment 11

[James Teh [:Jamie]]

Comment on attachment 9329773 [details]
Bug 1828373: Create accs for elements with overflow:scroll and overflow:auto styling r?nlapre,Jamie

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Needed to fix various accessibility regressions introduced by the Cache the World project which shipped in 115.
• User impact if declined: Screen reader mouse and touch tracking problems on popular sites such as LinkedIn.
• Fix Landed on Version: 116
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): These patches did cause regressions. However, these have since been addressed and those fixes have been baking on release since at least 117.

Comment 12

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr115/rev/beb04a5c3f9a

Comment 13

[Donal Meehan [:dmeehan]]

Comment on attachment 9329773 [details]
Bug 1828373: Create accs for elements with overflow:scroll and overflow:auto styling r?nlapre,Jamie

Approved for 115.4esr.