1543575 - Fission a11y: Add browser tests for a11y tree and events on OOP iframes

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task, P3)

Description

[James Teh [:Jamie]]

We have browser chrome tests which exercise e10s accessibility. For Fission, we will need some tests which exercise the a11y tree and events where an embedded iframe document is in a different process to its embedder iframe.

Comment 1

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add fission specific tests.

Comment 2

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - stop using frame scripts across accessible/ browser chrome tests. r=jamie

Comment 3

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - stop using ContentTask across accessible/ browser chrome tests. r=jamie

Depends on D47386

Comment 4

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - provide an option for running accessibility browser chrome e10s tests in fission process. r=jamie

Depends on D47387

Comment 5

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add fission coverage for accessibility b-c e10s event tests. r=jamie

Depends on D48549

Comment 6

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add fission coverage for accessibility b-c e10s caching tests. r=jamie

Depends on D48550

Comment 7

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add fission coverage for accessibility b-c e10s tree update tests. r=jamie

Depends on D48551

Comment 8

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add fission specific accessibility tests. r=jamie

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/472cbb99f5ce
stop using frame scripts across accessible/ browser chrome tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/5e5a6ce39829
stop using ContentTask across accessible/ browser chrome tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/5b54e1361ca1
provide an option for running accessibility browser chrome e10s tests in fission process. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/d7d27d08162d
add fission coverage for accessibility b-c e10s event tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/8b92e98ee616
add fission coverage for accessibility b-c e10s caching tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/2dbafadffbd9
add fission coverage for accessibility b-c e10s tree update tests. r=Jamie

Comment 10

[Dorel Luca [:dluca]]

Backed out 6 changesets (bug 1543575) for Mochitest failures in toolkit/content/tests/chrome/test_popup_button.xul. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271761981&repo=autoland&lineNumber=25249

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&selectedJob=271761981&revision=2dbafadffbd9114592c927136992571d57b86bae

Backout:
https://hg.mozilla.org/integration/autoland/rev/078a8b5628639322e0d0353435318072c9a64eb4

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/da16728f9f8c
stop using frame scripts across accessible/ browser chrome tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/af6f7a56fac2
stop using ContentTask across accessible/ browser chrome tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/8f82c25fa54a
provide an option for running accessibility browser chrome e10s tests in fission process. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/5768872b10a0
add fission coverage for accessibility b-c e10s event tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/b698e86b632a
add fission coverage for accessibility b-c e10s caching tests. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/f688dcef278a
add fission coverage for accessibility b-c e10s tree update tests. r=Jamie

Comment 12

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/da16728f9f8c
https://hg.mozilla.org/mozilla-central/rev/af6f7a56fac2
https://hg.mozilla.org/mozilla-central/rev/8f82c25fa54a
https://hg.mozilla.org/mozilla-central/rev/5768872b10a0
https://hg.mozilla.org/mozilla-central/rev/b698e86b632a
https://hg.mozilla.org/mozilla-central/rev/f688dcef278a

Comment 13

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add missing resources to browser.ini files to be able to run b-c a11y tests independently. r=jamie

Comment 14

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add coverage for bounds b-c tests with fission enabled. r=jamie

Depends on D51144

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/45bd2185d996
add missing resources to browser.ini files to be able to run b-c a11y tests independently. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/6d52b0976c8e
add coverage for bounds b-c tests with fission enabled. r=Jamie

Comment 16

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/45bd2185d996
https://hg.mozilla.org/mozilla-central/rev/6d52b0976c8e

Comment 17

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1543575 - add coverage for hit tests b-c tests with fission enabled. r=jamie

Comment 18

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f5a86c52bec6
add fission specific accessibility tests. r=Jamie

Comment 19

[Natalia Csoregi [:nataliaCs]]

Backed out for failures on /browser_hidden_iframe.js

backout: https://hg.mozilla.org/integration/autoland/rev/23e460d5c0212fc7999cd5055841ce7d2055a146

push: https://treeherder.mozilla.org/#/jobs?repo=autoland&searchStr=linux%2Cx64%2Cdebug%2Cmochitests%2Cwith%2Cfission%2Cenabled%2Ctest-linux64%2Fdebug-mochitest-browser-chrome-fis-e10s-7%2Cm-fis%28bc7%29&tochange=d410872412d5564635e2975ebccc2d589baa50a2&fromchange=121d1852d7cc67b9b0aaf2676d0d7753b02b65d6&selectedJob=275400741

failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=275400741&repo=autoland&lineNumber=1774

[task 2019-11-08T20:11:06.423Z] 20:11:06 INFO - TEST-PASS | accessible/tests/browser/fission/browser_hidden_iframe.js | fission document is hidden and should be accessible -
[task 2019-11-08T20:11:06.424Z] 20:11:06 INFO - Show the IFRAME and check that it's now available in the accessibility tree.
[task 2019-11-08T20:11:06.425Z] 20:11:06 INFO - TEST-PASS | accessible/tests/browser/fission/browser_hidden_iframe.js | Recieved reorder event -
[task 2019-11-08T20:11:06.426Z] 20:11:06 INFO - Buffered messages finished
[task 2019-11-08T20:11:06.427Z] 20:11:06 INFO - TEST-UNEXPECTED-FAIL | accessible/tests/browser/fission/browser_hidden_iframe.js | Test timed out -
[task 2019-11-08T20:11:06.428Z] 20:11:06 INFO - GECKO(1170) | MEMORY STAT | vsize 2954MB | residentFast 343MB | heapAllocated 102MB
[task 2019-11-08T20:11:06.429Z] 20:11:06 INFO - TEST-OK | accessible/tests/browser/fission/browser_hidden_iframe.js | took 90428ms

Comment 20

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a79182facdfb
add fission specific accessibility tests. r=Jamie

Comment 21

[Dorel Luca [:dluca]]

Backed out changeset a79182facdfb (bug 1543575) for Browser-chrome failure in tools/profiler/tests/browser/browser_test_profile_single_frame_page_info.js

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=275522310&repo=autoland&lineNumber=25054

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=a79182facdfb66acc72c0be720c8543978d2af5d

Backout:
https://hg.mozilla.org/integration/autoland/rev/72c52c0101cfd102b207224646c3007d6872e65c

Comment 22

[Yura Zenevich [:yzen] (please use "needinfo")]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=63103d5a6fc8282dad50b669f7ba0b7603027269

Comment 23

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/14095bfec9df
add fission specific accessibility tests. r=Jamie

Comment 24

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/14095bfec9df

Comment 25

[Phabricator Automation]

Comment on attachment 9106634 [details]
Bug 1543575 - add coverage for hit tests b-c tests with fission enabled. r=jamie

Revision D51878 was moved to bug 1595979. Setting attachment 9106634 [details] to obsolete.