Receipts aren't exposed as an App property

VERIFIED FIXED in mozilla16

Status

VERIFIED FIXED
7 years ago
a year ago

People

(Reporter: Mardak, Assigned: andy+bugzilla)

Tracking

Trunk
mozilla16
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 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
Component: Web Apps → DOM: Mozilla Extensions
Product: Firefox → Core
QA Contact: webapps → general
(Reporter)

Comment 1

7 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

7 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/mozilla-central/rev/c75aedd63695
Assignee: nobody → amckay
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
(Assignee)

Comment 6

7 years ago
Working for me in nightly, thanks.
Status: RESOLVED → VERIFIED
Component: DOM: Mozilla Extensions → DOM: Apps

Updated

a year ago
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.