SIM lock shows a transparent status bar

RESOLVED FIXED in Firefox OS v2.0

Status

Firefox OS
Gaia::System
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: nhirata, Assigned: gmarty)

Tracking

unspecified
2.0 S6 (18july)
x86
Mac OS X

Firefox Tracking Flags

(blocking-b2g:2.0+, b2g-v2.0 fixed, b2g-v2.1 fixed)

Details

(Whiteboard: [systemsfe])

Attachments

(2 attachments)

Created attachment 8442479 [details]
2014-06-18-23-20-26.png

1. go to settings -> SIM manager -> sim security
2. turn sim pin on 
3. create a passcode
4. reboot phone
5. unlock screen to get to SIM Pin

Expected: black status bar
Actual: transparent status bar

Gaia      83844c7679b3b9f6e7f1116c1eeec2d1e7a64eec
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/55679dc2e72b
BuildID   20140618000202
Version   32.0a2
ro.build.version.incremental=108
ro.build.date=Tue Jun 10 19:40:40 CST 2014
Flame
UX clarified in a separate bug that any instance of a transparent status bar showing in a dialog was a blocker for visual refresh for 2.0, so noming this.
blocking-b2g: --- → 2.0?
Whiteboard: [systemsfe]
blocking-b2g: 2.0? → 2.0+

Comment 2

3 years ago
I'll have a look at this this afternoon.
Assignee: nobody → aus
Status: NEW → ASSIGNED
Target Milestone: --- → 2.0 S5 (4july)

Comment 3

3 years ago
Ugh! My SIM I got from IT already has a PIN set on it and I don't know what it is! Someone else should grab this in the meantime. :(
Assignee: aus → nobody
Target Milestone: 2.0 S5 (4july) → ---
(Assignee)

Updated

3 years ago
Assignee: nobody → gmarty
Target Milestone: --- → 2.0 S5 (4july)
(Assignee)

Comment 4

3 years ago
Created attachment 8447235 [details]
Github PR

The events that open/close the sim pin dialog are listened and the status bar appearance is updated accordingly.

Etienne, can you have a look at this one?
Attachment #8447235 - Flags: review?(etienne)
Comment on attachment 8447235 [details]
Github PR

The sim pin dialog can be displayed on top of an app.

We should check that on simpinclose to make sure we're not making the statusbar transparent while the dialer is displayed for example.

Also, this change should be easily covered by tests :)
Attachment #8447235 - Flags: review?(etienne)
(Assignee)

Updated

3 years ago
Attachment #8447235 - Flags: review?(etienne)
Comment on attachment 8447235 [details]
Github PR

Almost there, and the tests are good.

But I think you still need to listen to the simpinopen event.
STR:
- On homescreen
- turn airplane mode on
- turn airplane mode off
- the sim dialog pops up but the status bar is still transparent
Attachment #8447235 - Flags: review?(etienne)
(Assignee)

Comment 7

3 years ago
Comment on attachment 8447235 [details]
Github PR

I updated the code and tests to listen to simpinshow event and check this scenario is now OK.
Attachment #8447235 - Flags: review?(etienne)
Comment on attachment 8447235 [details]
Github PR

The patch that keeps on giving :/

Okay, broken STR with the last version:
- skip the sim pin dialog the first time
- be at the top of the homescreen (transparent statusbar)
- open the dialer (black status bar + sim pin dialog)
- skip the sim pin dialog again (status bar still black)
- go back to the homescreen -> status bar black instead of transparent :/

This was a bit of a headscratcher, but basically when an app is active the status bar background will be black anyway.

So we need to *always* call  |this.setAppearance('semi-transparent');| when we get a simpinclose event.

This should fix all the scenarios and simplify the patch quite a bit.

Sorry for the earlier misinformation :/ Having a hard time wrapping my head around this part of the code.
Attachment #8447235 - Flags: review?(etienne)
(Assignee)

Comment 9

3 years ago
Comment on attachment 8447235 [details]
Github PR

The latest version of the patch fixes this edge case.
Hope that'll be ok that time :-)
Attachment #8447235 - Flags: review?(etienne)
Comment on attachment 8447235 [details]
Github PR

All good, congrats! This was harder than it looked :)

(PS: the patch needs rebasing, but it's an easy rebase)
Attachment #8447235 - Flags: review?(etienne) → review+
(Assignee)

Comment 11

3 years ago
Thanks Etienne! It's now rebased, waiting on Travis and Try.
(Assignee)

Comment 12

3 years ago
Just merged in:
https://github.com/mozilla-b2g/gaia/commit/3c5a0ffdc81419b50b84e24e73bdaa0187c8ded6
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
v2.0: https://github.com/mozilla-b2g/gaia/commit/16410831c2e513b5cabc550363f28a023dc256be
status-b2g-v2.0: --- → fixed
status-b2g-v2.1: --- → fixed
Target Milestone: 2.0 S5 (4july) → 2.0 S6 (18july)
Reverted from v2.0 for Gaia unit test failures.
v2.0: https://github.com/mozilla-b2g/gaia/commit/e935f4ff190b76c70d9b2af8856c542a6e4a7546

https://tbpl.mozilla.org/php/getParsedLog.php?id=43267991&tree=Mozilla-Aurora
status-b2g-v2.0: fixed → affected
Flags: needinfo?(gmarty)
Keywords: branch-patch-needed
(Assignee)

Comment 15

3 years ago
I remove the declaration of |originalLocked| because it's not used in master, though it is in 2.0.

The updated PR is https://github.com/mozilla-b2g/gaia/pull/21491, waiting on Travis and Try.
Flags: needinfo?(gmarty)

Updated

3 years ago
Duplicate of this bug: 1035485
Looks like this got merged to v2.0 already.
v2.0: https://github.com/mozilla-b2g/gaia/commit/4b7733d0a7a1efaffb4fd1540b1008afc83788bb

gmarty, please don't forget to mark the bugs when you land.
status-b2g-v2.0: affected → fixed
Keywords: branch-patch-needed
You need to log in before you can comment on or make changes to this bug.