991453 - [Sora][Clock]MS flashed a white screen when press power key to light screen

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Clock, defect, P2)

Description

[sync-1]

the latest id: Mozilla build ID: 20140323004002  FFOS: 1.3
 
 DEFECT DESCRIPTION:
  MS flashed a white screen when press power key to light screen
 
  REPRODUCING PROCEDURES:
  Precondition: Disable screen lock.
  1. open "clock" -> Press power key to unlight screen -> Press power key again to light screen ->  MS flashed a white screen(K.O)
 
  EXPECTED BEHAVIOUR:
 MS should not flashed a white screen
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE: 100%

Comment 1

[Warren WU]

I think it is duplicate with 990493.

Comment 2

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

I also think we should block on this one, the screen flash is quite obvious if you disable the screen lock and press the power key in certain apps, for example:

1. Contact
2. Message
3. Setting
4. Alarm - This one is very easy to reproduce

Maybe eventually we won't block on this, but i think this issue is worth investigating to see if there is any way we can improve.

Comment 4

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Jason, I don't think we should just close this one, as I mentioned in Comment#2, we can reproduce this issue in different apps, so to me it is like a general window management problem, also although this issue can be reproduced in 1.1, it is getting worse in 1.3 now. I really think we should at least find someone to check this problem first, if it is too risky to fix, then sure let's put that into backlog, but maybe it is just one or two line fix? I do believe fixing this one will bring better user experience to end user. 

Does this make sense to you?

Thanks

Vance

Comment 5

[Jason Smith [:jsmith]]

(In reply to Vance Chen [:vchen][[e-mail]] from comment #4)
> Hi Jason, I don't think we should just close this one, as I mentioned in
> Comment#2, we can reproduce this issue in different apps, so to me it is
> like a general window management problem, also although this issue can be
> reproduced in 1.1, it is getting worse in 1.3 now. I really think we should
> at least find someone to check this problem first, if it is too risky to
> fix, then sure let's put that into backlog, but maybe it is just one or two
> line fix? I do believe fixing this one will bring better user experience to
> end user. 
> 
> Does this make sense to you?
> 
> Thanks
> 
> Vance

It's still a dupe. The root cause of the problem is an issue with when you power on the device without the lockscreen enabled causes a white flash. I believe that's a known issue.

Comment 6

[Jason Smith [:jsmith]]

According to Jack this is a different bug since bug 990493 is present on 1.1, where as this is a regression. So I'm reopening to track this separately.

Can we confirm this on the Moz side on 1.3?

Comment 7

[Duane Dixon [:ddixon]]

Confirming that white screen flashing issue occurs on the latest Buri 1.3 build on the Moz side.  

Repro Steps:
1. Select Clock>Timer>Lock button>unlock screen>lock button (quickly)

Issue is about 4/5 or 80% repro when using these steps.

Comment 8

[Tim Guan-tin Chien [:timdream] (please needinfo)]

The flash of "white" makes it likely a Gecko issue.

Comment 9

[Jack Liu]

Dear Vance,

It's blocking issue to us, please help fix it.

Comment 10

[sync-1]

(In reply to comment #8)
 > Comment from Mozilla:The flash of "white" makes it likely a Gecko issue.
 > 
 I tried to fixed this bug. 
 /clock/style/clock.css 13 line: background-color: 000;
                                 background-color: #000;
 
 Then MS become normally. Is the CSS issue ?

Comment 11

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Tim -

Our partner just have a workaround solution for the Alarm white screen flashing, could you kindly help to comment? Only a workaround to reduce the symptom

Comment 12

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Yes, you could hide the default background-color by changing the background-color, and yes, that 000 is not a right color value.

Please file another bug under Clock to fix the CSS if that doesn't solve the real issue here.

Comment 13

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Jack -

Per Comment#12, it is OK to fix the clock white screen issue with the CSS modification you proposed on Comment#10.

Comment 14

[Jack Liu]

(In reply to Vance Chen [:vchen][[e-mail]] from comment #13)
> Hi Jack -
> 
> Per Comment#12, it is OK to fix the clock white screen issue with the CSS
> modification you proposed on Comment#10.
Hi Vance,
Thanks, I'm glad this bug can be fixed and won't annoy end user any more.

Comment 15

[Marcus Cavanaugh [:mcav]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/18591

Comment 16

[Gareth Aye [:gaye] (back from PTO)]

Comment on attachment 8411038 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/18591

LGTM. Also noting here that the black body background thing came out of https://bugzilla.mozilla.org/show_bug.cgi?id=948208.

Comment 17

[Marcus Cavanaugh [:mcav]]

master: https://github.com/mozilla-b2g/gaia/commit/18fd4d436797ede7503cd753a22b5a9804282042

Comment 18

[Edward Chen[:edchen]]

[Environment]
Gaia      265e1ac4ee71ad6190335c974bfce33f783edfce
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/36f67ce46855
BuildID   20140427000203
Version   30.0a2
ro.build.version.incremental=324
ro.build.date=Thu Dec 19 14:04:55 CST 2013


[Result]
PASS

Comment 19

[liangxin]

(In reply to Marcus Cavanaugh [:mcav] <[e-mail]> from comment #17)
> master:
> https://github.com/mozilla-b2g/gaia/commit/
> 18fd4d436797ede7503cd753a22b5a9804282042

This patch can not resolve other apps flashed a white screen when press power key to light screen.Foe example:
1. Contact
2. Message
3. Setting
It is not a complete solution.

Comment 20

[liangxin]

(In reply to Edward Chen[:edchen] from comment #18)
> [Environment]
> Gaia      265e1ac4ee71ad6190335c974bfce33f783edfce
> Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/36f67ce46855
> BuildID   20140427000203
> Version   30.0a2
> ro.build.version.incremental=324
> ro.build.date=Thu Dec 19 14:04:55 CST 2013
> 
> 
> [Result]
> PASS

This patch can not resolve this pr. It does not works.

Comment 21

[Marcus Cavanaugh [:mcav]]

> This patch can not resolve other apps flashed a white screen when press
> power key to light screen.

If there are problems in other apps, they should be resolved in a separate bug.