Closed Bug 1724626 Opened 3 years ago Closed 3 years ago

Perma css/cssom/getComputedStyle-pseudo.html | Unknown pseudo-elements throw - assert_throws_js: getComputedStyle with an unknown pseudo-element throws function "() => getComputedStyle(div, "totallynotapseudo")" when Gecko 92 merges to Beta on 2021-08-09

Categories

(Core :: DOM: CSS Object Model, defect)

Product:
Core
Component:
DOM: CSS Object Model
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
93 Branch
Tracking Flags:
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- unaffected
firefox90 --- unaffected
firefox91 --- unaffected
firefox92 + fixed
firefox93 + fixed

People

(Reporter: imoraru, Assigned: aryx)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1724626 - set test expectation to unknown pseudo-element as not throwing outside Nightly. r=emilio DONTBUILD
48 bytes, text/x-phabricator-request
Details | Review

[Tracking Requested - why for this release]:

Central-as-beta simulation: https://treeherder.mozilla.org/jobs?repo=try&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=df952ec0723137f54a2585d58e1615d7c43eef56&selectedTaskRun=eEVRU_l-RtmKu75NyD1ROA.0
How to run these simulations: https://wiki.mozilla.org/Sheriffing/How_To/Beta_simulations#TRUNK_AS_EARLY_BETA
Failure log: https://treeherder.mozilla.org/logviewer?job_id=347777229&repo=try&lineNumber=3879

[task 2021-08-08T10:51:02.035Z] 10:51:02     INFO - TEST-START | /css/cssom/getComputedStyle-pseudo.html
[task 2021-08-08T10:51:02.043Z] 10:51:02     INFO - Closing window d9348008-94bb-495e-ba2e-c2daa0e0138a
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-PASS | /css/cssom/getComputedStyle-pseudo.html | Resolution of width is correct for ::before and ::after pseudo-elements 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-PASS | /css/cssom/getComputedStyle-pseudo.html | Resolution of width is correct for ::before and ::after pseudo-elements of display: contents elements 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-PASS | /css/cssom/getComputedStyle-pseudo.html | Resolution of nonexistent pseudo-element styles 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-PASS | /css/cssom/getComputedStyle-pseudo.html | Resolution of pseudo-element styles in display: none elements 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-PASS | /css/cssom/getComputedStyle-pseudo.html | Item-based blockification of pseudo-elements 
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - TEST-FAIL | /css/cssom/getComputedStyle-pseudo.html | Item-based blockification of nonexistent pseudo-elements - assert_equals: Pseudo-styles of display: flex elements should get blockified expected "block" but got "inline"
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:108:18
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:107:25
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2087:25
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:557:30
[task 2021-08-08T10:51:02.500Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:105:5
[task 2021-08-08T10:51:02.501Z] 10:51:02     INFO - TEST-FAIL | /css/cssom/getComputedStyle-pseudo.html | display: contents on pseudo-elements - assert_equals: display: contents in :before should get reflected on CSSOM expected "contents" but got "block"
[task 2021-08-08T10:51:02.501Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:115:18
[task 2021-08-08T10:51:02.501Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:114:25
[task 2021-08-08T10:51:02.501Z] 10:51:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2087:25
[task 2021-08-08T10:51:02.502Z] 10:51:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:557:30
[task 2021-08-08T10:51:02.502Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:112:5
[task 2021-08-08T10:51:02.503Z] 10:51:02     INFO - TEST-FAIL | /css/cssom/getComputedStyle-pseudo.html | Dynamically change to display: contents on pseudo-elements - assert_equals: display: contents in :before should get reflected on CSSOM expected "contents" but got "block"
[task 2021-08-08T10:51:02.503Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:131:18
[task 2021-08-08T10:51:02.504Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:130:25
[task 2021-08-08T10:51:02.505Z] 10:51:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2087:25
[task 2021-08-08T10:51:02.506Z] 10:51:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:557:30
[task 2021-08-08T10:51:02.507Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:123:5
[task 2021-08-08T10:51:02.507Z] 10:51:02     INFO - TEST-UNEXPECTED-FAIL | /css/cssom/getComputedStyle-pseudo.html | Unknown pseudo-elements throw - assert_throws_js: getComputedStyle with an unknown pseudo-element throws function "() => getComputedStyle(div, "totallynotapseudo")" did not throw
[task 2021-08-08T10:51:02.508Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:141:19
[task 2021-08-08T10:51:02.509Z] 10:51:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2087:25
[task 2021-08-08T10:51:02.509Z] 10:51:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:557:30
[task 2021-08-08T10:51:02.510Z] 10:51:02     INFO - @http://web-platform.test:8000/css/cssom/getComputedStyle-pseudo.html:139:5
[task 2021-08-08T10:51:02.511Z] 10:51:02     INFO - TEST-OK | /css/cssom/getComputedStyle-pseudo.html | took 460ms
[task 2021-08-08T10:51:02.514Z] 10:51:02     INFO - PID 4767 | 1628419862496	Marionette	INFO	Stopped listening on port 55846
[task 2021-08-08T10:51:02.768Z] 10:51:02     INFO - PID 4767 | JavaScript error: resource:///modules/Interactions.jsm, line 230: NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIUserIdleService.removeIdleObserver]
[task 2021-08-08T10:51:02.769Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.771Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.781Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.783Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.785Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.785Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.962Z] 10:51:02     INFO - PID 4767 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-08-08T10:51:02.971Z] 10:51:02     INFO - PID 6310 | 1628419862970	Marionette	INFO	Marionette enabled
[task 2021-08-08T10:51:04.194Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.194Z] 10:51:04     INFO - PID 4767 | Suppressions used:
[task 2021-08-08T10:51:04.194Z] 10:51:04     INFO - PID 4767 |   count      bytes template
[task 2021-08-08T10:51:04.194Z] 10:51:04     INFO - PID 4767 |      12        384 nsComponentManagerImpl
[task 2021-08-08T10:51:04.194Z] 10:51:04     INFO - PID 4767 |       2        288 libfontconfig.so
[task 2021-08-08T10:51:04.195Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.380Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.383Z] 10:51:04     INFO - PID 4767 | Suppressions used:
[task 2021-08-08T10:51:04.383Z] 10:51:04     INFO - PID 4767 |   count      bytes template
[task 2021-08-08T10:51:04.384Z] 10:51:04     INFO - PID 4767 |      12        384 nsComponentManagerImpl
[task 2021-08-08T10:51:04.385Z] 10:51:04     INFO - PID 4767 |       2        288 libfontconfig.so
[task 2021-08-08T10:51:04.385Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.482Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.483Z] 10:51:04     INFO - PID 4767 | Suppressions used:
[task 2021-08-08T10:51:04.483Z] 10:51:04     INFO - PID 4767 |   count      bytes template
[task 2021-08-08T10:51:04.484Z] 10:51:04     INFO - PID 4767 |      12        384 nsComponentManagerImpl
[task 2021-08-08T10:51:04.484Z] 10:51:04     INFO - PID 4767 |       2        288 libfontconfig.so
[task 2021-08-08T10:51:04.485Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.818Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:04.818Z] 10:51:04     INFO - PID 4767 | Suppressions used:
[task 2021-08-08T10:51:04.818Z] 10:51:04     INFO - PID 4767 |   count      bytes template
[task 2021-08-08T10:51:04.818Z] 10:51:04     INFO - PID 4767 |      12        384 nsComponentManagerImpl
[task 2021-08-08T10:51:04.819Z] 10:51:04     INFO - PID 4767 |       2        288 libfontconfig.so
[task 2021-08-08T10:51:04.819Z] 10:51:04     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:06.198Z] 10:51:06     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:06.199Z] 10:51:06     INFO - PID 4767 | Suppressions used:
[task 2021-08-08T10:51:06.199Z] 10:51:06     INFO - PID 4767 |   count      bytes template
[task 2021-08-08T10:51:06.200Z] 10:51:06     INFO - PID 4767 |       9        280 nsComponentManagerImpl
[task 2021-08-08T10:51:06.200Z] 10:51:06     INFO - PID 4767 |     633      18083 libfontconfig.so
[task 2021-08-08T10:51:06.201Z] 10:51:06     INFO - PID 4767 |       1          9 libglib-2.0.so
[task 2021-08-08T10:51:06.201Z] 10:51:06     INFO - PID 4767 |       1        192 mozJSComponentLoader
[task 2021-08-08T10:51:06.202Z] 10:51:06     INFO - PID 4767 |       2         48 XPCWrappedNativeJSOps
[task 2021-08-08T10:51:06.202Z] 10:51:06     INFO - PID 4767 | -----------------------------------------------------
[task 2021-08-08T10:51:06.279Z] 10:51:06     INFO - Browser exited with return code 0
[task 2021-08-08T10:51:06.282Z] 10:51:06     INFO - Closing logging queue
[task 2021-08-08T10:51:06.282Z] 10:51:06     INFO - queue closed
[task 2021-08-08T10:51:06.300Z] 10:51:06     INFO - INFO | runtests.py | ASan using symbolizer at /builds/worker/workspace/build/application/firefox/llvm-symbolizer
[task 2021-08-08T10:51:06.307Z] 10:51:06     INFO - LSan enabled.
[task 2021-08-08T10:51:06.307Z] 10:51:06     INFO - INFO | runtests.py | ASan running in default memory configuration
[task 2021-08-08T10:51:06.308Z] 10:51:06     INFO - UBSan enabled.
[task 2021-08-08T10:51:06.308Z] 10:51:06     INFO - Application command: /builds/worker/workspace/build/application/firefox/firefox --marionette about:blank -profile /tmp/tmpse8y6ghy
[task 2021-08-08T10:51:06.315Z] 10:51:06     INFO - Starting runner
[task 2021-08-08T10:51:07.979Z] 10:51:07     INFO - PID 6472 | 1628419867977	Marionette	INFO	Marionette enabled
[task 2021-08-08T10:51:09.845Z] 10:51:09     INFO - PID 6310 | console.warn: SearchSettings: "get: No settings file exists, new profile?" (new NotFoundError("Could not open the file at /tmp/tmptw4iv8if/search.json.mozlz4", (void 0)))
[task 2021-08-08T10:51:11.515Z] 10:51:11     INFO - PID 6310 | 1628419871514	Marionette	INFO	Listening on port 39131
[task 2021-08-08T10:51:11.897Z] 10:51:11     INFO - TEST-START | /dom/traversal/NodeFilter-constants.html
Flags: needinfo?(emilio)
tracking-firefox92: --- → ?

Bug 1723921 updated the code in line with the spec to throw for invalid
pseudo-elements. This update is only enabled for Nightly, the expectation to
fail to throw for an unknown pseudo-element must be added back for Beta and
Release.

Assignee: nobody → aryx.bugmail
Status: NEW → ASSIGNED
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/2743d27b31a7
set test expectation to unknown pseudo-element as not throwing outside Nightly. r=emilio DONTBUILD
Flags: needinfo?(emilio)

https://hg.mozilla.org/mozilla-central/rev/2743d27b31a7

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox93: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: