Bug 1282427 - Reps: handle userDisplayName property on Function grips; draft
authorgasolin <gasolin@gmail.com>
Wed, 13 Jul 2016 14:44:01 +0800
changeset 386989 24d8a3b45acd0ac1b638c1d5ef77a4025e4c9689
parent 386922 2dc915c67fb09596c0e01bcf7b4d8b40d3e6c327
child 525271 826aa1662fe501f1421e8baa88fe07838958978c
push id22876
push userbmo:gasolin@mozilla.com
push dateWed, 13 Jul 2016 06:44:09 +0000
bugs1282427
milestone50.0a1
Bug 1282427 - Reps: handle userDisplayName property on Function grips; MozReview-Commit-ID: 2DPfSuHeFo9
devtools/client/shared/components/reps/function.js
devtools/client/shared/components/test/mochitest/test_reps_function.html
--- a/devtools/client/shared/components/reps/function.js
+++ b/devtools/client/shared/components/reps/function.js
@@ -29,17 +29,17 @@ define(function (require, exports, modul
         return this.props.objectLink({
           object: grip
         }, "function");
       }
       return "";
     },
 
     summarizeFunction: function (grip) {
-      let name = grip.displayName || grip.name || "function";
+      let name = grip.userDisplayName || grip.displayName || grip.name || "function";
       return cropString(name + "()", 100);
     },
 
     render: function () {
       let grip = this.props.object;
 
       return (
         ObjectBox({className: "function"},
--- a/devtools/client/shared/components/test/mochitest/test_reps_function.html
+++ b/devtools/client/shared/components/test/mochitest/test_reps_function.html
@@ -47,16 +47,32 @@ window.onload = Task.async(function* () 
         mode: undefined,
         expectedOutput: defaultOutput,
       }
     ];
 
     testRepRenderModes(modeTests, testName, componentUnderTest, getGripStub(testName));
   }
 
+  function testUserNamed() {
+    // Test declaration: `function testName{ let innerVar = "foo" }`
+    const testName = "testUserNamed";
+
+    const defaultOutput = `testUserName()`;
+
+    const modeTests = [
+      {
+        mode: undefined,
+        expectedOutput: defaultOutput,
+      }
+    ];
+
+    testRepRenderModes(modeTests, testName, componentUnderTest, getGripStub(testName));
+  }
+
   function testVarNamed() {
     // Test declaration: `let testVarName = function() { }`
     const testName = "testVarNamed";
 
     const defaultOutput = `testVarName()`;
 
     const modeTests = [
       {
@@ -113,16 +129,33 @@ window.onload = Task.async(function* () 
           "name": "testName",
           "displayName": "testName",
           "location": {
             "url": "debugger eval code",
             "line": 1
           }
         };
 
+      case "testUserNamed":
+        return {
+          "type": "object",
+          "class": "Function",
+          "actor": "server1.conn6.obj35",
+          "extensible": true,
+          "frozen": false,
+          "sealed": false,
+          "name": "testName",
+          "userDisplayName": "testUserName",
+          "displayName": "testName",
+          "location": {
+            "url": "debugger eval code",
+            "line": 1
+          }
+        };
+
       case "testVarNamed":
         return {
           "type": "object",
           "class": "Function",
           "actor": "server1.conn7.obj41",
           "extensible": true,
           "frozen": false,
           "sealed": false,