897169 - Inter App Communication API - prompting on the Gaia side

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System, defect)

Comment 1

[Fernando Jiménez Moreno [:ferjm]]

Attachment: wip

Comment 2

[Fernando Jiménez Moreno [:ferjm]]

Attachment: wip

Comment 3

[Fernando Jiménez Moreno [:ferjm]]

Attachment: wip

Comment 4

[Fernando Jiménez Moreno [:ferjm]]

Attachment: Gecko mock

I've used this Gecko patch to test the Gaia side. It emulates a connection request when trying to install an app.

Comment 5

[Fernando Jiménez Moreno [:ferjm]]

Attachment: wip screenshot

Comment 6

[Gene Lian (I already quit Mozilla)]

Comment on attachment 782509 [details] [diff] [review]
Gecko mock

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

Great! I almost have the same design as yours. I'll align the Gecko codes with your proposal here. Btw, let's do some nits as below:

::: b2g/chrome/content/shell.js
@@ +916,5 @@
> +         });
> +
> +         shell.sendChromeEvent({
> +           "type": "inter-app-comms-permission",
> +           "chromeEventId": "chromeEventId",

s/chromeEventId/chromeEventID

@@ +926,5 @@
> +             "permissionDescription": "Bla bla bla"
> +           }, {
> +             "manifestURL": "manifestURL",
> +             "name": "Peer2",
> +             "permissionDescription": "Bla bla bla2"

s/permissionDescription/description

Comment 7

[Gene Lian (I already quit Mozilla)]

Also,

s/inter-app-comms-permission/inter-app-comm-permission
s/chromeEventId/chromeEventID

Comment 8

[Gene Lian (I already quit Mozilla)]

This Gaia bug also needs to consider the ability of cancelling the previously allowed connections in the Settings app. Or fire a separate bug for that.

Comment 9

[Fernando Jiménez Moreno [:ferjm]]

Thanks for the feedback Gene!

(In reply to Gene Lian [:gene] from comment #8)
> This Gaia bug also needs to consider the ability of cancelling the
> previously allowed connections in the Settings app. Or fire a separate bug
> for that.

I'll be glad to add the Settings side in this bug too. But first we need some UX design for the whole feature. Let me write some emails :)

Comment 10

[BMO Automation]

Firefox OS is not being worked on