Closed Bug 1659653 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 25064 - [scroll-animations] Use longhands in high-specificity rule

Tracking

(firefox82 fixed)

Status:

RESOLVED FIXED

Milestone:

82 Branch

Tracking Flags:

Tracking

Status

firefox82

---

fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

Assignee

Description

•

5 years ago

Sync web-platform-tests PR 25064 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/25064
Details from upstream follow.

Anders Hartvoll Ruud <andruud@chromium.org> wrote:

[scroll-animations] Use longhands in high-specificity rule

The selector '#container > div' has higher specificity than a single
#id selector, so in a browser which supports setting the animation-
timeline via the animation shorthand (which we don't support yet but
will soon), the animation-timeline always becomes auto. To fix
this, use longhands instead.

This prepares for animation-timeline-in-shorthand support.

Bug: 1074052
Change-Id: I7d8a5ad2a315eb251cc0f42f5a54b1cf3b54ee45

Reviewed-on: https://chromium-review.googlesource.com/2360374
WPT-Export-Revision: 2e8b6a82a4d26fe3126bd4d92923a0c84d377efe

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

Assignee

Updated

•

5 years ago

Whiteboard: [wptsync downstream] → [wptsync downstream error]

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

Assignee

Updated

•

5 years ago

Whiteboard: [wptsync downstream error] → [wptsync downstream]

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

Assignee

Comment 1

•

5 years ago

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=cdf4e82c570470754b4ea33bdbe808090ddd649f

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

Assignee

Comment 2

•

5 years ago

CI Results

Ran 12 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 4 tests and 53 subtests

Status Summary

Firefox

OK : 4
PASS: 5
FAIL: 51

Chrome

OK : 4
PASS: 5
FAIL: 51

Safari

OK : 4
PASS: 5
FAIL: 51

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/scroll-animations/css/at-scroll-timeline-source.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt] (Chrome: OK, Safari: OK)
Source selector(<id-selector>) selects an element: FAIL (Chrome: FAIL, Safari: FAIL)
Source none causes inactive timeline: FAIL (Chrome: FAIL, Safari: FAIL)
Unknown source causes inactive timeline: FAIL (Chrome: FAIL, Safari: FAIL)
Source with no layout box causes inactive timeline: FAIL (Chrome: FAIL, Safari: FAIL)
Source auto selects scrollingElement of the document: FAIL (Chrome: FAIL, Safari: FAIL)
Unspecified source behaves like auto: FAIL (Chrome: FAIL, Safari: FAIL)
/scroll-animations/css/at-scroll-timeline-orientation.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt] (Chrome: OK, Safari: OK)
Orientation vertical behaves as expected: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation block behaves as expected in horizontal writing-mode: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation inline behaves as expected in horizontal writing-mode: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation auto behaves as expected: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation horizontal behaves as expected: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation block behaves as expected in vertical writing-mode: FAIL (Chrome: FAIL, Safari: FAIL)
Orientation inline behaves as expected in vertical writing-mode: FAIL (Chrome: FAIL, Safari: FAIL)
/scroll-animations/css/at-scroll-timeline-time-range.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt] (Chrome: OK, Safari: OK)
Timerange mapped correctly (1s): FAIL (Chrome: FAIL, Safari: FAIL)
Timerange mapped correctly (10s): FAIL (Chrome: FAIL, Safari: FAIL)
Timerange mapped correctly (1000ms): FAIL (Chrome: FAIL, Safari: FAIL)
/scroll-animations/css/at-scroll-timeline-start-end.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt] (Chrome: OK, Safari: OK)
Scroll at offset 131 updates animation correctly [element_50px_auto]: FAIL
Scroll at offset 49 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 51 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 195 updates animation correctly [element_50px_auto]: FAIL
Scroll at offset 169 updates animation correctly [element_25p_75p]: FAIL
Scroll at offset 212 updates animation correctly [element_50px_auto]: FAIL
Scroll at offset 49 updates animation correctly [element_50px_auto]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 106 updates animation correctly [element_auto_auto]: FAIL
Scroll at offset 0 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 100 updates animation correctly [element_auto_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 21 updates animation correctly [element_25p_75p]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 0 updates animation correctly [element_50px_auto]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 50 updates animation correctly [element_auto_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 116 updates animation correctly [element_25p_75p]: FAIL
Scroll at offset 99 updates animation correctly [element_0px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 50 updates animation correctly [element_0px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 100 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 116 updates animation correctly [element_calc_calc]: FAIL
Scroll at offset 100 updates animation correctly [element_0px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 99 updates animation correctly [element_auto_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 1 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 0 updates animation correctly [element_25p_75p]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 190 updates animation correctly [element_auto_auto]: FAIL
Scroll at offset 101 updates animation correctly [element_0px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 63 updates animation correctly [element_calc_calc]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 101 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 1 updates animation correctly [element_0px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 21 updates animation correctly [element_auto_auto]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 212 updates animation correctly [element_25p_75p]: FAIL
Scroll at offset 42 updates animation correctly [element_25p_75p]: FAIL
Scroll at offset 99 updates animation correctly [element_50px_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 179 updates animation correctly [element_calc_calc]: FAIL
Scroll at offset 1 updates animation correctly [element_auto_100px]: FAIL (Chrome: FAIL, Safari: FAIL)
Scroll at offset 212 updates animation correctly [element_auto_auto]: FAIL
Scroll at offset 74 updates animation correctly [element_25p_75p]: FAIL

Tests Disabled in Gecko Infrastructure

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

Assignee

Updated

•

5 years ago

Depends on: 1660313

Pulsebot

Comment 3

•

5 years ago

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f8659f7f1e9c [wpt PR 25064] - [scroll-animations] Use longhands in high-specificity rule, a=testonly

Atila Butkovits

Comment 4

•

5 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/f8659f7f1e9c

Status: NEW → RESOLVED

Closed: 5 years ago

status-firefox82: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → 82 Branch

You need to log in before you can comment on or make changes to this bug.

Bugzilla

[wpt-sync] Sync PR 25064 - [scroll-animations] Use longhands in high-specificity rule

Categories

(Testing :: web-platform-tests, task, P4)

Tracking

(firefox82 fixed)

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Crash Data

Security

(public)

User Story

Description

Updated

Updated

Comment 1

Comment 2

CI Results

Status Summary

Firefox

Chrome

Safari

Links

Details

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

Updated

Comment 3

Comment 4