Changeset 95504 in webkit for trunk/Source/JavaScriptCore/ChangeLog

Timestamp:

Sep 19, 2011, 6:41:42 PM (14 years ago)

Author:

[email protected]

Message:

String#split is buggy
​https://bugs.webkit.org/show_bug.cgi?id=68348

Reviewed by Sam Weinig.

Source/JavaScriptCore:

• runtime/StringPrototype.cpp:

(JSC::jsStringWithReuse):

• added helper function to reuse original JSString value.

(JSC::stringProtoFuncSplit):

• Rewritten from the spec.

• tests/mozilla/ecma/String/15.5.4.8-2.js:

(getTestCases):

• This test is not ES5 compliant.

LayoutTests:

• fast/js/script-tests/string-split-conformance.js: Added.
• fast/js/string-split-conformance-expected.txt: Added.
• fast/js/string-split-conformance.html: Added.
  • Added new Layout test based on:

​http://stevenlevithan.com/demo/split.cfm

• sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.14_String.prototype.split/S15.5.4.14_A1_T6-expected.txt:
• sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.14_String.prototype.split/S15.5.4.14_A1_T7-expected.txt:
• sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.14_String.prototype.split/S15.5.4.14_A1_T8-expected.txt:
• sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.14_String.prototype.split/S15.5.4.14_A1_T9-expected.txt:
• sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.14_String.prototype.split/S15.5.4.14_A2_T7-expected.txt:
  • Check in failing results for these 5 tests; they are all wrong (see ​https://bugs.ecmascript.org/show_bug.cgi?id=61).

File:

• trunk/Source/JavaScriptCore/ChangeLog (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2011-09-19  Gavin Barraclough  <[email protected]>
+
+        String#split is buggy
+        https://bugs.webkit.org/show_bug.cgi?id=68348
+
+        Reviewed by Sam Weinig.
+
+        * runtime/StringPrototype.cpp:
+        (JSC::jsStringWithReuse):
+            - added helper function to reuse original JSString value.
+        (JSC::stringProtoFuncSplit):
+            - Rewritten from the spec.
+        * tests/mozilla/ecma/String/15.5.4.8-2.js:
+        (getTestCases):
+            - This test is not ES5 compliant.
+
 2011-09-19  Geoffrey Garen  <[email protected]>