New issue
Advanced search Search tips

Issue 635693 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

EME: Add new event handler attributes

Project Member Reported by ddorwin@chromium.org, Aug 9 2016

Issue description

Version : 54.0.2824.0

Steps to reproduce: 
1. https://www.w3c-test.org/encrypted-media/idlharness.html

Observed behavior: 
Three tests fail with "object must have a property" for:
* onwaitingforkey
* onkeystatuseschange
* onmessage

Expected behavior: 
These tests pass.

Additional comments: 
These event handler attributes were added for https://github.com/w3c/encrypted-media/issues/176.

We already support onencrypted.
 
Labels: -M-55 M-54
Owner: ddorwin@chromium.org
Status: Started (was: Assigned)
Labels: -M-54 M-55
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17

commit 6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17
Author: ddorwin <ddorwin@chromium.org>
Date: Wed Sep 07 17:48:56 2016

EME: Add new event handler attributes

Adds onwaitingforkey to HTMLMediaElement.
Adds onkeystatuseschange and onmessage to MediaKeySession.

BUG= 635693 

TEST=The three tests for these attributes in https://www.w3c-test.org/encrypted-media/idlharness.html

Review-Url: https://codereview.chromium.org/2236483005
Cr-Commit-Position: refs/heads/master@{#416988}

[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/android_webview/tools/system_webview_shell/test/data/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/LayoutTests/virtual/stable/webexposed/element-instance-property-listing-expected.txt
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/LayoutTests/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/LayoutTests/webexposed/element-instance-property-listing-expected.txt
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.h
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.idl
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.h
[modify] https://crrev.com/6e3647df59a9b8a2c77d49ce5681e2a5b3ab7d17/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.idl

Status: Fixed (was: Started)

Sign in to add a comment