403852 - Unable to access nsIDOMFile using [] operator

Status: RESOLVED FIXED

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

Description

[Biju]

see  bug 371432

Unable to access nsIDOMFile by
   document.getElementById('f').fileList[0]
as mentioned in bug 371432 Comment 62
or http://developer.mozilla.org/en/docs/nsIDOMFileList

and accessing using [] operator is common practice like
* arguments[]
* document.links[]
* selection.options[]

PS: I can get document.getElementById('f').fileList.item(0)

Comment 1

[Alex Vincent [:WeirdAl]]

Probably just needs some DOMClassInfo love:
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/dom/src/base/nsDOMClassInfo.cpp&rev=1.491#7157

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Should probably fix this before release. There is some classinfo love there, not sure what's missing.

It should be doable to write a mochitest for this actually, if you request UniversalXPConnect access you can set .value as you please. The tricky part would be finding what file-name to set :(

Comment 3

[Damon Sicore (:damons)]

+'ing this and setting to P4 as there is a work around.

Comment 4

[Dave Camp (:dcamp)]

Attachment: use ARRAY_SCRIPTABLE_FLAGS

Comment 5

[Dave Camp (:dcamp)]

Attachment: and a test

mochitest for the array syntax.

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 293217 [details] [diff] [review]
use ARRAY_SCRIPTABLE_FLAGS

Sweet

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 293227 [details] [diff] [review]
and a test

Technically, you don't need to do this from onload. You could just write the code inline and not bother with the SimpleTest.waitForExplicitFinish/finish fluff.

Comment 8

[Dave Camp (:dcamp)]

Checking in content/base/test/Makefile.in;
/cvsroot/mozilla/content/base/test/Makefile.in,v  <--  Makefile.in
new revision: 1.37; previous revision: 1.36
done
RCS file: /cvsroot/mozilla/content/base/test/test_bug403852.html,v
done
Checking in content/base/test/test_bug403852.html;
/cvsroot/mozilla/content/base/test/test_bug403852.html,v  <--  test_bug403852.html
initial revision: 1.1
done
Checking in dom/src/base/nsDOMClassInfo.cpp;
/cvsroot/mozilla/dom/src/base/nsDOMClassInfo.cpp,v  <--  nsDOMClassInfo.cpp
new revision: 1.500; previous revision: 1.499
done

Comment 9

[Dave Camp (:dcamp)]

(In reply to comment #7)
> (From update of attachment 293227 [details] [diff] [review])
> Technically, you don't need to do this from onload. You could just write the
> code inline and not bother with the SimpleTest.waitForExplicitFinish/finish
> fluff.

I changed that.