Closed
Bug 1873960
Opened 1 year ago
Closed 1 year ago
Land test for Bug 1857430
Categories
(Firefox :: Security, task, P1)
Firefox
Security
Tracking
()
RESOLVED
FIXED
123 Branch
People
(Reporter: emz, Assigned: emz)
References
Details
(Keywords: sec-other, Whiteboard: [adv-main123-])
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•1 year ago
|
||
Depends on D192370
Updated•1 year ago
|
Assignee: nobody → pbz
Status: NEW → ASSIGNED
Pushed by pzuhlcke@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/056eee9ef326
Test. r=Gijs
![]() |
||
Comment 3•1 year ago
|
||
Backed out for mochitest and eslint failures:
https://hg.mozilla.org/integration/autoland/rev/1c986738e75771e534650c275b3127323750fa2b
browser/base/content/test/fullscreen/browser_fullscreen_permissions_prompt.js:0 | This file needs formatting with Prettier (use 'mach lint --fix <path>'). (prettier)
[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)
Assignee | ||
Comment 4•1 year ago
|
||
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)
Pushed by pzuhlcke@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/96407685e757
Test. r=Gijs
![]() |
||
Comment 6•1 year ago
|
||
Group: firefox-core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox123:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
Updated•1 year ago
|
status-firefox121:
--- → wontfix
status-firefox122:
--- → wontfix
Updated•1 year ago
|
QA Whiteboard: [post-critsmash-triage]
Flags: qe-verify-
Updated•1 year ago
|
Updated•1 year ago
|
Whiteboard: [adv-main123-]
Updated•5 months ago
|
Group: core-security-release
You need to log in
before you can comment on or make changes to this bug.
Description
•