910018 - [Buri][Data usage]Set data use alert,when exceed the data,it will not remind you right now.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Cost Control, defect, P2)

Description

[sync-1]

Firefox os  v1.1
 Mozilla build ID:20130817041203
 
 Created an attachment (id=501905)
 logcat.log
 
 DEFECT DESCRIPTION:
  Set data use alert.When exceed the data,it will not remind you right now,until you enter the APP.
 
  REPRODUCING PROCEDURES:
  Set data use alert:5MB.Use the mobile usage,when exceed 5MB,there is not presentation until you enter the APP--KO
 
  EXPECTED BEHAVIOUR:
  it will remind you right now，if the mobile usage exceed 5MB 
 
  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 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: logcat.log

Comment 3

[Leo]

I think this bug depends on Bug-901416 (#Comment 19).So adding Jlebar and Salva in CC.

Comment 4

[Salvador de la Puente González [:salva]]

Let me review the problem but if the network events are being received as expected, the only thing we can do is to lower the threshold.

Comment 5

[bhavana bajaj [:bajaj]]

needinfo :kyle here to see if its fallout from Bug-901416 (#Comment 19) ?

Comment 6

[Preeti Raghunath(:Preeti)]

QA,

Please test if this is consistently happening on our latest builds. Please check if it is a messaging issue or an actual issue.

Comment 7

[Sarah Parsons]

This issue is reproducible on the Buri 1.2 Build ID: 20130904040205 by setting the data alert to 5MB then surfing the browser. The user will only be notified with an alert if they pull down the notifications bar or open the usage app. Otherwise, the user is not reminded.

Environmental Variables
Build ID: 20130904040205
Gecko: http://hg.mozilla.org/mozilla-central/rev/7ff96bd19c1c
Gaia: b6c5bf1d24230bfed4a8f680e625fa2175001f82
Platform Version: 26.0a1

Also, reproduced this issue on the Buri 1.1 Build ID: 20130904041204

Environmental Variables
Build ID: 20130904041204
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/307824edadd7
Gaia: d0a415bbf23e5d01c2b287d9fca708e167cfe70d
Platform Version: 18.1

And reproduced this issue on the Leo 1.1 Build ID: 20130904041204

Environmental Variables
Build ID: 20130904041204
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/307824edadd7
Gaia: d0a415bbf23e5d01c2b287d9fca708e167cfe70d
Platform Version: 18.1
RIL Version: 01.01.00.019.211

Comment 8

[Preeti Raghunath(:Preeti)]

Daniel,

Please check if this is a cert issue for TEF

Comment 9

[Salvador de la Puente González [:salva]]

Hello

We have two problems here. 

1- 250 threshold is very high. I was trying with lower values. 75 seems enough.
2- For some reason, video does not trigger network events at a proper rate to reach the thresholds.

I continue working on it because I detected that the alarm is not triggered in every situation.

Comment 10

[Salvador de la Puente González [:salva]]

Oh, and we found this bug during Peru certification process.

Comment 11

[Salvador de la Puente González [:salva]]

I'm going to low the threshold to 75 and provide a patch. Then I'm gonna add qawanted to test the patch.

Comment 12

[Salvador de la Puente González [:salva]]

Attachment: Lowering the data check threshold.

Comment 13

[Salvador de la Puente González [:salva]]

Master: 88275d6f6194c7d0d8c5bbe47c9fcd7b257635ba

Comment 14

[Natalya Kot [:nkot]]

I'm assuming the patch from comment 12 is included in 09/10 m-r build... the issue doesn't reproduce on this build, when I set data alert for 5MB and then browse web using cell data, I do receive notification when usage exceeds 5MB without having to check it manually by pulling down notification bar. It's slightly delayed, I have 7.98MB usage when notification arrives, not sure if it's OK.

Buri, US_20130903_QC206
Build ID: 20130910040201
Gecko: http://hg.mozilla.org/mozilla-central/rev/be1053dc223b
Gaia: 6deda9d7c51f278443f704217eaed722044a03e7
Platform Version: 26.0a1

Comment 15

[Salvador de la Puente González [:salva]]

Asking for leo? because this bug has been reported as a certification issue.

Comment 16

[Preeti Raghunath(:Preeti)]

Not a regression issue and it seems to be a feature enhancement. Will be available in 1.2

Comment 17

[Salvador de la Puente González [:salva]]

No, this is a regression because before bug 901416 this worked well (i.e. the notification delay was acceptable). After fixing that platform bug, the delay becomes too much. Renominating. WDYT?

Comment 18

[sync-1]

Dear Salvador,

In this version if only adjust 250 -> 75, it still ko.
Firefox os  v1.1
 Mozilla build ID:20130902041201

Comment 19

[Salvador de la Puente González [:salva]]

First, note this has not been included in the official branch. 

If you applied the patch and it is not working, please, enable the extended traces (go to costcontrol/js/utils/debug.js and set DEBUGGING to true). Then repeat the tests and attach the logs.

Comment 20

[sync-1]

Attachment: costcontrol.log

Comment 21

[sync-1]

Dear Salvador,

Please check attachment, thanks.

Comment 22

[Jason Smith [:jsmith]]

Blocking+ - it's critically important the user is aware of when data limit is exceeded and this is a regression.

Comment 23

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

Uplifted 88275d6f6194c7d0d8c5bbe47c9fcd7b257635ba to:
v1-train: edd44ef32a8fcae6e937ee349a11e7f847456ed4

Comment 24

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

v1.1.0hd: edd44ef32a8fcae6e937ee349a11e7f847456ed4