Closed
Bug 1424657
Opened 3 years ago
Closed 3 years ago
After bug 1422201, webextension popup documents no longer accessible
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | fixed |
firefox59 | --- | verified |
People
(Reporter: Jamie, Assigned: Jamie)
References
Details
(Keywords: regression)
Attachments
(2 files)
59 bytes,
text/x-review-board-request
|
eeejay
:
review+
|
Details |
3.56 KB,
patch
|
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Spun off bug 1422201 comment 21. STR (with NVDA): 1. Try to open grease monkey, or lastpass from the bar of addons. Expected: a browse mode document with NVDA. Actual: NVDA just reports unknown. These popup documents have their own HWND. Therefore, the root accessible of that HWND needs to handle accChild for ids in remote documents therein. Unfortunately, this isn't the "root" accessible, since the root accessible is the root of the main HWND. The root check needs to be tweaked so that it allows for the root accessible for any HWND.
Assignee | ||
Updated•3 years ago
|
status-firefox58:
--- → affected
status-firefox59:
--- → affected
Assignee | ||
Updated•3 years ago
|
status-firefox57:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment hidden (mozreview-request) |
Comment 2•3 years ago
|
||
mozreview-review |
Comment on attachment 8936327 [details] Bug 1424657: Make IAccessible::accChild handle remote ids for popup windows. https://reviewboard.mozilla.org/r/207052/#review213016 This looks great!
Attachment #8936327 -
Flags: review?(eitan) → review+
Pushed by jteh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b27e2fb6c88f Make IAccessible::accChild handle remote ids for popup windows. r=eeejay
Comment 4•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b27e2fb6c88f
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee | ||
Comment 5•3 years ago
|
||
Bug 1422201 changed GetIAccessibleFor so it only handles remote ids when called on the root accessible. However, this breaks webextension popup documents. These popups have their own HWND, so the root accessible of that HWND needs to handle accChild for ids in remote documents within that HWND. Therefore, expand the restriction to cover the root accessible of any HWND, not just the main HWND. MozReview-Commit-ID: 69v4XSeQLcS
Assignee | ||
Comment 6•3 years ago
|
||
Verified fixed in Firefox 59.0a1 20171213220121 64 bit.
Assignee | ||
Comment 7•3 years ago
|
||
Comment on attachment 8936658 [details] [diff] [review] Make IAccessible::accChild handle remote ids for popup windows. (Patch for 58.) r=eeejay Approval Request Comment [Feature/Bug causing the regression]: Bug 1422201. [User impact if declined]: Accessibility clients will not be able to access webextension popup documents. [Is this code covered by automated tests?]: No, because we don't have a mechanism for platform accessibility testing. [Has the fix been verified in Nightly?]: Yes. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: None. [Is the change risky?]: Low risk. [Why is the change risky/not risky?]: Only affects accessibility. Straightforward patch. Easy to verify. [String changes made/needed]: None.
Attachment #8936658 -
Flags: approval-mozilla-beta?
Comment on attachment 8936658 [details] [diff] [review] Make IAccessible::accChild handle remote ids for popup windows. (Patch for 58.) r=eeejay Fix for recent regression in a11y, let's uplift to beta.
Attachment #8936658 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
![]() |
||
Comment 9•3 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/59d404cb212d
Updated•2 years ago
|
Assignee: nobody → jteh
You need to log in
before you can comment on or make changes to this bug.
Description
•