Bug 191999 - [EME][ClearKey][GTK] Update ClearKey tests expectations
Summary: [EME][ClearKey][GTK] Update ClearKey tests expectations
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-27 01:40 PST by Yacine Bandou
Modified: 2020-05-13 06:46 PDT (History)
6 users (show)

See Also:


Attachments
Patch (10.10 KB, patch)
2018-11-27 05:54 PST, Yacine Bandou
no flags Details | Formatted Diff | Diff
Patch (5.52 KB, patch)
2018-11-27 09:56 PST, Yacine Bandou
calvaris: review+
calvaris: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews124 for ios-simulator-wk2 (2.38 MB, application/zip)
2018-11-27 12:01 PST, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yacine Bandou 2018-11-27 01:40:32 PST
webkit.org/b/190738 media/encrypted-media/clearKey/clearKey-cenc-video-playback-mse.html [ Timeout ]
webkit.org/b/190738 media/encrypted-media/clearKey/clearKey-webm-video-playback-mse.html [ Timeout ]

As bug 190738 is resolved, these tests are now pass.
Comment 1 Yacine Bandou 2018-11-27 05:54:13 PST
Created attachment 355726 [details]
Patch
Comment 2 Charlie Turner 2018-11-27 08:05:54 PST
Comment on attachment 355726 [details]
Patch

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

Thanks Yacine! Just a small issue with the DRT output in this patch, but the rest LGTM. We'll need to wait for a formal review before landing though :)

> LayoutTests/platform/gtk/TestExpectations:-1212
> -webkit.org/b/190738 imported/w3c/web-platform-tests/encrypted-media/resources/clearkey-retrieve-persistent-license.html [ Missing ]

Please keep these two [ Missing ] expectations, they are not tests, but rather helper files, and including them introduces the DRT output you have included in this patch.

> LayoutTests/platform/gtk/imported/w3c/web-platform-tests/encrypted-media/resources/clearkey-retrieve-destroy-persistent-license-expected.txt:1
> +layer at (0,0) size 800x600

All these DRT outputs can be removed, (see above)
Comment 3 Yacine Bandou 2018-11-27 09:56:40 PST
Created attachment 355744 [details]
Patch
Comment 4 Charlie Turner 2018-11-27 10:03:04 PST
Perfect, thanks again. Informal r+ from me.
Comment 5 EWS Watchlist 2018-11-27 12:01:03 PST
Comment on attachment 355744 [details]
Patch

Attachment 355744 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/10169132

New failing tests:
media/no-fullscreen-when-hidden.html
Comment 6 EWS Watchlist 2018-11-27 12:01:05 PST
Created attachment 355756 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 7 Xabier Rodríguez Calvar 2018-12-02 23:27:51 PST
Comment on attachment 355744 [details]
Patch

Are there no changes in the WPE expectations?
Comment 8 Yacine Bandou 2018-12-03 01:27:55 PST
(In reply to Xabier Rodríguez Calvar from comment #7)
> Comment on attachment 355744 [details]
> Patch
> 
> Are there no changes in the WPE expectations?

Currently I think no.
Comment 9 Charlie Turner 2020-05-13 06:46:09 PDT
Not sure why this stalled, but it's out of date now.