Revert "use byte array instead of string for code fetch (#1307)" (#1455)

This reverts commit e976b3e0414dc768624b77e431ee7f55b03b76a4.

There is nothing technically wrong with this commit, but it's adding
complexity to a big refactor (native ES modules #975). Since it's not
necessary and simply a philosophical preference, I will revert for now
and try to bring it back later.

1 files changed, 3 insertions, 8 deletions

diff --git a/js/os.ts b/js/os.ts
index 7fc37e46a..e56aab574 100644
--- a/js/os.ts
+++ b/js/os.ts
@@ -4,7 +4,6 @@ import { assert } from "./util";
 import * as util from "./util";
 import * as flatbuffers from "./flatbuffers";
 import { sendSync } from "./dispatch";
-import { TextDecoder } from "./text_encoding";
 
 interface CodeInfo {
   moduleName: string | undefined;
@@ -46,17 +45,13 @@ export function codeFetch(specifier: string, referrer: string): CodeInfo {
   assert(baseRes!.inner(codeFetchRes) != null);
   // flatbuffers returns `null` for an empty value, this does not fit well with
   // idiomatic TypeScript under strict null checks, so converting to `undefined`
-  const sourceCode = codeFetchRes.sourceCodeArray() || undefined;
-  const outputCode = codeFetchRes.outputCodeArray() || undefined;
-  const sourceMap = codeFetchRes.sourceMapArray() || undefined;
-  const decoder = new TextDecoder();
   return {
     moduleName: codeFetchRes.moduleName() || undefined,
     filename: codeFetchRes.filename() || undefined,
     mediaType: codeFetchRes.mediaType(),
-    sourceCode: sourceCode && decoder.decode(sourceCode),
-    outputCode: outputCode && decoder.decode(outputCode),
-    sourceMap: sourceMap && decoder.decode(sourceMap)
+    sourceCode: codeFetchRes.sourceCode() || undefined,
+    outputCode: codeFetchRes.outputCode() || undefined,
+    sourceMap: codeFetchRes.sourceMap() || undefined
   };
 }