Receipts aren't exposed as an App property

VERIFIED FIXED in mozilla16

Status

()

Core
DOM: Apps
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: Mardak, Assigned: andym)

Tracking

Trunk
mozilla16
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Not sure why dom/tests/mochitest/webapps/test_install_receipts.xul isn't failing because it shouldn't be able to access pending.result.receipts

Updated

5 years ago
Component: Web Apps → DOM: Mozilla Extensions
Product: Firefox → Core
QA Contact: webapps → general
(Reporter)

Comment 1

5 years ago
Oh I see why it isn't failing..

p = navigator.mozApps.install("http://ed.agadak.net/manifest.webapp", {receipts: ["Woot!"]}); p.onsuccess = function() console.log("success", this.result.receipts); p.onerror = function() console.log("error");

Doing that from content pages runs onsuccess but receipts is undefined. Doing that from about: (chrome privileged page) works okay.
yes, we forgot to add |receipts| to __exposedProps__
(Assignee)

Comment 3

5 years ago
Created attachment 631460 [details] [diff] [review]
Add in receipts to the __exposedProps__
Attachment #631460 - Flags: review?
Attachment #631460 - Attachment is patch: true
Attachment #631460 - Flags: review? → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c75aedd63695
https://hg.mozilla.org/mozilla-central/rev/c75aedd63695
Assignee: nobody → amckay
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
(Assignee)

Comment 6

5 years ago
Working for me in nightly, thanks.
Status: RESOLVED → VERIFIED

Updated

5 years ago
Component: DOM: Mozilla Extensions → DOM: Apps
You need to log in before you can comment on or make changes to this bug.