Mark Banner (:standard8) Reporter
	Description • 7 years ago

Picked up via lint whilst starting to investigate bug 1369722.

http://searchfox.org/mozilla-central/rev/ed212c79cfe86357e9a5740082b9364e7f6e526f/toolkit/modules/GMPInstallManager.jsm#432

status is undefined. It looks like it has been that way since the code was introduced in bug 1009816.

AFAICT there's no functional effect here, as the results of the rejected promise aren't used from what I see.

I'm not sure what the original intent was though.

	Stephen A Pohl [:spohl] (OOO until 9/8) Assignee
	Comment 1 • 7 years ago

From the looks of it, |status| was intended for HTTP status codes, of which we don't have any in this case. This must have been copy/pasted from one of the other rejected Promises and never removed.

	Stephen A Pohl [:spohl] (OOO until 9/8) Assignee
	Comment 2 • 7 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/5fa423b95ce4c40101046f0b3d382b61a1293f3e
Bug 1415340: Remove undefined (and unnecessary) property when rejecting a Promise in GMPDownloader. r=gfritzsche

	Bogdan Tara[:bogdan_tara | bogdant]
	Comment 3 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/5fa423b95ce4