Closed Bug 425062 Opened 12 years ago Closed 12 years ago

FUEL: Error: Application is not defined

Categories

(Firefox :: General, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 3 beta5

People

(Reporter: zeniko, Assigned: Gavin)

References

Details

(Keywords: regression)

Attachments

(1 file)

Regression from bug 407963 in today's nightly (last known good: 2008032405). On the first start on a new profile I get...

Error: i is undefined
Source File: file:///C:/Programme/Mozilla%20Firefox/modules/XPCOMUtils.jsm
Line: 115
 ----------
Failed to load XPCOM component: C:\Programme\Mozilla Firefox\components\fuelApplication.js

... and trying to use FUEL results in...

Error: Application is not defined

Could this be a packing issue (as e.g. the new xpts missing)?
Flags: blocking-firefox3?
(In reply to comment #0)
> Could this be a packing issue (as e.g. the new xpts missing)?

Eek, probably.
Priority: -- → P1
Target Milestone: --- → Firefox 3 beta5
Flags: blocking-firefox3? → blocking-firefox3+
Assignee: nobody → gavin.sharp
(In reply to comment #1)
> Eek, probably.

Don't we have any test machine which installs builds instead of compiling them? This wouldn't be the first time new components aren't properly packed even though all tests are passing...
Attachment #311651 - Flags: approval1.9b5?
Comment on attachment 311651 [details] [diff] [review]
package extApplication.js and exthelper.xpt

a1.9b5=beltzner
Flags: blocking-firefox3+ → blocking-firefox3?
Priority: P1 → --
Target Milestone: Firefox 3 beta5 → ---
(In reply to comment #2)
> (In reply to comment #1)
> > Eek, probably.
> 
> Don't we have any test machine which installs builds instead of compiling them?

Not yet, see bug 383136.
Status: NEW → ASSIGNED
Component: General → Build Config
OS: Windows XP → All
QA Contact: general → build.config
Component: Build Config → General
OS: All → Windows XP
Priority: -- → P1
QA Contact: build.config → general
Target Milestone: --- → Firefox 3 beta5
OS: Windows XP → All
Hardware: PC → All
Comment on attachment 311651 [details] [diff] [review]
package extApplication.js and exthelper.xpt

 ; JavaScript components
 bin/components/FeedProcessor.js
 bin/components/FeedConverter.js
 bin/components/FeedWriter.js
+bin/components/extApplication.js

We preprocess #include extApplication.js into fuelApplication.js, so I don't think it needs to be packaged.
mozilla/browser/installer/unix/packages-static 	1.159
mozilla/browser/installer/windows/packages-static 	1.161 
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
(In reply to comment #6)
> We preprocess #include extApplication.js into fuelApplication.js, so I don't
> think it needs to be packaged.

Oops, you're right. I undid that change.

Attachment #311651 - Flags: approval1.9b5? → approval1.9b5+
Flags: blocking-firefox3? → blocking-firefox3+
You need to log in before you can comment on or make changes to this bug.