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

Unified Diff: test/mjsunit/harmony/reflect-enumerate-opt.js

Issue 1417063011: [runtime] support new Proxy() instead of Proxy.create and install getPrototypeOf trap (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: removing unreachable code Created 5 years, 1 month 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/harmony/proxy/proxy-getPrototypeOf.js ('k') | test/mjsunit/harmony/regress/regress-2219.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/harmony/reflect-enumerate-opt.js
diff --git a/test/mjsunit/harmony/reflect-enumerate-opt.js b/test/mjsunit/harmony/reflect-enumerate-opt.js
index 34cd660c8ff0b93497e4fb322bd6de09e0b0096d..ccd1845c78ebf940f295eb293ed88f718552df9a 100644
--- a/test/mjsunit/harmony/reflect-enumerate-opt.js
+++ b/test/mjsunit/harmony/reflect-enumerate-opt.js
@@ -47,31 +47,32 @@ var handler = {
}
};
-
-var proxy = Proxy.create(handler);
-var o = {__proto__: proxy};
-
-function f2(o) {
- var result = [];
- for (var i of Reflect.enumerate(o)) {
- result.push(i);
- }
- return result;
-}
-
-function check_f2() {
- assertEquals(keys, f2(o));
- assertEquals(keys, has_keys);
- has_keys.length = 0;
-}
-
-check_f2();
-check_f2();
+// TODO(neis,cbruni): Enable once the enumerate proxy trap is properly
+// implemented.
+// var proxy = new Proxy({}, handler);
+// var o = {__proto__: proxy};
+//
+// function f2(o) {
+// var result = [];
+// for (var i of Reflect.enumerate(o)) {
+// result.push(i);
+// }
+// return result;
+// }
+//
+// function check_f2() {
+// assertEquals(keys, f2(o));
+// assertEquals(keys, has_keys);
+// has_keys.length = 0;
+// }
+//
+// check_f2();
+// check_f2();
// Test lazy deopt after GetPropertyNamesFast
-%OptimizeFunctionOnNextCall(f2);
-deopt_enum = true;
-check_f2();
+// %OptimizeFunctionOnNextCall(f2);
+// deopt_enum = true;
+// check_f2();
// Test lazy deopt after FILTER_KEY
-%OptimizeFunctionOnNextCall(f2);
-deopt_has = true;
-check_f2();
+// %OptimizeFunctionOnNextCall(f2);
+// deopt_has = true;
+// check_f2();
« no previous file with comments | « test/mjsunit/harmony/proxy/proxy-getPrototypeOf.js ('k') | test/mjsunit/harmony/regress/regress-2219.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698