827663 - [Settings][Data Roaming] Can't close data roaming after turned on it.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: General, defect, P2)

Description

[Walter Chen[:ypwalter][:wachen]]

2012-1-7 b2g18(v1.0) build from pvt server
2012-12-28 date(v1.1) build from pvt server

STR:
  0. Make sure you are using a international SIM card
  1. Go to "Settings" > "Cellular & Data"
  2. Turn on "Data Connection"
  3. Turn on "Data Roaming"
  4. Wait till 3G/E/H sign came out
  5. Turn off "Data Roaming"
  6. Go to "Browser" and browse the internet

expected result: 
  Should show no internet

actual result:
  3G, E, or H sign is still there near the signal icon. You are still able to do data roaming!


That's not good at all. You think you closed the data roaming, but it actually doesn't get closed.

Comment 1

[Walter Chen[:ypwalter][:wachen]]

For v1.0, it should either disable the function if it is OOS so that it won't affect the normal use of this phone. Or, someone should fix it.

For v1.1, it is a blocker since we are going to ship with data roaming in no time.

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

I'm pretty sure this is a platform bug. Let's add it on their radar.

Comment 3

[Shian-Yow Wu [:swu]]

Attachment: V1

Disconnect data call if roaming turned off when device is roaming.

Comment 4

[Shian-Yow Wu [:swu]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/383de4872c30

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/383de4872c30

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/77ebb9d27dfe

Comment 7

[Walter Chen[:ypwalter][:wachen]]

Wait, this is strange. Why firefox19 20 wont be fixed?
The Firefox OS won't upgrade soon to firefox 19 or 20?
Why not put the patch in 19 20?

Comment 8

[Ryan VanderMeulen [:RyanVM]]

No, uplifts are only going onto b2g18. The post-b2g18 FirefoxOS release will come from a later Gecko revision (TBD).

Comment 9

[Walter Chen[:ypwalter][:wachen]]

Since it's TBD, why not just patch on those two version to prevent from possible regression?

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Because the release drivers said so.

Comment 12

[Walter Chen[:ypwalter][:wachen]]

Verified in pvt b2g-18 nightly build 	2013-01-18-23-02-02