Created attachment 269786 [details] [diff] [review]
Looks like there is a bug on the 1.8 branch that prevents the patch that fixed multipart/form-data on the trunk from working on the branch. When we go to set the nsXFormsAtoms::uploadFileProperty on the clone of the instance data node that originally contained it, it is failing to work correctly. By that point we've already done the ImportNode, but not the AppendChild. ImportNode should set up the owner document for the cloned node to be the submission document, but that doesn't seem to be the case since the owner document before appendchild and after are different.
So to work around this 1.8 bug we'll just set the property AFTER we've done the call to AppendChild.
checked into trunk
Reopening bug. I didn't move the statement high enough to fix uploads bound to attributes.
Created attachment 270088 [details] [diff] [review]
just asking for one review. It is a pretty minor change, more of a correction of the first change. But I don't want to be accused of breaking the rules :-)
checked patch2 into trunk and patches 1 and 2 into 1.8 branch.