Changeset 238004 in webkit


Ignore:
Timestamp:
Nov 8, 2018 3:40:25 PM (5 years ago)
Author:
Ross Kirsling
Message:

U+180E is no longer a whitespace character
​https://bugs.webkit.org/show_bug.cgi?id=191415

Reviewed by Saam Barati.

JSTests:

  • ChakraCore/test/es5/regexSpace.baseline:
  • ChakraCore/test/es6/unicode_whitespace.js:

Update tests to latest version.
(See ​https://github.com/Microsoft/ChakraCore/commit/7c097b698de1e400286f9b957597b2a81fc6f80b.)

  • test262.yaml:
  • test262/config.yaml:
  • test262/expectations.yaml:

Update expectations.

Source/JavaScriptCore:

Mongolian Vowel Separator stopped being a valid whitespace character as of ES2016.
(​https://github.com/tc39/ecma262/pull/300)

  • parser/Lexer.h:

(JSC::Lexer<UChar>::isWhiteSpace):

  • runtime/ParseInt.h:

(JSC::isStrWhiteSpace):

  • yarr/create_regex_tables:

LayoutTests:

  • js/ToNumber-expected.txt:
  • js/parseFloat-expected.txt:
  • js/script-tests/ToNumber.js:
  • js/script-tests/parseFloat.js:

Update tests and expectations.

  • sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt:
  • sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt:
  • sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt:
  • sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A1_T1-expected.txt:
  • sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A2_T1-expected.txt:
  • sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.2_parseInt/S15.1.2.2_A2_T10-expected.txt:
  • sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.3_parseFloat/S15.1.2.3_A2_T10-expected.txt:
  • sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt:
  • sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt:
  • sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt:
  • sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt:
  • sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt:
  • sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt:
  • sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt:

Let outdated sputnik checks fail.

Location:
trunk
Files:
31 edited

Legend:

Unmodified
Added
Removed

  • trunk/JSTests/ChakraCore/test/es5/regexSpace.baseline

    r205387 r238004  
    88\u00a0
    99\u1680
    10 \u180e
    1110\u2000
    1211\u2001

  • trunk/JSTests/ChakraCore/test/es6/unicode_whitespace.js

    r205387 r238004  
    1717    '\u00a0',
    1818    '\u1680',
    19     '\u180e',
    2019    '\u2000',
    2120    '\u2001',

  • trunk/JSTests/ChangeLog

    r237972 r238004  
     12018-11-08  Ross Kirsling  <ross.kirsling@sony.com>
     2
     3        U+180E is no longer a whitespace character
     4        https://bugs.webkit.org/show_bug.cgi?id=191415
     5
     6        Reviewed by Saam Barati.
     7
     8        * ChakraCore/test/es5/regexSpace.baseline:
     9        * ChakraCore/test/es6/unicode_whitespace.js:
     10        Update tests to latest version.
     11        (See https://github.com/Microsoft/ChakraCore/commit/7c097b698de1e400286f9b957597b2a81fc6f80b.)
     12
     13        * test262.yaml:
     14        * test262/config.yaml:
     15        * test262/expectations.yaml:
     16        Update expectations.
     17
    1182018-11-07  Caio Lima  <ticaiolima@gmail.com>
    219

  • trunk/JSTests/test262.yaml

    r235882 r238004  
    2427124271  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2427224272- path: test262/test/built-ins/Number/S9.3.1_A2_U180E.js
    24273   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     24273  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    2427424274- path: test262/test/built-ins/Number/S9.3.1_A2_U180E.js
    24275   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     24275  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2427624276- path: test262/test/built-ins/Number/S9.3.1_A30.js
    2427724277  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    2429124291  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2429224292- path: test262/test/built-ins/Number/S9.3.1_A3_T1_U180E.js
    24293   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     24293  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    2429424294- path: test262/test/built-ins/Number/S9.3.1_A3_T1_U180E.js
    24295   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     24295  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2429624296- path: test262/test/built-ins/Number/S9.3.1_A3_T2.js
    2429724297  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    2429924299  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2430024300- path: test262/test/built-ins/Number/S9.3.1_A3_T2_U180E.js
    24301   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     24301  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    2430224302- path: test262/test/built-ins/Number/S9.3.1_A3_T2_U180E.js
    24303   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     24303  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    2430424304- path: test262/test/built-ins/Number/S9.3.1_A4_T1.js
    2430524305  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    4057740577  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    4057840578- path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T1.js
    40579   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     40579  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    4058040580- path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T1.js
    40581   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     40581  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    4058240582- path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T2.js
    4058340583  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    4525345253  cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict]
    4525445254- path: test262/test/built-ins/RegExp/u180e.js
    45255   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     45255  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    4525645256- path: test262/test/built-ins/RegExp/u180e.js
    45257   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     45257  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    4525845258- path: test262/test/built-ins/RegExp/unicode_identity_escape.js
    4525945259  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    5019550195  cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js", "../../../../../harness/propertyHelper.js"], [:strict]
    5019650196- path: test262/test/built-ins/String/prototype/trim/u180e.js
    50197   cmd: runTest262 :fail, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], []
     50197  cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], []
    5019850198- path: test262/test/built-ins/String/prototype/trim/u180e.js
    50199   cmd: runTest262 :fail, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict]
     50199  cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict]
    5020050200- path: test262/test/built-ins/String/prototype/trimEnd/length.js
    5020150201  cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js", "../../../../../harness/propertyHelper.js"], []
    … …  
    5934359343  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    5934459344- path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T10_U180E.js
    59345   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     59345  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    5934659346- path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T10_U180E.js
    59347   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     59347  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    5934859348- path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T2.js
    5934959349  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    5957159571  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    5957259572- path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T10_U180E.js
    59573   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     59573  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    5957459574- path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T10_U180E.js
    59575   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     59575  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    5957659576- path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T2.js
    5957759577  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    … …  
    117209117209  cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    117210117210- path: test262/test/language/white-space/mongolian-vowel-separator-eval.js
    117211   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     117211  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    117212117212- path: test262/test/language/white-space/mongolian-vowel-separator-eval.js
    117213   cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     117213  cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
    117214117214- path: test262/test/language/white-space/mongolian-vowel-separator.js
    117215   cmd: runTest262 :fail, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], []
     117215  cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], []
    117216117216- path: test262/test/language/white-space/mongolian-vowel-separator.js
    117217   cmd: runTest262 :fail, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]
     117217  cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict]

  • trunk/JSTests/test262/config.yaml

    r235712 r238004  
    1717    # https://bugs.webkit.org/show_bug.cgi?id=174931
    1818    - regexp-lookbehind
    19     # https://bugs.webkit.org/show_bug.cgi?id=126608
    20     - u180e
    2119  files:
    2220    - test/built-ins/Array/prototype/reverse/length-exceeding-integer-limit-with-object.js

  • trunk/JSTests/test262/expectations.yaml

    r237299 r238004  
    11051105  default: 'Test262Error: Expected a TypeError to be thrown but no exception was thrown at all'
    11061106  strict mode: 'Test262Error: Expected a TypeError to be thrown but no exception was thrown at all'
    1107 test/built-ins/RegExp/S15.10.2.12_A2_T1.js:
    1108   default: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff'
    1109   strict mode: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff'
    11101107test/built-ins/RegExp/named-groups/groups-object-subclass-sans.js:
    11111108  default: 'Test262Error: Expected SameValue(«b», «$<a>») to be true'

  • trunk/LayoutTests/ChangeLog

    r238002 r238004  
     12018-11-08  Ross Kirsling  <ross.kirsling@sony.com>
     2
     3        U+180E is no longer a whitespace character
     4        https://bugs.webkit.org/show_bug.cgi?id=191415
     5
     6        Reviewed by Saam Barati.
     7
     8        * js/ToNumber-expected.txt:
     9        * js/parseFloat-expected.txt:
     10        * js/script-tests/ToNumber.js:
     11        * js/script-tests/parseFloat.js:
     12        Update tests and expectations.
     13
     14        * sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt:
     15        * sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt:
     16        * sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt:
     17        * sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A1_T1-expected.txt:
     18        * sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A2_T1-expected.txt:
     19        * sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.2_parseInt/S15.1.2.2_A2_T10-expected.txt:
     20        * sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.3_parseFloat/S15.1.2.3_A2_T10-expected.txt:
     21        * sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt:
     22        * sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt:
     23        * sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt:
     24        * sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt:
     25        * sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt:
     26        * sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt:
     27        * sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt:
     28        * sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt:
     29        * sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt:
     30        Let outdated sputnik checks fail.
     31
    1322018-11-07  Myles C. Maxfield  <mmaxfield@apple.com>
    233

  • trunk/LayoutTests/js/ToNumber-expected.txt

    r63120 r238004  
    5555PASS +ps is 0
    5656PASS +oghamSpaceMark is 0
    57 PASS +mongolianVowelSeparator is 0
     57PASS +mongolianVowelSeparator is NaN
    5858PASS +enQuad is 0
    5959PASS +emQuad is 0
    … …  
    7979PASS +(ps + '1') is 1
    8080PASS +(oghamSpaceMark + '1') is 1
    81 PASS +(mongolianVowelSeparator + '1') is 1
     81PASS +(mongolianVowelSeparator + '1') is NaN
    8282PASS +(enQuad + '1') is 1
    8383PASS +(emQuad + '1') is 1
    … …  
    103103PASS +('1' + ps) is 1
    104104PASS +('1' + oghamSpaceMark) is 1
    105 PASS +('1' + mongolianVowelSeparator) is 1
     105PASS +('1' + mongolianVowelSeparator) is NaN
    106106PASS +('1' + enQuad) is 1
    107107PASS +('1' + emQuad) is 1

  • trunk/LayoutTests/js/parseFloat-expected.txt

    r63120 r238004  
    3333PASS parseFloat(ps + '1') is 1
    3434PASS parseFloat(oghamSpaceMark + '1') is 1
    35 PASS parseFloat(mongolianVowelSeparator + '1') is 1
     35PASS parseFloat(mongolianVowelSeparator + '1') is NaN
    3636PASS parseFloat(enQuad + '1') is 1
    3737PASS parseFloat(emQuad + '1') is 1

  • trunk/LayoutTests/js/script-tests/ToNumber.js

    r98407 r238004  
    8383shouldBe("+ps", "0");
    8484shouldBe("+oghamSpaceMark", "0");
    85 shouldBe("+mongolianVowelSeparator", "0");
     85shouldBe("+mongolianVowelSeparator", "NaN");
    8686shouldBe("+enQuad", "0");
    8787shouldBe("+emQuad", "0");
    … …  
    107107shouldBe("+(ps + '1')", "1");
    108108shouldBe("+(oghamSpaceMark + '1')", "1");
    109 shouldBe("+(mongolianVowelSeparator + '1')", "1");
     109shouldBe("+(mongolianVowelSeparator + '1')", "NaN");
    110110shouldBe("+(enQuad + '1')", "1");
    111111shouldBe("+(emQuad + '1')", "1");
    … …  
    131131shouldBe("+('1' + ps)", "1");
    132132shouldBe("+('1' + oghamSpaceMark)", "1");
    133 shouldBe("+('1' + mongolianVowelSeparator)", "1");
     133shouldBe("+('1' + mongolianVowelSeparator)", "NaN");
    134134shouldBe("+('1' + enQuad)", "1");
    135135shouldBe("+('1' + emQuad)", "1");

  • trunk/LayoutTests/js/script-tests/parseFloat.js

    r98407 r238004  
    5959shouldBe("parseFloat(ps + '1')", "1");
    6060shouldBe("parseFloat(oghamSpaceMark + '1')", "1");
    61 shouldBe("parseFloat(mongolianVowelSeparator + '1')", "1");
     61shouldBe("parseFloat(mongolianVowelSeparator + '1')", "NaN");
    6262shouldBe("parseFloat(enQuad + '1')", "1");
    6363shouldBe("parseFloat(emQuad + '1')", "1");

  • trunk/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt

    r63120 r238004  
    11S9.3.1_A2
    22
    3 PASS
     3FAIL SputnikError: #1.1: Number("\u0009\u000C\u0020\u00A0\u000B\u000A\u000D\u2028\u2029\u1680\u180E\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200A\u202F\u205F\u3000") === 0. Actual: NaN
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt

    r63120 r238004  
    11S9.3.1_A3_T1
    22
    3 PASS
     3FAIL SputnikError: #1: Number("\u0009\u000C\u0020\u00A0\u000B\u000A\u000D\u2028\u2029\u1680\u180E\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200A\u202F\u205F\u3000") === Number("")
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt

    r63120 r238004  
    11S9.3.1_A3_T2
    22
    3 PASS
     3FAIL SputnikError: #1: Number("\u0009\u000C\u0020\u00A0\u000B"+"\u000A\u000D\u2028\u2029\u1680\u180E\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200A\u202F\u205F\u3000") === Number("")
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A1_T1-expected.txt

    r73056 r238004  
    11S15.10.2.12_A1_T1
    22
    3 FAIL SputnikError: #63: Error matching character class s between character fc00 and ffff
     3FAIL SputnikError: #6: Error matching character class s between character 1800 and 1bff
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/15_Native_Objects/15.10_RegExp/15.10.2/15.10.2.12_CharacterClassEscape/S15.10.2.12_A2_T1-expected.txt

    r73056 r238004  
    11S15.10.2.12_A2_T1
    22
    3 FAIL SputnikError: #63: Error matching character class S between character fc00 and ffff
     3FAIL SputnikError: #6: Error matching character class S between character 1800 and 1bff
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.2_parseInt/S15.1.2.2_A2_T10-expected.txt

    r58534 r238004  
    11S15.1.2.2_A2_T10
    22
    3 PASS
     3FAIL SputnikError: #1.180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.2/15.1.2.3_parseFloat/S15.1.2.3_A2_T10-expected.txt

    r63120 r238004  
    11S15.1.2.3_A2_T10
    22
    3 PASS
     3FAIL SputnikError: #1.180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A1_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A2_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt

    r58534 r238004  
    11S7.2_A1.6_T1
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A1_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A2_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt

    r58534 r238004  
    11S7.2_A1.6_T1
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A1_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt

    r58534 r238004  
    11S15.10.2.12_A2_T6
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/LayoutTests/sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt

    r58534 r238004  
    11S7.2_A1.6_T1
    22
    3 PASS
     3FAIL SputnikError: #180E
    44
    55TEST COMPLETE

  • trunk/Source/JavaScriptCore/ChangeLog

    r238000 r238004  
     12018-11-08  Ross Kirsling  <ross.kirsling@sony.com>
     2
     3        U+180E is no longer a whitespace character
     4        https://bugs.webkit.org/show_bug.cgi?id=191415
     5
     6        Reviewed by Saam Barati.
     7
     8        Mongolian Vowel Separator stopped being a valid whitespace character as of ES2016.
     9        (https://github.com/tc39/ecma262/pull/300)
     10
     11        * parser/Lexer.h:
     12        (JSC::Lexer<UChar>::isWhiteSpace):
     13        * runtime/ParseInt.h:
     14        (JSC::isStrWhiteSpace):
     15        * yarr/create_regex_tables:
     16
    1172018-11-08  Keith Miller  <keith_miller@apple.com>
    218

  • trunk/Source/JavaScriptCore/parser/Lexer.h

    r225799 r238004  
    241241ALWAYS_INLINE bool Lexer<UChar>::isWhiteSpace(UChar ch)
    242242{
    243     // 0x180E used to be in Zs category before Unicode 6.3, and EcmaScript says that we should keep treating it as such.
    244     return (ch < 256) ? Lexer<LChar>::isWhiteSpace(static_cast<LChar>(ch)) : (u_charType(ch) == U_SPACE_SEPARATOR || ch == 0x180E || ch == 0xFEFF);
     243    return (ch < 256) ? Lexer<LChar>::isWhiteSpace(static_cast<LChar>(ch)) : (u_charType(ch) == U_SPACE_SEPARATOR || ch == 0xFEFF);
    245244}
    246245

  • trunk/Source/JavaScriptCore/runtime/ParseInt.h

    r236697 r238004  
    111111    case 0x0020:
    112112    case 0x00A0:
    113     case 0x180E: // This character used to be in Zs category before Unicode 6.3, and EcmaScript says that we should keep treating it as such.
    114113    case 0x2028:
    115114    case 0x2029:

  • trunk/Source/JavaScriptCore/yarr/create_regex_tables

    r222499 r238004  
    3333    "nonwordUnicodeIgnoreCaseChar": { "UseTable" : False, "Inverse": "wordUnicodeIgnoreCaseChar", "data": ['`', (0, ord('0') - 1), (ord('9') + 1, ord('A') - 1), (ord('Z') + 1, ord('_') - 1), (ord('z') + 1, 0x017e), (0x0180, 0x2129), (0x212b, 0x10ffff)]},
    3434    "newline": { "UseTable" : False, "data": ['\n', '\r', 0x2028, 0x2029]},
    35     "spaces": { "UseTable" : True, "data": [' ', ('\t', '\r'), 0xa0, 0x1680, 0x180e, 0x2028, 0x2029, 0x202f, 0x205f, 0x3000, (0x2000, 0x200a), 0xfeff]},
     35    "spaces": { "UseTable" : True, "data": [' ', ('\t', '\r'), 0xa0, 0x1680, 0x2028, 0x2029, 0x202f, 0x205f, 0x3000, (0x2000, 0x200a), 0xfeff]},
    3636    "nonspaces": { "UseTable" : True, "Inverse": "spaces", "data": [(0, ord('\t') - 1), (ord('\r') + 1, ord(' ') - 1), (ord(' ') + 1, 0x009f), (0x00a1, 0x167f), (0x1681, 0x180d), (0x180f, 0x1fff), (0x200b, 0x2027), (0x202a, 0x202e), (0x2030, 0x205e), (0x2060, 0x2fff), (0x3001, 0xfefe), (0xff00, 0x10ffff)]},
    3737    "digits": { "UseTable" : False, "data": [('0', '9')]},
Note: See TracChangeset for help on using the changeset viewer.