Closed Bug 1702820 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 28358 - Removed scroll start and end offset properties of ScrollTimeline

Categories

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

task

Tracking

(firefox90 fixed)

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Olga Gerchikov <gerchiko@microsoft.com>' wrote:

Removed scroll start and end offset properties of ScrollTimeline

Made modifications spec'ed in
https://github.com/w3c/csswg-drafts/pull/5803:

  • Removed startOffset and endOffset attributes.
  • Updated procedures based on spec changes.

Bug: 1094014
Change-Id: Id08c9ce1818bd561ac291ca3e1f99c22dbb15674

Reviewed-on: https://chromium-review.googlesource.com/2801290
WPT-Export-Revision: 7930dd528890e6c56cdc3f784a1022060ed42b01

CI Results

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

Total 3 tests and 2 subtests

Status Summary

Firefox

OK : 2
FAIL : 22
ERROR: 1

Chrome

OK : 3
PASS : 74
FAIL : 1

Safari

OK : 2
FAIL : 22
ERROR: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/scroll-animations/constructor.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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: ERROR)
A ScrollTimeline can be created with a scrollSource: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline can be created with a non-scrolling scrollSource: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created with a null scrollSource should have no scrollSource: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created without a scrollSource should use the document.scrollingElement: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created with the default orientation should default to 'block': FAIL (Chrome: PASS, Safari: FAIL)
'block' is a valid orientation value: FAIL (Chrome: PASS, Safari: FAIL)
'inline' is a valid orientation value: FAIL (Chrome: PASS, Safari: FAIL)
'horizontal' is a valid orientation value: FAIL (Chrome: PASS, Safari: FAIL)
'vertical' is a valid orientation value: FAIL (Chrome: PASS, Safari: FAIL)
Creating a ScrollTimeline with an invalid orientation value should throw: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created with the default scrollOffsets should default to []: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created with empty scrollOffsets should resolve to []: FAIL (Chrome: PASS, Safari: FAIL)
A ScrollTimeline created with last 'auto' offset in scrollOffsets should be allowed.: FAIL (Chrome: PASS, Safari: FAIL)
Creating a ScrollTimeline with an invalid scrollOffsets value should throw: FAIL (Chrome: PASS, Safari: FAIL)
Creating a ScrollTimeline with an scrollOffsets value of [CSS.percent(20), 'auto', CSS.percent(50)] should throw: FAIL (Chrome: PASS, Safari: FAIL)
/scroll-animations/css/at-scroll-timeline-ignored.tentative.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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)
Changing animation-timeline changes the timeline (sanity check): FAIL (Chrome: PASS, Safari: FAIL)
animation-timeline ignored after setting timeline with JS (ScrollTimeline from JS): FAIL (Chrome: PASS, Safari: FAIL)
animation-timeline ignored after setting timeline with JS (ScrollTimeline from CSS): FAIL (Chrome: PASS, Safari: FAIL)
animation-timeline ignored after setting timeline with JS (document timeline): FAIL (Chrome: PASS, Safari: FAIL)
animation-timeline ignored after setting timeline with JS (null): FAIL (Chrome: PASS, Safari: FAIL)
/scroll-animations/multiple-scroll-offsets.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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)
currentTime calculations when multiple scroll offsets are specified: FAIL (Chrome: PASS, Safari: FAIL)
currentTime calculations when overlapping scroll offsets are specified: FAIL (Chrome: FAIL, Safari: FAIL)

Tests Disabled in Gecko Infrastructure

/scroll-animations/constructor.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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: ERROR)
/scroll-animations/css/at-scroll-timeline-ignored.tentative.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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-animations/multiple-scroll-offsets.html: OK [GitHub], SKIP [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-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)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2f45c48914bf [wpt PR 28358] - Removed scroll start and end offset properties of ScrollTimeline, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.