| Index: media/webm/webm_tracks_parser.cc
|
| diff --git a/media/webm/webm_tracks_parser.cc b/media/webm/webm_tracks_parser.cc
|
| index 29b4f33077051f83f3f9b5e6bb0b023cc0bd1f64..7cd956961b63993c7fb26625f067f85c65781c6a 100644
|
| --- a/media/webm/webm_tracks_parser.cc
|
| +++ b/media/webm/webm_tracks_parser.cc
|
| @@ -9,21 +9,39 @@
|
|
|
| namespace media {
|
|
|
| +static const base::TimeDelta kNoDuration =
|
| + base::TimeDelta::FromMicroseconds(-1);
|
| +
|
| WebMTracksParser::WebMTracksParser(int64 timecode_scale)
|
| : timecode_scale_(timecode_scale),
|
| track_type_(-1),
|
| track_num_(-1),
|
| track_default_duration_(-1),
|
| audio_track_num_(-1),
|
| - audio_default_duration_(base::TimeDelta::FromMicroseconds(-1)),
|
| + audio_default_duration_(kNoDuration),
|
| video_track_num_(-1),
|
| - video_default_duration_(base::TimeDelta::FromMicroseconds(-1)) {
|
| + video_default_duration_(kNoDuration) {
|
| }
|
|
|
| WebMTracksParser::~WebMTracksParser() {}
|
|
|
| int WebMTracksParser::Parse(const uint8* buf, int size) {
|
| - return WebMParseListElement(buf, size, kWebMIdTracks, 1, this);
|
| + track_type_ =-1;
|
| + track_num_ = -1;
|
| + track_default_duration_ = -1;
|
| + audio_track_num_ = -1;
|
| + audio_default_duration_ = kNoDuration;
|
| + video_track_num_ = -1;
|
| + video_default_duration_ = kNoDuration;
|
| +
|
| + WebMListParser parser(kWebMIdTracks);
|
| + int result = parser.Parse(buf, size, this);
|
| +
|
| + if (result <= 0)
|
| + return result;
|
| +
|
| + // For now we do all or nothing parsing.
|
| + return parser.IsParsingComplete() ? result : 0;
|
| }
|
|
|
|
|
| @@ -40,9 +58,9 @@ bool WebMTracksParser::OnListStart(int id) {
|
| bool WebMTracksParser::OnListEnd(int id) {
|
| if (id == kWebMIdTrackEntry) {
|
| if (track_type_ == -1 || track_num_ == -1) {
|
| - VLOG(1) << "Missing TrackEntry data"
|
| - << " TrackType " << track_type_
|
| - << " TrackNum " << track_num_;
|
| + DVLOG(1) << "Missing TrackEntry data"
|
| + << " TrackType " << track_type_
|
| + << " TrackNum " << track_num_;
|
| return false;
|
| }
|
|
|
| @@ -57,7 +75,7 @@ bool WebMTracksParser::OnListEnd(int id) {
|
| audio_track_num_ = track_num_;
|
| audio_default_duration_ = default_duration;
|
| } else {
|
| - VLOG(1) << "Unexpected TrackType " << track_type_;
|
| + DVLOG(1) << "Unexpected TrackType " << track_type_;
|
| return false;
|
| }
|
|
|
| @@ -86,7 +104,7 @@ bool WebMTracksParser::OnUInt(int id, int64 val) {
|
| }
|
|
|
| if (*dst != -1) {
|
| - VLOG(1) << "Multiple values for id " << std::hex << id << " specified";
|
| + DVLOG(1) << "Multiple values for id " << std::hex << id << " specified";
|
| return false;
|
| }
|
|
|
| @@ -95,7 +113,7 @@ bool WebMTracksParser::OnUInt(int id, int64 val) {
|
| }
|
|
|
| bool WebMTracksParser::OnFloat(int id, double val) {
|
| - VLOG(1) << "Unexpected float for id" << std::hex << id;
|
| + DVLOG(1) << "Unexpected float for id" << std::hex << id;
|
| return false;
|
| }
|
|
|
| @@ -108,7 +126,7 @@ bool WebMTracksParser::OnString(int id, const std::string& str) {
|
| return false;
|
|
|
| if (str != "A_VORBIS" && str != "V_VP8") {
|
| - VLOG(1) << "Unexpected CodecID " << str;
|
| + DVLOG(1) << "Unexpected CodecID " << str;
|
| return false;
|
| }
|
|
|
|
|