Commit 2cae49cc authored by Victor Gomes's avatar Victor Gomes Committed by Commit Bot

[builtins] Update arguments size DCHECK in CPP builtins

Change-Id: I468d64df5d1a06a395249d16c8974d3dec85fe7b
Bug: chromium:1138197, v8:10201
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2470570
Commit-Queue: Victor Gomes <victorgomes@chromium.org>
Commit-Queue: Leszek Swirski <leszeks@chromium.org>
Auto-Submit: Victor Gomes <victorgomes@chromium.org>
Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
Cr-Commit-Position: refs/heads/master@{#70505}
parent 19076e1a
...@@ -38,7 +38,7 @@ BUILTIN(ObjectPrototypePropertyIsEnumerable) { ...@@ -38,7 +38,7 @@ BUILTIN(ObjectPrototypePropertyIsEnumerable) {
// ES6 section 19.1.2.3 Object.defineProperties // ES6 section 19.1.2.3 Object.defineProperties
BUILTIN(ObjectDefineProperties) { BUILTIN(ObjectDefineProperties) {
HandleScope scope(isolate); HandleScope scope(isolate);
DCHECK_EQ(3, args.length()); DCHECK_LE(3, args.length());
Handle<Object> target = args.at(1); Handle<Object> target = args.at(1);
Handle<Object> properties = args.at(2); Handle<Object> properties = args.at(2);
...@@ -49,7 +49,7 @@ BUILTIN(ObjectDefineProperties) { ...@@ -49,7 +49,7 @@ BUILTIN(ObjectDefineProperties) {
// ES6 section 19.1.2.4 Object.defineProperty // ES6 section 19.1.2.4 Object.defineProperty
BUILTIN(ObjectDefineProperty) { BUILTIN(ObjectDefineProperty) {
HandleScope scope(isolate); HandleScope scope(isolate);
DCHECK_EQ(4, args.length()); DCHECK_LE(4, args.length());
Handle<Object> target = args.at(1); Handle<Object> target = args.at(1);
Handle<Object> key = args.at(2); Handle<Object> key = args.at(2);
Handle<Object> attributes = args.at(3); Handle<Object> attributes = args.at(3);
......
...@@ -19,7 +19,7 @@ namespace internal { ...@@ -19,7 +19,7 @@ namespace internal {
// ES6 section 26.1.3 Reflect.defineProperty // ES6 section 26.1.3 Reflect.defineProperty
BUILTIN(ReflectDefineProperty) { BUILTIN(ReflectDefineProperty) {
HandleScope scope(isolate); HandleScope scope(isolate);
DCHECK_EQ(4, args.length()); DCHECK_LE(4, args.length());
Handle<Object> target = args.at(1); Handle<Object> target = args.at(1);
Handle<Object> key = args.at(2); Handle<Object> key = args.at(2);
Handle<Object> attributes = args.at(3); Handle<Object> attributes = args.at(3);
...@@ -49,7 +49,7 @@ BUILTIN(ReflectDefineProperty) { ...@@ -49,7 +49,7 @@ BUILTIN(ReflectDefineProperty) {
// ES6 section 26.1.7 Reflect.getOwnPropertyDescriptor // ES6 section 26.1.7 Reflect.getOwnPropertyDescriptor
BUILTIN(ReflectGetOwnPropertyDescriptor) { BUILTIN(ReflectGetOwnPropertyDescriptor) {
HandleScope scope(isolate); HandleScope scope(isolate);
DCHECK_EQ(3, args.length()); DCHECK_LE(3, args.length());
Handle<Object> target = args.at(1); Handle<Object> target = args.at(1);
Handle<Object> key = args.at(2); Handle<Object> key = args.at(2);
...@@ -75,7 +75,7 @@ BUILTIN(ReflectGetOwnPropertyDescriptor) { ...@@ -75,7 +75,7 @@ BUILTIN(ReflectGetOwnPropertyDescriptor) {
// ES6 section 26.1.11 Reflect.ownKeys // ES6 section 26.1.11 Reflect.ownKeys
BUILTIN(ReflectOwnKeys) { BUILTIN(ReflectOwnKeys) {
HandleScope scope(isolate); HandleScope scope(isolate);
DCHECK_EQ(2, args.length()); DCHECK_LE(2, args.length());
Handle<Object> target = args.at(1); Handle<Object> target = args.at(1);
if (!target->IsJSReceiver()) { if (!target->IsJSReceiver()) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment