WebMobileConnection: Use webidl event generator for the event that is dispatched by mozMobileConnection

RESOLVED FIXED in mozilla29

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: edgar, Assigned: edgar)

Tracking

unspecified
mozilla29
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

After moving to use webidl event generator, below interfaces can be removed,
- dom/network/interfaces/nsIDOMMozEmergencyCbModeEvent.idl
- dom/network/interfaces/nsIDOMMozOtaStatusEvent.idl
- dom/network/interfaces/nsIDOMUSSDReceivedEvent.idl
- dom/network/interfaces/nsIDOMDataErrorEvent.idl
- dom/network/interfaces/nsIDOMCFStateChangeEvent.idl
Comment on attachment 8355451 [details] [diff] [review]
Patch, v1

Hi smaug, need your help to review this, thank you.
Attachment #8355451 - Flags: review?(bugs)
Comment on attachment 8355451 [details] [diff] [review]
Patch, v1

Great, thanks!
Attachment #8355451 - Flags: review?(bugs) → review+
Depends on: 956723
https://hg.mozilla.org/mozilla-central/rev/66ffd474bf5b
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
So we don't want to condition these on MOZ_B2G_RIL anymore?
(In reply to Peter Van der Beken [:peterv] from comment #7)
> So we don't want to condition these on MOZ_B2G_RIL anymore?

We condition these on prefs 'dom.mobileconnection.enabled'.
And this prefs will be configured based on MOZ_B2G_RIL.
Sure, but that means that we're increasing the binary size with code that's never used (unless dom.mobileconnection.enabled is somehow ever true on non-b2g builds).
You need to log in before you can comment on or make changes to this bug.