Closed
Bug 879881
Opened 11 years ago
Closed 11 years ago
[Camera][MMS] Limit size of "pick" activity captured video for MMS
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect, P1)
Tracking
(blocking-b2g:leo+, b2g18 fixed, b2g-v1.1hd fixed)
People
(Reporter: gnarf, Assigned: daleharvey)
References
Details
(Whiteboard: TaipeiWW)
Attachments
(1 file, 1 obsolete file)
Per https://bugzilla.mozilla.org/show_bug.cgi?id=868225#c2: - For video capture, an MMS Video Limit mode is supported The camera application must support video capture that stops automatically before the file reaches a maximum size to match the handset's maximum MMS attachment file size, being configurable as part of the variant process according to the network provider's network limits. The SMS side will add a "maxFileSize" property to the "pick" MozActivity, we need to make sure that the video capture in camera uses this.
Updated•11 years ago
|
blocking-b2g: leo? → leo+
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → dale
Comment 1•11 years ago
|
||
share some investigation, currently camera app did not support pick video, and not support open in record mode yet (camera always open as camera mode, tap video icon to switch to record mode). bug 853349 'open in record mode when trigger camera with 'record' activity by video type' is similar to that 'pick' video action.
Assignee | ||
Comment 2•11 years ago
|
||
I used the label 'maxFileSizeBytes' to be consistent with previous usage, I also think its clearer
Attachment #759577 -
Flags: review?(dflanagan)
Comment 3•11 years ago
|
||
Comment on attachment 759577 [details]
Pointer to GH
r=djf for github commit 3ce908c55452cfb26d8e25408a0aae8e962e5344
Attachment #759577 -
Flags: review?(dflanagan) → review+
Reporter | ||
Comment 4•11 years ago
|
||
Hrm, we already committed maxFileSize as a key in SMS - I suppose we can sneak the word "Bytes" in there... I think that is a tad redundant though don't you? What else would you measure FileSize in?
Comment 5•11 years ago
|
||
(In reply to Corey Frang (@ conference until Jun 14) [:gnarf] [:gnarf37] from comment #4) > > Hrm, we already committed maxFileSize as a key in SMS - I suppose we can > sneak the word "Bytes" in there... I think that is a tad redundant though > don't you? What else would you measure FileSize in? Kilobytes? Megabytes? My intention was for it to parallel maxVideoLengthMs member, which is much more ambiguous without the units on it.
Comment 6•11 years ago
|
||
Does this need checkin-needed added? What are we waiting on if this is r+?
Flags: needinfo?(dale)
Assignee | ||
Comment 7•11 years ago
|
||
It is waiting on https://bugzilla.mozilla.org/show_bug.cgi?id=853349 to be landed, its a fairly minor ammendment that can be committed as soon as that patch lands (I will probably do the patch for sms at the same time once it has landed)
Flags: needinfo?(dale)
Updated•11 years ago
|
Whiteboard: TaipeiWW
Assignee | ||
Comment 8•11 years ago
|
||
The previous patch was r+'d, but this has some further changes so requesting new review.
Attachment #759577 -
Attachment is obsolete: true
Attachment #767026 -
Flags: review?(dflanagan)
Comment 9•11 years ago
|
||
r=djf for github commit 882778c You didn't attached that github PR to this bug, so I can't actually set an r+ flag here. The code looks good. I was unable to actually exercise it, however. It looks like maybe the contacts app is not actually setting a size limit? I'm using gaia master with b2g18, so my build may be incorrect. If you've tested it and it works, then go ahead and land.
Comment 10•11 years ago
|
||
Comment on attachment 767026 [details] [review] Pointer to GH Mid-air collision with the new PR attachment. Now I can set a real r+
Attachment #767026 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 11•11 years ago
|
||
Merged in: https://github.com/mozilla-b2g/gaia/commit/caedc9f9de79605da705697790f92287c9c361fe Uplifted to v1-train: https://github.com/mozilla-b2g/gaia/commit/05ff7b42362ac81216921b97f1f606568156dfb1
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 13•11 years ago
|
||
1.1hd: 05ff7b42362ac81216921b97f1f606568156dfb1
status-b2g-v1.1hd:
--- → fixed
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•