AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[debugger] be more lenient about argument type in %DebugGetProperty.

Browse Source 
R=mstarzinger@chromium.org
BUG=chromium:629996

Review-Url: https://codereview.chromium.org/2166123003
Cr-Commit-Position: refs/heads/master@{#37942}
This commit is contained in:
yangguo 2016-07-21 04:27:52 -07:00 committed by Commit bot
parent f04fd635b6
commit 07a0f9ebff
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/runtime/runtime-debug.cc
View File

@ -408,7 +408,7 @@ RUNTIME_FUNCTION(Runtime_DebugGetProperty) {
DCHECK(args.length() == 2);
CONVERT_ARG_HANDLE_CHECKED(JSObject, obj, 0);
CONVERT_ARG_HANDLE_CHECKED(Object, obj, 0);
CONVERT_ARG_HANDLE_CHECKED(Name, name, 1);
LookupIterator it(obj, name);

9
test/mjsunit/regress/regress-crbug-629996.js Normal file
View File

@ -0,0 +1,9 @@
// Copyright 2016 the V8 project authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// // Flags: --expose-debug-as debug
var mirror = debug.MakeMirror(new Proxy({}, {}));
// As long as we have no special mirror for proxies, we use an object mirror.
assertEquals("object", mirror.type());