[webkit-changes] [WebKit/WebKit] 252224: [WPT][WebVTT] Move some WebVTT rendering tests int...

Jer Noble noreply at github.com
Sun Sep 29 12:48:01 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 25222453815607ad36719b01666f6ca52bc821b3
      https://github.com/WebKit/WebKit/commit/25222453815607ad36719b01666f6ca52bc821b3
  Author: Jer Noble <jer.noble at apple.com>
  Date:   2024-09-29 (Sun, 29 Sep 2024)

  Changed paths:
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering-expected.html
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering.html
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering.webvtt
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/basic-cue-rendering-expected.html
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/basic-cue-rendering.html
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/basic-cue-rendering.webvtt
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/reference.css
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/webvtt-rendering-test.js
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/white-space-pre-line.webvtt
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/white-space-pre-line-expected.html
    A LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/white-space-pre-line.html
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/resources/reference.css
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/resources/webvtt-rendering-test.js
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line-expected.html
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line.html
    R LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line.webvtt
    M LayoutTests/platform/glib/TestExpectations

  Log Message:
  -----------
  [WPT][WebVTT] Move some WebVTT rendering tests into the correct location
https://bugs.webkit.org/show_bug.cgi?id=280436
<rdar://problem/136782503>

Reviewed by Eric Carlson.

Other tests inside web-platform-tests/webvtt/rendering are inside the subdirectory
cues-with-video/processing-model/, so move our own tests to that location to
prepare for upstreaming.

Additionally, make some stylistic changes to the test, and conform to WPT mechanisms
for waiting for tests to complete.

* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering-expected.html: Removed.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering.html: Removed.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/basic-cue-rendering-expected.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/basic-cue-rendering.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/basic-cue-rendering.webvtt: Renamed from LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/basic-cue-rendering.webvtt.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/reference.css: Renamed from LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/resources/reference.css.
(@font-face):
(.test-diff):
(video-reference, cue):
(cue-background):
(video, video-reference):
(::cue, cue):
(::cue, cue-background):
(cue):
(video-reference[expected] > cue > cue-background):
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/webvtt-rendering-test.js: Added.
(async event):
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/support/white-space-pre-line.webvtt: Renamed from LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line.webvtt.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/white-space-pre-line-expected.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/cues-with-video/processing-model/white-space-pre-line.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/resources/webvtt-rendering-test.js: Removed.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line-expected.html: Removed.
* LayoutTests/imported/w3c/web-platform-tests/webvtt/rendering/white-space-pre-line.html: Removed.

Canonical link: https://commits.webkit.org/284436@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list