Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Side by Side Diff: test/mjsunit/es6/promise-lookup-getter-setter.js

Issue 2592013003: Align __lookupGetter__/__lookupSetter__ behavior with the spec (Closed)
Patch Set: Add test Created 3 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/builtins/builtins-object.cc ('k') | test/test262/test262.status » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2016 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 let get = () => {};
6 let set = () => {};
7 let target = {};
8 let handler = {
9 getOwnPropertyDescriptor(target, prop) {
10 let configurable = true;
11 if (prop == "both") {
12 return { get, set, configurable };
13 } else if (prop == "get") {
14 return { get, configurable };
15 } else if (prop == "set") {
16 return { set, configurable };
17 } else if (prop == "data") {
18 return { value: 42, configurable };
19 } else {
20 return Reflect.getOwnPropertyDescriptor(target, prop);
21 }
22 }
23 };
24
25 // Test behavior on own properties.
26 let proxy = new Proxy(target, handler);
27 assertSame(get, proxy.__lookupGetter__("both"));
28 assertSame(get, proxy.__lookupGetter__("get"));
29 assertSame(undefined, proxy.__lookupGetter__("set"));
30 assertSame(undefined, proxy.__lookupGetter__("data"));
31 assertSame(set, proxy.__lookupSetter__("both"));
32 assertSame(undefined, proxy.__lookupSetter__("get"));
33 assertSame(set, proxy.__lookupSetter__("set"));
34 assertSame(undefined, proxy.__lookupSetter__("data"));
35
36 // Test behavior on the prototype chain.
37 let object = { __proto__: proxy };
38 assertSame(get, object.__lookupGetter__("both"));
39 assertSame(get, object.__lookupGetter__("get"));
40 assertSame(undefined, object.__lookupGetter__("set"));
41 assertSame(undefined, object.__lookupGetter__("data"));
42 assertSame(set, object.__lookupSetter__("both"));
43 assertSame(undefined, object.__lookupSetter__("get"));
44 assertSame(set, object.__lookupSetter__("set"));
45 assertSame(undefined, object.__lookupSetter__("data"));
46
47 // Test being shadowed while on prototype chain.
48 let shadower = { __proto__: proxy, both: 1, get: 2, set: 3, data: 4 };
49 assertSame(undefined, shadower.__lookupGetter__("both"));
50 assertSame(undefined, shadower.__lookupGetter__("get"));
51 assertSame(undefined, shadower.__lookupGetter__("set"));
52 assertSame(undefined, shadower.__lookupGetter__("data"));
53 assertSame(undefined, shadower.__lookupSetter__("both"));
54 assertSame(undefined, shadower.__lookupSetter__("get"));
55 assertSame(undefined, shadower.__lookupSetter__("set"));
56 assertSame(undefined, shadower.__lookupSetter__("data"));
57
58 // Test getPrototypeOf trap.
59 let getFoo = () => {};
60 let setFoo = () => {};
61 let proto = {};
62 Reflect.defineProperty(proto, "foo", { get: getFoo, set: setFoo });
63 Reflect.setPrototypeOf(target, proto);
64 assertSame(getFoo, proxy.__lookupGetter__("foo"));
65 assertSame(setFoo, proxy.__lookupSetter__("foo"));
66 handler.getPrototypeOf = () => null;
67 assertSame(undefined, proxy.__lookupGetter__("foo"));
68 assertSame(undefined, proxy.__lookupSetter__("foo"));
69 handler.getPrototypeOf = () => proto;
70 assertSame(getFoo, proxy.__lookupGetter__("foo"));
71 assertSame(setFoo, proxy.__lookupSetter__("foo"));
72
73 // Test shadowing the prototype.
74 Reflect.defineProperty(proto, "data", { get: getFoo, set: setFoo });
75 assertSame(undefined, proxy.__lookupGetter__("data"));
76 assertSame(undefined, proxy.__lookupSetter__("data"));
OLDNEW
« no previous file with comments | « src/builtins/builtins-object.cc ('k') | test/test262/test262.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698