[ConvertScriptString] is used by FileReaderSync.idl only. The difference between [ConvertScriptString] and [TreatReturnedNullStringAs=Null] is whether the returned string is generated by jsString() or jsOwnedString() in JSC (No difference in V8). I am not sure why (only) FileReaderSync.idl requires jsOwnedString(). If it is not necessary, I would like to replace [ConvertScriptString] with [TreatReturnedNullStringAs=Null], and remove [ConvertScriptString] completely from code generators. Any advice?
The spec says that FileReadSync should throw NOT_FOUND_ERR if a given blob is invalid: http://www.w3.org/TR/FileAPI/#FileReaderSync By throwing NOT_FOUND_ERR if a given blob is invalid, we can remove [ConvertScriptString] from FileReadSync.idl.
Created attachment 126699 [details] Patch
Comment on attachment 126699 [details] Patch LGTM.
Comment on attachment 126699 [details] Patch Clearing flags on attachment: 126699 Committed r107994: <http://trac.webkit.org/changeset/107994>
All reviewed patches have been landed. Closing bug.