Created attachment 8758357 [details] [diff] [review] blob.patch
Comment on attachment 8758357 [details] [diff] [review] blob.patch So this is a change to the behavior (DOMString -> USVString) Please test how other browsers work, if unusual string data is passed. With that, r+. https://github.com/w3c/FileAPI/issues/30 is a recent spec change. Ms2ger, did anyone check what browsers actually do before the spec change was done.
BlobSet::AppendString uses NS_ConvertUTF16toUTF8, which already replaces unpaired surrogates by U+FFFD, so there's no behaviour change in using USVString here.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/045cdefdb941 Use BlobPart in the Blob/File webidl interface, r=smaug
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e08645cf7793 part 2 - WPT fixed, r=me
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.