Bug 185277

Summary: [wpe] update the TestExpectations for encrypted-media after some fixes
Product: WebKit Reporter: Yacine Bandou <bandou.yacine>
Component: WPE WebKitAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris, ews-watchlist, olivier.blin
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 181855, 185242, 185244    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews115 for mac-sierra
none
Patch
none
Patch none

Description Yacine Bandou 2018-05-03 16:30:45 PDT
Once the decryptor moved from AppendPipeline to Playbackpipeline and patches 185242 and 185244 will be fixed, 
WPT LayoutTests will no longer crash and some tests will pass.
Comment 1 Yacine Bandou 2018-05-03 19:06:40 PDT
Created attachment 339504 [details]
Patch
Comment 2 EWS Watchlist 2018-05-04 02:05:26 PDT
Comment on attachment 339504 [details]
Patch

Attachment 339504 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/7561756

New failing tests:
imported/w3c/web-platform-tests/html/syntax/parsing/html5lib_tests21.html
Comment 3 EWS Watchlist 2018-05-04 02:05:28 PDT
Created attachment 339531 [details]
Archive of layout-test-results from ews115 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 4 Xabier Rodríguez Calvar 2018-05-07 01:27:49 PDT
Comment on attachment 339504 [details]
Patch

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

> LayoutTests/platform/wpe/TestExpectations:983
> +webkit.org/b/181225 media/encrypted-media/mock-MediaKeySystemAccess.html [ Crash ]
> +
> +[ Debug ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-multiple-times-with-the-same-mediakeys.https.html [ Crash ]

Why are these two tests crashing?

I guess the other ones are timing out because they they go further without crashing, right?
Comment 5 Yacine Bandou 2018-05-07 01:57:22 PDT
(In reply to Xabier Rodríguez Calvar from comment #4)
> Comment on attachment 339504 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=339504&action=review
> 
> > LayoutTests/platform/wpe/TestExpectations:983
> > +webkit.org/b/181225 media/encrypted-media/mock-MediaKeySystemAccess.html [ Crash ]
> > +
> > +[ Debug ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-multiple-times-with-the-same-mediakeys.https.html [ Crash ]
> 
> Why are these two tests crashing?

I did not investigate, its crashed before my pachese

> 
> I guess the other ones are timing out because they they go further without
> crashing, right?

Yes, most tests are waiting for "waitingforkey" event
Comment 6 Xabier Rodríguez Calvar 2018-05-07 02:21:17 PDT
(In reply to Yacine Bandou from comment #5)
> > Why are these two tests crashing?
> 
> I did not investigate, its crashed before my pachese
> 
> Yes, most tests are waiting for "waitingforkey" event

Ok, we'll land after the dependencies then.
Comment 7 Yacine Bandou 2018-05-07 08:20:44 PDT
Created attachment 339722 [details]
Patch
Comment 8 Yacine Bandou 2018-05-09 03:23:56 PDT
Created attachment 339949 [details]
Patch
Comment 9 Xabier Rodríguez Calvar 2018-05-10 00:11:49 PDT
Comment on attachment 339949 [details]
Patch

Clearing flags on attachment: 339949

Committed r231634: <https://trac.webkit.org/changeset/231634>
Comment 10 Xabier Rodríguez Calvar 2018-05-10 00:11:53 PDT
All reviewed patches have been landed.  Closing bug.