353159 - upload control fails to update filename/mediatype if bound to attribute

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[Steve Speicher]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060816 Minefield/3.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060816 Minefield/3.0a1

When an upload control is bound to an attribute, the elements filename and mediatype don't work (i.e. the associated instance node value are not updated).

This is fails since the qi on nsIContent fails in nsXFormsUploadElement::SetFile()
  nsCOMPtr<nsIContent> content = do_QueryInterface(mBoundNode);
  

Reproducible: Always

Steps to Reproduce:
\

Comment 1

[Steve Speicher]

Attachment: test case

Comment 2

[Steve Speicher]

Mark as blocking 0.7 release.  This is causing a proof of concept of mine to not work well as the instances' use attributes and is controlled by a standards organization that can easily change it.

Comment 3

[Steve Speicher]

me and my fat fingers

Comment 4

[Steve Speicher]

Attachment: patch (if they could all be this easy)

Comment 5

[aaronr]

Comment on attachment 239190 [details] [diff] [review]
patch (if they could all be this easy)

nit: change content to node or something even more descriptive, please

r=me

Comment 6

[Steve Speicher]

Attachment: patch for checkin

aaronr, checkin please?

Comment 7

[aaronr]

checked into trunk for sspeiche

Comment 8

[aaronr]

Steve, please open a new bug to re-do this patch so that it can work on the branches (might as well keep the code common between trunk and branches if at all possible).  nsINode doesn't exist in the branches (came into being in early 2006).  Thanks.

Comment 9

[aaronr]

checked into 1.8.0 on 2006/09/27

Comment 10

[aaronr]

checked into 1.8 branch on 2006/11/21