This layout test tests the contents of the clearKey message event and checks its KID.
Created attachment 329651 [details] Patch
Comment on attachment 329651 [details] Patch The test looks mainly ok and it does not deserve an r- but I think we can improve it, specially thinking of the future, if any. I see two things. First, I'd prefer to use const and let instead of var. Second, this cenc message test should be enough already, but I am not going to complain only for one test. If you're planning to file more dual tests like this that are mostly the same (almost everything but the video and test initialization is the same) we should think of a common way of running these tests. My suggestion would be merging both tests at least to share the biggest part of the code and maybe thinking of a way of writing a small utility to run the same code through MSE and non-MSE paths if only initialization is different).
Created attachment 329922 [details] Patch
Created attachment 330805 [details] Patch
Comment on attachment 330805 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=330805&action=review Most important part is removing the [ Pass ] from the WPE expectations. Now, I'd encourage you to use more const and let instead of var. I see you have converted many var into let but some of them can be const and there are some var left. > LayoutTests/platform/wpe/TestExpectations:1003 > +media/encrypted-media/clearKey/clearKey-message-cenc-event.html [ Pass ] > +media/encrypted-media/clearKey/clearKey-message-cenc-event-mse.html [ Pass ] > + This is not needed since we are not skipping globally.
Created attachment 330818 [details] Patch
Comment on attachment 330818 [details] Patch Clearing flags on attachment: 330818 Committed r226639: <https://trac.webkit.org/changeset/226639>
All reviewed patches have been landed. Closing bug.
<rdar://problem/36376406>