1746007 - Intermittent /css/css-cascade/revert-layer-008.html | 'revert-layer' should revert font-size to 20px and trigger a smooth transition - assert_equals: expected "15px" but got "20px"

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: malexandru [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=361200321&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/K75L3Ma2RFWzrgq3OwdplA/runs/0/artifacts/public/logs/live_backing.log

---

[task 2021-12-14T16:46:03.057Z] 16:46:03     INFO - TEST-START | /css/css-cascade/revert-layer-008.html
[task 2021-12-14T16:46:03.067Z] 16:46:03     INFO - Closing window 03cf833b-c0b1-4178-a01d-7f32fe823c8a
[task 2021-12-14T16:46:03.355Z] 16:46:03     INFO - 
[task 2021-12-14T16:46:03.355Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/layer-statement-before-import.html | insert other rules before the first layer statement without imports 
[task 2021-12-14T16:46:03.355Z] 16:46:03     INFO - TEST-FAIL | /css/css-cascade/layer-statement-before-import.html | deleteRule before imports - can't access property "deleteRule", sheet is null
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - operations@http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:111:7
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - @http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:176:14
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2534:25
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - test@http://web-platform.test:8000/resources/testharness.js:572:30
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - @http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:175:7
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-FAIL | /css/css-cascade/layer-statement-before-import.html | deleteRule after imports - can't access property "deleteRule", sheet is null
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - operations@http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:130:7
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - @http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:176:14
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2534:25
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - test@http://web-platform.test:8000/resources/testharness.js:572:30
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - @http://web-platform.test:8000/css/css-cascade/layer-statement-before-import.html:175:7
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/layer-vs-inline-style.html | Normal inline style > normal layered style 
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/layer-vs-inline-style.html | Normal inline style < important layered style 
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/layer-vs-inline-style.html | Important inline style > normal layered style 
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/layer-vs-inline-style.html | Important inline style > important layered style 
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/presentational-hints-cascade.html | Presentational hints have lower precedence than regular author style sheets 
[task 2021-12-14T16:46:03.356Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/presentational-hints-cascade.html | Presentational hints have lower precedence than the style attribute 
[task 2021-12-14T16:46:03.357Z] 16:46:03     INFO - TEST-PASS | /css/css-cascade/presentational-hints-cascade.html | Presentational hints have lower precedence than layered style 
[task 2021-12-14T16:46:03.357Z] 16:46:03     INFO - TEST-UNEXPECTED-FAIL | /css/css-cascade/revert-layer-008.html | 'revert-layer' should revert font-size to 20px and trigger a smooth transition - assert_equals: expected "15px" but got "20px"
[task 2021-12-14T16:46:03.357Z] 16:46:03     INFO - @http://web-platform.test:8000/css/css-cascade/revert-layer-008.html:39:16
[task 2021-12-14T16:46:03.439Z] 16:46:03     INFO - TEST-OK | /css/css-cascade/revert-layer-008.html | took 382ms
[task 2021-12-14T16:46:04.161Z] 16:46:04     INFO - STDOUT: cleanup aborted: args: /builds/worker/fetches/android-sdk-linux/platform-tools/adb wait-for-device shell rm /data/data/org.mozilla.geckoview.test_runner/files/mozilla/profiles.ini; echo adb_returncode=$?, exitcode: 1, stdout: rm: /data/data/org.mozilla.geckoview.test_runner/files/mozilla/profiles.ini: No such file or directory
[task 2021-12-14T16:46:04.163Z] 16:46:04     INFO - Closing logging queue
[task 2021-12-14T16:46:04.163Z] 16:46:04     INFO - queue closed
[task 2021-12-14T16:46:04.179Z] 16:46:04     INFO - Setting up ssl
[task 2021-12-14T16:46:04.194Z] 16:46:04     INFO - certutil | b''
[task 2021-12-14T16:46:04.209Z] 16:46:04     INFO - certutil | b''
[task 2021-12-14T16:46:04.221Z] 16:46:04     INFO - certutil | b'\nCertificate Nickname                                         Trust Attributes\n                                                             SSL,S/MIME,JAR/XPI\n\nweb-platform-tests                                           CT,, \n'
[task 2021-12-14T16:46:09.572Z] 16:46:09     INFO - STDOUT: timed out waiting for profiles.ini
[task 2021-12-14T16:46:09.737Z] 16:46:09     INFO - adb Granting important runtime permissions to org.mozilla.geckoview.test_runner
[task 2021-12-14T16:46:11.250Z] 16:46:11     INFO - adb launch_application: am start -W -n org.mozilla.geckoview.test_runner/org.mozilla.geckoview.test_runner.TestRunnerActivity -a android.intent.action.MAIN --es env0 MOZ_CRASHREPORTER=1 --es env1 MOZ_CRASHREPORTER_NO_REPORT=1 --es env2 MOZ_CRASHREPORTER_SHUTDOWN=1 --es env3 MOZ_HIDE_RESULTS_TABLE=1 --es env4 MOZ_IN_AUTOMATION=1 --es env5 MOZ_LOG=signaling:3,mtransport:4,DataChannel:4,jsep:4 --es env6 R_LOG_LEVEL=6 --es env7 R_LOG_DESTINATION=stderr --es env8 R_LOG_VERBOSE=1 --es env9 MOZ_PROCESS_LOG=/tmp/tmp1i5al7i0pidlog --es env10 MOZ_DISABLE_NONLOCAL_CONNECTIONS=1 --es env11 STYLO_THREADS=1 --es env12 MOZ_WEBRENDER=1 --es arg0 -no-remote --es arg1 -profile --es arg2 /data/local/tmp/test_root/profile --es arg3 --marionette --es arg4 about:blank --ez use_multiprocess True
[task 2021-12-14T16:46:12.348Z] 16:46:12     INFO - Starting runner
[task 2021-12-14T16:46:13.093Z] 16:46:13     INFO - TEST-START | /css/css-cascade/revert-val-003.html

Comment 1

[Emilio Cobos Álvarez (:emilio)]

The test passes locally and in http://wpt.live/css/css-cascade/revert-layer-008.html, so I suspect what's going on is that automation is very slow to trigger the transition or something? Hiro do you know what's the best way to deal with this? Maybe we should just make the transition duration and delay large enough or so...

Comment 2

[Hiroyuki Ikezoe (:hiro)]

We need to make sure that after-change style is different from before-change style, I am assuming the test author expected this await raf() to ensure the difference, but it doesn't seem to work. Adding an explicit flush should solve it. A patch is coming.

https://treeherder.mozilla.org/jobs?repo=try&revision=1fff525a7b51e8cd0ec54ed14bb183a115f471b6

Comment 3

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1746007 - Do an explicit style flush before triggering a CSS transition. r?emilio

It ensures that after-change style is different from before-change style.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2189fa5e8f0a
Do an explicit style flush before triggering a CSS transition. r=emilio

Comment 5

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/32136 for changes under testing/web-platform/tests

Comment 6

[Intermittent Failures Robot]

41 failures in 3263 pushes (0.013 failures/push) were associated with this bug in the last 7 days.

This is the #25 most frequent failure this week.

** This failure happened more than 30 times this week! Resolving this bug is a high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 2 weeks, the affected test(s) may be disabled. **

Repository breakdown:

• autoland: 16
• mozilla-central: 18
• try: 7

Platform and build breakdown:

• android-em-7-0-x86_64-lite-qr: 3
  • opt: 3
• windows10-32-2004-qr: 3
  • opt: 3
• android-em-7-0-x86_64-shippable-qr: 5
  • opt: 5
• android-em-7-0-x86_64-shippable-lite-qr: 10
  • opt: 10
• macosx1015-64-qr: 4
  • opt: 4
• windows10-64-2004-qr: 4
  • opt: 3
  • debug: 1
• android-em-7-0-x86_64-qr: 1
  • opt: 1
• windows10-64-2004-devedition-qr: 1
  • opt: 1
• windows10-64-2004-shippable-qr: 4
  • opt: 4
• macosx1015-64-shippable-qr: 2
  • opt: 2
• windows10-32-2004-shippable-qr: 3
  • opt: 3
• windows10-64-2004-ccov-qr: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1746007&startday=2021-12-13&endday=2021-12-19&tree=all

Comment 7

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/2189fa5e8f0a

Comment 8

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot