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

Unified Diff: test/mjsunit/harmony/debug-stepin-proxies.js

Issue 1717893002: Remove the Proxy enumerate trap (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix tests Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/mjsunit/for-in-opt.js ('k') | test/mjsunit/harmony/proxies-enumerate.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/harmony/debug-stepin-proxies.js
diff --git a/test/mjsunit/harmony/debug-stepin-proxies.js b/test/mjsunit/harmony/debug-stepin-proxies.js
index 0689801a4f64b44040849f1fa79d2d20dac85d6f..8595f404f00d8fd5f76488140a9a8d6ab2134e4a 100644
--- a/test/mjsunit/harmony/debug-stepin-proxies.js
+++ b/test/mjsunit/harmony/debug-stepin-proxies.js
@@ -36,13 +36,6 @@ var handler = {
set: function(target, name, value) {
return false; // l
}, // m
- enumerate: function(target) {
- function* keys() { // n
- yield "foo"; // o
- yield "bar"; // p
- } // q
- return keys(); // r
- }, // s
}
var proxy = new Proxy(target, handler);
@@ -52,9 +45,6 @@ debugger; // a
var has = "step" in proxy; // b
var get = proxy.step; // c
proxy.step = 43; // d
-for (var i in proxy) { // e
- log.push(i); // f
-}
Debug.setListener(null); // g
@@ -67,12 +57,5 @@ assertEquals([
"b0", "h4b20", "i2b20", // [[Has]]
"c0", "j4c15", "k2c15", // [[Get]]
"d0", "l4d11", "m2d11", // [[Set]]
- "e14", "r4e14", "q4r11e14", "s2e14", // for-in [[Enumerate]]
- "o6e14", "q4e14", "p6e14", "q4e14", "q4e14", // exhaust iterator
- "e9", // for-in-body
- "h4e9","i2e9", // [[Has]] property
- "f2","foo", "e9", // for-in-body
- "h4e9","i2e9", // [[Has]]property
- "f2","bar", "e9", // for-in-body
"g0"
], log);
« no previous file with comments | « test/mjsunit/for-in-opt.js ('k') | test/mjsunit/harmony/proxies-enumerate.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698