979995 - accessible/tests/mochitest/hittest/test_canvas_hitregion.html fails in automation, is disabled

Status: RESOLVED FIXED

(Core :: Graphics: Canvas2D, defect)

Description

[Gregory Szorc [:gps]]

+++ This bug was initially created as a clone of Bug #966591 +++

accessible/tests/mochitest/hittest/test_canvas_hitregion.html was defined as "text_canvas_hitregion.html" (note "text" and not "test") in a11y.ini and thus this test was never getting installed nor executed.

In bug 979640, we changed the build system so typos like this result in a build error. Unfortunately, enabling this test resulted in error. See https://tbpl.mozilla.org/?tree=Fx-Team&rev=593206fbd97e and https://tbpl.mozilla.org/php/getParsedLog.php?id=35671645&tree=Fx-Team&full=1

I disabled the test universally via the manifest to unblock the check-in of the build system patch.

10:56:24     INFO -  412 INFO TEST-START | chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html
10:56:24     INFO -  413 INFO TEST-INFO | dumping last 1 message(s)
10:56:24     INFO -  414 INFO TEST-INFO | if you need more context, please use SimpleTest.requestCompleteLog() in your test
10:56:24     INFO -  415 INFO TEST-INFO | chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html | must wait for load
10:56:24     INFO -  416 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html | Can't get DOM element for hittest
10:56:24     INFO -  417 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html | uncaught exception - TypeError: getNode(...) is null at chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html:46
10:56:24     INFO -  JavaScript error: chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html, line 46: getNode(...) is null
10:56:24     INFO -  418 INFO TEST-INFO | MEMORY STAT vsize after test: 3366707200
10:56:24     INFO -  419 INFO TEST-INFO | MEMORY STAT residentFast after test: 264212480
10:56:24     INFO -  420 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 76100968
10:56:24     INFO -  421 INFO TEST-END | chrome://mochitests/content/a11y/accessible/tests/mochitest/hittest/test_canvas_hitregion.html | finished in 66ms

Comment 1

[Rik Cabanier]

Attachment: Fix for reference to missing element

try run: https://tbpl.mozilla.org/?tree=Try&rev=6b6f84a8ac83

Comment 2

[Gregory Szorc [:gps]]

This patch needs to be rebased on top of https://hg.mozilla.org/integration/fx-team/rev/5e6934522047

Comment 3

[Rik Cabanier]

(In reply to Gregory Szorc [:gps] from comment #2)
> This patch needs to be rebased on top of
> https://hg.mozilla.org/integration/fx-team/rev/5e6934522047

Do I need to pull down that tree and apply the patch there?

Comment 4

[Gregory Szorc [:gps]]

Or wait an hour or two for this to be merged to central and inbound.

Comment 5

[Rik Cabanier]

(In reply to Gregory Szorc [:gps] from comment #4)
> Or wait an hour or two for this to be merged to central and inbound.

Ok.
It will take a couple of hours for the try build anyway :-)

Comment 6

[Rik Cabanier]

Attachment: Fix for reference to missing element

Comment 7

[Rik Cabanier]

try build: https://tbpl.mozilla.org/?tree=Try&rev=98ac5f6f41ea

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0f23c9b295ff

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0f23c9b295ff