Bug 99546

Summary: files attribute should be read only in HTMLInputElement
Product: WebKit Reporter: Li Yin <li.yin>
Component: DOMAssignee: Li Yin <li.yin>
Status: RESOLVED WONTFIX    
Severity: Normal CC: abarth, haraken, kenneth, mifenton, ojan, tkent, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch tkent: review-, buildbot: commit-queue-

Li Yin
Reported 2012-10-16 23:02:53 PDT
Spec:http://dev.w3.org/html5/spec/single-page.html#the-input-element files should be read only attribute. And Firefox 16 and IE 10 keep the same behavior with spec.
Attachments
Patch (4.28 KB, patch)
2012-10-17 01:32 PDT, Li Yin 		tkent: review-
buildbot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Li Yin
Comment 1 2012-10-17 01:32:50 PDT
Created attachment 169120 [details] Patch
Build Bot
Comment 2 2012-10-17 01:36:52 PDT
Comment on attachment 169120 [details] Patch Attachment 169120 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14386492
Kentaro Hara
Comment 3 2012-10-17 01:37:54 PDT
Comment on attachment 169120 [details] Patch Looks reasonable. Please fix the build failures.
Kent Tamura
Comment 4 2012-10-17 01:38:34 PDT
Comment on attachment 169120 [details] Patch This is intentional. See https://bugs.webkit.org/show_bug.cgi?id=87154#c15 and so on.
Adam Barth
Comment 5 2012-10-17 08:46:54 PDT
Why didn't fast/forms/file/input-file-write-files.html fail?
Alexey Proskuryakov
Comment 6 2012-10-17 10:51:00 PDT
+1. Seems particularly important to have a working test case for this issue.
Li Yin
Comment 7 2012-10-21 20:17:08 PDT
fast/forms/file/input-file-write-files.html should do more sub tests to verify the correct setting. Currently, it tracked the special case such as: 1. when files is illegal. 2. when value is set. It lacked verification of normal FileList object.
Li Yin
Comment 8 2012-10-23 01:10:41 PDT
I filed a new bug to refine the test. See https://bugs.webkit.org/show_bug.cgi?id=100085
Note You need to log in before you can comment on or make changes to this bug.