997149 - Gonk video-size-values fallback on preview-size-values does not work as expected

Status: RESOLVED DUPLICATE of bug 987068

(Core :: DOM: Device Interfaces, defect)

Description

[:gerard-majax]

Attachment: Quick hack

Enabling video recording mode on Nexus S and Desire Z makes the Camera app not working.

After some investigation, I found out that the |video-size-values| camera parameter would fail because it is non-existent.

As far as I understand this code, this case is expected to be correctly handled by |nsGonkCameraControl::Get| at http://dxr.mozilla.org/mozilla-central/source/dom/camera/GonkCameraControl.cpp#374

Adding debug to the code, I do see this path being hit and handled properly at least once. But at some point later, SetVideoSize() gets called and we trigger another request for |video-size-values| and the proper |preview-size-values| replacement is not being done.

This results in no recording size being used, and recording broken.

Quick hack attached does the trick of bringing back the recording mode on Nexus S.

Comment 1

[Mike Habicher [:mikeh] (high bugzilla latency)]

I believe this is/will be addressed in bug 987068.