Closed
Bug 897863
Opened 11 years ago
Closed 11 years ago
[Buri][camera]It display abnormally when plug in USB to preview video
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect, P1)
Firefox OS Graveyard
Gaia::Camera
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sync-1, Assigned: johnhu)
Details
Attachments
(2 files)
AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.164 Firefox os v1.1 Mozilla build ID:20130715070218 Created an attachment (id=467620) PIC DEFECT DESCRIPTION: It display abnormally when plug in USB to preview video REPRODUCING PROCEDURES: 1.Enable USB storage 2.Launch camera app 3.Switch to Camcorder 4.Record a video 5.Tap the video on the left top to preview 6.Plug in USB,then press play key->the screen display abnormally->KO EXPECTED BEHAVIOUR: It should have notification that can't preview ASSOCIATE SPECIFICATION: TEST PLAN REFERENCE: TOOLS AND PLATFORMS USED: USER IMPACT: Mid REPRODUCING RATE: 5/5 For FT PR, Please list reference mobile's behavior:
Assignee | ||
Updated•11 years ago
|
Component: Gaia::Video → Gaia::Camera
Comment 3•11 years ago
|
||
I confirmed this happens on v1.1 hamachi. It seems that camera app does not care about this use case.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → johu
Assignee | ||
Comment 4•11 years ago
|
||
The correct behavior should be to show an overlay saying storage is mounted as USB device. And preventing user to play the video. This bug has two sub-issues: 1. Message overlay should be higher than media frame. They are at the same level 100. So, I make message overlay(110) higher than media frame(100). 2. When media frame is shown, it may be a case of playing video. Camera app may be killed because of locked file when user plug USB in. So, I change the code to go back to camera mode when sdcard becomes unavailable or shared.
Attachment #781538 -
Flags: review?(dflanagan)
Comment 5•11 years ago
|
||
Comment on attachment 781538 [details]
change css z-index and make camera go back to camera mode while usb is plugged
r+. But please see the name change suggetions on github. You can land this patch as it is if you want, but I'd prefer to take this opportunity to fix the very confusing "returnToCamera" variable.
Attachment #781538 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 781538 [details]
change css z-index and make camera go back to camera mode while usb is plugged
David,
Sorry to reset the review flag from + to ?. Besides last changes, I added some changes of variables and functions renaming to make code more clear. So, it is nice to review it again.
Change items:
1. make camera go back to camera mode while usb is plugged
2. remove returnToCamera variable, use Filmstrip.isPreviewShown() to replace it.
3. rename the returnToCameraMode to hidePreview()
4. add isPreviewShown and hidePreview to Filmstrip
5. rename the screenWakeLock and screenTimeout to requestScreenWakeLock and releaseScreenWakeLock.
6. I had rebase to the gaia master before changing it. The css change seems useless, someone already changed it.
Attachment #781538 -
Flags: review+ → review?(dflanagan)
Comment 7•11 years ago
|
||
Comment on attachment 781538 [details]
change css z-index and make camera go back to camera mode while usb is plugged
This is so much better! Again, I'm assuming you've tested the changes. Thanks!
Attachment #781538 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 8•11 years ago
|
||
merged to master: https://github.com/mozilla-b2g/gaia/commit/d9a2f8ff576a253de3527b650da45040b3d0d2d2
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•11 years ago
|
||
Can we merge to v1.1??? We may need to mark this bug as leo+. If it is, John Ford will help to do that..
You need to log in
before you can comment on or make changes to this bug.
Description
•