StevenLee[:slee] Reporter
	Description • 11 years ago

Current MediaEngineWebRTCVideoSource::OnNewPreviewFrame rotates the video frames according to the phone is in portrait or landscape mode. It may introduce some problems, such as performance and some unnecessary rotation in MediaRecorder. This bug is to make MediaRecorder read the rotation angle set in bug 976397 and do what it want to do.

	Randy Lin [:rlin] - Be Mozillian on 2015/01/26
	Comment 1 • 10 years ago

Only apply to b2g platform.

	Randy Lin [:rlin] - Be Mozillian on 2015/01/26
	Comment 2 • 10 years ago

Can try this comment:
https://bugzilla.mozilla.org/show_bug.cgi?id=976397#c24
 
Per discussion when we landed the patch with DYNAMIC_GUM_ROTATION turned off, what the MediaRecorder needs to do is: 
if rotation == initial rotation, just encode
if rotation != initial rotation, scale-and-inset-with-black-bars (90 degree rotates).  Note that whether the black bars are top/bottom or left/right will depend on the initial rotation.  I'm assuming 180 rotate can be handled automatically as there's no resolution change.

Note that PeerConnection would really benefit from DYNAMIC_GUM_ROTATION

	Randy Lin [:rlin] - Be Mozillian on 2015/01/26
	Comment 3 • 10 years ago

Free it if anyone wants.

	Bulk Bug Changes for mreavy's org
	Comment 4 • 7 years ago

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

	Sylvestre Ledru [:Sylvestre]
	Comment 5 • 6 years ago

No assignee, updating the status.

	Sylvestre Ledru [:Sylvestre]
	Comment 6 • 6 years ago

No assignee, updating the status.

	Sylvestre Ledru [:Sylvestre]
	Comment 7 • 6 years ago

No assignee, updating the status.