rename promise handlers

Author: lkwinta

test/test_other.py

@@ -3951,15 +3951,15 @@ def test_file_packager_embed(self):
     self.assertContained('hello data', output)
 
   def test_file_packager_export_es6(self):
-    create_file('data.txt', 'hello data')
-    self.run_process([FILE_PACKAGER, 'test.data', '--export-es6', '--preload', 'data.txt', '--js-output=dataFileLoader.js', '--no-node'])
+    create_file('smth.txt', 'hello data')
+    self.run_process([FILE_PACKAGER, 'test.data', '--export-es6', '--preload', 'smth.txt', '--js-output=dataFileLoader.js'])
 
     create_file('test.c', '''
     #include <stdio.h>
     #include <emscripten.h>
 
     EMSCRIPTEN_KEEPALIVE int test_fun() {
-      FILE* f = fopen("data.txt", "r");
+      FILE* f = fopen("smth.txt", "r");
       char buf[64] = {0};
       int rtn = fread(buf, 1, 64, f);
       buf[rtn] = '\\0';

tools/file_packager.py

@@ -634,11 +634,7 @@ def generate_js(data_target, data_files, metadata):
   else:
     if options.export_es6:
       ret = '''export default function loadDataFile(Module) {
-  var readyPromiseResolve, readyPromiseReject;
-  var readyPromise = new Promise((resolve, reject) => {
-    readyPromiseResolve = resolve;
-    readyPromiseReject = reject;
-  });'''
+  return new Promise((loadDataResolve, loadDataReject) => {'''
 
     else:
       ret = '''
@@ -710,7 +706,7 @@ def generate_js(data_target, data_files, metadata):
           Module['FS_createDataFile'](this.name, null, byteArray, true, true, true);
           Module['removeRunDependency'](`fp ${that.name}`);'''
     ready_promise = '''
-          readyPromiseResolve();'''
+          loadDataResolve();'''
 
     if not options.lz4:
       # Data requests - for getting a block of data out of the big archive - have
@@ -987,7 +983,7 @@ def generate_js(data_target, data_files, metadata):
         }'''.strip()
 
     reject_promise = '''
-          readyPromiseReject();'''
+          loadDataReject();'''
 
     ret += '''
       function fetchRemotePackage(packageName, packageSize, callback, errback) {
@@ -1170,7 +1166,7 @@ def generate_js(data_target, data_files, metadata):
 
   if options.export_es6:
     ret += '''
-  return readyPromise;
+  });
 }
 // END the loadDataFile function
 '''