• 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
..
api Loading commit data...
asmjs Loading commit data...
ast Loading commit data...
base Loading commit data...
builtins Loading commit data...
codegen Loading commit data...
common Loading commit data...
compiler Loading commit data...
compiler-dispatcher Loading commit data...
d8 Loading commit data...
date Loading commit data...
debug Loading commit data...
deoptimizer Loading commit data...
diagnostics Loading commit data...
execution Loading commit data...
extensions Loading commit data...
flags Loading commit data...
handles Loading commit data...
heap Loading commit data...
ic Loading commit data...
init Loading commit data...
inspector Loading commit data...
interpreter Loading commit data...
json Loading commit data...
libplatform Loading commit data...
libsampler Loading commit data...
logging Loading commit data...
numbers Loading commit data...
objects Loading commit data...
parsing Loading commit data...
profiler Loading commit data...
protobuf Loading commit data...
regexp Loading commit data...
roots Loading commit data...
runtime Loading commit data...
sanitizer Loading commit data...
snapshot Loading commit data...
strings Loading commit data...
tasks Loading commit data...
third_party Loading commit data...
torque Loading commit data...
tracing Loading commit data...
trap-handler Loading commit data...
utils Loading commit data...
wasm Loading commit data...
zone Loading commit data...
DEPS Loading commit data...
OWNERS Loading commit data...