only lazify array, fix the double borrow issue

Author: jasonwilliams

core/engine/src/builtins/mod.rs

@@ -206,12 +206,15 @@ impl Realm {
         AsyncIterator::init(self);
         AsyncFromSyncIterator::init(self);
         ForInIterator::init(self);
+        Math::init(self);
+        Json::init(self);
         ArrayIterator::init(self);
         Proxy::init(self);
         ArrayBuffer::init(self);
         SharedArrayBuffer::init(self);
         BigInt::init(self);
         Boolean::init(self);
+        Date::init(self);
         DataView::init(self);
         Map::init(self);
         MapIterator::init(self);
@@ -223,7 +226,9 @@ impl Realm {
         Eval::init(self);
         Set::init(self);
         SetIterator::init(self);
+        String::init(self);
         StringIterator::init(self);
+        RegExp::init(self);
         RegExpStringIterator::init(self);
         BuiltinTypedArray::init(self);
         Int8Array::init(self);

core/engine/src/context/intrinsics.rs

@@ -257,7 +257,7 @@ impl StandardConstructors {
             )),
             async_generator_function: StandardConstructor::default(),
             proxy: StandardConstructor::default(),
-            date: StandardConstructor::lazy(Date::init, realm_inner),
+            date: StandardConstructor::default(),
             function: StandardConstructor {
                 constructor: JsFunction::empty_intrinsic_function(true),
                 prototype: JsFunction::empty_intrinsic_function(false).into(),
@@ -270,8 +270,11 @@ impl StandardConstructors {
             boolean: StandardConstructor::with_prototype(JsObject::from_proto_and_data(
                 None, false,
             )),
-            string: StandardConstructor::lazy(String::init, realm_inner),
-            regexp: StandardConstructor::lazy(RegExp::init, realm_inner),
+            string: StandardConstructor::with_prototype(JsObject::from_proto_and_data(
+                None,
+                js_string!(),
+            )),
+            regexp: StandardConstructor::default(),
             symbol: StandardConstructor::default(),
             error: StandardConstructor::default(),
             type_error: StandardConstructor::default(),
@@ -1165,8 +1168,8 @@ impl IntrinsicObjects {
     pub(crate) fn uninit(realm_inner: &WeakGc<RealmInner>) -> Option<Self> {
         Some(Self {
             reflect: JsObject::default(),
-            math: JsObject::lazy(Math::init, realm_inner),
-            json: JsObject::lazy(Json::init, realm_inner),
+            math: JsObject::default(),
+            json: JsObject::default(),
             throw_type_error: JsFunction::empty_intrinsic_function(false),
             array_prototype_values: JsFunction::empty_intrinsic_function(false),
             array_prototype_to_string: JsFunction::empty_intrinsic_function(false),

core/engine/src/native_function.rs

@@ -343,14 +343,14 @@ pub(crate) fn native_function_call(
     argument_count: usize,
     context: &mut Context,
 ) -> JsResult<CallValue> {
-    native_function_call_inner(
-        obj,
-        &obj.downcast_ref::<NativeFunctionObject>()
-            .expect("the object should be a native function object")
-            .clone(),
-        argument_count,
-        context,
-    )
+    let native_function = &obj
+        .clone()
+        .downcast::<NativeFunctionObject>()
+        .expect("the object should be a native function object")
+        .borrow_mut()
+        .data
+        .clone();
+    native_function_call_inner(obj, native_function, argument_count, context)
 }
 
 /// Call this object.