303625 - nsXFormsSubmissionElement::LoadReplaceAll uses wrong document

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

The code gets the ownerDocument and a container from that (in order to load a
new document there).

It seems to me like the code really wants to use the current document
(GetCurrentDoc()) of the node.

Comment 1

[Allan Beaufour]

(In reply to comment #0)
> The code gets the ownerDocument and a container from that (in order to load a
> new document there).
> 
> It seems to me like the code really wants to use the current document
> (GetCurrentDoc()) of the node.

Nobody ever did anything about this. Why should we use GetCurrentDoc() instead? Mind you that it is an XTF element.

Comment 2

[Boris Zbarsky [:bzbarsky]]

If the element comes from an XBL binding (could it?) then the owner doc and current doc would be different in XBL2 -- the owner doc would be the XBL binding document while the current doc would be the bound document.

More to the point, anything involving presentation (including containers) should be using the current document, since it's depending on messing with the "view" the element is in, not where the element hails from.

Comment 3

[Allan Beaufour]

(In reply to comment #2)
> If the element comes from an XBL binding (could it?)

No.

> then the owner doc and current doc would be different in XBL2 -- the owner doc
> would be the XBL binding document while the current doc would be the bound 
> document.
> 
> More to the point, anything involving presentation (including containers)
> should be using the current document, since it's depending on messing with the
> "view" the element is in, not where the element hails from.

But the submission element has no visual representation.

Comment 4

[Boris Zbarsky [:bzbarsky]]

> No.

So if XBL anonymous content includes a submission element nothing will happen?  What if XBL anonymous content includes a complete XForms form?

> But the submission element has no visual representation.

That's a matter of its CSS display type, no?

Comment 5

[Allan Beaufour]

(In reply to comment #4)
> > No.
> 
> So if XBL anonymous content includes a submission element nothing will happen? 
> What if XBL anonymous content includes a complete XForms form?

Ah, sorry, understood that the wrong way around. submission's not going to be very happy unless it's a child of a model, but that said, you could stash it inside XBL anonymous content, yes.

> > But the submission element has no visual representation.
> 
> That's a matter of its CSS display type, no?

Hmmm, it inherits from nsIXTFGenericElement which iirc should have no visual representation.

Comment 6

[Boris Zbarsky [:bzbarsky]]

> which iirc should have no visual representation.

It's just treated as "display: none !important" in the UA style sheet, basically.  Except hardcoded in C++ in a hacky way.

But back to the XBL anonymous content issue.  Would you be wanting to do the load in the XBL binding document, or the document the binding is attached to?  That's the line between GetOwnerDoc and GetCurrentDoc.

Comment 7

[Allan Beaufour]

(In reply to comment #6)
> > which iirc should have no visual representation.
> 
> It's just treated as "display: none !important" in the UA style sheet,
> basically.  Except hardcoded in C++ in a hacky way.
> 
> But back to the XBL anonymous content issue.  Would you be wanting to do the
> load in the XBL binding document, or the document the binding is attached to? 
> That's the line between GetOwnerDoc and GetCurrentDoc.

(oops, this disappeared in my bugmail) We want to load in the document that the binding is attached to.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Then you want GetCurrentDoc() and this bug is valid.

Comment 9

[Allan Beaufour]

Attachment: Patch

Comment 10

[Allan Beaufour]

Fixed on trunk.