Closed
Bug 894815
Opened 11 years ago
Closed 11 years ago
[Buri] Alarm screen and permission screen displays abnormally on the top of Apps
Categories
(Firefox OS Graveyard :: Gaia::System, defect, P1)
Firefox OS Graveyard
Gaia::System
Tracking
(blocking-b2g:leo+, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix, b2g-v1.1hd fixed)
RESOLVED
FIXED
blocking-b2g | leo+ |
People
(Reporter: sync-1, Assigned: rexboy)
References
Details
(Whiteboard: [TD-350479])
Attachments
(6 files)
AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.152
Firefox os v1.1
Mozilla build ID:20130702230206
Created an attachment (id=463326)
PIC
DEFECT DESCRIPTION:
The screen display abnormally on he top in camera
REPRODUCING PROCEDURES:
Prerequisites:
Geolocation option needs to be enabled in Settings.
1.Launch the Settings app.
2.Scroll down to Phone Lock and enable Lock Screen and enable Passcode Lock with a valid passcode.
3.Exit Settings to Home screen then tap the Power button to lock the phone.
4.Tap the power button again then tap the Camera icon on the lock screen.
5.Wait a couple seconds.
6.The screen display abnormally on he top->KO
EXPECTED BEHAVIOUR:
The screen should display correctly
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:
Comment 2•11 years ago
|
||
Can you please attach a screenshot?
Comment 5•11 years ago
|
||
(In reply to buri.blff from comment #4)
> Created attachment 778262 [details]
> sometimes,the system will reset
buri.blff, can you create another bug for it. It is a different bug.
Flags: needinfo?(buri.blff)
Comment 6•11 years ago
|
||
This ui behavior is controlled by System App. It seems like System app's bug. The crash seems a camera related bug.
Component: Gaia::Camera → Gaia::System
Comment 7•11 years ago
|
||
FYI, mikeh is PTO until Aug 5.
(In reply to Sotaro Ikeda [:sotaro] from comment #5)
> (In reply to buri.blff from comment #4)
> > Created attachment 778262 [details]
> > sometimes,the system will reset
>
> buri.blff, can you create another bug for it. It is a different bug.
OK, I will file a new bug.
Flags: needinfo?(buri.blff)
Comment 9•11 years ago
|
||
Triage - Buri can you confirm after closing the dialogue in the attached image, the camera functions correctly?
Flags: needinfo?(sync-1)
Comment 10•11 years ago
|
||
(In reply to Wayne Chang [:wchang] from comment #9)
> Triage - Buri can you confirm after closing the dialogue in the attached
> image, the camera functions correctly?
The camera functions is OK.
Flags: needinfo?(sync-1)
Comment 13•11 years ago
|
||
(In reply to Alive Kuo [:alive] from comment #12)
>
> Taken. UI polish.
BTW, alive: I notice that the text in the message says "_System_ would like to know your location." This is because the lockscreen runs in the System app; but I hate to say it: this will probably be confusing to the user. Can we make it say _Camera_ instead?
Flags: needinfo?(mhabicher)
Comment 14•11 years ago
|
||
Ya, I knew this before...and I am afraid there's no way to fix it now. The name is provided from gecko side.
Comment 17•11 years ago
|
||
Still not a blocker given the workaround in comment #10.Pushing this to 1.2 for consideration.
blocking-b2g: leo? → koi?
Comment 19•11 years ago
|
||
Please re-evaluate on blocking status considering bug 902814. This can be reproduced with applications like Gallery, Video and many more 3rd Party Apps like Notes, Cut the rope.
blocking-b2g: koi? → leo?
Updated•11 years ago
|
Summary: [Buri][Camera]The screen display abnormally on the top in camera → [Buri][Camera] The screen displays abnormally on the top of Apps
Comment 20•11 years ago
|
||
This is a screenshot with Geolocation permission request view and Alarm "OnRing" screen on top of camera for Leo device.
Comment 21•11 years ago
|
||
(In reply to Alive Kuo [:alive] from comment #14)
> Ya, I knew this before...and I am afraid there's no way to fix it now. The
> name is provided from gecko side.
Hi Alive,
Wasn't this issue fixed by 889175 before?
Somehow I can reproduce the issue now.
Flags: needinfo?(alive)
Comment 22•11 years ago
|
||
(In reply to hanj.kim25 from comment #21)
> (In reply to Alive Kuo [:alive] from comment #14)
> > Ya, I knew this before...and I am afraid there's no way to fix it now. The
> > name is provided from gecko side.
>
> Hi Alive,
>
> Wasn't this issue fixed by 889175 before?
> Somehow I can reproduce the issue now.
I'm replying this:
(In reply to Mike Habicher [:mikeh] from comment #13)
> BTW, alive: I notice that the text in the message says "_System_ would like
> to know your location." This is because the lockscreen runs in the System
> app; but I hate to say it: this will probably be confusing to the user. Can
> we make it say _Camera_ instead?
And even the original issue is different from 889175.
Do you still repro 889175?
Flags: needinfo?(alive)
Comment 23•11 years ago
|
||
(In reply to Alive Kuo [:alive] from comment #22)
I see. I thought this is the same issue as 889175.
Yes, for some reason I can repro 889175. I think, I verified the patch before..
Updated•11 years ago
|
Summary: [Buri][Camera] The screen displays abnormally on the top of Apps → [Buri] The screen displays abnormally on the top of Apps
Comment 24•11 years ago
|
||
Hmm, I cannot repro using "Mozilla"'s v1-train.
Could you use mozilla v1-train to repro?
Keywords: qawanted
Comment 25•11 years ago
|
||
(In reply to Alive Kuo [:alive] from comment #24)
> Hmm, I cannot repro using "Mozilla"'s v1-train.
> Could you use mozilla v1-train to repro?
I have tested with the latest v1-train and I have seen the statusbar becoming transparent. Please take a look at a new bug I created, 903271.
Updated•11 years ago
|
Keywords: qawanted
Summary: [Buri] The screen displays abnormally on the top of Apps → [Buri][v1-train] The screen displays abnormally on the top of Apps
Comment 27•11 years ago
|
||
This seems only reproducible in v1-train
Assignee | ||
Comment 29•11 years ago
|
||
Just hesitated whether I should add a .fullscreen-app to #screen when switching
to lockscreen camera.. In the end I decide just do css polish after looking around
current codes.
Alive may you review the patch?
Attachment #788151 -
Flags: review?(alive)
Assignee | ||
Comment 30•11 years ago
|
||
The patch fixes both
- Alarm screen on secure camera (fail on both master and v1-train)
- Permission screen on secure camera (fail on v1-train)
The latter one simply doesn't jump up on master.
Summary: [Buri][v1-train] The screen displays abnormally on the top of Apps → [Buri] Alarm screen and permission screen displays
Assignee | ||
Comment 31•11 years ago
|
||
Assignee | ||
Comment 32•11 years ago
|
||
Comment 33•11 years ago
|
||
What about the symptoms of bug 903271 for Incoming Call (duplicated to this bug)?
Flags: needinfo?(rexboy)
Comment 34•11 years ago
|
||
Comment on attachment 788151 [details]
patch
r=me, thanks.
Attachment #788151 -
Flags: review?(alive) → review+
Comment 35•11 years ago
|
||
(In reply to Scoobidiver from comment #33)
> What about the symptoms of bug 903271 for Incoming Call (duplicated to this
> bug)?
Different issue I think. Rex do you want to take it too?
Comment 36•11 years ago
|
||
(In reply to Alive Kuo [:alive] from comment #35)
> (In reply to Scoobidiver from comment #33)
> > What about the symptoms of bug 903271 for Incoming Call (duplicated to this
> > bug)?
> Different issue I think. Rex do you want to take it too?
If it's too different, you can undupe bug 903271.
Assignee | ||
Comment 37•11 years ago
|
||
Yes it's different issue. This bug doesn't help bug 903271.
Seems there's already a patch by Leo? Anyway we can take a look there, let's
close this bug first.
Flags: needinfo?(rexboy)
Summary: [Buri] Alarm screen and permission screen displays → [Buri] Alarm screen and permission screen displays abnormally on the top of Apps
Assignee | ||
Comment 38•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
status-b2g18:
--- → affected
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
status-b2g-v1.1hd:
--- → affected
Resolution: --- → FIXED
Comment 39•11 years ago
|
||
Uplifted 94a40401610c28fb47419d44726f199a0fae742e to:
v1-train: 4cd749e4136473c3835667689ecccfa1961e4e64
Comment 40•11 years ago
|
||
(In reply to KM Lee [:rexboy] from comment #37)
> Yes it's different issue. This bug doesn't help bug 903271.
>
> Seems there's already a patch by Leo? Anyway we can take a look there, let's
> close this bug first.
(In reply to Alive Kuo [:alive] from comment #34)
> Comment on attachment 788151 [details]
> patch
>
> r=me, thanks.
I have tested with the latest v1-train and the issue is still reproducible.
The last commit of v1-train was 0c6b0cefeee504b7ed42ba36b983f155b8c6d666.
Steps to repro:
1. Launch Clock app and set an alarm for the next minute.
2. Launch Gallery app and wait until the alarm set off.
The statuabar is still transparent..
Comment 41•11 years ago
|
||
v1.1.0hd: 4cd749e4136473c3835667689ecccfa1961e4e64
Comment 42•11 years ago
|
||
(In reply to KM Lee [:rexboy] from comment #37)
> Yes it's different issue. This bug doesn't help bug 903271.
>
> Seems there's already a patch by Leo? Anyway we can take a look there, let's
> close this bug first.
The patch suggested by Leo was to hide the statusbar, instead of transparent statusbar area. I guess the patch is not acceptable because it changes the behavior..
Assignee | ||
Comment 43•11 years ago
|
||
>
> Steps to repro:
> 1. Launch Clock app and set an alarm for the next minute.
> 2. Launch Gallery app and wait until the alarm set off.
>
> The statuabar is still transparent..
This symptom looks similar but it needs a different patch. Secure camera is an
individual component inside system app, whereas gallery app is an full-screen app.
Since they don't share the same layout rule, I think we can open another
bug for it.
Comment 44•11 years ago
|
||
(In reply to KM Lee [:rexboy] from comment #43)
Then, does the Bug 903271 cover the Gallery app case?
Bug 903271 - [Statusbar] becomes transparent on incoming call using fullscreen app
No need to open a new bug?
Assignee | ||
Comment 45•11 years ago
|
||
I can try to ensure the STR in comment #40 in bug 903271.
You need to log in
before you can comment on or make changes to this bug.
Description
•