hermes: update to hermes v1

Author: ammarahm-ed

.vscode/settings.json

@@ -80,7 +80,8 @@
         "stdlib.h": "c",
         "assert.h": "c",
         "js_native_api.h": "c",
-        "javascriptcore.h": "c"
+        "javascriptcore.h": "c",
+        "codecvt": "cpp"
     },
     "cmake.sourceDirectory": "/Volumes/DataDrive/Android/android-runtime/test-app/runtime"
 }

build.gradle

@@ -20,6 +20,8 @@ def hasNdkVersion = project.hasProperty("ndkVersion")
 
 def hasHostObjects = project.hasProperty("useHostObjects")
 
+def asNapiModule = project.hasProperty("asNapiModule")
+
 if (hasNdkVersion) {
     println "Using NDK version " + ndkVersion
 }
@@ -213,6 +215,10 @@ def getAssembleReleaseBuildArguments = { ->
          arguments.add("-PuseHostObjects")
     }
 
+    if (asNapiModule) {
+         arguments.add("-PasNapiModule")
+    }
+
     arguments += ["--warning-mode", "all"]
 
     return arguments

test-app/.idea/caches/deviceStreaming.xml

@@ -1,6 +1,6 @@
 {
-    "kotlinUsage": {
-        "hasUseKotlinPropertyInApp": true,
-        "hasKotlinRuntimeClasses": false
-    }
+  "kotlinUsage": {
+    "hasUseKotlinPropertyInApp": true,
+    "hasKotlinRuntimeClasses": true
+  }
 }

test-app/.idea/deploymentTargetSelector.xml

@@ -214,6 +214,7 @@ def computeNamespace = { ->
 }
 
 android {
+
     namespace computeNamespace()
 
     applyBeforePluginGradleConfiguration()
@@ -350,7 +351,7 @@ dependencies {
     }
 
     if (ns_engine == "HERMES") {
-        implementation 'com.facebook.fbjni:fbjni:0.3.0'
+        implementation 'com.facebook.fbjni:fbjni:0.7.0'
     }
 
     def androidXMaterialVersion = "${ns_default_androidx_material_version}"

test-app/analytics/build-statistics.json

@@ -41,19 +41,19 @@ var TerminalReporter = require('../jasmine-reporters/terminal_reporter').Termina
 
     it: function(desc, func) {
 //    if (global.__engine === "QuickJS") {
-//          if (func.length > 0) {
-//            return env.it(desc, func);
-//          } else {
-//            return env.it(desc, function(done) {
-//                func();
-//                __ns__setTimeout(() => {
-//                    done();
-//                }, 1)
-//
-//            })
-//          }
+          if (func.length > 0) {
+            return env.it(desc, func);
+          } else {
+            return env.it(desc, function(done) {
+                func();
+                __ns__setTimeout(() => {
+                    done();
+                }, 1)
+
+            })
+          }
 //    } else {
-        return env.it(desc, func);
+      //  return env.it(desc, func);
 //    }
     },
 

test-app/app/build.gradle

@@ -18,7 +18,8 @@ describe("Runtime exposes", function () {
         var timeEnd = __time();
         var dateDelta = dateTimeEnd - dateTimeStart;
         var timeDelta = timeEnd - timeStart;
-        expect(Math.abs(dateDelta - timeDelta) < dateDelta * 0.25).toBe(true);
+
+        expect(Math.abs(dateDelta - timeDelta) <= dateDelta * 0.25).toBe(true);
         break;
       } catch(e) {
         if (i == 9) {

test-app/app/src/main/assets/app/Infrastructure/Jasmine/jasmine-2.0.1/boot.js

@@ -204,7 +204,7 @@
   }
 
   global.setNativeArrayProp = (target, prop, value, receiver) => {
-    if (!isNaN(prop)) {
+     if (typeof prop !== "symbol" && !isNaN(prop)) {
       receiver.setValueAtIndex(parseInt(prop), value);
       return true;
     }
@@ -213,7 +213,7 @@
   };
 
   global.getNativeArrayProp = (target, prop, receiver) => {
-    if (!isNaN(prop)) {
+    if (typeof prop !== "symbol" && !isNaN(prop)) {
       return receiver.getValueAtIndex(parseInt(prop));
     }