searching for reviewer(yzen)
532dcdd149e5a6af44b0fa75bd069066c75b2bb4: Bug 1673882 - [devtools] Fix DOMNode icon in a11y panel sidebar. r=yzen.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Thu, 29 Oct 2020 14:52:11 +0000 - rev 555090
Push 37903 by apavel@mozilla.com at Fri, 30 Oct 2020 03:48:30 +0000
Bug 1673882 - [devtools] Fix DOMNode icon in a11y panel sidebar. r=yzen. The panel was using the `highlighterFront` property of the nodeFront, but it was removed in Bug 1623667, and as a result, highlighting was broken. To be consistent with the other panels having such capabilities (console, debugger, …), we now use the toolbox `getHighlighter` function. This wasn't caught because there wasn't any test covering this behavior; this patch adds one, that was failing without the fix in Accessible. Differential Revision: https://phabricator.services.mozilla.com/D95030
cc8134ef17713f45d589222281945b9f8352333f: Bug 1672937 - [devtools] Remove l10n fixtures in accessibility jest tests. r=yzen.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Sat, 24 Oct 2020 06:05:07 +0000 - rev 554775
Push 37898 by abutkovits@mozilla.com at Wed, 28 Oct 2020 09:24:21 +0000
Bug 1672937 - [devtools] Remove l10n fixtures in accessibility jest tests. r=yzen. Those fixtures aren't needed anymore since the l10n module supports Node environment. Tests and stubs were updated accordingly so we now have the actual properties values instead of their keys. Depends on D94571 Differential Revision: https://phabricator.services.mozilla.com/D94572
9d74ce5f7e0476e7bcde908d0cb495a145364026: Bug 1665751 Part 3: Rewrite textSelection/test_general.html to use async await instead of event queue, r=yzen DONTBUILD
Marco Zehe <mzehe@mozilla.com> - Mon, 21 Sep 2020 18:34:45 +0000 - rev 549552
Push 37802 by rmaries@mozilla.com at Tue, 22 Sep 2020 09:45:38 +0000
Bug 1665751 Part 3: Rewrite textSelection/test_general.html to use async await instead of event queue, r=yzen DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D90860
ac97b9243102a1e12d633f8b4f676eb8536ed01f: Bug 1665751 Part 2: Factor out the Select All And Focus element sequence into a utility function for use by other test files, r=yzen DONTBUILD
Marco Zehe <mzehe@mozilla.com> - Mon, 21 Sep 2020 18:34:27 +0000 - rev 549551
Push 37802 by rmaries@mozilla.com at Tue, 22 Sep 2020 09:45:38 +0000
Bug 1665751 Part 2: Factor out the Select All And Focus element sequence into a utility function for use by other test files, r=yzen DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D90859
6917838bb7739d6467c634e0b46611f987e58c32: Bug 1665751 - Convert test_userinput.html to promisified events, r=yzen DONTBUILD
Marco Zehe <mzehe@mozilla.com> - Sat, 19 Sep 2020 06:49:35 +0000 - rev 549364
Push 37795 by ccoroiu@mozilla.com at Sat, 19 Sep 2020 09:34:53 +0000
Bug 1665751 - Convert test_userinput.html to promisified events, r=yzen DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D90682
ab4e18b490a3c3ca286c5e8cfd318a6729a12dc4: Bug 1660359 part 2: Allow a11y focus on a remote print preview OuterDocAccessible. r=jwatt,yzen
James Teh <jteh@mozilla.com> - Thu, 27 Aug 2020 06:12:25 +0000 - rev 546902
Push 37737 by csabou@mozilla.com at Sat, 29 Aug 2020 09:12:26 +0000
Bug 1660359 part 2: Allow a11y focus on a remote print preview OuterDocAccessible. r=jwatt,yzen Print preview documents don't get DocAccessibles because this currently causes crashes and doesn't provide much value. However, we still want to tell a11y clients something useful when a print preview document gets focus, rather than a11y focus just going nowhere. Therefore, we allow a11y focus to land on the OuterDocAccessible (browser element) in this case. Differential Revision: https://phabricator.services.mozilla.com/D87997
e3b7b1a14ab1d0f25622d030bc882451915e62f0: Bug 1660359 part 1: Unify rejection of a11y focus on remote OuterDocAccessibles. r=yzen
James Teh <jteh@mozilla.com> - Thu, 27 Aug 2020 06:12:29 +0000 - rev 546711
Push 37736 by apavel@mozilla.com at Fri, 28 Aug 2020 15:31:26 +0000
Bug 1660359 part 1: Unify rejection of a11y focus on remote OuterDocAccessibles. r=yzen Before Fission, FocusManager::FocusedDOMNode rejected a11y focus on remote XUL browsers by checking EventStateManager::IsRemoteTarget. In bug 1594623, code was added to FocusManager::ProcessDOMFocus to prevent a11y focus on OuterDocAccessibles in order to reject focus on OOP iframes. In bug 1635784, EventStateManager::IsRemoteTarget was renamed to EventStateManager::IsTopLevelRemoteTarget, and EventStateManager::IsRemoteTarget now checks for OOP iframes as well. This allows us to unify rejection of a11y focus on remote OuterDocAccessibles in FocusManager::FocusedDOMNode. Differential Revision: https://phabricator.services.mozilla.com/D87996
697accb3ceb382cb613d9ded02f115caa3d71de0: Bug 1660359 part 2: Allow a11y focus on a remote print preview OuterDocAccessible. r=jwatt,yzen
James Teh <jteh@mozilla.com> - Wed, 26 Aug 2020 13:49:43 +0000 - rev 546367
Push 37734 by ccoroiu@mozilla.com at Thu, 27 Aug 2020 09:30:43 +0000
Bug 1660359 part 2: Allow a11y focus on a remote print preview OuterDocAccessible. r=jwatt,yzen Print preview documents don't get DocAccessibles because this currently causes crashes and doesn't provide much value. However, we still want to tell a11y clients something useful when a print preview document gets focus, rather than a11y focus just going nowhere. Therefore, we allow a11y focus to land on the OuterDocAccessible (browser element) in this case. Differential Revision: https://phabricator.services.mozilla.com/D87997
fc650bd7d6b8ae03713e90193c111f9d9aa83f9a: Bug 1660359 part 1: Unify rejection of a11y focus on remote OuterDocAccessibles. r=yzen
James Teh <jteh@mozilla.com> - Mon, 24 Aug 2020 17:17:19 +0000 - rev 546366
Push 37734 by ccoroiu@mozilla.com at Thu, 27 Aug 2020 09:30:43 +0000
Bug 1660359 part 1: Unify rejection of a11y focus on remote OuterDocAccessibles. r=yzen Before Fission, FocusManager::FocusedDOMNode rejected a11y focus on remote XUL browsers by checking EventStateManager::IsRemoteTarget. In bug 1594623, code was added to FocusManager::ProcessDOMFocus to prevent a11y focus on OuterDocAccessibles in order to reject focus on OOP iframes. In bug 1635784, EventStateManager::IsRemoteTarget was renamed to EventStateManager::IsTopLevelRemoteTarget, and EventStateManager::IsRemoteTarget now checks for OOP iframes as well. This allows us to unify rejection of a11y focus on remote OuterDocAccessibles in FocusManager::FocusedDOMNode. Differential Revision: https://phabricator.services.mozilla.com/D87996
9e2d306e37b5fe088bafa7edea23de9771bd42f8: Bug 1629890 - Convert accessible isDestroyed getter to Actor::isDestroyed() r=yzen,nchevobbe
Julian Descottes <jdescottes@mozilla.com> - Mon, 10 Aug 2020 18:56:38 +0000 - rev 544387
Push 37694 by dluca@mozilla.com at Wed, 12 Aug 2020 21:49:48 +0000
Bug 1629890 - Convert accessible isDestroyed getter to Actor::isDestroyed() r=yzen,nchevobbe Depends on D86324 Differential Revision: https://phabricator.services.mozilla.com/D86325
b9141033e4a4a4ef463f31477396aae72858a2d1: Bug 1639328 - Fix browser_hidden_iframe.js. r=yzen
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 11 Aug 2020 02:17:15 +0000 - rev 544288
Push 37692 by cbrindusan@mozilla.com at Wed, 12 Aug 2020 03:44:18 +0000
Bug 1639328 - Fix browser_hidden_iframe.js. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D86591
23130282a937f703c23003353b362a503be6e182: Bug 1639328 - Fix browser_hidden_iframe.js. r=yzen
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 10 Aug 2020 23:00:10 +0000 - rev 544185
Push 37689 by ncsoregi@mozilla.com at Tue, 11 Aug 2020 09:17:31 +0000
Bug 1639328 - Fix browser_hidden_iframe.js. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D86591
f3a845bd10aaabd0bfe8711bb4c6ac2df5d927a6: Bug 1657744 - Don't fail when forcing e10s off on a11y browser tests. r=yzen DONTBUILD
Eitan Isaacson <eitan@monotonous.org> - Thu, 06 Aug 2020 23:19:23 +0000 - rev 543686
Push 37679 by btara@mozilla.com at Fri, 07 Aug 2020 09:31:58 +0000
Bug 1657744 - Don't fail when forcing e10s off on a11y browser tests. r=yzen DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D86271
3e441eedb6579084d36b847b7c5eab06d1449445: Bug 1647666: test_doc_busy.html: Handle the case where the busy change and not-busy change events are coalesced out of existence. r=yzen
James Teh <jteh@mozilla.com> - Sun, 12 Jul 2020 14:56:23 +0000 - rev 540095
Push 37591 by nbeleuzu@mozilla.com at Sun, 12 Jul 2020 21:44:58 +0000
Bug 1647666: test_doc_busy.html: Handle the case where the busy change and not-busy change events are coalesced out of existence. r=yzen Also removed debug logging, as well as about:blank check which turned out not to be the cause of the timeout. Differential Revision: https://phabricator.services.mozilla.com/D82137
5db8f5164ed6731aca5fb2c4adf1c5eb0d7f6fdc: Bug 1648069 - Make the Changes view a document for screen readers, r=yzen
Marco Zehe <mzehe@mozilla.com> - Fri, 26 Jun 2020 00:15:21 +0000 - rev 537554
Push 37544 by dluca@mozilla.com at Fri, 26 Jun 2020 09:45:40 +0000
Bug 1648069 - Make the Changes view a document for screen readers, r=yzen Also make it focusable so screen readers will pick up even the "no changes yet" scenario and won't skip over it. Differential Revision: https://phabricator.services.mozilla.com/D80865
7ba87dd39ebb9cfa253e8f6fe4455516dd2d502c: Bug 1630208 part 4: Re-enable accessible/tests/browser/hittest/browser_test_zoom_text.js. r=yzen
James Teh <jteh@mozilla.com> - Wed, 17 Jun 2020 18:42:48 +0000 - rev 536226
Push 37517 by malexandru@mozilla.com at Thu, 18 Jun 2020 04:43:29 +0000
Bug 1630208 part 4: Re-enable accessible/tests/browser/hittest/browser_test_zoom_text.js. r=yzen The intermittent leak has been fixed, so we can now re-enable this. Differential Revision: https://phabricator.services.mozilla.com/D79782
11f1206f750698c94dc6384973a0e4d6d019858f: Bug 1630208 part 3: Clean up the remote XPC document accessible cache on shutdown. r=yzen
James Teh <jteh@mozilla.com> - Wed, 17 Jun 2020 18:42:30 +0000 - rev 536225
Push 37517 by malexandru@mozilla.com at Thu, 18 Jun 2020 04:43:29 +0000
Bug 1630208 part 3: Clean up the remote XPC document accessible cache on shutdown. r=yzen The cache should already be empty by this point unless there's a bug, but we should still clean up regardless. Also, the hashtable itself was previously never cleaned up even if it was empty. Differential Revision: https://phabricator.services.mozilla.com/D79781
ff44409be7643400d9fe5d6c5bbb203abc382862: Bug 1630208 part 2: Drop events for DocAccessibleParents which are already shut down. r=yzen
James Teh <jteh@mozilla.com> - Wed, 17 Jun 2020 18:40:42 +0000 - rev 536224
Push 37517 by malexandru@mozilla.com at Thu, 18 Jun 2020 04:43:29 +0000
Bug 1630208 part 2: Drop events for DocAccessibleParents which are already shut down. r=yzen We can receive IPC events after a DocAccessibleParent is shut down if the BrowserParent is in the process of being destroyed (probably the tab closed) but there are still events in the IPC queue. Most Recv*Event methods in BrowserParent check mShutdown, but a few don't. For the events that don't, if the event is for a document, we'll successfully fire the platform event, and then successfully get and cache an xpcAccessibleDocument. However, that xpcAccessibleDocument will never be removed from the cache because it's already shut down, so NotifyOfRemoteDocShutdown (which would normally remove it from the XPC cache) won't get called. This results in a leaked object. Thus, it's important that all Recv*Event methods drop the event if mShutdown is true. This patch adds that check to the methods which didn't have it already. Differential Revision: https://phabricator.services.mozilla.com/D79780
085812b8255e76f7de4f9f05b035fa2b97b0b52f: Bug 1630208 part 1: Assert that we don't add a shut down DocAccessibleParent to the remote XPC cache in DocManager::GetXPCDocument. r=yzen
James Teh <jteh@mozilla.com> - Wed, 17 Jun 2020 18:37:58 +0000 - rev 536223
Push 37517 by malexandru@mozilla.com at Thu, 18 Jun 2020 04:43:29 +0000
Bug 1630208 part 1: Assert that we don't add a shut down DocAccessibleParent to the remote XPC cache in DocManager::GetXPCDocument. r=yzen We really want to know about this because the document will never be removed from the cache, since NotifyOfRemoteDocShutdown (which would normally remove it from the XPC cache) won't get called. Differential Revision: https://phabricator.services.mozilla.com/D79779
3435100c8a79141bc3ac99ded0cb2fecbae97326: Bug 1644669 part 2 - Make the flex container view more accessible to screen reader users, r=yzen
Marco Zehe <mzehe@mozilla.com> - Fri, 12 Jun 2020 17:39:44 +0000 - rev 535543
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644669 part 2 - Make the flex container view more accessible to screen reader users, r=yzen This patch does the following: 1. Both the flex-direction and flex-wrap badges are being treated as illustrations, not generic containers, so screen readers immediately pick up the full meaning from the title. 2. Turns the flex item list sub heading into an actual heading for screen readers. It is level 3 because the accordeon headings are all level 2. Differential Revision: https://phabricator.services.mozilla.com/D79129
f495c9163247176a0c7c45016a763ae11095e6c5: Bug 1644669 Part 1 - Label the button to return to the flex container view for screen readers, r=yzen
Marco Zehe <mzehe@mozilla.com> - Fri, 12 Jun 2020 14:50:30 +0000 - rev 535542
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644669 Part 1 - Label the button to return to the flex container view for screen readers, r=yzen Differential Revision: https://phabricator.services.mozilla.com/D79128
3fefa1d023a6f62cc899f02539cce31edd88bd41: Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen
Morgan Reschenberg <mreschenberg@mozilla.com> - Fri, 12 Jun 2020 17:02:45 +0000 - rev 535518
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen Differential Revision: https://phabricator.services.mozilla.com/D72184
f55dacce61a217fbf9ff3b54c85c4db2ed7d0032: Bug 1644693 - Grid accordeon: Give the section headers proper heading roles and levels for better structure, r=yzen
Marco Zehe <mzehe@mozilla.com> - Fri, 12 Jun 2020 15:18:40 +0000 - rev 535485
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644693 - Grid accordeon: Give the section headers proper heading roles and levels for better structure, r=yzen Differential Revision: https://phabricator.services.mozilla.com/D79131
2909e1179d0d1c6de3d1d2be92c3aee08d3384fd: Bug 1644343 Part 3 - Make the box model understandable to screen reader users, r=yzen,flod
Marco Zehe <mzehe@mozilla.com> - Fri, 12 Jun 2020 12:08:09 +0000 - rev 535455
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644343 Part 3 - Make the box model understandable to screen reader users, r=yzen,flod This patch adds IDs to all the box divs so they can be referenced by ARIA attributes. The actual legend spans also act as region landmarks with an associated level so the nesting is relayed. In addition, each of those boxes gets the buttons for its dimensions rendered in-place so for assistive technologies, this all follows a logical flow. aria-owns changes the accessibility tree so the nesting and associated properties are grouped together. The actual buttons also receive a localizable aria-label that is a combination of the button text and title, so screen reader users not only hear "0 button" without knowing immediately what that button actually belongs to. Title is secondary information. Differential Revision: https://phabricator.services.mozilla.com/D78953
d4f83af5e5246671a95ff70c8bbbd67bc2c02fc7: Bug 1644343 Part 2 - Make the dimension information in the box model more explicit for screen readers, r=yzen,flod
Marco Zehe <mzehe@mozilla.com> - Thu, 11 Jun 2020 10:54:45 +0000 - rev 535454
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644343 Part 2 - Make the dimension information in the box model more explicit for screen readers, r=yzen,flod This patch turns the info that holds the dimensions and position info into a region. Because the visual representation is without explanatory text, an aria-label is constructed for that region to hold a more explicit version of the displayed information. Differential Revision: https://phabricator.services.mozilla.com/D78952
f9f69dac57bee05981f9f38944f184c7e53140cc: Bug 1644343 Part 1 - Make layout properties accessible to screen readers, r=yzen
Marco Zehe <mzehe@mozilla.com> - Thu, 11 Jun 2020 15:31:08 +0000 - rev 535453
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1644343 Part 1 - Make layout properties accessible to screen readers, r=yzen 1. Fix the twisty so it has a role and name that reflects the action the button will take when clicked (show or hide). 2. Give the section itself a heading so it can easily be found. Make it a level 3 so it is clear that this is a sub section of the box model view. 3. Order the properties themselves in a table with rows for each property, and cells for the name and value. 4. Make the property names and values, which are not editable, not clickable. Differential Revision: https://phabricator.services.mozilla.com/D78951
f1f9e78e6a1290ab7427776bb59f2a87311e6dab: Bug 1596018 - Mark all presentational divs that might become scrollable, with a tabindex of -1 so they are not reached when tabbing, r=yzen
Marco Zehe <mzehe@mozilla.com> - Fri, 12 Jun 2020 11:11:50 +0000 - rev 535437
Push 37501 by nbeleuzu@mozilla.com at Sat, 13 Jun 2020 03:21:52 +0000
Bug 1596018 - Mark all presentational divs that might become scrollable, with a tabindex of -1 so they are not reached when tabbing, r=yzen Differential Revision: https://phabricator.services.mozilla.com/D79019
04b11bcd330ec9e5803cdf54f35673e506a7685c: Bug 1596018 - Mark all presentational divs that might become scrollable, with a tabindex of -1 so they are not reached when tabbing, r=yzen
Marco Zehe <mzehe@mozilla.com> - Thu, 11 Jun 2020 14:55:24 +0000 - rev 535109
Push 37498 by apavel@mozilla.com at Fri, 12 Jun 2020 03:05:25 +0000
Bug 1596018 - Mark all presentational divs that might become scrollable, with a tabindex of -1 so they are not reached when tabbing, r=yzen Differential Revision: https://phabricator.services.mozilla.com/D79019
b1ea583f8cb86bf703b37e0f6bbee151e9a85c1a: Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen
Morgan Reschenberg <mreschenberg@mozilla.com> - Wed, 10 Jun 2020 20:40:33 +0000 - rev 534970
Push 37496 by btara@mozilla.com at Thu, 11 Jun 2020 04:04:33 +0000
Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen Differential Revision: https://phabricator.services.mozilla.com/D72184
c15ccd143f3267081423ac6f59e5341918e344a9: Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen
Morgan Reschenberg <mreschenberg@mozilla.com> - Tue, 09 Jun 2020 21:56:01 +0000 - rev 534771
Push 37494 by nbeleuzu@mozilla.com at Wed, 10 Jun 2020 14:02:18 +0000
Bug 1630391: Add telemetry test for backplate and a11y theme scalars. r=chutten,yzen Differential Revision: https://phabricator.services.mozilla.com/D72184
3d1e9c77a42dec977bd7a22e2668af56b2587145: Bug 1644127 - Turn the layout side pane into a document for screen readers, r=yzen
Marco Zehe <mzehe@mozilla.com> - Mon, 08 Jun 2020 16:53:03 +0000 - rev 534579
Push 37491 by apavel@mozilla.com at Tue, 09 Jun 2020 09:21:34 +0000
Bug 1644127 - Turn the layout side pane into a document for screen readers, r=yzen This will make it navigable with screen reader commands. Depends on D78674 Differential Revision: https://phabricator.services.mozilla.com/D78675
be296969313c0ff1a85e6eb1811f89643c38f715: Bug 977244 - Make the rules view easier to navigate with a screen reader, r=yzen
Marco Zehe <mzehe@mozilla.com> - Mon, 08 Jun 2020 17:08:53 +0000 - rev 534578
Push 37491 by apavel@mozilla.com at Tue, 09 Jun 2020 09:21:34 +0000
Bug 977244 - Make the rules view easier to navigate with a screen reader, r=yzen To give the whole rules view more structure: 1. Make the container a document so screen readers invoke their reading mode. 2. Give the different section headers a proper heading role. 3. Make each separate item an article so screen readers can navigate them more easily. 4. Give the expander twisty for the pseudo classes a proper label that adjusts depending on whether pseudo classes are expanded or collapsed. Also make it a button for screen readers. This patch does not attempt to simulate any advanced navigation via the keyboard. Differential Revision: https://phabricator.services.mozilla.com/D78674
13b2c57cb1fe9ae6735f898a72c4858dc4d5d225: Bug 1151648: Construct accessible objects for SVG elements which have title or desc. r=MarcoZ,yzen
James Teh <jteh@mozilla.com> - Wed, 03 Jun 2020 13:18:38 +0000 - rev 534040
Push 37482 by nbeleuzu@mozilla.com at Fri, 05 Jun 2020 14:35:19 +0000
Bug 1151648: Construct accessible objects for SVG elements which have title or desc. r=MarcoZ,yzen According to SVG Accessibility API Mappings, an SVG element which has title or desc element must be exposed. https://w3c.github.io/svg-aam/#include_elements Previously, we didn't expose <g> elements at all even if they had a title/desc, and we unconditionally exposed some other SVG elements even when they didn't. This removes the Dev Tools A11y Panel code which explicitly allowed unlabelled descendants of role="img" <svg> elements, since we don't create descendants if they don't have a label now anyway. The associated tests had to be tweaked as well, since now we don't create unlabelled descendants. Original patch by Takeshi Kurosawa. Differential Revision: https://phabricator.services.mozilla.com/D77763
f65636dc2b12ab0480672a418adccc64219d8d7a: Bug 1427304: Return the oleacc window accessible for the outer HWND of windows created with popup=yes. r=yzen
James Teh <jteh@mozilla.com> - Fri, 29 May 2020 14:16:14 +0000 - rev 533096
Push 37462 by malexandru@mozilla.com at Sat, 30 May 2020 09:46:43 +0000
Bug 1427304: Return the oleacc window accessible for the outer HWND of windows created with popup=yes. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D77442
553af70f2690c9709bd5492034df0b54de43a526: Bug 1572852: a11y::Logging::LogDocShellTree: Make it clear that this is in-process DocShell tree info. r=yzen
James Teh <jteh@mozilla.com> - Mon, 11 May 2020 14:38:53 +0000 - rev 529831
Push 37416 by csabou@mozilla.com at Thu, 14 May 2020 16:15:31 +0000
Bug 1572852: a11y::Logging::LogDocShellTree: Make it clear that this is in-process DocShell tree info. r=yzen For an OOP iframe, the logged parent will be null and the logged root won't be the real top level document. This is just the way things are and this might still be useful info sometimes, so we still log it, but make it clear in the log message that it's about the in-process tree. Differential Revision: https://phabricator.services.mozilla.com/D74587
fed64af04da307da62a504be6d0b5554afb18566: Bug 1628582: Set DocAccessibleParent::mDocProxyStream on the child document, not the parent document. r=yzen
James Teh <jteh@mozilla.com> - Mon, 11 May 2020 14:41:57 +0000 - rev 529757
Push 37415 by btara@mozilla.com at Thu, 14 May 2020 09:40:44 +0000
Bug 1628582: Set DocAccessibleParent::mDocProxyStream on the child document, not the parent document. r=yzen DocAccessibleParent::AddChildDoc sends a COM proxy for the embedded document to the embedder process hosting the iframe. This gets returned as the child of the embedder OuterDocAccessible. Whenever we send a proxy to a content process, we must hold a PreservedStreamPtr in the parent process. Previously, we set mDocProxyStream on the parent (embedder) document. However, if a document had multiple OOP iframes, this meant we ended up losing the PreservedStreamPtr for all but the last added child OOP document. We now set mDocProxyStream on the child document instead, since there can only be one embedder OuterDocAccessible per child document. Differential Revision: https://phabricator.services.mozilla.com/D74584
ebf2a8fae0c6b0bf1d864a8c7e262c4761b1c071: Bug 1617237 - Part 3: Add the keypress listener for just DevTools' tab bar r=yzen,daisuke
Micah Tigley <mtigley@mozilla.com> - Thu, 07 May 2020 17:22:12 +0000 - rev 528727
Push 37395 by malexandru@mozilla.com at Fri, 08 May 2020 09:35:43 +0000
Bug 1617237 - Part 3: Add the keypress listener for just DevTools' tab bar r=yzen,daisuke `_onToolbarArrowKeypress` is only concerned with moving focus between tabs in the toolbar when the right/left arrow keys are pressed. This patch ensures the listener is only added to the container of the tab elements (.devtools-tabbar). Depends on D70279 Differential Revision: https://phabricator.services.mozilla.com/D72227
4c1410654ab76a76d4b050227b31597e79b3e28e: Bug 1620098: Show HTMLTooltip in html document as XUL popup. r=bgrins,jdescottes,mtigley,yzen,bdahl,nchevobbe
Daisuke Akatsuka <daisuke@birchill.co.jp> - Thu, 07 May 2020 23:33:57 +0000 - rev 528717
Push 37395 by malexandru@mozilla.com at Fri, 08 May 2020 09:35:43 +0000
Bug 1620098: Show HTMLTooltip in html document as XUL popup. r=bgrins,jdescottes,mtigley,yzen,bdahl,nchevobbe Differential Revision: https://phabricator.services.mozilla.com/D67983
358ce0a1518ff49872961866a853977720cc801e: Bug 1617237 - Part 3: Add the keypress listener for just DevTools' tab bar r=yzen,daisuke
Micah Tigley <mtigley@mozilla.com> - Thu, 07 May 2020 15:19:29 +0000 - rev 528647
Push 37392 by apavel@mozilla.com at Thu, 07 May 2020 21:43:47 +0000
Bug 1617237 - Part 3: Add the keypress listener for just DevTools' tab bar r=yzen,daisuke `_onToolbarArrowKeypress` is only concerned with moving focus between tabs in the toolbar when the right/left arrow keys are pressed. This patch ensures the listener is only added to the container of the tab elements (.devtools-tabbar). Depends on D70279 Differential Revision: https://phabricator.services.mozilla.com/D72227
6b397048940f8f6b6dbcacf1d1ddc51f0e46e37e: Bug 1620098: Show HTMLTooltip in html document as XUL popup. r=bgrins,jdescottes,mtigley,yzen,bdahl
Daisuke Akatsuka <daisuke@birchill.co.jp> - Thu, 07 May 2020 02:04:17 +0000 - rev 528540
Push 37389 by dluca@mozilla.com at Thu, 07 May 2020 08:52:31 +0000
Bug 1620098: Show HTMLTooltip in html document as XUL popup. r=bgrins,jdescottes,mtigley,yzen,bdahl Differential Revision: https://phabricator.services.mozilla.com/D67983
fc77f6e49447cef619d28d973120e28271561cda: Bug 1593937 - Disable browser_accessibility_tree_nagivation_oop.js as it fails intermittenly on Fission. r=yzen,jdescottes
Alexandre Poirot <poirot.alex@gmail.com> - Thu, 30 Apr 2020 07:59:44 +0000 - rev 526962
Push 37366 by cbrindusan@mozilla.com at Fri, 01 May 2020 09:42:47 +0000
Bug 1593937 - Disable browser_accessibility_tree_nagivation_oop.js as it fails intermittenly on Fission. r=yzen,jdescottes It also triggers a C++ assertion. Differential Revision: https://phabricator.services.mozilla.com/D72182
8dddeb7310cae05f291e65004d460febe610c383: Bug 1593937 - Prevent fetching NodeFront for already destroyed AccessibleFront. r=yzen
Alexandre Poirot <poirot.alex@gmail.com> - Thu, 30 Apr 2020 07:59:40 +0000 - rev 526960
Push 37366 by cbrindusan@mozilla.com at Fri, 01 May 2020 09:42:47 +0000
Bug 1593937 - Prevent fetching NodeFront for already destroyed AccessibleFront. r=yzen When navigating between process, we happen to destroy the AccessibleFront in middle of this call to `Target.getFront("inspector")`. Before calling updateDetails, we already check if AccessibleFront is already destroyed. This code also checks after the call to getFront. Differential Revision: https://phabricator.services.mozilla.com/D70842
613cf14725fe55f0a4ae9a4fa0f4dc50bcc18f29: Bug 1630391: Remove zoom telemetry, extend backplate and theme to 80. r=yzen
Morgan Reschenberg <mreschenberg@mozilla.com> - Thu, 30 Apr 2020 00:07:54 +0000 - rev 526801
Push 37364 by apavel@mozilla.com at Thu, 30 Apr 2020 08:26:21 +0000
Bug 1630391: Remove zoom telemetry, extend backplate and theme to 80. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D73161
9393d59d234af6b8df0582e8c9360942df1a6d0e: Bug 1619506 part 2: Don't expose NODE_CHILD_OF relation for OOP iframe documents. r=yzen
James Teh <jteh@mozilla.com> - Wed, 08 Apr 2020 18:30:01 +0000 - rev 523021
Push 37296 by csabou@mozilla.com at Thu, 09 Apr 2020 09:55:00 +0000
Bug 1619506 part 2: Don't expose NODE_CHILD_OF relation for OOP iframe documents. r=yzen For OOP iframe documents, we can't support NODE_CHILD_OF in the usual way, since the iframe resides in a different process. Previously, we incorrectly returned the application accessible. This was breaking focus ancestry walking for JAWS. Instead, don't expose NODE_CHILD_OF at all in this case. This is fine because the client will then request the parent instead, which will be correctly handled by platform/AccessibleOrProxy code. Differential Revision: https://phabricator.services.mozilla.com/D70163
2289e59ea19f8124202fc877501efd01a51cfc5a: Bug 1619506 part 1: Implement QueryService to SID_IAccessibleContentDocument for OOP iframes. r=yzen
James Teh <jteh@mozilla.com> - Wed, 08 Apr 2020 18:28:08 +0000 - rev 523020
Push 37296 by csabou@mozilla.com at Thu, 09 Apr 2020 09:55:00 +0000
Bug 1619506 part 1: Implement QueryService to SID_IAccessibleContentDocument for OOP iframes. r=yzen For OOP iframes, the top level document lives in a different process. Previously, we incorrectly returned the top level document in the same process. This was causing JAWS to incorrectly identify OOP iframe documents as separate tab documents. To fix this, we must send the real top level document down from the parent process and return that when requested in the content process. Differential Revision: https://phabricator.services.mozilla.com/D70162
b20baffe6b38d4bc3109f64718f8be1f0bf0593f: Bug 1625396: AccessibleOrProxy::ChildAtPoint: Accessible::ChildAtPoint can return null, so don't try to check for a remote child doc in that case. r=yzen
James Teh <jteh@mozilla.com> - Mon, 30 Mar 2020 13:19:15 +0000 - rev 521204
Push 37268 by apavel@mozilla.com at Tue, 31 Mar 2020 09:35:27 +0000
Bug 1625396: AccessibleOrProxy::ChildAtPoint: Accessible::ChildAtPoint can return null, so don't try to check for a remote child doc in that case. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D68739
6ff6a0401a22efc9e140a3fcdd085349ef642fc4: Bug 1598299 part 2: Remove ProxyAccessible::AccessibleAtPoint. r=yzen,nika
James Teh <jteh@mozilla.com> - Sun, 29 Mar 2020 23:08:44 +0000 - rev 520991
Push 37263 by aciure@mozilla.com at Mon, 30 Mar 2020 09:47:47 +0000
Bug 1598299 part 2: Remove ProxyAccessible::AccessibleAtPoint. r=yzen,nika This wasn't useful cross-platform. ATK was the only consumer of this and it now uses ProxyAccessible::ChildAtPoint. This also means the related aNeedsScreenCoords functionality in PDocAccessible::AccessibleAtPoint is no longer needed and has thus been removed. Finally, this renames PDocAccessible::AccessibleAtPoint to PDocAccessible::ChildAtPoint for consistency with Accessible::ChildAtPoint now that the functionality is mirrored. Differential Revision: https://phabricator.services.mozilla.com/D67987
7951281fed61b6cfd85c79a623324dfe26ecb754: Bug 1598299 part 1: Make atk_component_ref_accessible_at_point return the deepest child instead of the direct child. r=yzen
James Teh <jteh@mozilla.com> - Thu, 26 Mar 2020 15:27:45 +0000 - rev 520990
Push 37263 by aciure@mozilla.com at Mon, 30 Mar 2020 09:47:47 +0000
Bug 1598299 part 1: Make atk_component_ref_accessible_at_point return the deepest child instead of the direct child. r=yzen This changes the ATK code to use AccessibleOrProxy::ChildAtPoint, since that already handles walking into OuterDocAccessibles appropriately. Previously, atk_component_ref_accessible_at_point didn't work at all on OuterDocAccessibles. Also, for ProxyAccessibles, we no longer adjust the coordinates for ATK_XY_WINDOW in the content process, as this depends on stuff that doesn't exist cross-platform and thus can't be used with AccessibleOrProxy. Instead, we now handle this in the parent process before making the IPC call. Differential Revision: https://phabricator.services.mozilla.com/D67986
6edd2e43a3e77dbf3f64233b8a22f778fffe5e87: Bug 1624729 - Notify observer service of mac accessibility events. r=morgan,yzen
Eitan Isaacson <eitan@monotonous.org> - Fri, 27 Mar 2020 17:11:44 +0000 - rev 520852
Push 37259 by dluca@mozilla.com at Sat, 28 Mar 2020 09:32:56 +0000
Bug 1624729 - Notify observer service of mac accessibility events. r=morgan,yzen Differential Revision: https://phabricator.services.mozilla.com/D68127
803031c78559f8ff53bbb77214152da9b5e379c8: Bug 1624729 - Introduce nsIAccessibleMacInterface as mac native xpc interface. r=yzen,morgan
Eitan Isaacson <eitan@monotonous.org> - Fri, 27 Mar 2020 17:15:31 +0000 - rev 520848
Push 37259 by dluca@mozilla.com at Sat, 28 Mar 2020 09:32:56 +0000
Bug 1624729 - Introduce nsIAccessibleMacInterface as mac native xpc interface. r=yzen,morgan Differential Revision: https://phabricator.services.mozilla.com/D68123