Use existing unicode.sky instead of re-inventing

I somehow missed this function the first time.

This is already tested by
tests/framework/xmlhttprequest/unicode-post.sky

R=esprehn@chromium.org
BUG=

Review URL: https://codereview.chromium.org/806043002
This commit is contained in:
Eric Seidel 2014-12-17 12:11:42 -08:00
parent 58d3a54d7b
commit 0b8c9cf3ac

View File

@ -21,17 +21,6 @@ class Private {
}
}
// Somewhat hacky, but works.
function stringToUTF8Buffer(string) {
var string = unescape(encodeURIComponent(string));
var charList = string.split('');
var uintArray = [];
for (var i = 0; i < charList.length; i++) {
uintArray.push(charList[i].charCodeAt(0));
}
return new Uint8Array(uintArray);
}
// https://xhr.spec.whatwg.org
class XMLHttpRequest {
constructor() {
@ -89,7 +78,8 @@ class XMLHttpRequest {
var bodyAsBufferView = null;
if (typeof(body) === "string") {
this.setRequestHeader("Content-Type", "text/plain;charset=UTF-8");
bodyAsBufferView = stringToUTF8Buffer(body);
var bodyAsBufferView = new Uint8Array(unicode.utf8Length(body));
unicode.encodeUtf8String(body, bodyAsBufferView);
} else {
bodyAsBufferView = new Uint8Array(body);
}