48264 - Mispelled error message: 'qulaified' on any scrollbar click

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Decklin Foster]

Using linux CVS from 080913, I get the following error message several (3) times
whenever I click on a (gfx) scrollbar:

Possible DOM Error: .name, .nodeName or .tagName requested on a namespace
element/attribute with the qulaified name 'xul:spring', is this OK?

'qulaified' should probably read 'qualified'. Ideally, this error should go away.

Comment 1

[John Morrison]

I have a two-line patch to fix the output of these messages (both for 
when using the scrollbars (actually when starting a drag), and for middle 
mouse on the scrollbar). I'll attach it in a moment. They both occur when
the event starts in XBL content that is 'xul:' namespaced (i.e., the xul:thumb,
xul:slider, and xul:spring of scrollbar)

(I think the spelling error can stay; that warning is going away completely  
someday I believe.)

Comment 2

[John Morrison]

Attachment: patch; use .localName, not .nodeName, as nodes may be from XBL/XUL

Comment 3

[John Morrison]

jst -- the patch clears up the warnings (and does the right thing). I ran this
on mac/linux/win32 (the last two with current builds) and checked scrolling and 
D&D, plus pre-checkin tests. Since I have no CVS account, would you check this
in (if review of the two-line change looks good :-]). 

Comment 4

[Daniel Bratell]

Adding keyword "review".

Comment 5

[Doron Rosenberg (IBM)]

it's been a bit, what's the status for this? [e-mail], can you check
this in? Or has this been checked already?

Comment 6

[John Morrison]

jst checked this in last night. Marking FIXED. Thanks jst.

Comment 7

[Johnny Stenback (:jst)]

Yup, this was fixed, the error message is all gone now and I also checked in the
attached patch some time back. Thanks for the patch and sorry for forgetting to
update this bug.

Comment 8

[Decklin Foster]

Verifying fix