Closed Bug 897169 Opened 11 years ago Closed 6 years ago

Inter App Communication API - prompting on the Gaia side

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ferjm, Unassigned)

References

Details

Attachments

(3 files, 2 obsolete files)

      No description provided.
Assignee: nobody → ferjmoreno
Depends on: inter-app-comm-api
Attached patch wip (obsolete) — Splinter Review
Attached patch wip (obsolete) — Splinter Review
Attachment #780245 - Attachment is obsolete: true
Attached patch wipSplinter Review
Attachment #782295 - Attachment is obsolete: true
Attached patch Gecko mockSplinter Review
I've used this Gecko patch to test the Gaia side. It emulates a connection request when trying to install an app.
Attached image wip screenshot
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
Also,

s/inter-app-comms-permission/inter-app-comm-permission
s/chromeEventId/chromeEventID
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.
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 :)
Assignee: ferjmoreno → nobody
Summary: Inter App Communication API - Gaia side → Inter App Communication API - prompting on the Gaia side
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: