Closed Bug 1873960 Opened 1 year ago Closed 1 year ago

Land test for Bug 1857430

Categories

(Firefox :: Security, task, P1)

task

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox-esr115 --- fixed
firefox121 --- wontfix
firefox122 --- wontfix
firefox123 --- fixed

People

(Reporter: emz, Assigned: emz)

References

Details

(Keywords: sec-other, Whiteboard: [adv-main123-])

Attachments

(1 file)

No description provided.

Depends on D192370

Assignee: nobody → pbz
Status: NEW → ASSIGNED

Backed out for mochitest and eslint failures:
https://hg.mozilla.org/integration/autoland/rev/1c986738e75771e534650c275b3127323750fa2b

Push with failures

Failure log eslint

browser/base/content/test/fullscreen/browser_fullscreen_permissions_prompt.js:0 | This file needs formatting with Prettier (use 'mach lint --fix <path>'). (prettier)

Failure log mochitest

[task 2024-01-10T17:54:58.536Z] 17:54:58     INFO - TEST-PASS | browser/base/content/test/fullscreen/browser_fullscreen_permissions_prompt.js | at least one notification displayed - 
[task 2024-01-10T17:54:58.538Z] 17:54:58     INFO - Triggering main command for notification web-notifications-notification
[task 2024-01-10T17:54:58.545Z] 17:54:58     INFO - TEST-INFO | started process screentopng
[task 2024-01-10T17:54:58.710Z] 17:54:58     INFO - TEST-INFO | screentopng: exit 0
[task 2024-01-10T17:54:58.710Z] 17:54:58     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/fullscreen/browser_fullscreen_permissions_prompt.js | Notification should still be open because we clicked during the security delay. - 
[task 2024-01-10T17:54:58.710Z] 17:54:58     INFO - Stack trace:
[task 2024-01-10T17:54:58.710Z] 17:54:58     INFO - chrome://mochikit/content/browser-test.js:test_ok:1591
[task 2024-01-10T17:54:58.710Z] 17:54:58     INFO - chrome://mochitests/content/browser/browser/base/content/test/fullscreen/browser_fullscreen_permissions_prompt.js:test_permission_prompt_closes_fullscreen_and_extends_security_delay:228
Flags: needinfo?(pbz)

I think this happens because there is no full screen transition on Linux in CI. Unless we can find a way to force enable the transition, we can probably skip the test there.

Flags: needinfo?(pbz)
Group: firefox-core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
QA Whiteboard: [post-critsmash-triage]
Flags: qe-verify-
Whiteboard: [adv-main123-]
Group: core-security-release
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: