B2G MobileConnection: Hook up to permissions manager

RESOLVED FIXED in mozilla17

Status

()

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

People

(Reporter: marshall_law, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
mozilla17
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment)

WiP
4.04 KB, patch
Justin Lebar (not reading bugmail)
: review+
Details | Diff | Splinter Review
(Reporter)

Description

5 years ago
Need to remove the dom.mobileconnection.whitelist pref, and properly hook up to PermissionManager API
blocking-basecamp: --- → ?
Blocks: 764618
blocking-basecamp: ? → +
Marshall can you take this bug?
Created attachment 652306 [details] [diff] [review]
WiP
(Reporter)

Comment 3

5 years ago
(In reply to Gregor Wagner [:gwagner] from comment #2)
> Created attachment 652306 [details] [diff] [review]
> WiP

Arg .. just saw this :(. Sorry, I've been buried in update work this week. If you like, I'd be happy to provide feedback/review..
Comment on attachment 652306 [details] [diff] [review]
WiP

My emulator crashes after 2 sec so I couldn't test it with marionette. 
The phone still works with this patch :)
Attachment #652306 - Flags: review?(justin.lebar+bug)
Comment on attachment 652306 [details] [diff] [review]
WiP

r=me
Attachment #652306 - Flags: review?(justin.lebar+bug) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c1fa02d9465

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/7c1fa02d9465
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.