Avoid JavaScript strict warning in AppsUtils.jsm during startup

RESOLVED FIXED in Firefox 48

Status

Core Graveyard
DOM: Apps
RESOLVED FIXED
2 years ago
8 months ago

People

(Reporter: jwatt, Assigned: marco)

Tracking

unspecified
mozilla48

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

During startup I get:

JavaScript strict warning: resource://gre/modules/AppsUtils.jsm, line 839: ReferenceError: reference to undefined property this._manifest[aProp]

(aProp == "widgetPages")

It would be really nice to get this clutter out of the startup output to make debugging real startup issues easier.
Fabrice, this is code you wrote. Any idea how to properly fix this?

https://mxr.mozilla.org/mozilla-central/source/dom/apps/AppsUtils.jsm?rev=a85b572e0d8c#842
Flags: needinfo?(fabrice)
(Assignee)

Comment 2

2 years ago
Created attachment 8739431 [details] [diff] [review]
avoid_localeProp_warning
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #8739431 - Flags: review?(fabrice)
Attachment #8739431 - Flags: review?(fabrice) → review+
Flags: needinfo?(fabrice)
(Assignee)

Comment 4

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ac43dab284afd189ed2b0f4c1906a45e0c1736dd
Bug 1263158 - Check if key is present in manifest object before using it. r=fabrice

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ac43dab284af
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48

Updated

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