871437 - [Settings] Data connection turn off automatically after booting with low battery

Status: RESOLVED FIXED

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

Description

[Leo]

1. Title : Data connection turn off automatically after booting out of batter
2. Precondition : Data connection turn on and low battery
3. Tester's Action : let it be until out of battery
4. Detailed Symptom : After reboot, go to data connection menu. Data connection turn off
5. Expected : Data connection keep the status as on
6. Reproducibility: Y
   - Frequency Rate : 100%
7.Gaia Master/v1-train : No
8. Version Info
Gaia : 5cbb19e4bb78a7ad879fbe4b9a841e1c35714f5c
Gecko : e4a677c13bf35253fea99e6b85df1c707038d119
Mozilla build ID : 20130421070203

Comment 1

[Ken Chang[:kenkai]]

Is this scenario the origianl design?

Comment 2

[Shian-Yow Wu [:swu]]

(In reply to Ken Chang from comment #1)
> Is this scenario the origianl design?

No, it's not expected behavior.

Comment 3

[Shian-Yow Wu [:swu]]

I suspect it could be settings broken, and goes back to factory default value.
Are other settings normal when out of battery?

Comment 4

[hyuna.cho]

(In reply to Shian-Yow Wu from comment #3)
> I suspect it could be settings broken, and goes back to factory default
> value.
> Are other settings normal when out of battery?

Other settings doesn't change when out of battery.
Sometimes it works well.

Comment 5

[Mike Habicher [:mikeh] (high bugzilla latency)]

Just to clarify: the cellular radio is ON while data is OFF?

Comment 6

[hyuna.cho]

(In reply to Mike Habicher [:mikeh] from comment #5)
> Just to clarify: the cellular radio is ON while data is OFF?

The cellular radio is connected and only data is off.

Comment 7

[Ken Chang[:kenkai]]

(In reply to hyuna.cho from comment #6)
> (In reply to Mike Habicher [:mikeh] from comment #5)
> > Just to clarify: the cellular radio is ON while data is OFF?
> 
> The cellular radio is connected and only data is off.

Can you please provide a log?

Comment 8

[hyuna.cho]

(In reply to Shian-Yow Wu from comment #3)
> I suspect it could be settings broken, and goes back to factory default
> value.
> Are other settings normal when out of battery?

I checked all setting values when load Setting app.
Some values are changed like the below.
 geolocation.enabled
 geolocation.suspended
 powersave.enabled
 ril.data.enabled
 time.nitz.available

'powersave.enabled' value is changed from false to true. so 'ril.data.enable' also change from true to false.

I will attach a log soon

Comment 9

[John Hu [:johnhu][:johu][:醬糊小弟]]

What's the value in Settings App -> Battery -> Power save mode -> Turn on automatically option???

If it is not "never", that's correct to turn on powersave.enable and turn off the following settings:

  wifi.enabled
  ril.data.enabled
  bluetooth.enabled
  geolocation.enabled

Comment 10

[hyuna.cho]

(In reply to John Hu [:johnhu] from comment #9)
> What's the value in Settings App -> Battery -> Power save mode -> Turn on
> automatically option???
> 
> If it is not "never", that's correct to turn on powersave.enable and turn
> off the following settings:
> 
>   wifi.enabled
>   ril.data.enabled
>   bluetooth.enabled
>   geolocation.enabled

The default values are same with the follow
 powersave.enabled : false
 powersave.threshold : 0

After booting with low battery,
 powersave.enabled : true
 powersave.threshold : 0

Comment 11

[John Hu [:johnhu][:johu][:醬糊小弟]]

I need to focus on another bug about email.

Comment 12

[Leo]

Attachment: ADB_log

It's adb log file.

Comment 13

[Alex Keybl [:akeybl]]

Do we have everything we need for the investigation here? Any ETA for a patch?

Comment 14

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Yes.
Will provide a patch today.

Thanks.

Comment 15

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9985

Pointer to Github pull-request

Comment 16

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Comment on attachment 753741 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9985

I think this is because the ambiguity of the threshold value, it is "0" originally, and I thought it won't reach that since we would power off the phone when the battery level < 2%.

And now I changed it to -1 for safety.

Hi Alive,

Please help review this simple patch.
Thanks.

Comment 17

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

Comment on attachment 753741 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/9985

I miss this...makes sense.

Comment 18

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Gaia master,
https://github.com/mozilla-b2g/gaia/commit/f2930cf440e0c8c21b48f7b0b01fac48b941e3fc


--
Alive, thanks.

Comment 19

[Jason Smith [:jsmith]]

If you want this considered for 1.01, then you'll need a tef+. Although I don't think I would block on this for 1.01 at this point.

Comment 20

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted f2930cf440e0c8c21b48f7b0b01fac48b941e3fc to:
v1-train: 055c80563dae10f75e6e970cbafcae2949f8a0f2

Comment 21

[Darren]

Wrongly added flag and whiteboard, looks like the build checked on just is having data connection issue not pertaining to low battery.