Bug 71378 – Remove <track>-related LayoutTests that don't actually work yet

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Anna Cavender]

There are a number of tests in LayoutTests that don't succeed because the feature is a work in progress and not all functionality is there yet.  This patch removes all the tests that don't work so that they can be more easily added with patches that implement the functionality they test.

[Anna Cavender]

Created attachment 113340 [details]
Patch

[Eric Carlson]

Comment on attachment 113340 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=113340&action=review

> LayoutTests/platform/qt/Skipped:-260
> -media/track/track-webvtt-tc027-empty-cue.html
> -media/track/track-webvtt-tc028-unsupported-markup.html
> -

Maybe you should add "media/track" because the feature isn't enabled in this port?

> LayoutTests/platform/wk2/Skipped:-908
> -media/track/track-webvtt-tc027-empty-cue.html
> -media/track/track-webvtt-tc028-unsupported-markup.html
> -

Ditto

[Anna Cavender]

Comment on attachment 113340 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=113340&action=review

>> LayoutTests/platform/qt/Skipped:-260
>> -
> 
> Maybe you should add "media/track" because the feature isn't enabled in this port?

I think track IS enabled on qt.  Notice media/track/track-constants.html and media/track/track-load-* are not skipped and seem to be running fine.

>> LayoutTests/platform/wk2/Skipped:-908
>> -
> 
> Ditto

Ditto for wk2, too.

[WebKit Review Bot]

Comment on attachment 113340 [details]
Patch

Clearing flags on attachment: 113340

Committed r99110: <http://trac.webkit.org/changeset/99110>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.