Closed Bug 1179726 Opened 5 years ago Closed 4 years ago

[B2G][Camera] Prefer lower recording resolutions over 4kuhd as the default profile

Categories

(Firefox OS Graveyard :: Gaia::Camera, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox45 fixed)

RESOLVED FIXED
FxOS-S10 (30Oct)
Tracking Status
firefox45 --- fixed

People

(Reporter: aosmond, Assigned: aosmond)

References

Details

Attachments

(2 files, 1 obsolete file)

4kuhd is made the default recording profile if supported by the platform. This results in videos that cannot be played on older / lower memory devices and out of sync audio. We should use a lower resolution as the default if supported (i.e. 1080p).
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
See Also: → 1177250
Make the app prefer the default (note: gecko default needs to be changed too).
Attachment #8628837 - Flags: review?(jdarcangelo)
Attachment #8628841 - Flags: review?(dhylands)
Attachment #8628837 - Flags: review?(jdarcangelo)
Did not notice the pull request in bug 1177250.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Comment on attachment 8628841 [details] [diff] [review]
[gecko] add priority modifier to recording profiles to not always prefer largest

Review of attachment 8628841 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm
Attachment #8628841 - Flags: review+
Attachment #8628837 - Attachment is obsolete: true
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Attachment #8680816 - Flags: review?(jdarcangelo)
Blocks: 1177250
Attachment #8680816 - Flags: review?(jdarcangelo) → review+
https://hg.mozilla.org/mozilla-central/rev/364e247ed7d1
Status: REOPENED → RESOLVED
Closed: 5 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S10 (30Oct)
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
You need to log in before you can comment on or make changes to this bug.