Closed Bug 914602 Opened 11 years ago Closed 11 years ago

Make video size restriction customisable

Categories

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

Product:
Firefox OS Graveyard
Component:
Gaia::Camera
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:leo+, b2g18 fixed)

Status:
RESOLVED FIXED
Project Flags:
blocking-b2g leo+
Tracking Flags:
Tracking Status
b2g18 --- fixed

People

(Reporter: brg, Unassigned)

References

Details

Video resolution is restricted now. Please make video size resolution customisable, this issue had been reported through aftersales channels. Nominating to leo+ This issue is similar to bug 900399.
This dupes to bug 896425, which has a patch reviewed and landed on HD. If this gets granted leo+ we can simply land the patch from bug 896425 to v1-train and dupe this.
Blocks: 900399
To fulfill the partner's request in bug 900399
blocking-b2g: leo? → leo+
hi David, IIRC, we limit the picture to 2MP to avoid gallery crashes. is this still true for v1.1 on a 256MB RAM device? thanks
Flags: needinfo?(dflanagan)
Hi I had a pull request for v1-train which was already reviewed on bug 896425. feel free to merge it if needed. https://github.com/mozilla-b2g/gaia/pull/12079
(In reply to Joe Cheng [:jcheng] from comment #3) > hi David, IIRC, we limit the picture to 2MP to avoid gallery crashes. is > this still true for v1.1 on a 256MB RAM device? thanks I see the code in Gallery has lifted picture size to 5MB. https://github.com/mozilla-b2g/gaia/blob/v1-train/apps/gallery/js/MetadataParser.js#L19
Just had a offline conversation with Joe, he think it's okay if we have lifted the restriction to 5MP. I'm going to merge Yuren's PR on comment 4, since my WIP patch for bug 900399 depends on it.
v1-train: https://github.com/mozilla-b2g/gaia/commit/52d4e499261a0a27a5341a51b751a68b0ddf64ce
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Flags: needinfo?(dflanagan)
You need to log in before you can comment on or make changes to this bug.