New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 636999 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Ensure that all stream parsers assign correct track ids to emitted buffers

Project Member Reported by servolk@chromium.org, Aug 11 2016

Issue description

Currently only the WebM stream parser assigns the correct track bytestream ids for each emitted buffer. We need to ensure that other stream parsers (MP4, mpeg2ts, etc) do the same.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/06a15b52d5f3f5cf05559afc0f03a60e4f289e21

commit 06a15b52d5f3f5cf05559afc0f03a60e4f289e21
Author: servolk <servolk@chromium.org>
Date: Fri Aug 12 01:03:21 2016

Assign correct track ids to buffers emitted from StreamParsers

Currently only the WebM stream parser assigns correct track bytestream
ids for each emitted buffer. We need to ensure that other stream parsers
(MP4, mpeg2ts, etc) do the same.

BUG= 636999 

Review-Url: https://codereview.chromium.org/2239013002
Cr-Commit-Position: refs/heads/master@{#411498}

[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/common/stream_parser_test_base.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/common/stream_parser_test_base.h
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp2t/es_parser_adts.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp2t/es_parser_h264.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp2t/es_parser_mpeg1audio.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp2t/mp2t_common.h
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp2t/mp2t_stream_parser_unittest.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp4/mp4_stream_parser.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mp4/mp4_stream_parser_unittest.cc
[modify] https://crrev.com/06a15b52d5f3f5cf05559afc0f03a60e4f289e21/media/formats/mpeg/mpeg_audio_stream_parser_base.cc

Status: Fixed (was: Started)

Sign in to add a comment