Closed Bug 936763 Opened 11 years ago Closed 11 years ago

B2G MMS: the connected/disconnected network can come from 3G not MMS, which throws exceptions when accessing MMS proxy settings

Categories

(Firefox OS Graveyard :: RIL, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: airpingu, Assigned: airpingu)

References

Details

Attachments

(1 file, 2 obsolete files)

      No description provided.
Steps:

1. Turn off 3G.
2. Send an MMS.
3. MmsService.js will create a connection for that and cache it for 30 seconds.
4. Within this 30 seconds, turn on 3G.
5. Log shows:

E/GeckoConsole(  111): [JavaScript Error: "NS_ERROR_UNEXPECTED: 'Unexpected error' when calling method: [nsIRilNetworkInterface::mmsc]" {file: "jar:file:///system/b2g/omni.ja!/components/MmsService.js" line: 165}]
I don't have a better solution by far since APN can be shared for 3G and MMS, unless we can expose something like .inConnectedTypes(...).
Attached patch Patch (obsolete) — Splinter Review
Attachment #829706 - Flags: review?(vyang)
Blocks: b2g-mms
Bug 928861 can help this a lot, which provides a callback not observer for connected. This patch is just a work-around.
Summary: B2g MMS: the connected network interface can com from 3G not MMS, which throws exceptions when accessing MMS proxy settings → B2G MMS: the connected/disconnected network can come from 3G not MMS, which throws exceptions when accessing MMS proxy settings
Attached patch Patch, V2 (obsolete) — Splinter Review
A better solution. Please see the comment. Thanks!
Attachment #829706 - Attachment is obsolete: true
Attachment #829706 - Flags: review?(vyang)
Attachment #829719 - Flags: review?(vyang)
Attached patch Patch, V2.1Splinter Review
Polish the comment.
Attachment #829719 - Attachment is obsolete: true
Attachment #829719 - Flags: review?(vyang)
Attachment #829720 - Flags: review?(vyang)
Attachment #829720 - Flags: review?(vyang) → review+
https://hg.mozilla.org/mozilla-central/rev/6d7dd3e97d68
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Hi skao, please take a look at this bug which might be related to what you're working for NetworkManager.acquireConnection(). Well, if it's lucky enough it might also solve bug 937013.

Btw, thanks Vicamo for the review and check-in. :)
Blocks: 928861
Hi Gene,

I tried a build including this patch and didn't solve bug 937013 :(
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: