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

RESOLVED FIXED in Firefox 45

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: aosmond, Assigned: aosmond)

Tracking

unspecified
FxOS-S10 (30Oct)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(firefox45 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

4 years ago
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)

Updated

4 years ago
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
See Also: → bug 1177250
(Assignee)

Comment 1

4 years ago
Created attachment 8628837 [details] [review]
[gaia] aosmond:bug1179726 > mozilla-b2g:master

Make the app prefer the default (note: gecko default needs to be changed too).
Attachment #8628837 - Flags: review?(jdarcangelo)
(Assignee)

Comment 2

4 years ago
Created attachment 8628841 [details] [diff] [review]
[gecko] add priority modifier to recording profiles to not always prefer largest

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=dc0bd735d255
Attachment #8628841 - Flags: review?(dhylands)
(Assignee)

Updated

4 years ago
Attachment #8628841 - Flags: review?(dhylands)
(Assignee)

Updated

4 years ago
Attachment #8628837 - Flags: review?(jdarcangelo)
(Assignee)

Comment 3

4 years ago
Did not notice the pull request in bug 1177250.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 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+
Created attachment 8680816 [details] [review]
[gaia] aosmond:bug1179726 > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Attachment #8628837 - Attachment is obsolete: true
(Assignee)

Updated

3 years ago
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
(Assignee)

Updated

3 years ago
Attachment #8680816 - Flags: review?(jdarcangelo)
(Assignee)

Updated

3 years ago
Blocks: 1177250
Attachment #8680816 - Flags: review?(jdarcangelo) → review+

Comment 8

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/364e247ed7d1
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago3 years ago
status-firefox45: --- → fixed
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
status-b2g-v2.5: fixed → ---
You need to log in before you can comment on or make changes to this bug.