Intent to Deprecate: MediaError.MEDIA_ERR_ENCRYPTED

240 views
Skip to first unread message

Philip Jägenstedt

unread,
Jan 29, 2014, 1:22:09 AM1/29/14
to blink-dev, David Dorwin

Primary eng (and PM) emails

phi...@opera.com


Summary

Deprecate the "const unsigned short MEDIA_ERR_ENCRYPTED" member of the MediaError interface.

Motivation

It has been removed from the Encrypted Media Extensions spec:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=21203#c30

I sent a review for a plain (MeasureAs) UseCounter:

David Dorwin commented:
"DeprecateAs? This has no purpose anymore, and I don't think it's being used, so I think it's fine to display a warning. This should only be tracked if this constant name is used in JS anyway (and not a comparison to '5')."

I checked and David is correct. It is never actually used in Blink, the constant on the interface is the only Web-facing part there is.

Usage information from UseCounter

As per above, no UseCounter yet, I mean to add it as a deprecated feature directly.

Compatibility Risk

None for deprecation. For removal, any page that checks for an EME error using code like "video.error.code == MediaError.MEDIA_ERR_ENCRYPTED" will act as if there was no error, and probably misbehave.

Row on feature dashboard?

http://www.chromestatus.com/features/6578378068983808


Requesting approval to remove too?

No.

Eric Seidel

unread,
Jan 29, 2014, 9:29:53 PM1/29/14
to Philip Jägenstedt, blink-dev, David Dorwin
No approval needed since it's just deprecate, but LGTM. :) Looking
forward to the usecounter data.
Reply all
Reply to author
Forward
0 new messages