• Joyee Cheung's avatar
    [class] fix undefined private name access in computed property keys · 7fa12e2a
    Joyee Cheung authored
    This patch implements https://github.com/tc39/proposal-class-fields/pull/269
    and makes sure we always throw TypeError when there is invalid private
    name access in computed property keys.
    
    Before this patch, private name variables of private fields and methods
    are initialized together with computed property keys in the order they
    are declared. Accessing undefined private names in the computed property
    keys thus fail silently.
    
    After this patch, we initialize the private name variables of private
    fields before we initialize the computed property keys, so that invalid
    access to private fields in the computed keys can be checked in the IC.
    We now also initialize the brand early, so that invalid access to private
    methods or accessors in the computed keys throw TypeError during brand
    checks - and since these accesses are guarded by brand checks, we can
    create the private methods and accessors after the class is
    defined, and merge the home object setting with the creation
    of the closures.
    
    Bug: v8:8330, v8:9611
    Change-Id: I01363f7befac6cf9dd28ec229b99a99102bcf012
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1846571
    Commit-Queue: Joyee Cheung <joyee@igalia.com>
    Reviewed-by: 's avatarMythri Alle <mythria@chromium.org>
    Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#64225}
    7fa12e2a
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-api-tests 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...