Check capabilities for SMS access

RESOLVED FIXED in mozilla17

Status

()

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

People

(Reporter: cjones, Assigned: cjones)

Tracking

Trunk
mozilla17
Points:
---

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Comment hidden (empty)
Created attachment 645228 [details] [diff] [review]
Check "sms" capability for PSms

This patch doesn't work currently because sms isn't hooked up to the permission manager.  (I get UNKNOWN_ACTION when running this.)  But cross-process sms also crashes, so this doesn't break things any further.

I'm also assuming here that we want a coarse-grained "access all of the SMS API" permission, not something finer-grained like reading/writing.  If that assumption is wrong, let me know.
Assignee: nobody → jones.chris.g
Attachment #645228 - Flags: review?(mounir)
Created attachment 645686 [details] [diff] [review]
Check "sms" capability for PSms, v2

Same assumption applies.
Attachment #645228 - Attachment is obsolete: true
Attachment #645228 - Flags: review?(mounir)
Attachment #645686 - Flags: review?(mounir)
Comment on attachment 645686 [details] [diff] [review]
Check "sms" capability for PSms, v2

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

I would prefer to review that when all the other plumbing will be done (ie. capabilities and sms permissions). r+ a patch that doesn't work/isn't testable doesn't make that much sense. Even more when it's that trivial ;)
Attachment #645686 - Flags: review?(mounir)
I have a working patch built on bug 776649 that I've been testing with the last two days, but will hold off posting per mounir's request.  Doesn't really matter, it's a trivial patch.
Created attachment 646414 [details] [diff] [review]
Check "sms" capability for PSms, v3
Attachment #645686 - Attachment is obsolete: true
blocking-basecamp: --- → +
Comment on attachment 646414 [details] [diff] [review]
Check "sms" capability for PSms, v3

Hope you don't mind pinch-reviewing for mounir here, but this patch is deliciously trivial.
Attachment #646414 - Flags: review?(justin.lebar+bug)
Attachment #646414 - Flags: review?(justin.lebar+bug) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/769750c602b2

Comment 8

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