[*kit] fulfil promise immediately

I think this would only commonly happen for cases which files aren't
being loaded (debug asmjs), but could happen if multiple ready() calls
are made (which is discouraged).

Bug: skia:
Change-Id: If848aad8603f997661502563136ac590cc9a6d5a
Reviewed-on: https://skia-review.googlesource.com/c/181409
Reviewed-by: Kevin Lubick <[email protected]>

Author: kjlubick

experimental/canvaskit/CHANGELOG.md

@@ -6,6 +6,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+ - Potential bug in `ready()` if already loaded.
 
 ## [0.3.1] - 2019-01-04
 ### Added

experimental/canvaskit/ready.js

@@ -5,9 +5,13 @@ Module['ready'] = function() {
   return new Promise(function (resolve, reject) {
     delete Module['then'];
     Module['onAbort'] = reject;
-    addOnPostRun(function () {
-      resolve(Module)
-    });
+    if (runtimeInitialized) {
+      resolve(Module);
+    } else {
+      addOnPostRun(function() {
+        resolve(Module);
+      });
+    }
   });
 }
 // TODO(kjlubick): Shut .then() entirely off in 0.4.0 by uncommenting below.

modules/pathkit/CHANGELOG.md

@@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+ - Potential bug in `ready()` if already loaded.
+
 ## [0.5.1] 2019-01-04
 
 ### Changed

modules/pathkit/ready.js

@@ -5,9 +5,13 @@ Module['ready'] = function() {
   return new Promise(function (resolve, reject) {
     delete Module['then'];
     Module['onAbort'] = reject;
-    addOnPostRun(function () {
-      resolve(Module)
-    });
+    if (runtimeInitialized) {
+      resolve(Module);
+    } else {
+      addOnPostRun(function() {
+        resolve(Module);
+      });
+    }
   });
 }
 // TODO(kjlubick): Shut .then() entirely off in 0.6.0 by uncommenting below.

modules/pathkit/tests/util.spec.js

@@ -64,4 +64,4 @@ describe('PathKit\'s CubicMap Behavior', function() {
         }));
     });
 
-});
+});