Update EncryptionScheme |
|||
Issue descriptionCurrently EncryptionScheme has CipherMode and EncryptionPattern fields. They should be separated out so that EncryptionPattern can be optional. This also allows Audio/VideoDecoderConfig to note that the pattern (if specified) is the default for the file. And CipherMode should be replaced by EncyptionMode from DecryptConfig (which should land shortly) so that it references the actual scheme used by the media, and not the cipher mode directly. The cipher can be derived from EncryptionMode. This will allow future encryption schemes to be supported.
,
Jun 12 2018
I think we should merge EncryptionScheme with EncryptionMode + EncryptionPattern: 1. In AudioDecoderConfig and VideoDecoderConfig, pass EncryptionMode instead of EncryptionScheme. Note that this means we are not passing the pattern in *DecoderConfig. The pattern will be passed in DecoderBuffer at decryption/decoding time. 2. Remove EncryptionScheme. 3. Rename EncryptionMode to EncryptionScheme and move it to encrytion_scheme.* yucliu: WDYT? Will this work for ChromeCast, especially step 1? Also, we are busy and since this is touching an existing cast path the most, would it be possible for you to pick up this work?
,
Aug 7
Lowering priority as this is code cleanup. |
|||
►
Sign in to add a comment |
|||
Comment 1 by xhw...@chromium.org
, Apr 19 2018