• Sathya Gunasekaran's avatar
    [fni] Mark computed props as computed, not anonymous function · ab24897c
    Sathya Gunasekaran authored
    I thought about potentially adding the identifer ref to the error but
    that would require allocating a new string or at the very least
    increasing the size of the resulting cons string. Given that the
    parser is pretty performance sensitive, I've decided to not display
    the identifier.
    
    Previously, the error was:
      _test.js:3: Error
      a[foo].c = () => { throw Error(); };
                         ^
      Error
        at a.(anonymous function).c (_test.js:3:26)
        at _test.js:5:1
    
    With this patch, the error becomes:
      _test.js:3: Error
      a[foo].c = () => { throw Error(); };
                         ^
      Error
        at a.<computed>.c (_test.js:3:26)
        at _test.js:5:1
    
    Bug: v8:8823
    Change-Id: I557b3517e317652c447ca06c5a400e9625353d9b
    Reviewed-on: https://chromium-review.googlesource.com/c/1495017
    Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org>
    Reviewed-by: 's avatarMathias Bynens <mathias@chromium.org>
    Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#59985}
    ab24897c
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...