From 04db7c8793d50bc64125a31539a458153f14f8aa Mon Sep 17 00:00:00 2001 From: "adamk@chromium.org" Date: Wed, 22 Oct 2014 17:21:53 +0000 Subject: [PATCH] Fix {get,set}ter-on-elements tests to run through all creation functions setter-on-elements had the wrong length hardcoded in a for loop over the creation functions (getter-on-elements had the right length, but seemed worth future-proofing). R=mvstanton@chromium.org Review URL: https://codereview.chromium.org/643143005 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24813 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- test/mjsunit/getters-on-elements.js | 2 +- test/mjsunit/setters-on-elements.js | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/mjsunit/getters-on-elements.js b/test/mjsunit/getters-on-elements.js index 3bc360f..7f2c98b 100644 --- a/test/mjsunit/getters-on-elements.js +++ b/test/mjsunit/getters-on-elements.js @@ -176,7 +176,7 @@ var cf = [create_func_smi, create_func_double, create_func_fast]; -for(var c = 0; c < 3; c++) { +for(var c = 0; c < cf.length; c++) { base_getter_test(cf[c]); } diff --git a/test/mjsunit/setters-on-elements.js b/test/mjsunit/setters-on-elements.js index dd3fabf..001906c 100644 --- a/test/mjsunit/setters-on-elements.js +++ b/test/mjsunit/setters-on-elements.js @@ -191,8 +191,8 @@ var cf = [create_func_smi, var values = [3, 3.5, true]; -for(var c = 0; c < 3; c++) { - for(var s = 0; s < 3; s++) { +for(var c = 0; c < cf.length; c++) { + for(var s = 0; s < values.length; s++) { base_setter_test(cf[c], 0, values[s]); base_setter_test(cf[c], 1, values[s]); } -- 2.7.4