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

Side by Side Diff: media/formats/mp2t/mp2t_stream_parser_unittest.cc

Issue 2496723002: [MSAN] Initialize video_ and audio_track_id_ in media_unittest. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "media/formats/mp2t/mp2t_stream_parser.h" 5 #include "media/formats/mp2t/mp2t_stream_parser.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 public: 57 public:
58 Mp2tStreamParserTest() 58 Mp2tStreamParserTest()
59 : segment_count_(0), 59 : segment_count_(0),
60 config_count_(0), 60 config_count_(0),
61 audio_frame_count_(0), 61 audio_frame_count_(0),
62 video_frame_count_(0), 62 video_frame_count_(0),
63 has_video_(true), 63 has_video_(true),
64 audio_min_dts_(kNoDecodeTimestamp()), 64 audio_min_dts_(kNoDecodeTimestamp()),
65 audio_max_dts_(kNoDecodeTimestamp()), 65 audio_max_dts_(kNoDecodeTimestamp()),
66 video_min_dts_(kNoDecodeTimestamp()), 66 video_min_dts_(kNoDecodeTimestamp()),
67 video_max_dts_(kNoDecodeTimestamp()) { 67 video_max_dts_(kNoDecodeTimestamp()),
68 audio_track_id_(0),
69 video_track_id_(0) {
68 bool has_sbr = false; 70 bool has_sbr = false;
69 parser_.reset(new Mp2tStreamParser(has_sbr)); 71 parser_.reset(new Mp2tStreamParser(has_sbr));
70 } 72 }
71 73
72 protected: 74 protected:
73 std::unique_ptr<Mp2tStreamParser> parser_; 75 std::unique_ptr<Mp2tStreamParser> parser_;
74 int segment_count_; 76 int segment_count_;
75 int config_count_; 77 int config_count_;
76 int audio_frame_count_; 78 int audio_frame_count_;
77 int video_frame_count_; 79 int video_frame_count_;
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 parser_->Flush(); 319 parser_->Flush();
318 EXPECT_EQ(audio_frame_count_, 40); 320 EXPECT_EQ(audio_frame_count_, 40);
319 EXPECT_EQ(video_frame_count_, 0); 321 EXPECT_EQ(video_frame_count_, 0);
320 // This stream has no mid-stream configuration change. 322 // This stream has no mid-stream configuration change.
321 EXPECT_EQ(config_count_, 1); 323 EXPECT_EQ(config_count_, 1);
322 EXPECT_EQ(segment_count_, 1); 324 EXPECT_EQ(segment_count_, 1);
323 } 325 }
324 326
325 } // namespace mp2t 327 } // namespace mp2t
326 } // namespace media 328 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698