use CreateElementWithNameSpace() for anonymous content nodes

VERIFIED FIXED in M9

Status

()

P3
normal
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: waterson, Assigned: eric)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: Back to engineer for verification...)

(Reporter)

Description

19 years ago
Using just "CreateElement()" will break as soon as someone uses something
besides XUL for the default namespace. Use "CreateElementWithNameSpace()"
instead.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M9
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → INVALID
(Assignee)

Comment 1

19 years ago
I manually set the namespace via SetNameSpace.

Updated

19 years ago
Status: RESOLVED → REOPENED

Comment 2

19 years ago
That won't work right for HTML content.  You need to get the namespace right up
front.  I use this in nsMenuFrame.cpp when I make anonymous content.  It's
trivial for you to look at that and use it as a guide for switching your stuff
over.
(Reporter)

Comment 3

19 years ago
Eric and I talked about this. In his case, I think it's okay for him to
manually create XML nodes (_not_ XUL nodes), and then just twiddle them
himself. Eric, this is what you're doing, isn't it?

Updated

19 years ago
Resolution: INVALID → ---

Comment 4

19 years ago
Clearing Invalid resolution due to reopen.
(Assignee)

Comment 5

19 years ago
I don't make HTML or XUL content nodes. I use XML nodes... And it seems to work
fine. Am I missing something?

Updated

19 years ago
Status: REOPENED → RESOLVED
Last Resolved: 19 years ago19 years ago
Resolution: --- → FIXED

Comment 6

19 years ago
Ok, that's fine then.

Updated

19 years ago
Whiteboard: Back to engineer for verification...

Comment 7

19 years ago
Chris, could you verify this?  Thanks!
-Chris
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 8

19 years ago
BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.
Component: XUL → XP Toolkit/Widgets: XUL

Updated

10 years ago
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: ckritzer → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.