• vabr's avatar
    Fix TypeError message for Reflect.construct · b478e9c1
    vabr authored
    If the Reflect.construct receives an argument expected to be a constructor,
    and the argument is not a constructor, V8 currently declares that
    Reflect.construct is not a function. It should instead say that the offending
    argument is not a constructor.
    
    This is the case for all ports of builtins
    (Builtins::Generate_ReflectConstruct). All of them make an
    attempt to at least pass the right argument to the TypeError parametrised
    message, calling out the offending Reflect.construct argument. However,
    Runtime::kThrowCalledNonCallable extracts the callsite from those arguments,
    discarding the precise information.
    
    This CL adds Runtime::kNotConstructor, which reports the arguments passed
    to it, and the CL also modifies the ports of builtins to make use of
    Runtime::kNotConstructor
    
    BUG=v8:5671
    
    Review-Url: https://codereview.chromium.org/2688393003
    Cr-Commit-Position: refs/heads/master@{#43182}
    b478e9c1
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...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...