[File API] feedback on August 1/5 draft

Thanks for the update to the draft! Below some feedback:

In the table of contents the link to the filedata URL scheme is broken.


The Web IDL syntax needs to be updated. E.g. FileList can be simply described as

  typedef sequence<any> FileList;


I have not received any feedback on my comments as to why getAsDataURL is actually needed. I still think it should be dropped.


I think getAsURL() should become an attribute instead. E.g.

  readonly attribute DOMString localURL;

Since it is just a reference there is no need this needs to be asynchronous and there is also no need for it to be method.


The constants of FileError need to be actually placed on the FileError object and renumbered as to make sense. They are not DOM exceptions so it does not make sense to align with that in any way.


The filedata URI scheme should allow the use of fragment identifiers on the resource. Also, I think we should drop the // from the scheme. I do not see why that is needed here.


Last time I also made comments regarding the details of discovering the encoding of a file etc. Those still seem to apply.


-- 
Anne van Kesteren
http://annevankesteren.nl/

Received on Thursday, 6 August 2009 11:05:28 UTC