B2G MMS: Change gMmsConnection into an interface based object.

RESOLVED WONTFIX

Status

Firefox OS
RIL
RESOLVED WONTFIX
5 years ago
3 months ago

People

(Reporter: ctai, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Currently, gMmsConnection is an object in MmsService.js and don't have a good network manager yet. We should create an interface for all the connection functions for MMS. That will provide more flexible way for different connection logic. For example, we can replace the original MMS connection with test-based connection.
Created attachment 782370 [details] [diff] [review]
bug-897817.patch-v1.0
Attachment #782370 - Flags: feedback?(vyang)
Comment on attachment 782370 [details] [diff] [review]
bug-897817.patch-v1.0

I think that's only an option, not something necessary for now.  We can re-examine the necessity when we have a working MMS test framework.  Shall we?
Attachment #782370 - Flags: feedback?(vyang)
Created attachment 812955 [details] [diff] [review]
bug-897817.patch v1.1

Per talk with vyang, we need this for MMS test framework.
Attachment #782370 - Attachment is obsolete: true
Attachment #812955 - Flags: feedback?(vyang)
Comment on attachment 812955 [details] [diff] [review]
bug-897817.patch v1.1

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

Let's keep it in |MmsService.js| first, so that we don't install new components.
Attachment #812955 - Flags: feedback?(vyang)
Component: DOM: Device Interfaces → RIL
Product: Core → Firefox OS
Assignee: ctai → nobody

Comment 5

3 months ago
Firefox OS is not being worked on
Status: NEW → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.