1615544 - Re-enable test_videocontrols.html for Fission

Status: RESOLVED FIXED

(Toolkit :: Video/Audio Controls, defect, P3)

Description

[Henri Sivonen (:hsivonen)]

Re-enable test_videocontrols.html for Fission.

Comment 1

[Henri Sivonen (:hsivonen)]

Filed ahead of landing bug 1556627 to get the number for this bug.

Comment 2

[:Gijs (out for now; he/him)]

Henri, do you plan on working on this once bug 1556627 lands?

Comment 3

[Henri Sivonen (:hsivonen)]

At least not specifically on this immediately. I'll work on bug 1615548 and bug 1614297 and see what tests those end up fixing.

Comment 4

[Chris Peterson [:cpeterson]]

Tracking for Fission mochitests (M4.1)

nsFocusManager::SetFocus bug 1556627 regressed and disabled this test for Fission, so we now need to fix and re-enable it.

Comment 5

[Henri Sivonen (:hsivonen)]

This bug may or may not actually depend on bug 1614297 or bug 1615548, but those are the ones that have the most direct repro mechanics, so it doesn't make sense to pursue this one before those are fixed.

Comment 6

[:Gijs (out for now; he/him)]

All the deps are fixed, do you know if we can re-enable or if more work is needed here?

Comment 7

[Henri Sivonen (:hsivonen)]

I think it doesn't make sense to spend time on a round of looking at the disabled tests before the know path to bug 1614268 has been fixed.

Comment 8

[Henri Sivonen (:hsivonen)]

Attachment: Bug 1615544 - Re-enable test_videocontrols.html for Fission.

Comment 9

[Henri Sivonen (:hsivonen)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=994f924c6d3e6d410bb0e52aec959afa0770ce87

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1fe75c06b2d7
Re-enable test_videocontrols.html for Fission. r=Gijs

Comment 11

[Cosmin Sabou [:CosminS]]

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