1124630 - Remove CameraPreviewMediaStream

Status: NEW

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

Description

[Andreas Pehrson [:pehrsons]]

Since CameraPreviewMediaStream is a temporary fix let's use this bug to track its removal.

Comment 1

[Sotaro Ikeda [:sotaro]]

The reason why CameraPreviewMediaStream is implemented is latency of MediaStreamGraph. MediaStreamGraph added lags to camera preview. I am not sure current MediaStreamGraph supports no latency mode.

Comment 2

[Sotaro Ikeda [:sotaro]]

roc, can you answer to Comment 1?

Comment 3

[Sotaro Ikeda [:sotaro]]

And if we remove CameraPreviewMediaStream, we need to evaluate the latency about whether there is regression.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

(In reply to Sotaro Ikeda [:sotaro] from comment #1)
> The reason why CameraPreviewMediaStream is implemented is latency of
> MediaStreamGraph. MediaStreamGraph added lags to camera preview. I am not
> sure current MediaStreamGraph supports no latency mode.

Yes. Another issue is that the current MSG code can hang onto multiple Images in the graph, causing the camera's pool of gralloc capture buffers to run out. I don't think we can remove CameraPreviewMediaStream yet, at least not without carefully testing on our supported FxOS devices.

Comment 5

[Bulk Bug Changes for mreavy's org]

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