980816 - [MediaEncoder] TrackEncoder should filter out the MediaSegment with zero duration?

Status: RESOLVED WONTFIX

(Core :: Audio/Video: Recording, defect, P3)

Description

[Benjamin Chen [:bechen]]

http://dxr.mozilla.org/mozilla-central/source/content/media/TrackUnionStream.h?from=trackunionstream&case=true#176

TrackEncoder will receive MediaSegment with zero duration because the segment only carries the TRACK_EVENT_CREATED event. We should filter out this kind of segments instead pending it to vp8/omx encoder?

Comment 1

[u459114]

We handle track end notification in VideoTrackEncoder::NotifyQueuedTrackChanges
184   if (aTrackEvents == MediaStreamListener::TRACK_EVENT_ENDED) {
185     LOG("[VideoTrackEncoder]: Receive TRACK_EVENT_ENDED .");
186     NotifyEndOfStream();
187   }

For me, it makes sense to handle MediaStreamListener::TRACK_EVENT_CREATED in that function as well

Comment 2

[Bulk Bug Changes for mreavy's org]

Mass change P2->P3 to align with new Mozilla triage process.

Comment 3

[Andreas Pehrson [:pehrsons]]

Overtaken by events.