Fix encrypted AAC in BMFF by adding subsample info.

Fix encrypted AAC in BMFF by adding subsample info.

AAC audio is sent to the Source Buffer in ADTS format, which requires
adding a header to each media sample. Encrypted subsample information
was not being adjusted to compensate for the extra clear bytes.

TEST=Manual, pending audio decryption support in decoder
BUG=132351
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=153854

[strobe_, 2012-08-28 23:07:32]

Small fix for encrypted audio. Tested on two platforms (GStreamer/ClearKey and
[major SOC vendor]/PlayReady).

[acolwell GONE FROM CHROMIUM, 2012-08-28 23:28:42]

LGTM % minor nit

https://chromiumcodereview.appspot.com/10886022/diff/1/media/mp4/aac.h
File media/mp4/aac.h (right):

https://chromiumcodereview.appspot.com/10886022/diff/1/media/mp4/aac.h#newcode50
media/mp4/aac.h:50: static const size_t kADTSHeaderSize = 7;
nit: Please update ConvertEsdsToADTS() to use this constant instead of 7 when
computing size & the insert offset to help ensure this stays in sync with the
code.

[strobe_, 2012-08-28 23:44:09]

https://chromiumcodereview.appspot.com/10886022/diff/1/media/mp4/aac.h
File media/mp4/aac.h (right):

https://chromiumcodereview.appspot.com/10886022/diff/1/media/mp4/aac.h#newcode50
media/mp4/aac.h:50: static const size_t kADTSHeaderSize = 7;
On 2012/08/28 23:28:42, acolwell wrote:
> nit: Please update ConvertEsdsToADTS() to use this constant instead of 7 when
> computing size & the insert offset to help ensure this stays in sync with the
> code.

Done.

[commit-bot: I haz the power, 2012-08-28 23:45:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/strobe@google.com/10886022/5001

[commit-bot: I haz the power, 2012-08-29 04:24:18]

Change committed as 153854