change Jsx.addKeyProp to external binding

Author: mununki

jscomp/main/builtin_cmi_datasets.ml

@@ -38,8 +38,5 @@ type 'props component = ('props, element) componentLike
 external component : ('props, element) componentLike -> 'props component
   = "%identity"
 
-let addKeyProp (p : 'props) (k : string) : 'props =
-  (* Use Js_obj.assign, not Js.Obj.assign, otherwise the dependency will not be picked up correctly
-     in the ninja file. *)
-  Obj.magic (Js_obj.assign (Obj.magic p) [%obj { key = k }])
-  [@@inline]
+external addKeyProp : 'props -> string -> 'props = "Object.assign"
+  [@@inline] [@@bs.val]

jscomp/main/builtin_cmj_datasets.ml

@@ -63,7 +63,7 @@ o others/js_vector.cmj : cc_cmi others/js_vector.ml | others/belt_internals.cmi
 o others/js_vector.cmi : cc others/js_vector.mli | others/belt_internals.cmi others/js.cmi $bsc
 o others/js_weakmap.cmi others/js_weakmap.cmj : cc others/js_weakmap.ml | others/belt_internals.cmi others/js.cmi $bsc
 o others/js_weakset.cmi others/js_weakset.cmj : cc others/js_weakset.ml | others/belt_internals.cmi others/js.cmi $bsc
-o others/jsx.cmi others/jsx.cmj : cc others/jsx.ml | others/belt_internals.cmi others/js.cmi others/js_obj.cmj $bsc
+o others/jsx.cmi others/jsx.cmj : cc others/jsx.ml | others/belt_internals.cmi others/js.cmi $bsc
 o others/jsxDOM.cmi others/jsxDOM.cmj : cc others/jsxDOM.ml | others/belt_internals.cmi others/js.cmi others/jsx.cmj others/jsxDOMStyle.cmj others/jsxEvent.cmj $bsc
 o others/jsxDOMStyle.cmi others/jsxDOMStyle.cmj : cc others/jsxDOMStyle.ml | others/belt_internals.cmi others/js.cmi $bsc
 o others/jsxEvent.cmi others/jsxEvent.cmj : cc others/jsxEvent.ml | others/belt_internals.cmi others/js.cmi $bsc

jscomp/others/jsx.ml

@@ -1,13 +1 @@
-
-
-
-function addKeyProp(p, k) {
-  return Object.assign(p, {
-              key: k
-            });
-}
-
-export {
-  addKeyProp ,
-}
-/* No side effect */
+/* This output is empty. Its source's type definitions, externals and/or unused code got optimized away. */

jscomp/others/release.ninja

@@ -1,11 +1 @@
-'use strict';
-
-
-function addKeyProp(p, k) {
-  return Object.assign(p, {
-              key: k
-            });
-}
-
-exports.addKeyProp = addKeyProp;
-/* No side effect */
+/* This output is empty. Its source's type definitions, externals and/or unused code got optimized away. */