Title: [197417] trunk/LayoutTests
Revision
197417
Author
[email protected]
Date
2016-03-01 15:40:46 -0800 (Tue, 01 Mar 2016)

Log Message

FTL should simplify StringReplace with an empty replacement string
https://bugs.webkit.org/show_bug.cgi?id=154871

Reviewed by Michael Saboff.

Really add this new test.

* js/regress/script-tests/string-replace-empty.js: Added.
* js/regress/string-replace-empty-expected.txt: Added.
* js/regress/string-replace-empty.html: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (197416 => 197417)


--- trunk/LayoutTests/ChangeLog	2016-03-01 23:35:05 UTC (rev 197416)
+++ trunk/LayoutTests/ChangeLog	2016-03-01 23:40:46 UTC (rev 197417)
@@ -1,3 +1,16 @@
+2016-03-01  Filip Pizlo  <[email protected]>
+
+        FTL should simplify StringReplace with an empty replacement string
+        https://bugs.webkit.org/show_bug.cgi?id=154871
+
+        Reviewed by Michael Saboff.
+
+        Really add this new test.
+
+        * js/regress/script-tests/string-replace-empty.js: Added.
+        * js/regress/string-replace-empty-expected.txt: Added.
+        * js/regress/string-replace-empty.html: Added.
+
 2016-02-29  Filip Pizlo  <[email protected]>
 
         Turn String.prototype.replace into an intrinsic

Added: trunk/LayoutTests/js/regress/script-tests/string-replace-empty.js (0 => 197417)


--- trunk/LayoutTests/js/regress/script-tests/string-replace-empty.js	                        (rev 0)
+++ trunk/LayoutTests/js/regress/script-tests/string-replace-empty.js	2016-03-01 23:40:46 UTC (rev 197417)
@@ -0,0 +1,8 @@
+(function() {
+    var result;
+    for (var i = 0; i < 400000; ++i) {
+        result = "foo".replace(/f/g, "");
+    }
+    if (result != "oo")
+        throw "Error: bad result: "+ result;
+})();

Added: trunk/LayoutTests/js/regress/string-replace-empty-expected.txt (0 => 197417)


--- trunk/LayoutTests/js/regress/string-replace-empty-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/js/regress/string-replace-empty-expected.txt	2016-03-01 23:40:46 UTC (rev 197417)
@@ -0,0 +1,10 @@
+JSRegress/string-replace-empty
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS no exception thrown
+PASS successfullyParsed is true
+
+TEST COMPLETE
+

Added: trunk/LayoutTests/js/regress/string-replace-empty.html (0 => 197417)


--- trunk/LayoutTests/js/regress/string-replace-empty.html	                        (rev 0)
+++ trunk/LayoutTests/js/regress/string-replace-empty.html	2016-03-01 23:40:46 UTC (rev 197417)
@@ -0,0 +1,12 @@
+<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
+<html>
+<head>
+<script src=""
+</head>
+<body>
+<script src=""
+<script src=""
+<script src=""
+<script src=""
+</body>
+</html>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to