Closed Bug 1075189 Opened 10 years ago Closed 10 years ago

"ReferenceError: Task is not defined" when installing an app from another app

Categories

(Firefox Graveyard :: Webapp Runtime, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 35

People

(Reporter: myk, Assigned: marco)

References

Details

(Keywords: regression)

Attachments

(1 file)

Trying to install an app from another app fails with this error in the JS Console: > Timestamp: 2014/09/30, 14:47:34 > Error: ReferenceError: Task is not defined > Source File: resource://app/modules/WebappManager.jsm > Line: 90
Attached patch PatchSplinter Review
Attachment #8497812 - Flags: review?(myk)
Attachment #8497812 - Flags: review?(myk) → review+
No tests for this?
(In reply to :Gavin Sharp [email: gavin@gavinsharp.com] from comment #2) > No tests for this? Hmm, I thought we did have a test (in the suite that we're working to enable in automation over in bug 899707), but I just checked, and it looks like we don't. Marco: can you add a test for this? In the meantime, this run should at least ensure we haven't broken anything in the browser: https://tbpl.mozilla.org/?tree=Try&rev=1843090935ad
Flags: needinfo?(mar.castelluccio)
There's a bug opened to add such a test (bug 1006394), I had some problems with it that may have gone away in the meantime (the transition to WebIDL may have fixed them). Can we land this and track the addition of a test in bug 1006394?
Flags: needinfo?(mar.castelluccio)
(In reply to Marco Castelluccio [:marco] from comment #4) > Can we land this and track the addition of a test in bug 1006394? Yes, that's reasonable, given the circumstances.
Assignee: nobody → mar.castelluccio
Keywords: regression
Keywords: checkin-needed
Just fyi, I had to add the bug number to the commit message to meet the commit hook requirements.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 35
Blocks: 1078777
Flags: qe-verify-
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: