Revert "[inspector] RemoteObject.description should be empty for primitive type"

This reverts commit 003159e7.

Reason for revert: breaks roll into Chromium: https://ci.chromium.org/p/chromium/builders/luci.chromium.try/linux_chromium_headless_rel/3140

Original change's description:
> [inspector] RemoteObject.description should be empty for primitive type
> 
> We currently report description field for numbers. On client side user
> can calculate description as remoteObject.unserializableValue ||
> (remoteObject.value + ''). Let's report description only for objects to
> simplify value -> remoteObject logic a bit.
> 
> R=​dgozman@chromium.org
> TBR=jgruber@chromium.org
> 
> Bug: chromium:595206
> Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
> Change-Id: I91356a44aa3024e20c8f966869abf4a41b88e4bc
> Reviewed-on: https://chromium-review.googlesource.com/737485
> Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
> Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#53453}

TBR=dgozman@chromium.org,pfeldman@chromium.org,kozyatinskiy@chromium.org

Change-Id: Ifc184e1ac158d9ea7034922a7250444448fac49f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: chromium:595206
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Reviewed-on: https://chromium-review.googlesource.com/1081207Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#53461}

src/inspector/injected-script-source.js

@@ -740,14 +740,14 @@ InjectedScript.RemoteObject = function(object, objectGroupName, doNotBind, force
 
         // Provide user-friendly number values.
         if (this.type === "number") {
-            var description = toStringDescription(object);
-            switch (description) {
+            this.description = toStringDescription(object);
+            switch (this.description) {
             case "NaN":
             case "Infinity":
             case "-Infinity":
             case "-0":
                 delete this.value;
-                this.unserializableValue = description;
+                this.unserializableValue = this.description;
                 break;
             }
         }
@@ -765,6 +765,7 @@ InjectedScript.RemoteObject = function(object, objectGroupName, doNotBind, force
     if (injectedScript._shouldPassByValue(object)) {
         this.value = object;
         this.subtype = injectedScript._subtype(object);
+        this.description = injectedScript._describeIncludingPrimitives(object);
         return;
     }
 

test/debugger/test-api.js

@@ -636,8 +636,8 @@ class DebugWrapper {
         break;
       }
       case "number": {
-        if (obj.unserializableValue) {
-          value = obj.unserializableValue * 1.0;
+        if (obj.description === "NaN") {
+          value = NaN;
         }
         break;
       }

test/inspector/console/scoped-variables-expected.txt

@@ -5,6 +5,7 @@ Evaluating 'let a = 42;'
 }
 Evaluating 'a'
 {
+    description : 42
     type : number
     value : 42
 }
@@ -34,6 +35,7 @@ Evaluating 'let a = 239;'
 }
 Evaluating 'a'
 {
+    description : 42
     type : number
     value : 42
 }

test/inspector/cpu-profiler/coverage-block-expected.txt

@@ -21,6 +21,7 @@ Running test: testPreciseCountCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -133,6 +134,7 @@ Running test: testPreciseCountCoverageIncremental
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -228,6 +230,7 @@ Running test: testPreciseCountCoverageIncremental
     id : <messageId>
     result : {
         result : {
+            description : 10946
             type : number
             value : 10946
         }
@@ -342,6 +345,7 @@ Running test: testPreciseCoverageFail
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -360,6 +364,7 @@ Running test: testBestEffortCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -385,6 +390,7 @@ Running test: testBestEffortCoverageWithPreciseBinaryEnabled
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -544,6 +550,7 @@ Running test: testBestEffortCoverageWithPreciseCountEnabled
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -737,6 +744,7 @@ Running test: testPreciseBinaryCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -822,6 +830,7 @@ Running test: testPreciseBinaryCoverage
     id : <messageId>
     result : {
         result : {
+            description : 10946
             type : number
             value : 10946
         }

test/inspector/cpu-profiler/coverage-expected.txt

@@ -21,6 +21,7 @@ Running test: testPreciseCountCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -113,6 +114,7 @@ Running test: testPreciseCoverageFail
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -131,6 +133,7 @@ Running test: testBestEffortCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -156,6 +159,7 @@ Running test: testBestEffortCoverageWithPreciseBinaryEnabled
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -315,6 +319,7 @@ Running test: testBestEffortCoverageWithPreciseCountEnabled
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -508,6 +513,7 @@ Running test: testPreciseBinaryCoverage
     id : <messageId>
     result : {
         result : {
+            description : 8
             type : number
             value : 8
         }
@@ -583,6 +589,7 @@ Running test: testPreciseBinaryCoverage
     id : <messageId>
     result : {
         result : {
+            description : 10946
             type : number
             value : 10946
         }

test/inspector/debugger/change-return-value-expected.txt

@@ -16,6 +16,7 @@ Update return value to 42..
 Dump actual return value
 {
     result : {
+        description : 42
         type : number
         value : 42
     }
@@ -27,6 +28,7 @@ Update return value to 239..
 Dump actual return value
 {
     result : {
+        description : 239
         type : number
         value : 239
     }

test/inspector/debugger/eval-scopes-expected.txt

@@ -9,6 +9,7 @@ Tests that variables introduced in eval scopes are accessible
                 isOwn : true
                 name : hest
                 value : {
+                    description : 420
                     type : number
                     value : 420
                 }

test/inspector/debugger/eval-without-codegen-expected.txt

@@ -36,6 +36,7 @@ Running test: testCallFunctionOn
     id : <messageId>
     result : {
         result : {
+            description : 3
             type : number
             value : 3
         }

test/inspector/debugger/evaluate-on-call-frame-in-module-expected.txt

@@ -33,48 +33,56 @@ Array =
 }
 c1 = 
 {
+    description : 12
     type : number
     value : 12
 }
 Evaluating: ++c1
 updated c1 = 
 {
+    description : 13
     type : number
     value : 13
 }
 Evaluating: --c1
 g1 = 
 {
+    description : 2
     type : number
     value : 2
 }
 Evaluating: ++g1
 updated g1 = 
 {
+    description : 3
     type : number
     value : 3
 }
 Evaluating: --g1
 a1 = 
 {
+    description : 10
     type : number
     value : 10
 }
 Evaluating: ++a1
 updated a1 = 
 {
+    description : 11
     type : number
     value : 11
 }
 Evaluating: --a1
 b1 = 
 {
+    description : 11
     type : number
     value : 11
 }
 Evaluating: ++b1
 updated b1 = 
 {
+    description : 12
     type : number
     value : 12
 }
@@ -115,24 +123,28 @@ Array =
 }
 c2 = 
 {
+    description : 22
     type : number
     value : 22
 }
 Evaluating: ++c2
 updated c2 = 
 {
+    description : 23
     type : number
     value : 23
 }
 Evaluating: --c2
 a2 = 
 {
+    description : 20
     type : number
     value : 20
 }
 Evaluating: ++a2
 updated a2 = 
 {
+    description : 21
     type : number
     value : 21
 }
@@ -146,12 +158,14 @@ foo1 =
 }
 b2 = 
 {
+    description : 21
     type : number
     value : 21
 }
 Evaluating: ++b2
 updated b2 = 
 {
+    description : 22
     type : number
     value : 22
 }
@@ -193,12 +207,14 @@ foo2 =
 }
 b3 = 
 {
+    description : 31
     type : number
     value : 31
 }
 Evaluating: ++b3
 updated b3 = 
 {
+    description : 32
     type : number
     value : 32
 }
@@ -233,12 +249,14 @@ Array =
 }
 a = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: ++a
 updated a = 
 {
+    description : 1
     type : number
     value : 1
 }
@@ -265,12 +283,14 @@ Array =
 }
 a = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: ++a
 updated a = 
 {
+    description : 1
     type : number
     value : 1
 }
@@ -318,48 +338,56 @@ Array =
 }
 b2 = 
 {
+    description : 21
     type : number
     value : 21
 }
 Evaluating: ++b2
 updated b2 = 
 {
+    description : 21
     type : number
     value : 21
 }
 Evaluating: --b2
 a = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: ++a
 updated a = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: --a
 b = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: ++b
 updated b = 
 {
+    description : 1
     type : number
     value : 1
 }
 Evaluating: --b
 c = 
 {
+    description : 0
     type : number
     value : 0
 }
 Evaluating: ++c
 updated c = 
 {
+    description : 1
     type : number
     value : 1
 }
@@ -394,24 +422,28 @@ Array =
 }
 y = 
 {
+    description : 5
     type : number
     value : 5
 }
 Evaluating: ++y
 updated y = 
 {
+    description : 6
     type : number
     value : 6
 }
 Evaluating: --y
 x = 
 {
+    description : 5
     type : number
     value : 5
 }
 Evaluating: ++x
 updated x = 
 {
+    description : 6
     type : number
     value : 6
 }
@@ -438,12 +470,14 @@ Array =
 }
 x = 
 {
+    description : 5
     type : number
     value : 5
 }
 Evaluating: ++x
 updated x = 
 {
+    description : 6
     type : number
     value : 6
 }

test/inspector/debugger/evaluate-on-call-frame-in-module.js

@@ -133,7 +133,7 @@ async function checkFrame(frame, i) {
         objectId: scopeChain.object.objectId});
     result.forEach(v => variables.add(v.name));
     result = result.map(v => v.value  ?
-       (v.name + ' = ' + (v.value.description || v.value.value + '')) : (v.name));
+       (v.name + ' = ' + v.value.description) : (v.name));
     InspectorTest.logMessage(result);
   }
 

test/inspector/debugger/scope-skip-variables-with-empty-name-expected.txt

@@ -9,6 +9,7 @@ Tests that scopes do not report variables with empty names
                 isOwn : true
                 name : a
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }

test/inspector/debugger/set-script-source-expected.txt

 Tests Debugger.setScriptSource
-Function evaluate: {"type":"number","value":6}
+Function evaluate: {"type":"number","value":6,"description":"6"}
 PASS, result value: 6
-Function evaluate: {"type":"number","value":8}
+Function evaluate: {"type":"number","value":8,"description":"8"}
 PASS, result value: 8
 Has error reported: PASS
 Reported error is a compile error: PASS

test/inspector/debugger/suspended-generator-scopes-expected.txt

@@ -8,6 +8,7 @@ Running test: testScopesPaused
         isOwn : true
         name : a
         value : {
+            description : 420
             type : number
             value : 420
         }
@@ -19,6 +20,7 @@ Running test: testScopesPaused
         isOwn : true
         name : b
         value : {
+            description : 42
             type : number
             value : 42
         }
@@ -34,6 +36,7 @@ Running test: testScopesNonPaused
         isOwn : true
         name : a
         value : {
+            description : 430
             type : number
             value : 430
         }

test/inspector/debugger/wasm-scope-info.js

@@ -103,7 +103,7 @@ async function getValueString(value) {
     var msg = await Protocol.Runtime.getProperties({objectId: value.objectId});
     printFailure(msg);
     let printProperty = elem => '"' + elem.name + '"' +
-        ': ' + elem.value.value + ' (' + elem.value.type + ')';
+        ': ' + elem.value.description + ' (' + elem.value.type + ')';
     return msg.result.result.map(printProperty).join(', ');
   }
   return JSON.stringify(value.value) + ' (' + value.type + ')';

test/inspector/runtime/await-promise-expected.txt

@@ -5,6 +5,7 @@ Running test: testResolvedPromise
     id : <messageId>
     result : {
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -43,6 +44,7 @@ Running test: testRejectedPromiseWithStack
         exceptionDetails : {
             columnNumber : 0
             exception : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -51,6 +53,7 @@ Running test: testRejectedPromiseWithStack
             text : Uncaught (in promise)
         }
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -89,6 +92,7 @@ Running test: testPendingPromise
     id : <messageId>
     result : {
         result : {
+            description : 239
             type : number
             value : 239
         }

test/inspector/runtime/call-function-on-async-expected.txt

@@ -114,6 +114,7 @@ Running test: testFunctionReturnNotPromise
     id : <messageId>
     result : {
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -188,6 +189,7 @@ Running test: testEvaluateOnExecutionContext
     id : <messageId>
     result : {
         result : {
+            description : 70
             type : number
             value : 70
         }

test/inspector/runtime/clear-of-command-line-api-expected.txt

@@ -4,6 +4,7 @@ Tests that CommandLineAPI is presented only while evaluation.
     id : <messageId>
     result : {
         result : {
+            description : 15
             type : number
             value : 15
         }
@@ -13,6 +14,7 @@ Tests that CommandLineAPI is presented only while evaluation.
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -23,6 +25,7 @@ setPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 14
             type : number
             value : 14
         }
@@ -32,6 +35,7 @@ setPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -41,6 +45,7 @@ setPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -51,6 +56,7 @@ defineValuePropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 14
             type : number
             value : 14
         }
@@ -60,6 +66,7 @@ defineValuePropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -69,6 +76,7 @@ defineValuePropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -79,6 +87,7 @@ definePropertiesForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 14
             type : number
             value : 14
         }
@@ -88,6 +97,7 @@ definePropertiesForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -97,6 +107,7 @@ definePropertiesForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -107,6 +118,7 @@ defineAccessorPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 14
             type : number
             value : 14
         }
@@ -116,6 +128,7 @@ defineAccessorPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -125,6 +138,7 @@ defineAccessorPropertyForMethod()
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -135,6 +149,7 @@ redefineGetOwnPropertyDescriptors()
     id : <messageId>
     result : {
         result : {
+            description : 14
             type : number
             value : 14
         }
@@ -144,6 +159,7 @@ redefineGetOwnPropertyDescriptors()
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }
@@ -153,6 +169,7 @@ redefineGetOwnPropertyDescriptors()
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }

test/inspector/runtime/command-line-api-expected.txt

@@ -56,6 +56,7 @@ Running test: testInspect
         hints : {
         }
         object : {
+            description : 239
             type : number
             value : 239
         }
@@ -67,6 +68,7 @@ Running test: testInspect
         hints : {
         }
         object : {
+            description : -0
             type : number
             unserializableValue : -0
         }
@@ -170,6 +172,7 @@ Is p: true
             queryObjects : true
         }
         object : {
+            description : 1
             type : number
             value : 1
         }
@@ -189,6 +192,7 @@ Running test: testEvaluationResult
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -198,6 +202,7 @@ Running test: testEvaluationResult
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -207,6 +212,7 @@ Running test: testEvaluationResult
     id : <messageId>
     result : {
         result : {
+            description : -0
             type : number
             unserializableValue : -0
         }
@@ -438,6 +444,7 @@ Running test: testDir
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }
@@ -544,6 +551,7 @@ Running test: testDirXML
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }
@@ -616,6 +624,7 @@ Running test: testTable
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }

test/inspector/runtime/console-assert-expected.txt

@@ -36,6 +36,7 @@ Checks that console.assert works and points to correct call frame.
     params : {
         args : [
             [0] : {
+                description : 1
                 type : number
                 value : 1
             }
@@ -68,10 +69,12 @@ Checks that console.assert works and points to correct call frame.
     params : {
         args : [
             [0] : {
+                description : 1
                 type : number
                 value : 1
             }
             [1] : {
+                description : 2
                 type : number
                 value : 2
             }

test/inspector/runtime/console-line-and-column-expected.txt

@@ -4,6 +4,7 @@ Tests line and column numbers in console messages
     params : {
         args : [
             [0] : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -29,6 +30,7 @@ Tests line and column numbers in console messages
     params : {
         args : [
             [0] : {
+                description : 239
                 type : number
                 value : 239
             }

test/inspector/runtime/console-log-doesnt-run-microtasks-expected.txt

 Check that console.log doesn't run microtasks.
 {
+    description : 42
     type : number
     value : 42
 }
 {
+    description : 43
     type : number
     value : 43
 }
 {
+    description : 239
     type : number
     value : 239
 }

test/inspector/runtime/console-memory-expected.txt

@@ -25,6 +25,7 @@ Running test: testWithMemory
     id : <messageId>
     result : {
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -36,6 +37,7 @@ Running test: testSetterDoesntOverride
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }

test/inspector/runtime/es6-module-expected.txt

@@ -161,6 +161,7 @@ console.log(239)
                 isOwn : true
                 name : a1
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -172,6 +173,7 @@ console.log(239)
                 isOwn : true
                 name : a2
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }

test/inspector/runtime/evaluate-async-expected.txt

@@ -5,6 +5,7 @@ Running test: testResolvedPromise
     id : <messageId>
     result : {
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -18,6 +19,7 @@ Running test: testRejectedPromise
         exceptionDetails : {
             columnNumber : 0
             exception : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -26,6 +28,7 @@ Running test: testRejectedPromise
             text : Uncaught (in promise)
         }
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -167,6 +170,7 @@ Running test: testExceptionInEvaluate
         exceptionDetails : {
             columnNumber : 0
             exception : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -176,6 +180,7 @@ Running test: testExceptionInEvaluate
             text : Uncaught
         }
         result : {
+            description : 239
             type : number
             value : 239
         }
@@ -187,6 +192,7 @@ Running test: testThenableJob
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -198,6 +204,7 @@ Running test: testLastEvaluatedResult
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -207,6 +214,7 @@ Running test: testLastEvaluatedResult
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }

test/inspector/runtime/evaluate-run-microtasks-expected.txt

@@ -4,6 +4,7 @@ Tests that microtasks run before the Runtime.evaluate response is sent
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }

test/inspector/runtime/evaluate-unserializable-expected.txt

@@ -4,6 +4,7 @@ Tests Runtime.evaluate with unserializable results.
     id : <messageId>
     result : {
         result : {
+            description : -0
             type : number
             unserializableValue : -0
         }
@@ -14,6 +15,7 @@ NaN
     id : <messageId>
     result : {
         result : {
+            description : NaN
             type : number
             unserializableValue : NaN
         }
@@ -24,6 +26,7 @@ Infinity
     id : <messageId>
     result : {
         result : {
+            description : Infinity
             type : number
             unserializableValue : Infinity
         }
@@ -34,6 +37,7 @@ Infinity
     id : <messageId>
     result : {
         result : {
+            description : -Infinity
             type : number
             unserializableValue : -Infinity
         }

test/inspector/runtime/evaluate-without-side-effects-expected.txt

@@ -4,6 +4,7 @@ Test throwOnSideEffect: false
     id : <messageId>
     result : {
         result : {
+            description : 2
             type : number
             value : 2
         }
@@ -41,6 +42,7 @@ Test expression without side-effect, with throwOnSideEffect: true
     id : <messageId>
     result : {
         result : {
+            description : 4
             type : number
             value : 4
         }
@@ -52,6 +54,7 @@ paused
     id : <messageId>
     result : {
         result : {
+            description : 2
             type : number
             value : 2
         }
@@ -62,6 +65,7 @@ Test that debug break does not trigger with throwOnSideEffect
     id : <messageId>
     result : {
         result : {
+            description : 2
             type : number
             value : 2
         }

test/inspector/runtime/exception-thrown-expected.txt

@@ -88,6 +88,7 @@ Check that exceptionThrown is supported by test runner.
         exceptionDetails : {
             columnNumber : 2
             exception : {
+                description : 239
                 type : number
                 value : 239
             }

test/inspector/runtime/function-scopes-expected.txt

@@ -6,6 +6,7 @@ Catch:
     isOwn : true
     name : a
     value : {
+        description : 1
         type : number
         value : 1
     }
@@ -18,6 +19,7 @@ With block:
     isOwn : true
     name : b
     value : {
+        description : 2
         type : number
         value : 2
     }
@@ -30,6 +32,7 @@ Closure (closure):
     isOwn : true
     name : c
     value : {
+        description : 3
         type : number
         value : 3
     }
@@ -42,6 +45,7 @@ Global:
     isOwn : true
     name : e
     value : {
+        description : 5
         type : number
         value : 5
     }

test/inspector/runtime/get-properties-on-proxy-expected.txt

@@ -176,6 +176,7 @@ Checking counter
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }

test/inspector/runtime/internal-properties-entries-expected.txt

@@ -50,6 +50,7 @@ expression: new Map([[1,2],[3,4]])
                 isOwn : true
                 name : length
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }
@@ -71,6 +72,7 @@ expression: new Map()
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -130,6 +132,7 @@ expression: new Map([[1,2],[3,4]]).entries()
                 isOwn : true
                 name : length
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }
@@ -169,6 +172,7 @@ expression: it = new Map([[1,2],[3,4]]).entries(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -207,6 +211,7 @@ expression: it = new Map([[1,2],[3,4]]).keys(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -245,6 +250,7 @@ expression: it = new Map([[1,2],[3,4]]).values(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -266,6 +272,7 @@ expression: it = new Map([[1,2],[3,4]]).entries(); it.next(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -323,6 +330,7 @@ expression: new Set([1,2])
                 isOwn : true
                 name : length
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }
@@ -344,6 +352,7 @@ expression: new Set()
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -401,6 +410,7 @@ expression: new Set([1,2]).values()
                 isOwn : true
                 name : length
                 value : {
+                    description : 2
                     type : number
                     value : 2
                 }
@@ -439,6 +449,7 @@ expression: it = new Set([1,2]).values(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -477,6 +488,7 @@ expression: it = new Set([1,2]).keys(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -515,6 +527,7 @@ expression: it = new Set([1,2]).entries(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -536,6 +549,7 @@ expression: it = new Set([1,2]).values(); it.next(); it.next(); it
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -559,6 +573,7 @@ expression: new WeakMap()
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -600,6 +615,7 @@ expression: new WeakMap([[{ a: 2 }, 42]])
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }
@@ -623,6 +639,7 @@ expression: new WeakSet()
                 isOwn : true
                 name : length
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -663,6 +680,7 @@ expression: new WeakSet([{a:2}])
                 isOwn : true
                 name : length
                 value : {
+                    description : 1
                     type : number
                     value : 1
                 }

test/inspector/runtime/internal-properties-expected.txt

@@ -9,6 +9,7 @@ expression: (function* foo() { yield 1 })
             [0] : {
                 name : [[FunctionLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -48,6 +49,7 @@ expression: (function foo() {})
             [0] : {
                 name : [[FunctionLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -80,6 +82,7 @@ expression: new Number(239)
             [0] : {
                 name : [[PrimitiveValue]]
                 value : {
+                    description : 239
                     type : number
                     value : 239
                 }
@@ -166,6 +169,7 @@ expression: Promise.resolve(42)
             [1] : {
                 name : [[PromiseValue]]
                 value : {
+                    description : 42
                     type : number
                     value : 42
                 }
@@ -229,6 +233,7 @@ expression: gen1
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -284,6 +289,7 @@ expression: gen1.next();gen1
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -339,6 +345,7 @@ expression: gen1.next();gen1
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -386,6 +393,7 @@ expression: gen2
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -441,6 +449,7 @@ expression: gen2.next();gen2
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -496,6 +505,7 @@ expression: gen2.next();gen2
             [3] : {
                 name : [[GeneratorLocation]]
                 value : {
+                    description : Object
                     subtype : internal#location
                     type : object
                     value : {
@@ -525,6 +535,7 @@ expression: (new Map([[1,2]])).entries()
             [1] : {
                 name : [[IteratorIndex]]
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }
@@ -564,6 +575,7 @@ expression: (new Set([[1,2]])).entries()
             [1] : {
                 name : [[IteratorIndex]]
                 value : {
+                    description : 0
                     type : number
                     value : 0
                 }

test/inspector/runtime/property-on-console-proto-expected.txt

@@ -3,6 +3,7 @@ Tests that property defined on console.__proto__ doesn't observable on other Obj
     id : <messageId>
     result : {
         result : {
+            description : 0
             type : number
             value : 0
         }

test/inspector/runtime/protocol-works-with-different-locale-expected.txt

@@ -6,6 +6,7 @@ Running test: consoleLogWithDefaultLocale
     params : {
         args : [
             [0] : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -62,6 +63,7 @@ set locale to fr_CA.UTF-8 (has comma as separator)
     params : {
         args : [
             [0] : {
+                description : 239
                 type : number
                 value : 239
             }
@@ -90,6 +92,7 @@ set locale to fr_CA.UTF-8 (has comma as separator)
     params : {
         args : [
             [0] : {
+                description : 239
                 type : number
                 value : 239
             }

test/inspector/runtime/runtime-disable-preserve-injected-script-expected.txt

@@ -5,6 +5,7 @@ Tests that Runtime.disable doesn't invalidate injected-script.
     isOwn : true
     name : a
     value : {
+        description : 1
         type : number
         value : 1
     }
@@ -17,6 +18,7 @@ Disabling agent..
     isOwn : true
     name : a
     value : {
+        description : 1
         type : number
         value : 1
     }

test/inspector/runtime/runtime-evaluate-with-dirty-context-expected.txt

@@ -4,6 +4,7 @@ Checks that Runtime.evaluate works with dirty context.
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }

test/inspector/runtime/runtime-restore-expected.txt

@@ -35,6 +35,7 @@ will reconnect..
     params : {
         args : [
             [0] : {
+                description : 42
                 type : number
                 value : 42
             }

test/inspector/runtime/terminate-execution-expected.txt

@@ -14,6 +14,7 @@ Terminate first evaluation (it forces injected-script-source compilation)
 }
 Checks that we reset termination after evaluation
 {
+    description : 42
     type : number
     value : 42
 }

test/inspector/sessions/runtime-command-line-api-expected.txt

@@ -5,6 +5,7 @@ Evaluating $0 in 1
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -25,6 +26,7 @@ Evaluating $0 in 1
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -35,6 +37,7 @@ Evaluating $0 in 2
     id : <messageId>
     result : {
         result : {
+            description : 17
             type : number
             value : 17
         }
@@ -46,6 +49,7 @@ Evaluating $_ in 1
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -66,6 +70,7 @@ Evaluating $_ in 1
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }
@@ -76,6 +81,7 @@ Evaluating $_ in 2
     id : <messageId>
     result : {
         result : {
+            description : 17
             type : number
             value : 17
         }
@@ -89,6 +95,7 @@ inspectRequested from 1
         hints : {
         }
         object : {
+            description : 42
             type : number
             value : 42
         }
@@ -102,6 +109,7 @@ inspectRequested from 1
         hints : {
         }
         object : {
+            description : 42
             type : number
             value : 42
         }
@@ -115,6 +123,7 @@ inspectRequested from 2
         hints : {
         }
         object : {
+            description : 17
             type : number
             value : 17
         }
@@ -128,6 +137,7 @@ inspectRequested from 2
         hints : {
         }
         object : {
+            description : 17
             type : number
             value : 17
         }
@@ -141,6 +151,7 @@ inspectRequested from 1
         hints : {
         }
         object : {
+            description : 42
             type : number
             value : 42
         }
@@ -152,6 +163,7 @@ Evaluating $0 in 2
     id : <messageId>
     result : {
         result : {
+            description : 17
             type : number
             value : 17
         }
@@ -162,6 +174,7 @@ Evaluating $_ in 2
     id : <messageId>
     result : {
         result : {
+            description : 17
             type : number
             value : 17
         }
@@ -175,6 +188,7 @@ inspectRequested from 2
         hints : {
         }
         object : {
+            description : 17
             type : number
             value : 17
         }
@@ -189,6 +203,7 @@ inspectRequested from 2
         hints : {
         }
         object : {
+            description : 17
             type : number
             value : 17
         }

test/inspector/sessions/runtime-console-api-called-expected.txt

@@ -6,6 +6,7 @@ From session 1
     params : {
         args : [
             [0] : {
+                description : 1
                 type : number
                 value : 1
             }
@@ -32,6 +33,7 @@ From session 2
     params : {
         args : [
             [0] : {
+                description : 1
                 type : number
                 value : 1
             }
@@ -59,6 +61,7 @@ From session 1
     params : {
         args : [
             [0] : {
+                description : 2
                 type : number
                 value : 2
             }
@@ -85,6 +88,7 @@ From session 2
     params : {
         args : [
             [0] : {
+                description : 2
                 type : number
                 value : 2
             }

test/inspector/sessions/runtime-evaluate-expected.txt

@@ -5,6 +5,7 @@ Evaluating in 2
     id : <messageId>
     result : {
         result : {
+            description : 42
             type : number
             value : 42
         }

test/inspector/sessions/runtime-remote-object-expected.txt

@@ -12,6 +12,7 @@ Retrieving properties in 2
                 isOwn : true
                 name : a
                 value : {
+                    description : 17
                     type : number
                     value : 17
                 }
@@ -44,6 +45,7 @@ Retrieving properties in 1
                 isOwn : true
                 name : a
                 value : {
+                    description : 42
                     type : number
                     value : 42
                 }
@@ -77,6 +79,7 @@ Retrieving properties in 1
                 isOwn : true
                 name : a
                 value : {
+                    description : 42
                     type : number
                     value : 42
                 }