Delete unused loadItemProperties function.

Author: nchevobbe

packages/devtools-reps/src/object-inspector/utils/load-properties.js

@@ -2,14 +2,6 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-const {
-  enumEntries,
-  enumIndexedProperties,
-  enumNonIndexedProperties,
-  getPrototype,
-  enumSymbols,
-} = require("./client");
-
 const {
   getClosestGripNode,
   getClosestNonBucketNode,
@@ -27,77 +19,10 @@ const {
 } = require("./node");
 
 import type {
-  GripProperties,
   LoadedProperties,
   Node,
-  NodeContents,
-  ObjectClient,
-  RdpGrip,
 } from "../types";
 
-function loadItemProperties(
-  item: Node,
-  createObjectClient: (RdpGrip | NodeContents) => ObjectClient,
-  loadedProperties: LoadedProperties
-) : Promise<GripProperties> | null {
-  const [start, end] = item.meta
-    ? [item.meta.startIndex, item.meta.endIndex]
-    : [];
-
-  let objectClient;
-  const getObjectClient = () => {
-    if (objectClient) {
-      return objectClient;
-    }
-
-    const gripItem = getClosestGripNode(item);
-    const value = getValue(gripItem);
-    return createObjectClient(value);
-  };
-
-  let loadingPromises = [];
-  if (shouldLoadItemIndexedProperties(item, loadedProperties)) {
-    loadingPromises.push(enumIndexedProperties(getObjectClient(), start, end));
-  }
-
-  if (shouldLoadItemNonIndexedProperties(item, loadedProperties)) {
-    loadingPromises.push(enumNonIndexedProperties(getObjectClient(), start, end));
-  }
-
-  if (shouldLoadItemEntries(item, loadedProperties)) {
-    loadingPromises.push(enumEntries(getObjectClient(), start, end));
-  }
-
-  if (shouldLoadItemPrototype(item, loadedProperties)) {
-    loadingPromises.push(getPrototype(getObjectClient()));
-  }
-
-  if (shouldLoadItemSymbols(item, loadedProperties)) {
-    loadingPromises.push(enumSymbols(getObjectClient(), start, end));
-  }
-
-  if (loadingPromises.length === 0) {
-    return null;
-  }
-
-  return Promise.all(loadingPromises)
-    .then(responses => responses.reduce((accumulator, res) => {
-      // Let's loop through the responses to build a single response object.
-      Object.entries(res).forEach(([k, v]) => {
-        if (accumulator.hasOwnProperty(k)) {
-          if (Array.isArray(accumulator[k])) {
-            accumulator[k].push(...v);
-          } else if (typeof accumulator[k] === "object") {
-            accumulator[k] = Object.assign({}, accumulator[k], v);
-          }
-        } else {
-          accumulator[k] = v;
-        }
-      });
-      return accumulator;
-    }, {}));
-}
-
 function shouldLoadItemIndexedProperties(
   item: Node,
   loadedProperties: LoadedProperties = new Map()
@@ -180,7 +105,6 @@ function shouldLoadItemSymbols(
 }
 
 module.exports = {
-  loadItemProperties,
   shouldLoadItemEntries,
   shouldLoadItemIndexedProperties,
   shouldLoadItemNonIndexedProperties,