231534 - ReferenceError: XMLHttpRequest is not defined @ <chrome://venkman/content/venkman-static.js> 651

Status: RESOLVED FIXED

(Other Applications Graveyard :: Venkman JS Debugger, defect)

Description

[timeless]

An exception occurred while initializing, please file a bug.

ReferenceError: XMLHttpRequest is not defined @
<chrome://venkman/content/venkman-static.js> 651

build flags:
ac_add_options
--enable-extensions=universalchardet,typeaheadfind,cookie,venkman,inspector

Comment 1

[timeless]

fwiw
vnk: Application venkman, 'JavaScript Debugger' loaded.
vnk: init {
vnk:   initDebugger {} initDebugger 2.077 sec
vnk:   Application venkman, 'JavaScript Debugger' unloading.

I never get the closed brace to match init {...

Comment 2

[James Ross]

This is because xmlextras wasn't compiled, isn't it? (Do we have enough reasons
yet to make it required?)

Comment 3

[timeless]

that's right. it's an extension. it's optional. even if it isn't optional, it
could still fail to load. please tolerate such a failure.

Comment 4

[James Ross]

Attachment: Catch and cope with XMLHttpRequest errors.

This catches any errors doing the XMLHttpRequest (e.g. it not existing because
of a lack of the xmlextras extension), and will display the normal message with
"<unknown>" (localised, naturally) in place of the global startup count.

Comment 5

[Robert Ginda]

Comment on attachment 145889 [details] [diff] [review]
Catch and cope with XMLHttpRequest errors.

r=rginda

Comment 6

[James Ross]

Checked in.