Replace Object.assign with the spread operator in object inspector.

Author: nchevobbe

packages/devtools-reps/src/object-inspector/component.js

@@ -267,9 +267,10 @@ class ObjectInspector extends Component {
       return {
         label: this.renderGrip(
           item,
-          Object.assign({}, this.props, {
+          {
+            ...this.props,
             functionName: label
-          })
+          }
         )
       };
     }
@@ -280,7 +281,7 @@ class ObjectInspector extends Component {
       || nodeIsMapEntry(item)
       || isPrimitive
     ) {
-      let repsProp = Object.assign({}, this.props);
+      let repsProp = {...this.props};
       if (depth > 0) {
         repsProp.mode = this.props.mode === MODE.LONG
           ? MODE.SHORT
@@ -416,11 +417,12 @@ class ObjectInspector extends Component {
     props: Props
   ) {
     const object = getValue(item);
-    return Rep(Object.assign({}, props, {
+    return Rep({
+      ...props,
       object,
       mode: props.mode || MODE.TINY,
       defaultRep: Grip,
-    }));
+    });
   }
 
   render() {

packages/devtools-reps/src/object-inspector/reducer.js

@@ -17,7 +17,7 @@ function reducer(
     data,
   } = action;
 
-  const cloneState = overrides => Object.assign({}, state, overrides);
+  const cloneState = overrides => ({ ...state, ...overrides});
 
   if (type === "NODE_EXPAND") {
     return cloneState({
@@ -56,7 +56,7 @@ function mergeResponses(responses: Array<Object>) : Object {
 
   for (const response of responses) {
     if (response.hasOwnProperty("ownProperties")) {
-      data.ownProperties = Object.assign({}, data.ownProperties, response.ownProperties);
+      data.ownProperties = {...data.ownProperties, ...response.ownProperties};
     }
 
     if (response.ownSymbols && response.ownSymbols.length > 0) {

packages/devtools-reps/src/object-inspector/store.js

@@ -14,12 +14,13 @@ import type {
 } from "./types";
 
 function createInitialState(overrides : Object) : State {
-  return Object.assign({
+  return {
     actors: new Set(),
     expandedPaths: new Set(),
     focusedItem: null,
     loadedProperties: new Map(),
-  }, overrides);
+    ...overrides,
+  };
 }
 
 module.exports = (props : Props) => {

packages/devtools-reps/src/object-inspector/utils/node.js

@@ -535,7 +535,7 @@ function makeNodesForProperties(
   const parentPath = parent.path;
   const parentValue = getValue(parent);
 
-  let allProperties = Object.assign({}, ownProperties, safeGetterValues);
+  let allProperties = {...ownProperties, ...safeGetterValues};
 
   // Ignore properties that are neither non-concrete nor getters/setters.
   const propertiesNames = sortProperties(Object.keys(allProperties)).filter(name => {