RIL: Data call fixes

RESOLVED FIXED in mozilla13

Status

()

Core
DOM: Device Interfaces
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: philikon, Assigned: philikon)

Tracking

Trunk
mozilla13
ARM
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Bug 713426 was a good start, but contains several typos (the code was never run, apparently) and some inconsistent behaviour wrt main thread notifications.
Created attachment 595919 [details] [diff] [review]
v1

Feex
Attachment #595919 - Flags: review?(kyle)

Comment 2

5 years ago
Comment on attachment 595919 [details] [diff] [review]
v1

Review of attachment 595919 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/system/b2g/ril_worker.js
@@ +1846,5 @@
> +          case DATACALL_INACTIVE:
> +            newDataCall.state = GECKO_NETWORK_STATE_DISCONNECTED;
> +            break;
> +          case DATACALL_ACTIVE_DOWN:
> +            newDataCall.state = GECKO_NETWORK_STATE_SUSPENDED;

On my SGS2, it is not actually suspended when a data call is active down.  And, it never goes to active up.  Is it a bug of rild extracted from SGS2?  (Bug #718903)
(In reply to Thinker Li [:sinker] from comment #2)
> ::: dom/system/b2g/ril_worker.js
> @@ +1846,5 @@
> > +          case DATACALL_INACTIVE:
> > +            newDataCall.state = GECKO_NETWORK_STATE_DISCONNECTED;
> > +            break;
> > +          case DATACALL_ACTIVE_DOWN:
> > +            newDataCall.state = GECKO_NETWORK_STATE_SUSPENDED;
> 
> On my SGS2, it is not actually suspended when a data call is active down. 
> And, it never goes to active up.  Is it a bug of rild extracted from SGS2? 
> (Bug #718903)

Good question. I will investigate that. We can always work around that with another "QUIRKS" thing if we find it's an SGS2-specific bug.
Comment on attachment 595919 [details] [diff] [review]
v1

Review of attachment 595919 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, r=me
Attachment #595919 - Flags: review?(kyle) → review+
Thanks!

https://hg.mozilla.org/mozilla-central/rev/6b5537115a23
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.