From 5c264ade8ee3f129cab7ddc4e4fccc8520b01d1d Mon Sep 17 00:00:00 2001 From: "yangguo@chromium.org" Date: Wed, 27 Feb 2013 15:36:02 +0000 Subject: [PATCH] Fix wrong test in r13766 (Insert conversion to string in string.replace). R=ulan@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/12315130 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13770 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- test/mjsunit/string-replace.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/mjsunit/string-replace.js b/test/mjsunit/string-replace.js index d9a274d..de92115 100644 --- a/test/mjsunit/string-replace.js +++ b/test/mjsunit/string-replace.js @@ -225,7 +225,7 @@ var side_effect_flag = false; var replace_obj_side_effects = { toString: function() { side_effect_flag = true; return "x" } } -assertEquals("abc", "abc".replace(/z/g, function() { return "x"; })); +assertEquals("abc", "abc".replace(/z/g, replace_obj_side_effects)); assertTrue(side_effect_flag); // Side effect triggers even without a match. var regexp99pattern = ""; -- 2.7.4