Source/modules/encryptedmedia/MediaKeySession.idl - Issue 124253003: Update MediaKeySession events to match latest EME spec

Side by Side Diff: Source/modules/encryptedmedia/MediaKeySession.idl

Issue 124253003: Update MediaKeySession events to match latest EME spec (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 6 years, 11 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« Source/modules/encryptedmedia/MediaKeySession.h ('K') | « Source/modules/encryptedmedia/MediaKeySession.cpp ('k') | Source/platform/drm/ContentDecryptionModuleSession.h » ('j') | Source/platform/drm/ContentDecryptionModuleSession.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 /*	1 /*

2 * Copyright (C) 2013 Apple Inc. All rights reserved.	2 * Copyright (C) 2013 Apple Inc. All rights reserved.

3 *	3 *

4 * Redistribution and use in source and binary forms, with or without	4 * Redistribution and use in source and binary forms, with or without

5 * modification, are permitted provided that the following conditions	5 * modification, are permitted provided that the following conditions

6 * are met:	6 * are met:

7 * 1. Redistributions of source code must retain the above copyright	7 * 1. Redistributions of source code must retain the above copyright

8 * notice, this list of conditions and the following disclaimer.	8 * notice, this list of conditions and the following disclaimer.

9 * 2. Redistributions in binary form must reproduce the above copyright	9 * 2. Redistributions in binary form must reproduce the above copyright

10 * notice, this list of conditions and the following disclaimer in the	10 * notice, this list of conditions and the following disclaimer in the

(...skipping 19 matching lines...) Expand all Loading...
30 readonly attribute MediaKeyError error;	30 readonly attribute MediaKeyError error;

31	31

32 // session properties	32 // session properties

33 readonly attribute DOMString keySystem;	33 readonly attribute DOMString keySystem;

34 readonly attribute DOMString sessionId;	34 readonly attribute DOMString sessionId;

35	35

36 // session operations	36 // session operations

37 [RaisesException] void update(Uint8Array key);	37 [RaisesException] void update(Uint8Array key);

38 void close();	38 void close();

39	39

40 // EventHandlers	40 // EventHandlers
	ddorwin 2014/01/07 23:44:41 Delete all. Delete all. jrummell 2014/01/08 18:05:52 Done. Show quoted text On 2014/01/07 23:44:41, ddorwin wrote: > Delete all. Done.
41 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onwebkitkeyadded;	41 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onerror;
xhwang 2014/01/03 23:38:20 Shall we keep the "webkit" prefix? I thought we ha Shall we keep the "webkit" prefix? I thought we have that prefix because EME is not a standard yet, but I could be wrong. +acolwell: In which case should we use "webkit" prefix, and at what point we can drop it? acolwell GONE FROM CHROMIUM 2014/01/06 20:34:10 We should not be using a webkit prefix on new APIs Show quoted text On 2014/01/03 23:38:20, xhwang wrote: > Shall we keep the "webkit" prefix? I thought we have that prefix because EME is > not a standard yet, but I could be wrong. > > +acolwell: In which case should we use "webkit" prefix, and at what point we can > drop it? We should not be using a webkit prefix on new APIs. The idea is that things should stay behind a runtime flag until they are stable enough for release.
42 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onwebkitkeyerror;	42 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onmessage;

43 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onwebkitkeymessage;	43 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onready;

	44 [RuntimeEnabled=EncryptedMedia] attribute EventHandler onclosed;

44 };	45 };

OLD	NEW