summaryrefslogtreecommitdiffstats
path: root/media/mp4/es_descriptor_unittest.cc
Commit message (Collapse)AuthorAgeFilesLines
* Add HE AAC support to ISO BMFF.xiaomings@google.com2012-07-191-0/+92
| | | | | | | | | | | | | | | | | | | | Was https://chromiumcodereview.appspot.com/10753005/. The original cl also includes abstraction of BitReader class used by H264Parser. It includes bugs that break the H264Parser. In this cl I removed the abstraction and make the BitReader used by H264Parser independent to the one used by esds parser because of: 1. The original cl introduce bugs. 2. Even if we fix the bugs, we may not be able to make the generalized class as fast as the old one while keeping the abstraction. 3. The H264 bit stream use very special syntax on escaping and trailing zero bits, which might not be a good candidate for abstraction. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10780026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147511 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 145769 (broke test stream, removed needed HasMoreRBSPData, added ↵fischman@chromium.org2012-07-171-92/+0
| | | | | | | | | | | | | | | | | | | | | extra ops) - Add HE AAC support to ISO BMFF. Also abstract common code in H264BitReader into BitReader for reusing. Was: https://chromiumcodereview.appspot.com/10710002/ In the submitted patch of the last issue, the media.gyp included a non-existing file. This error hadn't been caught by try but later got caught by the build process and reverted the commit. So I create this issue to submit the fixed patch. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10753005 TBR=xiaomings@google.com Review URL: https://chromiumcodereview.appspot.com/10779025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146927 0039d316-1c4b-4281-b951-d872f2087c98
* Add HE AAC support to ISO BMFF.xiaomings@google.com2012-07-091-0/+92
| | | | | | | | | | | | | | | | Also abstract common code in H264BitReader into BitReader for reusing. Was: https://chromiumcodereview.appspot.com/10710002/ In the submitted patch of the last issue, the media.gyp included a non-existing file. This error hadn't been caught by try but later got caught by the build process and reverted the commit. So I create this issue to submit the fixed patch. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10753005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145769 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 145617 - Add HE AAC support to ISO BMFF.cpu@chromium.org2012-07-061-92/+0
| | | | | | | | | | | | | | | | | | 1. Parse esds box to get HE AAC config. 2. Send audio config from AAC header to decoder instead of the one embedded in SampleDescription. 3. Convert raw audio data into ADTS before sending to decoder. 4. Abstract general bit stream code from H264BitReader into media::BitReader. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10710002 TBR=xiaomings@google.com Review URL: https://chromiumcodereview.appspot.com/10693115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145621 0039d316-1c4b-4281-b951-d872f2087c98
* Add HE AAC support to ISO BMFF.xiaomings@google.com2012-07-061-0/+92
1. Parse esds box to get HE AAC config. 2. Send audio config from AAC header to decoder instead of the one embedded in SampleDescription. 3. Convert raw audio data into ADTS before sending to decoder. 4. Abstract general bit stream code from H264BitReader into media::BitReader. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10710002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145617 0039d316-1c4b-4281-b951-d872f2087c98