Turns out that the non-javascript method will involve a full page refresh, which will clear the file input field anyway, so we can just do this in javascript.

When all the uploaded files are successful, and the lifecycle has not had any validation errors, then the fileEntry file selection will be cleared.

Subversion 24124

Discovered an issue on IE 7 and 8, and possibly 6 and 9 but not sure. Not an issue in Firefox or Safari.

A first upload would work, but then a second upload would fail with a javascript exception. The next would succeed, and the next fail, and so on. The exception was about a node not supporting a property, and the code showed that is was where we set the progress div className.

Investigation of previous code changes showed that it was the auto-clearing code that uses innerHTML. Commenting it out made everything work. At first it looked like a race condition between the innerHTML assignment and the progress updates. But, it wasn't the progress updates, it was just the className assignment before we even do an upload, and it was too tick/tock in behaviour to be a race condition.

Investigation online found that IE has an incredibly odd behaviour with innerHTML, where some node types will become the correct dom nodes, and others will just become text nodes. Investigation of the affected nodes found that the input element was becomming an input element, but the div elements were becoming text nodes. The text nodes lacked the className property of their div counterparts.

I solved this by wrapping the input element in its own div, to sequester it from the progress divs, so that I could localise the innerHTML effects, and leave the progress divs unmolested.

Subversion 24235