66907 2011-08-24 17:22:42 -0700 File input element does not fire onchange event when the selection changes a second time 2011-09-07 22:33:46 -0700 1 1 1 Unclassified WebKit Forms 528+ (Nightly build) PC All RESOLVED DUPLICATE 63929 P2 Normal --- 0 Mike.Fellows webkit-unassigned ap shinyak tkent oldest_to_newest 456547 0 105101 Mike.Fellows 2011-08-24 17:22:42 -0700 Created attachment 105101 A reduced test case for displaying an input of type file's onchange event behaviour If a file or set of files is selected via an input element of type file the onchange event will be fired for the first selection. If a new set of files is selected, and the first file in the new file list is the same as the first file in the old file list the onchange event will not be fired. Any additional information displayed by the input element about how many files are selected will change regardless of whether or not the onchange event is fired. If the second selection does not have the same first file as the first selection the onchange event will fire as expected. A step by step example for reproducing the problem: 1. Page is loaded, the Choose File button is clicked and the files selected are "1.jpg, 2.jpg". The input element lists "2 files" next to the Choose File button and the onchange event will be fired as expected. 2. The Choose File button is clicked a second time and the files selected are "1.jpg, 2.jpg, 3.jpg, 4.jpg". The input element now changes to list "4 files" next to the Choose File button but the onchange event will not be fired. I've attached a reduced test case to the issue the will display this behaviour. I've tested this in the following browsers: Google Chrome Linux version 13.0.782.215, WebKit: 535.1 (branches/chromium/782@93192) Google Chrome Windows version 13.0.782.215 WebKit: 535.1 (branches/chromium/782@93192) Safari for Windows version 5.1 (7534.50) (WebKit: 534.50 according to the User Agent String) I've seen the same behaviour in a Chrome v 12 browser as well so I think the issue has been around for awhile. According to the current draft html5 spec the change event should fire when the file list is changed by user input: http://www.w3.org/TR/html5/common-input-element-attributes.html#event-input-change 463890 1 shinyak 2011-09-07 22:31:43 -0700 I couldn't repro this. I think this was fixed in r90438. See also https://bugs.webkit.org/show_bug.cgi?id=63929 463891 2 tkent 2011-09-07 22:33:46 -0700 *** This bug has been marked as a duplicate of bug 63929 *** 105101 2011-08-24 17:22:42 -0700 2011-08-24 17:22:42 -0700 A reduced test case for displaying an input of type file's onchange event behaviour index-onchange.html text/html 2263 Mike.Fellows PCFET0NUWVBFIGh0bWwKICAgIFBVQkxJQyAiLS8vVzNDLy9EVEQgWEhUTUwgMS4wIFN0cmljdC8v RU4iCiAgICAiaHR0cDovL3d3dy53My5vcmcvVFIveGh0bWwxL0RURC94aHRtbDEtc3RyaWN0LmR0 ZCI+CjxodG1sIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hodG1sIiB4bWw6bGFuZz0i ZW4iIGxhbmc9ImVuIj4KICA8aGVhZD4KICAgIDx0aXRsZT5GaWxlIElucHV0IG9uY2hhbmdlIFRl c3QgQXBwPC90aXRsZT4KICAgIDxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KICAgICAg d2luZG93Lm9ubG9hZCA9IGZ1bmN0aW9uKCkKICAgICAgewogICAgICAgIHZhciBmaWxlc1NlbGVj dGVkRWxlbWVudCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJmaWxlc1NlbGVjdGVkIik7Cgog ICAgICAgIHZhciBpbnB1dEVsZW1lbnQgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiZmlsZUlu cHV0Iik7CiAgICAgICAgaW5wdXRFbGVtZW50Lm9uY2hhbmdlID0gZnVuY3Rpb24oKQogICAgICAg IHsKICAgICAgICAgIHZhciBvbmNoYW5nZVJlcG9ydCA9ICJGaWxlIElucHV0J3Mgb25jaGFuZ2Ug ZXZlbnQgY2FsbGJhY2sgY2FsbGVkIGF0OiAiICsgRGF0ZSgpOwogICAgICAgICAgdmFyIG9uY2hh bmdlUmVwb3J0RWxlID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgicCIpOwogICAgICAgICAgb25j aGFuZ2VSZXBvcnRFbGUuc2V0QXR0cmlidXRlKCJjbGFzcyIsICJoZWFkZXIiKTsKICAgICAgICAg IG9uY2hhbmdlUmVwb3J0RWxlLmFwcGVuZENoaWxkKGRvY3VtZW50LmNyZWF0ZVRleHROb2RlKG9u Y2hhbmdlUmVwb3J0KSk7CgogICAgICAgICAgZmlsZXNTZWxlY3RlZEVsZW1lbnQuYXBwZW5kQ2hp bGQob25jaGFuZ2VSZXBvcnRFbGUpOwoKICAgICAgICAgIGZvciAoaSA9IDA7IGkgPCB0aGlzLmZp bGVzLmxlbmd0aDsgaSsrKQogICAgICAgICAgewogICAgICAgICAgICBsaXN0U2VsZWN0ZWRGaWxl cyhpLCBmaWxlc1NlbGVjdGVkRWxlbWVudCwgdGhpcy5maWxlc1tpXSk7CiAgICAgICAgICB9Cgog ICAgICAgICAgZmlsZXNTZWxlY3RlZEVsZW1lbnQuYXBwZW5kQ2hpbGQoZG9jdW1lbnQuY3JlYXRl RWxlbWVudCgiYnIiKSk7CiAgICAgICAgfQoKICAgICAgICAvLyBMaXN0IHRoZSBnaXZlbiBmaWxl IG9uIHRoZSBtYWluIHBhZ2UuCiAgICAgICAgZnVuY3Rpb24gbGlzdFNlbGVjdGVkRmlsZXMoaW5k ZXgsIGxpc3RFbGUsIGZpbGUpCiAgICAgICAgewogICAgICAgICAgdmFyIGZpbGVTdGF0cyA9IChp bmRleCArIDEpICsgIi4gbmFtZTogIiArIGZpbGUubmFtZSArCiAgICAgICAgICAgICAgICAgICAg ICAgICAgIiwgc2l6ZTogIiArIGZpbGUuc2l6ZSArICIsIHR5cGU6ICIgKyBmaWxlLnR5cGUgKwog ICAgICAgICAgICAgICAgICAgICAgICAgICIsIGxhc3QgbW9kaWZpZWQ6ICIgKyBmaWxlLmxhc3RN b2RpZmllZERhdGU7CgogICAgICAgICAgdmFyIG15TGlzdEVsZSA9IGRvY3VtZW50LmNyZWF0ZUVs ZW1lbnQoInAiKTsKICAgICAgICAgIG15TGlzdEVsZS5hcHBlbmRDaGlsZChkb2N1bWVudC5jcmVh dGVUZXh0Tm9kZShmaWxlU3RhdHMpKTsKCiAgICAgICAgICBsaXN0RWxlLmFwcGVuZENoaWxkKG15 TGlzdEVsZSk7CiAgICAgICAgfQogICAgICB9CiAgICA8L3NjcmlwdD4KICAgIDxzdHlsZSB0eXBl PSJ0ZXh0L2NzcyI+CiAgICAgIGJvZHkgeyBmb250LWZhbWlseTogYXJpYWwsIHNhbnMtc2VyaWY7 IGZvbnQtc2l6ZTogMTNweDt9CiAgICAgIHAgeyBwYWRkaW5nOiAwcHg7IG1hcmdpbjogMHB4OyB9 CiAgICAgICNmaWxlU2VsZWN0ZWR7IG1hcmdpbjogNXB4OyB9CiAgICAgICNmaWxlSW5wdXQgeyBt YXJnaW46IDE1cHggMHB4IDE1cHggMHB4OyB9CiAgICAgIHAuaGVhZGVyIHsgbWFyZ2luLWJvdHRv bTogM3B4OyBjb2xvcjogZ3JlZW47IGZvbnQtd2VpZ2h0OiBib2xkOyB9CiAgICA8L3N0eWxlPgog IDwvaGVhZD4KCiAgPGJvZHk+CiAgICA8aDM+RmlsZSBJbnB1dCBvbmNoYW5nZSBUZXN0IEFwcDwv aDM+CiAgICA8cD4KICAgICAgU2VsZWN0IGZpbGVzIGFuZCB0aGV5IHdpbGwgYmUgbGlzdGVkIGJl bG93LiAgQ2hhbmdlIHRoZSBzZWxlY3Rpb24gYW5kIHRoZSBuZXcKICAgICAgc2VsZWN0aW9uIHNl dCBzaG91bGQgYWxzbyBiZSBsaXN0ZWQgYmVsb3cuCiAgICA8L3A+CiAgICA8aW5wdXQgaWQ9ImZp bGVJbnB1dCIgdHlwZT0iZmlsZSIgbXVsdGlwbGU9Im11bHRpcGxlIiAvPgogICAgPGRpdiBpZD0i ZmlsZXNTZWxlY3RlZCI+PC9kaXY+CiAgPC9ib2R5Pgo8L2h0bWw+Cg==