1121853 - Add [AvailableIn="CertifiedApps"] and [CheckPermissions="voicemail"] to Voicemail interfaces

Status: RESOLVED FIXED

(Firefox OS Graveyard :: RIL, defect)

Description

[Hsin-Yi Tsai (she/her)[:hsinyi]]

To hide the below from privileged or web content:
MozVoicemailEvent.webidl
MozVoicemailStatus.webidl
MozVoicemail.webidl

Comment 1

[Hsin-Yi Tsai (she/her)[:hsinyi]]

Attachment: patch

1) Add [AvailableIn="CertifiedApps"] and [CheckPermissions="voicemail"] 
2) Modify mochitests

Comment 2

[Hsin-Yi Tsai (she/her)[:hsinyi]]

Attachment: patch (v2)

Remove trailing w.s.

Comment 3

[Hsin-Yi Tsai (she/her)[:hsinyi]]

Comment on attachment 8559570 [details] [diff] [review]
patch (v2)

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

Hi Olli,
Another similar work :)

Voicemail API was designed for certified apps only. And I haven't heard of any plan to move it to privilege apps or web content yet. So this patch just adds webidl attributes to reflect the case properly.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8559570 [details] [diff] [review]
patch (v2)

All this stuff is certified only, so should be fine.
(not a mix of privileged and certified)

Comment 5

[Hsin-Yi Tsai (she/her)[:hsinyi]]

https://hg.mozilla.org/integration/b2g-inbound/rev/8cc9897d5642

Comment 6

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/mozilla-central/rev/8cc9897d5642