Closed Bug 891235 Opened 6 years ago Closed 6 years ago

WebSMS: remove navigator.mozSms

Categories

(Core :: DOM: Device Interfaces, defect, trivial)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: vicamo, Assigned: vicamo)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [fixed-in-birch])

Attachments

(1 file, 4 obsolete files)

We should have only navigator.mozMobileMessage from now on.
Attached patch WIP (obsolete) — Splinter Review
Attached patch patch (obsolete) — Splinter Review
https://tbpl.mozilla.org/?tree=Try&rev=d516adace020
Attachment #773227 - Attachment is obsolete: true
Attachment #773887 - Flags: review?
Attachment #773887 - Flags: review? → review?(mounir)
Comment on attachment 773887 [details] [diff] [review]
patch

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

rs=me

Though, some changes might no longer apply with the recent Navigator's WebIDL conversion.
Attachment #773887 - Flags: review?(mounir) → review+
Attached patch patch : v2 (obsolete) — Splinter Review
Rebase. https://tbpl.mozilla.org/?tree=Try&rev=3d9a65d66d2f
Attachment #773887 - Attachment is obsolete: true
Attachment #779746 - Flags: review+
Attached patch patch : v3 (obsolete) — Splinter Review
fix Mn bustage.
Attachment #779746 - Attachment is obsolete: true
Attachment #780138 - Flags: review+
Blocks: b2g-sms
Attached patch bug-891235.patchSplinter Review
Picked wrong patch file.
Attachment #780138 - Attachment is obsolete: true
Attachment #781623 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/5512e76c7a26
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Removed this bug from the site compat doc. My false positive.
Keywords: site-compat
You need to log in before you can comment on or make changes to this bug.