[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}

[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}
003159e7 · Alexey Kozyatinskiy · Commit Bot · 9eb56e82 · 003159e7 · 003159e7
Commit 003159e7 authored May 30, 2018 by Alexey Kozyatinskiy Committed by Commit Bot May 31, 2018
42 changed files
--- a/src/inspector/injected-script-source.js
+++ b/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") {
-            this.description = toStringDescription(object);
+            var description = toStringDescription(object);
-            switch (this.description) {
+            switch (description) {
            case "NaN":
            case "Infinity":
            case "-Infinity":
            case "-0":
                delete this.value;
-                this.unserializableValue = this.description;
+                this.unserializableValue = description;
                break;
            }
        }
@@ -765,7 +765,6 @@ 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;
    }

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

--- a/test/inspector/console/scoped-variables-expected.txt
+++ b/test/inspector/console/scoped-variables-expected.txt
@@ -5,7 +5,6 @@ Evaluating 'let a = 42;'
 }
 Evaluating 'a'
 {
-    description : 42
    type : number
    value : 42
 }
@@ -35,7 +34,6 @@ Evaluating 'let a = 239;'
 }
 Evaluating 'a'
 {
-    description : 42
    type : number
    value : 42
 }
--- a/test/inspector/cpu-profiler/coverage-block-expected.txt
+++ b/test/inspector/cpu-profiler/coverage-block-expected.txt
@@ -21,7 +21,6 @@ Running test: testPreciseCountCoverage
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -134,7 +133,6 @@ Running test: testPreciseCountCoverageIncremental
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -230,7 +228,6 @@ Running test: testPreciseCountCoverageIncremental
    id : <messageId>
    result : {
        result : {
-            description : 10946
            type : number
            value : 10946
        }
@@ -345,7 +342,6 @@ Running test: testPreciseCoverageFail
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -364,7 +360,6 @@ Running test: testBestEffortCoverage
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -390,7 +385,6 @@ Running test: testBestEffortCoverageWithPreciseBinaryEnabled
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -550,7 +544,6 @@ Running test: testBestEffortCoverageWithPreciseCountEnabled
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -744,7 +737,6 @@ Running test: testPreciseBinaryCoverage
    id : <messageId>
    result : {
        result : {
-            description : 8
            type : number
            value : 8
        }
@@ -830,7 +822,6 @@ Running test: testPreciseBinaryCoverage
    id : <messageId>
    result : {
        result : {
-            description : 10946
            type : number
            value : 10946
        }

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

--- a/test/inspector/debugger/change-return-value-expected.txt
+++ b/test/inspector/debugger/change-return-value-expected.txt
@@ -16,7 +16,6 @@ Update return value to 42..
 Dump actual return value
 {
    result : {
-        description : 42
        type : number
        value : 42
    }
@@ -28,7 +27,6 @@ Update return value to 239..
 Dump actual return value
 {
    result : {
-        description : 239
        type : number
        value : 239
    }

--- a/test/inspector/debugger/eval-scopes-expected.txt
+++ b/test/inspector/debugger/eval-scopes-expected.txt
@@ -9,7 +9,6 @@ Tests that variables introduced in eval scopes are accessible
                isOwn : true
                name : hest
                value : {
-                    description : 420
                    type : number
                    value : 420
                }

--- a/test/inspector/debugger/eval-without-codegen-expected.txt
+++ b/test/inspector/debugger/eval-without-codegen-expected.txt
@@ -36,7 +36,6 @@ Running test: testCallFunctionOn
    id : <messageId>
    result : {
        result : {
-            description : 3
            type : number
            value : 3
        }

--- a/test/inspector/debugger/evaluate-on-call-frame-in-module-expected.txt
+++ b/test/inspector/debugger/evaluate-on-call-frame-in-module-expected.txt
@@ -33,56 +33,48 @@ 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
 }
@@ -123,28 +115,24 @@ 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
 }
@@ -158,14 +146,12 @@ foo1 =
 }
 b2 = 
 {
-    description : 21
    type : number
    value : 21
 }
 Evaluating: ++b2
 updated b2 = 
 {
-    description : 22
    type : number
    value : 22
 }
@@ -207,14 +193,12 @@ foo2 =
 }
 b3 = 
 {
-    description : 31
    type : number
    value : 31
 }
 Evaluating: ++b3
 updated b3 = 
 {
-    description : 32
    type : number
    value : 32
 }
@@ -249,14 +233,12 @@ Array =
 }
 a = 
 {
-    description : 0
    type : number
    value : 0
 }
 Evaluating: ++a
 updated a = 
 {
-    description : 1
    type : number
    value : 1
 }
@@ -283,14 +265,12 @@ Array =
 }
 a = 
 {
-    description : 0
    type : number
    value : 0
 }
 Evaluating: ++a
 updated a = 
 {
-    description : 1
    type : number
    value : 1
 }
@@ -338,56 +318,48 @@ 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
 }
@@ -422,28 +394,24 @@ 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
 }
@@ -470,14 +438,12 @@ Array =
 }
 x = 
 {
-    description : 5
    type : number
    value : 5
 }
 Evaluating: ++x
 updated x = 
 {
-    description : 6
    type : number
    value : 6
 }

--- a/test/inspector/debugger/evaluate-on-call-frame-in-module.js
+++ b/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.name));
+       (v.name + ' = ' + (v.value.description || v.value.value + '')) : (v.name));
    InspectorTest.logMessage(result);
  }

--- a/test/inspector/debugger/scope-skip-variables-with-empty-name-expected.txt
+++ b/test/inspector/debugger/scope-skip-variables-with-empty-name-expected.txt
@@ -9,7 +9,6 @@ Tests that scopes do not report variables with empty names
                isOwn : true
                name : a
                value : {
-                    description : 2
                    type : number
                    value : 2
                }

--- a/test/inspector/debugger/set-script-source-expected.txt
+++ b/test/inspector/debugger/set-script-source-expected.txt
 Tests Debugger.setScriptSource
-Function evaluate: {"type":"number","value":6,"description":"6"}
+Function evaluate: {"type":"number","value":6}
 PASS, result value: 6
-Function evaluate: {"type":"number","value":8,"description":"8"}
+Function evaluate: {"type":"number","value":8}
 PASS, result value: 8
 Has error reported: PASS
 Reported error is a compile error: PASS

--- a/test/inspector/debugger/suspended-generator-scopes-expected.txt
+++ b/test/inspector/debugger/suspended-generator-scopes-expected.txt
@@ -8,7 +8,6 @@ Running test: testScopesPaused
        isOwn : true
        name : a
        value : {
-            description : 420
            type : number
            value : 420
        }
@@ -20,7 +19,6 @@ Running test: testScopesPaused
        isOwn : true
        name : b
        value : {
-            description : 42
            type : number
            value : 42
        }
@@ -36,7 +34,6 @@ Running test: testScopesNonPaused
        isOwn : true
        name : a
        value : {
-            description : 430
            type : number
            value : 430
        }

--- a/test/inspector/debugger/wasm-scope-info.js
+++ b/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.description + ' (' + elem.value.type + ')';
+        ': ' + elem.value.value + ' (' + elem.value.type + ')';
    return msg.result.result.map(printProperty).join(', ');
  }
  return JSON.stringify(value.value) + ' (' + value.type + ')';

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

--- a/test/inspector/runtime/call-function-on-async-expected.txt
+++ b/test/inspector/runtime/call-function-on-async-expected.txt
@@ -114,7 +114,6 @@ Running test: testFunctionReturnNotPromise
    id : <messageId>
    result : {
        result : {
-            description : 239
            type : number
            value : 239
        }
@@ -189,7 +188,6 @@ Running test: testEvaluateOnExecutionContext
    id : <messageId>
    result : {
        result : {
-            description : 70
            type : number
            value : 70
        }

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

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

--- a/test/inspector/runtime/console-assert-expected.txt
+++ b/test/inspector/runtime/console-assert-expected.txt
@@ -36,7 +36,6 @@ Checks that console.assert works and points to correct call frame.
    params : {
        args : [
            [0] : {
-                description : 1
                type : number
                value : 1
            }
@@ -69,12 +68,10 @@ 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
            }

--- a/test/inspector/runtime/console-line-and-column-expected.txt
+++ b/test/inspector/runtime/console-line-and-column-expected.txt
@@ -4,7 +4,6 @@ Tests line and column numbers in console messages
    params : {
        args : [
            [0] : {
-                description : 239
                type : number
                value : 239
            }
@@ -30,7 +29,6 @@ Tests line and column numbers in console messages
    params : {
        args : [
            [0] : {
-                description : 239
                type : number
                value : 239
            }

--- a/test/inspector/runtime/console-log-doesnt-run-microtasks-expected.txt
+++ b/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
 }

--- a/test/inspector/runtime/console-memory-expected.txt
+++ b/test/inspector/runtime/console-memory-expected.txt
@@ -25,7 +25,6 @@ Running test: testWithMemory
    id : <messageId>
    result : {
        result : {
-            description : 239
            type : number
            value : 239
        }
@@ -37,7 +36,6 @@ Running test: testSetterDoesntOverride
    id : <messageId>
    result : {
        result : {
-            description : 42
            type : number
            value : 42
        }

--- a/test/inspector/runtime/es6-module-expected.txt
+++ b/test/inspector/runtime/es6-module-expected.txt
@@ -161,7 +161,6 @@ console.log(239)
                isOwn : true
                name : a1
                value : {
-                    description : 1
                    type : number
                    value : 1
                }
@@ -173,7 +172,6 @@ console.log(239)
                isOwn : true
                name : a2
                value : {
-                    description : 2
                    type : number
                    value : 2
                }

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

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

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

--- a/test/inspector/runtime/evaluate-without-side-effects-expected.txt
+++ b/test/inspector/runtime/evaluate-without-side-effects-expected.txt
@@ -4,7 +4,6 @@ Test throwOnSideEffect: false
    id : <messageId>
    result : {
        result : {
-            description : 2
            type : number
            value : 2
        }
@@ -42,7 +41,6 @@ Test expression without side-effect, with throwOnSideEffect: true
    id : <messageId>
    result : {
        result : {
-            description : 4
            type : number
            value : 4
        }
@@ -54,7 +52,6 @@ paused
    id : <messageId>
    result : {
        result : {
-            description : 2
            type : number
            value : 2
        }
@@ -65,7 +62,6 @@ Test that debug break does not trigger with throwOnSideEffect
    id : <messageId>
    result : {
        result : {
-            description : 2
            type : number
            value : 2
        }

--- a/test/inspector/runtime/exception-thrown-expected.txt
+++ b/test/inspector/runtime/exception-thrown-expected.txt
@@ -88,7 +88,6 @@ Check that exceptionThrown is supported by test runner.
        exceptionDetails : {
            columnNumber : 2
            exception : {
-                description : 239
                type : number
                value : 239
            }

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

--- a/test/inspector/runtime/get-properties-on-proxy-expected.txt
+++ b/test/inspector/runtime/get-properties-on-proxy-expected.txt
@@ -176,7 +176,6 @@ Checking counter
    id : <messageId>
    result : {
        result : {
-            description : 0
            type : number
            value : 0
        }

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

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

--- a/test/inspector/runtime/property-on-console-proto-expected.txt
+++ b/test/inspector/runtime/property-on-console-proto-expected.txt
@@ -3,7 +3,6 @@ Tests that property defined on console.__proto__ doesn't observable on other Obj
    id : <messageId>
    result : {
        result : {
-            description : 0
            type : number
            value : 0
        }

--- a/test/inspector/runtime/protocol-works-with-different-locale-expected.txt
+++ b/test/inspector/runtime/protocol-works-with-different-locale-expected.txt
@@ -6,7 +6,6 @@ Running test: consoleLogWithDefaultLocale
    params : {
        args : [
            [0] : {
-                description : 239
                type : number
                value : 239
            }
@@ -63,7 +62,6 @@ set locale to fr_CA.UTF-8 (has comma as separator)
    params : {
        args : [
            [0] : {
-                description : 239
                type : number
                value : 239
            }
@@ -92,7 +90,6 @@ set locale to fr_CA.UTF-8 (has comma as separator)
    params : {
        args : [
            [0] : {
-                description : 239
                type : number
                value : 239
            }

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

--- a/test/inspector/runtime/runtime-evaluate-with-dirty-context-expected.txt
+++ b/test/inspector/runtime/runtime-evaluate-with-dirty-context-expected.txt
@@ -4,7 +4,6 @@ Checks that Runtime.evaluate works with dirty context.
    params : {
        args : [
            [0] : {
-                description : 42
                type : number
                value : 42
            }

--- a/test/inspector/runtime/runtime-restore-expected.txt
+++ b/test/inspector/runtime/runtime-restore-expected.txt
@@ -35,7 +35,6 @@ will reconnect..
    params : {
        args : [
            [0] : {
-                description : 42
                type : number
                value : 42
            }

--- a/test/inspector/runtime/terminate-execution-expected.txt
+++ b/test/inspector/runtime/terminate-execution-expected.txt
@@ -14,7 +14,6 @@ Terminate first evaluation (it forces injected-script-source compilation)
 }
 Checks that we reset termination after evaluation
 {
-    description : 42
    type : number
    value : 42
 }

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

--- a/test/inspector/sessions/runtime-console-api-called-expected.txt
+++ b/test/inspector/sessions/runtime-console-api-called-expected.txt
@@ -6,7 +6,6 @@ From session 1
    params : {
        args : [
            [0] : {
-                description : 1
                type : number
                value : 1
            }
@@ -33,7 +32,6 @@ From session 2
    params : {
        args : [
            [0] : {
-                description : 1
                type : number
                value : 1
            }
@@ -61,7 +59,6 @@ From session 1
    params : {
        args : [
            [0] : {
-                description : 2
                type : number
                value : 2
            }
@@ -88,7 +85,6 @@ From session 2
    params : {
        args : [
            [0] : {
-                description : 2
                type : number
                value : 2
            }

--- a/test/inspector/sessions/runtime-evaluate-expected.txt
+++ b/test/inspector/sessions/runtime-evaluate-expected.txt
@@ -5,7 +5,6 @@ Evaluating in 2
    id : <messageId>
    result : {
        result : {
-            description : 42
            type : number
            value : 42
        }

--- a/test/inspector/sessions/runtime-remote-object-expected.txt
+++ b/test/inspector/sessions/runtime-remote-object-expected.txt
@@ -12,7 +12,6 @@ Retrieving properties in 2
                isOwn : true
                name : a
                value : {
-                    description : 17
                    type : number
                    value : 17
                }
@@ -45,7 +44,6 @@ Retrieving properties in 1
                isOwn : true
                name : a
                value : {
-                    description : 42
                    type : number
                    value : 42
                }
@@ -79,7 +77,6 @@ Retrieving properties in 1
                isOwn : true
                name : a
                value : {
-                    description : 42
                    type : number
                    value : 42
                }