856916 - [Buri]LCD become dark after system sleep out

Status: RESOLVED WORKSFORME

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

Description

[sync-1]

AU_LINUX_GECKO_ICS_STRAWBERRY_V1.01.00.01.19.044
 Firefox os  v1.0.1
 Mozilla build ID: 20130319070203
 
 +++ This bug was initially created as a clone of Bug #432001 +++
 
 DEFECT DESCRIPTION:
 
 1. Set the brightness level to 100% and disable "adjust automatically".
 2. Wake up MS after system go to sleep(or press power key twice)
 
 ==> LCD brightness can't go back to 100% after sleep out.
 
 In driver layer, i can see level 127 setted(the maximum value is 255).
 
 
 REPRODUCING PROCEDURES:
 
 EXPECTED BEHAVIOUR:
 
 ASSOCIATE SPECIFICATION:
 
 TEST PLAN REFERENCE:
 
 TOOLS AND PLATFORMS USED:
 
 USER IMPACT:
 
 REPRODUCING RATE:
 
 For FT PR, Please list reference mobile's behavior:
 
 ++++++++++ end of initial bug #432001 description ++++++++++
 
 
 
 CONTACT INFO (Name,Phone number):
 
  DEFECT DESCRIPTION:
 
  REPRODUCING PROCEDURES:
 
  EXPECTED BEHAVIOUR:
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[Daniel Coloma:dcoloma]

I have tried with multiple devices (Buri, Unagi, Peak) and can't notice any difference. qawanted to check if it is only me the one not noticing it.

Comment 2

[sync-1]

(In reply to comment #1)
 > Comment from Mozilla:I have tried with multiple devices (Buri, Unagi, Peak) and
 > cant

Comment 3

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

We can check on Unagi within Mozilla. We don't have the other devices available to check immediately, so someone else will need to confirm there.

Comment 4

[sync-1]

Hi there,
 
 On the Buri, we turn off Adjust automatically and set Brightness to a value
 (like 1 or 0) manually, then turn off the screen or let the phone go to sleep
 after timeout, finally we turn on the screen (press the power key) and will
 find the brightness level is not what we set manually in the first step. If we
 want to go back to the Brightness value we have set before, we have to launch
 settings and set the Brightness again.
 
 Briefly, after wake up, the Brightness value changed to another value, maybe
 0.5, everytime.
 
 (Assume the Brightness if between 0 and 1)

Comment 5

[Daniel Coloma:dcoloma]

I cannot repro. Can you try to reproduce with the most recent build you have and tell us the tips of that build?

Comment 6

[sync-1]

Tried on latest build, still same problem.
 
 Firefox os  v1.0.1
 Mozilla build ID:20130331070206
 
 While, we will also check the problem at the same time.
 
 Any logs are helping?

Comment 7

[Daniel Coloma:dcoloma]

Joe, who from Mozilla could have a look at this? This is not occurring in any other device.

Comment 8

[Joe Cheng [:jcheng] (please needinfo)]

kkuo, any investigation help can possibly be provided? thanks

Comment 9

[viral [:viralwang]]

We can found this issue in build id 20130410185334
Due to the permission limited, we can only have logcat/dmesg to analysis but no useful information found.

We also test it in our lastest code in v1.0.1 and not found this issue.
It goods like the default setting of backlight is set after device suspend/resume.

Comment 10

[Daniel Coloma:dcoloma]

It seems we need support from reporter to progress with this

Comment 11

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot of brightness comparable brightness setting

I believe that it's not necessarily with the after sleeping.  If you set the device to automatic, it will be as dim as the setting shown in the screenshot regardless.

Gecko  http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/ada03bf95314
Gaia   3e7dda88950ae09166109783adea8181eb857d21
BuildID 20130412070205
Version 18.0
Inari

Comment 12

[xiaokang.chen]

(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #11)
> Created attachment 737067 [details]
> screenshot of brightness comparable brightness setting
> 
> I believe that it's not necessarily with the after sleeping.  If you set the
> device to automatic, it will be as dim as the setting shown in the
> screenshot regardless.
> 
> Gecko  http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/ada03bf95314
> Gaia   3e7dda88950ae09166109783adea8181eb857d21
> BuildID 20130412070205
> Version 18.0
> Inari

I think we should consider all the posibilities that users maybe encountered. Imagine this scene: the user don't want to use the "automatic adjust" and set a brightness they like manually, but everytime when they wake up the phone from sleep mode, the user always find the brightness is not what they set before. What a terrible experience!

I found the system called 
power.screenBrightness = brightness; 
in the gaia/apps/system/js/screen_manager.js, the brightness is the user set before, and the power.screenBrightness is actually set to brightness, but didn't take effect.

Comment 13

[Joe Cheng [:jcheng] (please needinfo)]

alive, can you take this per comment 12? if you are overlaoded, please feel free to reassign or unassign. thanks

Comment 14

[Alex Keybl [:akeybl]]

(In reply to sync-1 from comment #4)
>  Briefly, after wake up, the Brightness value changed to another value, maybe
>  0.5, everytime.

qawanted to confirm that this only occurs briefly (test Buri with the latest build). If yes, this is not a blocker.

Comment 15

[Tony Chung [:tchung]]

Reproduced, only on the Buri device does this happen.   Doesn't repro on the Ikura.

IMO, this is not a tef blocker, and should be tracking-b2g instead.  the dimming is very slight, and does not break the user experience.

Buri build: 2013041214825

Comment 16

[Alex Keybl [:akeybl]]

(In reply to Tony Chung [:tchung] from comment #15)
> IMO, this is not a tef blocker, and should be tracking-b2g instead.  the
> dimming is very slight, and does not break the user experience.

Thanks Tony - sounds like we honor the user's requested brightness (minus some small %). Agreed on tracking but not blocking.

Comment 17

[sync-1]

only reproduced on buri and brother closed

Comment 18

[(inactive after 6/18) Alive Kuo [:alive]]

Can't repro, closing