Closed Bug 897863 Opened 12 years ago Closed 12 years ago

[Buri][camera]It display abnormally when plug in USB to preview video

Categories

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

defect

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:
Clone from brother
Attached image PIC
Component: Gaia::Video → Gaia::Camera
I confirmed this happens on v1.1 hamachi. It seems that camera app does not care about this use case.
Assignee: nobody → johu
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 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+
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 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+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Hi David, Could you merge this patch to v1.1 ?
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.

Attachment

General

Creator:
Created:
Updated:
Size: