237726 - XUL elements aren't properly registered as DOM3 nodes

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[[e-mail]]

Error: uncaught exception: [Exception... "Illegal operation on WrappedNative
prototype object"  nsresult: "0x8057000c (NS_ERROR_XPC_BAD_OP_ON_WN_PROTO)" 
location: "JS frame :: javascript: top.document.documentElement.textContent ::
<TOP_LEVEL> :: line 1"  data: no]

I did a scan of a fair number of elements and attributes and the only attributes
I had problems with were baseURI and textContent of xul elements so I assume the
issue is with the nsIDOM3Node registration.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Comment 2

[Boris Zbarsky [:bzbarsky]]

You don't need nsIDOMElementCSSInlineStyle (.style property)?

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 144084 [details] [diff] [review]
Proposed patch

Good catch! r+sr=jst

Comment 4

[[e-mail]]

(In reply to comment #2)
>You don't need nsIDOMElementCSSInlineStyle (.style property)?
No, it's define on nsIDOMXULElement.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Oh, ugh.  Yay XUL dom being different just because.  :(

Comment 6

[[e-mail]]

Comment on attachment 144084 [details] [diff] [review]
Proposed patch

Low risk fix for erroneous exception accessing certain xul element attributes.

Comment 7

[chris hofmann]

Comment on attachment 144084 [details] [diff] [review]
Proposed patch

a=chofmann for 1.7.  wait until later today to make sure tag is cut and beta is
finished before checking in.

Comment 8

[[e-mail]]

Fix checked in.