Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(65)

Side by Side Diff: media/formats/mp4/mp4_stream_parser_unittest.cc

Issue 246853005: Fix SPS/PPS insertion logic in MP4StreamParser. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move typedef in test to try to make the Android bot happy. Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <algorithm> 5 #include <algorithm>
6 #include <string> 6 #include <string>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/bind_helpers.h" 9 #include "base/bind_helpers.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 512)); 207 512));
208 } 208 }
209 209
210 // Test an invalid file where there are encrypted samples, but 210 // Test an invalid file where there are encrypted samples, but
211 // SampleAuxiliaryInformation{Sizes|Offsets}Box (saiz|saio) are missing. 211 // SampleAuxiliaryInformation{Sizes|Offsets}Box (saiz|saio) are missing.
212 // The parser should fail instead of crash. See http://crbug.com/361347 212 // The parser should fail instead of crash. See http://crbug.com/361347
213 TEST_F(MP4StreamParserTest, MissingSampleAuxInfo) { 213 TEST_F(MP4StreamParserTest, MissingSampleAuxInfo) {
214 ParseMP4File("bear-1280x720-a_frag-cenc_missing-saiz-saio.mp4", 512); 214 ParseMP4File("bear-1280x720-a_frag-cenc_missing-saiz-saio.mp4", 512);
215 } 215 }
216 216
217 // Test a file where all video samples start with an Access Unit
218 // Delimiter (AUD) NALU.
219 TEST_F(MP4StreamParserTest, VideoSamplesStartWithAUDs) {
220 ParseMP4File("bear-1280x720-av_with-aud-nalus_frag.mp4", 512);
221 }
222
217 // TODO(strobe): Create and test media which uses CENC auxiliary info stored 223 // TODO(strobe): Create and test media which uses CENC auxiliary info stored
218 // inside a private box 224 // inside a private box
219 225
220 } // namespace mp4 226 } // namespace mp4
221 } // namespace media 227 } // namespace media
OLDNEW
« media/formats/mp4/mp4_stream_parser.cc ('K') | « media/formats/mp4/mp4_stream_parser.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698