Open Bug 1124630 Opened 7 years ago Updated 4 years ago

Remove CameraPreviewMediaStream

Categories

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

defect

Tracking

()

People

(Reporter: pehrsons, Unassigned)

References

Details

Since CameraPreviewMediaStream is a temporary fix let's use this bug to track its removal.
See Also: → 844248, 1124139
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.
roc, can you answer to Comment 1?
Flags: needinfo?(roc)
And if we remove CameraPreviewMediaStream, we need to evaluate the latency about whether there is regression.
(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.
Flags: needinfo?(roc)
Depends on: 1201363
Component: Audio/Video → Audio/Video: MSG/cubeb/GMP
Rank: 25
Priority: -- → P2
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.