WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
[patch]
Patch
bug-191415-20181108103024.patch (text/plain), 32.47 KB, created by
Ross Kirsling
on 2018-11-08 10:30:24 PST
(
hide
)
Description:
Patch
Filename:
MIME Type:
Creator:
Ross Kirsling
Created:
2018-11-08 10:30:24 PST
Size:
32.47 KB
patch
obsolete
>Subversion Revision: 237989 >diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog >index 4fdfa8b865cf382b516477c823b653611c920a35..fe8888c02587360b9b11f30a558ce0273db7f437 100644 >--- a/Source/JavaScriptCore/ChangeLog >+++ b/Source/JavaScriptCore/ChangeLog >@@ -1,3 +1,19 @@ >+2018-11-08 Ross Kirsling <ross.kirsling@sony.com> >+ >+ U+180E is no longer a whitespace character >+ https://bugs.webkit.org/show_bug.cgi?id=191415 >+ >+ Reviewed by NOBODY (OOPS!). >+ >+ 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: >+ > 2018-11-07 Yusuke Suzuki <yusukesuzuki@slowstart.org> > > Consider removing double load for accessing the MetadataTable from LLInt >diff --git a/Source/JavaScriptCore/parser/Lexer.h b/Source/JavaScriptCore/parser/Lexer.h >index f051bf6030a420dbfb6df54e17749468d88ec23e..32f1b0b2acb5eeffdf5b29836f0a8a966c97ae25 100644 >--- a/Source/JavaScriptCore/parser/Lexer.h >+++ b/Source/JavaScriptCore/parser/Lexer.h >@@ -240,8 +240,7 @@ ALWAYS_INLINE bool Lexer<LChar>::isWhiteSpace(LChar ch) > template <> > ALWAYS_INLINE bool Lexer<UChar>::isWhiteSpace(UChar ch) > { >- // 0x180E used to be in Zs category before Unicode 6.3, and EcmaScript says that we should keep treating it as such. >- return (ch < 256) ? Lexer<LChar>::isWhiteSpace(static_cast<LChar>(ch)) : (u_charType(ch) == U_SPACE_SEPARATOR || ch == 0x180E || ch == 0xFEFF); >+ return (ch < 256) ? Lexer<LChar>::isWhiteSpace(static_cast<LChar>(ch)) : (u_charType(ch) == U_SPACE_SEPARATOR || ch == 0xFEFF); > } > > template <> >diff --git a/Source/JavaScriptCore/runtime/ParseInt.h b/Source/JavaScriptCore/runtime/ParseInt.h >index a9cd412cced119c89dee8d27a8ddf7e4bd4d1b68..18d500d795988d3fba6ed36ea1a8e0914e55d060 100644 >--- a/Source/JavaScriptCore/runtime/ParseInt.h >+++ b/Source/JavaScriptCore/runtime/ParseInt.h >@@ -110,7 +110,6 @@ ALWAYS_INLINE static bool isStrWhiteSpace(UChar c) > case 0x000D: > case 0x0020: > case 0x00A0: >- 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. > case 0x2028: > case 0x2029: > case 0xFEFF: >diff --git a/Source/JavaScriptCore/yarr/create_regex_tables b/Source/JavaScriptCore/yarr/create_regex_tables >index 4c3dbbe3fbd494a23824b33418942a7a61ab78bc..992566db77b213123aafb16977e279656027bb08 100644 >--- a/Source/JavaScriptCore/yarr/create_regex_tables >+++ b/Source/JavaScriptCore/yarr/create_regex_tables >@@ -32,7 +32,7 @@ types = { > "nonwordchar": { "UseTable" : True, "Inverse": "wordchar", "data": ['`', (0, ord('0') - 1), (ord('9') + 1, ord('A') - 1), (ord('Z') + 1, ord('_') - 1), (ord('z') + 1, 0x10ffff)]}, > "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)]}, > "newline": { "UseTable" : False, "data": ['\n', '\r', 0x2028, 0x2029]}, >- "spaces": { "UseTable" : True, "data": [' ', ('\t', '\r'), 0xa0, 0x1680, 0x180e, 0x2028, 0x2029, 0x202f, 0x205f, 0x3000, (0x2000, 0x200a), 0xfeff]}, >+ "spaces": { "UseTable" : True, "data": [' ', ('\t', '\r'), 0xa0, 0x1680, 0x2028, 0x2029, 0x202f, 0x205f, 0x3000, (0x2000, 0x200a), 0xfeff]}, > "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)]}, > "digits": { "UseTable" : False, "data": [('0', '9')]}, > "nondigits": { "UseTable" : False, "Inverse": "digits", "data": [(0, ord('0') - 1), (ord('9') + 1, 0x10ffff)] } >diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog >index bf82c05c743468c50b5e956a5ca285eab19a1152..76325088f3b871700fdb21d000d13bb62aab0f5c 100644 >--- a/LayoutTests/ChangeLog >+++ b/LayoutTests/ChangeLog >@@ -1,3 +1,34 @@ >+2018-11-08 Ross Kirsling <ross.kirsling@sony.com> >+ >+ U+180E is no longer a whitespace character >+ https://bugs.webkit.org/show_bug.cgi?id=191415 >+ >+ Reviewed by NOBODY (OOPS!). >+ >+ * 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. >+ > 2018-11-08 Eric Carlson <eric.carlson@apple.com> > > [MediaStream] Use DeviceIdHashSaltStorage to generate device ID hash salt >diff --git a/LayoutTests/js/ToNumber-expected.txt b/LayoutTests/js/ToNumber-expected.txt >index f6ae4afa827e5cfb027e21f756b6097d9840a8f4..866c32df3dec66f0f06cfcb5b58bf1490517d2bf 100644 >--- a/LayoutTests/js/ToNumber-expected.txt >+++ b/LayoutTests/js/ToNumber-expected.txt >@@ -54,7 +54,7 @@ PASS +lf is 0 > PASS +ls is 0 > PASS +ps is 0 > PASS +oghamSpaceMark is 0 >-PASS +mongolianVowelSeparator is 0 >+PASS +mongolianVowelSeparator is NaN > PASS +enQuad is 0 > PASS +emQuad is 0 > PASS +enSpace is 0 >@@ -78,7 +78,7 @@ PASS +(lf + '1') is 1 > PASS +(ls + '1') is 1 > PASS +(ps + '1') is 1 > PASS +(oghamSpaceMark + '1') is 1 >-PASS +(mongolianVowelSeparator + '1') is 1 >+PASS +(mongolianVowelSeparator + '1') is NaN > PASS +(enQuad + '1') is 1 > PASS +(emQuad + '1') is 1 > PASS +(enSpace + '1') is 1 >@@ -102,7 +102,7 @@ PASS +('1' + lf) is 1 > PASS +('1' + ls) is 1 > PASS +('1' + ps) is 1 > PASS +('1' + oghamSpaceMark) is 1 >-PASS +('1' + mongolianVowelSeparator) is 1 >+PASS +('1' + mongolianVowelSeparator) is NaN > PASS +('1' + enQuad) is 1 > PASS +('1' + emQuad) is 1 > PASS +('1' + enSpace) is 1 >diff --git a/LayoutTests/js/parseFloat-expected.txt b/LayoutTests/js/parseFloat-expected.txt >index 8e00c639b0aa8601679749ebca37f4aa2657e493..d10c56af4265f47af1d07b46c20f0fa6f27f85ba 100644 >--- a/LayoutTests/js/parseFloat-expected.txt >+++ b/LayoutTests/js/parseFloat-expected.txt >@@ -32,7 +32,7 @@ PASS parseFloat(lf + '1') is 1 > PASS parseFloat(ls + '1') is 1 > PASS parseFloat(ps + '1') is 1 > PASS parseFloat(oghamSpaceMark + '1') is 1 >-PASS parseFloat(mongolianVowelSeparator + '1') is 1 >+PASS parseFloat(mongolianVowelSeparator + '1') is NaN > PASS parseFloat(enQuad + '1') is 1 > PASS parseFloat(emQuad + '1') is 1 > PASS parseFloat(enSpace + '1') is 1 >diff --git a/LayoutTests/js/script-tests/ToNumber.js b/LayoutTests/js/script-tests/ToNumber.js >index 82d4c417615a8641b5aa823a41c248251def874b..e51e0df2709f2a4bb3c108a2ca5a8f3d0eaccf4b 100644 >--- a/LayoutTests/js/script-tests/ToNumber.js >+++ b/LayoutTests/js/script-tests/ToNumber.js >@@ -82,7 +82,7 @@ shouldBe("+lf", "0"); > shouldBe("+ls", "0"); > shouldBe("+ps", "0"); > shouldBe("+oghamSpaceMark", "0"); >-shouldBe("+mongolianVowelSeparator", "0"); >+shouldBe("+mongolianVowelSeparator", "NaN"); > shouldBe("+enQuad", "0"); > shouldBe("+emQuad", "0"); > shouldBe("+enSpace", "0"); >@@ -106,7 +106,7 @@ shouldBe("+(lf + '1')", "1"); > shouldBe("+(ls + '1')", "1"); > shouldBe("+(ps + '1')", "1"); > shouldBe("+(oghamSpaceMark + '1')", "1"); >-shouldBe("+(mongolianVowelSeparator + '1')", "1"); >+shouldBe("+(mongolianVowelSeparator + '1')", "NaN"); > shouldBe("+(enQuad + '1')", "1"); > shouldBe("+(emQuad + '1')", "1"); > shouldBe("+(enSpace + '1')", "1"); >@@ -130,7 +130,7 @@ shouldBe("+('1' + lf)", "1"); > shouldBe("+('1' + ls)", "1"); > shouldBe("+('1' + ps)", "1"); > shouldBe("+('1' + oghamSpaceMark)", "1"); >-shouldBe("+('1' + mongolianVowelSeparator)", "1"); >+shouldBe("+('1' + mongolianVowelSeparator)", "NaN"); > shouldBe("+('1' + enQuad)", "1"); > shouldBe("+('1' + emQuad)", "1"); > shouldBe("+('1' + enSpace)", "1"); >diff --git a/LayoutTests/js/script-tests/parseFloat.js b/LayoutTests/js/script-tests/parseFloat.js >index ce614e947cba397c9b31efc43274ecf6bc90a8e0..9b8be7bd9d5ddd53d53e038d57de2a237e2c523c 100644 >--- a/LayoutTests/js/script-tests/parseFloat.js >+++ b/LayoutTests/js/script-tests/parseFloat.js >@@ -58,7 +58,7 @@ shouldBe("parseFloat(lf + '1')", "1"); > shouldBe("parseFloat(ls + '1')", "1"); > shouldBe("parseFloat(ps + '1')", "1"); > shouldBe("parseFloat(oghamSpaceMark + '1')", "1"); >-shouldBe("parseFloat(mongolianVowelSeparator + '1')", "1"); >+shouldBe("parseFloat(mongolianVowelSeparator + '1')", "NaN"); > shouldBe("parseFloat(enQuad + '1')", "1"); > shouldBe("parseFloat(emQuad + '1')", "1"); > shouldBe("parseFloat(enSpace + '1')", "1"); >diff --git a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt >index 5372461a67abc300d509b0ae3c39884eda7c1486..e35015252f4637e1882515a9acd3e55834b58050 100644 >--- a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt >+++ b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A2-expected.txt >@@ -1,6 +1,6 @@ > S9.3.1_A2 > >-PASS >+FAIL 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 > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt >index 7b7f56274dc6eef8cd27585b3b8a886d38d87d0e..e70c659885ed187a542ff3c9dc246f65f42fc634 100644 >--- a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt >+++ b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T1-expected.txt >@@ -1,6 +1,6 @@ > S9.3.1_A3_T1 > >-PASS >+FAIL 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("") > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt >index 8cce60c37c4fb24d16b6e6c62953cd7367652bcf..48c7b07f9703d4eb58edf43faed9ffbe478db29d 100644 >--- a/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt >+++ b/LayoutTests/sputnik/Conformance/09_Type_Conversion/9.3_ToNumber/9.3.1_ToNumber_from_String/S9.3.1_A3_T2-expected.txt >@@ -1,6 +1,6 @@ > S9.3.1_A3_T2 > >-PASS >+FAIL 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("") > > TEST COMPLETE > >diff --git a/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 b/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 >index dd66b3d26eb1eaae81bf5f37bc5d97f75815de02..aec1b8218161f958211113f7d34c977715220def 100644 >--- a/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 >+++ b/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 >@@ -1,6 +1,6 @@ > S15.10.2.12_A1_T1 > >-FAIL SputnikError: #63: Error matching character class s between character fc00 and ffff >+FAIL SputnikError: #6: Error matching character class s between character 1800 and 1bff > > TEST COMPLETE > >diff --git a/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 b/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 >index a4e01a203d850be94da7b0aae1ec0d53fe562fc3..4709a4fa5724e98029e678088ea1a6691df0d736 100644 >--- a/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 >+++ b/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 >@@ -1,6 +1,6 @@ > S15.10.2.12_A2_T1 > >-FAIL SputnikError: #63: Error matching character class S between character fc00 and ffff >+FAIL SputnikError: #6: Error matching character class S between character 1800 and 1bff > > TEST COMPLETE > >diff --git a/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 b/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 >index aea5fed8a34e5b796d4640416a053c16766b326f..2c1fbe5c89225fec776875d2889b4acf1d254c0f 100644 >--- a/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 >+++ b/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 >@@ -1,6 +1,6 @@ > S15.1.2.2_A2_T10 > >-PASS >+FAIL SputnikError: #1.180E > > TEST COMPLETE > >diff --git a/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 b/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 >index a6c56c05333a2975053d27996bf950b82497b9fa..78c7854f63fe3045c0926987b8362f3dc6d05204 100644 >--- a/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 >+++ b/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 >@@ -1,6 +1,6 @@ > S15.1.2.3_A2_T10 > >-PASS >+FAIL SputnikError: #1.180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt >index f6505e270f9e821d30aeb3e877b4e5cda2cb12c3..e3636d94e64e92b1800db5b3760172246b6afb76 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A1_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A1_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt >index 26ba77873351d44ff69d484f3bc10c97c6fab237..6cf9f94dad4fb4dd1c911accefe7ec26b29ed563 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_410/S15.10.2.12_A2_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A2_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt >index b2faa2a50b7f4d9a5dbaf409478c90243eb2d3c0..cb7f0f94ea5356f540f1b60a88de23443a4e61f0 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_410/S7.2_A1.6_T1-expected.txt >@@ -1,6 +1,6 @@ > S7.2_A1.6_T1 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt >index f6505e270f9e821d30aeb3e877b4e5cda2cb12c3..e3636d94e64e92b1800db5b3760172246b6afb76 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A1_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A1_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt >index 26ba77873351d44ff69d484f3bc10c97c6fab237..6cf9f94dad4fb4dd1c911accefe7ec26b29ed563 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_500/S15.10.2.12_A2_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A2_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt >index b2faa2a50b7f4d9a5dbaf409478c90243eb2d3c0..cb7f0f94ea5356f540f1b60a88de23443a4e61f0 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_500/S7.2_A1.6_T1-expected.txt >@@ -1,6 +1,6 @@ > S7.2_A1.6_T1 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt >index f6505e270f9e821d30aeb3e877b4e5cda2cb12c3..e3636d94e64e92b1800db5b3760172246b6afb76 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A1_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A1_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt >index 26ba77873351d44ff69d484f3bc10c97c6fab237..6cf9f94dad4fb4dd1c911accefe7ec26b29ed563 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_510/S15.10.2.12_A2_T6-expected.txt >@@ -1,6 +1,6 @@ > S15.10.2.12_A2_T6 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/LayoutTests/sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt b/LayoutTests/sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt >index b2faa2a50b7f4d9a5dbaf409478c90243eb2d3c0..cb7f0f94ea5356f540f1b60a88de23443a4e61f0 100644 >--- a/LayoutTests/sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt >+++ b/LayoutTests/sputnik/Unicode/Unicode_510/S7.2_A1.6_T1-expected.txt >@@ -1,6 +1,6 @@ > S7.2_A1.6_T1 > >-PASS >+FAIL SputnikError: #180E > > TEST COMPLETE > >diff --git a/JSTests/ChakraCore/test/es5/regexSpace.baseline b/JSTests/ChakraCore/test/es5/regexSpace.baseline >index 659341b38f8a0551f09bc526079e78874f98dd7e..35c7039045842726ef779bc2ad7085f91f9c102d 100644 >--- a/JSTests/ChakraCore/test/es5/regexSpace.baseline >+++ b/JSTests/ChakraCore/test/es5/regexSpace.baseline >@@ -7,7 +7,6 @@ start > \u0020 > \u00a0 > \u1680 >-\u180e > \u2000 > \u2001 > \u2002 >diff --git a/JSTests/ChakraCore/test/es6/unicode_whitespace.js b/JSTests/ChakraCore/test/es6/unicode_whitespace.js >index 6f8a8010468f979d2c50977d3625e75d9c7c1721..fafd94e590577a0493109eedf4b4e198e5245273 100644 >--- a/JSTests/ChakraCore/test/es6/unicode_whitespace.js >+++ b/JSTests/ChakraCore/test/es6/unicode_whitespace.js >@@ -16,7 +16,6 @@ var whitespace_characters = [ > '\u0020', > '\u00a0', > '\u1680', >- '\u180e', > '\u2000', > '\u2001', > '\u2002', >diff --git a/JSTests/ChangeLog b/JSTests/ChangeLog >index 42c179185b077cb0df5733c7bf85106f91f3bab7..514749d57f2b72b842e21fa376770037fd1db160 100644 >--- a/JSTests/ChangeLog >+++ b/JSTests/ChangeLog >@@ -1,3 +1,20 @@ >+2018-11-08 Ross Kirsling <ross.kirsling@sony.com> >+ >+ U+180E is no longer a whitespace character >+ https://bugs.webkit.org/show_bug.cgi?id=191415 >+ >+ Reviewed by NOBODY (OOPS!). >+ >+ * 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. >+ > 2018-11-07 Caio Lima <ticaiolima@gmail.com> > > [BigInt] Add support to BigInt into ValueAdd >diff --git a/JSTests/test262.yaml b/JSTests/test262.yaml >index db48372d03f1bc3acf9dfa878a8aaa8af5ed7108..22216fe5525f5e48c3d99edfcd5aa39d6674ca3f 100644 >--- a/JSTests/test262.yaml >+++ b/JSTests/test262.yaml >@@ -24270,9 +24270,9 @@ > - path: test262/test/built-ins/Number/S9.3.1_A29.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A2_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A2_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A30.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A30.js >@@ -24290,17 +24290,17 @@ > - path: test262/test/built-ins/Number/S9.3.1_A3_T1.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A3_T1_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A3_T1_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A3_T2.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A3_T2.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A3_T2_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A3_T2_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/Number/S9.3.1_A4_T1.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/Number/S9.3.1_A4_T1.js >@@ -40576,9 +40576,9 @@ > - path: test262/test/built-ins/RegExp/S15.10.2.12_A1_T5.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T1.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T1.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T2.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/RegExp/S15.10.2.12_A2_T2.js >@@ -45252,9 +45252,9 @@ > - path: test262/test/built-ins/RegExp/prototype/unicode/this-val-regexp.js > cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/RegExp/u180e.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/RegExp/u180e.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/RegExp/unicode_identity_escape.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/RegExp/unicode_identity_escape.js >@@ -50194,9 +50194,9 @@ > - path: test262/test/built-ins/String/prototype/trim/name.js > cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js", "../../../../../harness/propertyHelper.js"], [:strict] > - path: test262/test/built-ins/String/prototype/trim/u180e.js >- cmd: runTest262 :fail, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [] > - path: test262/test/built-ins/String/prototype/trim/u180e.js >- cmd: runTest262 :fail, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/String/prototype/trimEnd/length.js > cmd: runTest262 :normal, "NoException", ["../../../../../harness/assert.js", "../../../../../harness/sta.js", "../../../../../harness/propertyHelper.js"], [] > - path: test262/test/built-ins/String/prototype/trimEnd/length.js >@@ -59342,9 +59342,9 @@ > - path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T10.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T10_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T10_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T2.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/parseFloat/S15.1.2.3_A2_T2.js >@@ -59570,9 +59570,9 @@ > - path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T10.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T10_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T10_U180E.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T2.js > cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/built-ins/parseInt/S15.1.2.2_A2_T2.js >@@ -117208,10 +117208,10 @@ > - path: test262/test/language/white-space/S7.2_A5_T5.js > cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/language/white-space/mongolian-vowel-separator-eval.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/language/white-space/mongolian-vowel-separator-eval.js >- cmd: runTest262 :fail, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "NoException", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] > - path: test262/test/language/white-space/mongolian-vowel-separator.js >- cmd: runTest262 :fail, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [] >+ cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [] > - path: test262/test/language/white-space/mongolian-vowel-separator.js >- cmd: runTest262 :fail, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >+ cmd: runTest262 :normal, "SyntaxError", ["../../../harness/assert.js", "../../../harness/sta.js"], [:strict] >diff --git a/JSTests/test262/config.yaml b/JSTests/test262/config.yaml >index b14a9a9b65b625262a3c05eaf6a6d3c356206a9f..13ba99e02d50a68e5592b2621b76edcd3ad346a2 100644 >--- a/JSTests/test262/config.yaml >+++ b/JSTests/test262/config.yaml >@@ -16,8 +16,6 @@ skip: > - Symbol.matchAll > # https://bugs.webkit.org/show_bug.cgi?id=174931 > - regexp-lookbehind >- # https://bugs.webkit.org/show_bug.cgi?id=126608 >- - u180e > files: > - test/built-ins/Array/prototype/reverse/length-exceeding-integer-limit-with-object.js > - test/built-ins/Array/prototype/unshift/length-near-integer-limit.js >diff --git a/JSTests/test262/expectations.yaml b/JSTests/test262/expectations.yaml >index 08ca9b13ec6cd28cc5cdf99c6c3e7e9dfcf7ab9a..0cb01ac55eda87f725cdb480657460fe788a6eaf 100644 >--- a/JSTests/test262/expectations.yaml >+++ b/JSTests/test262/expectations.yaml >@@ -1104,9 +1104,6 @@ test/built-ins/Proxy/ownKeys/return-duplicate-entries-throws.js: > test/built-ins/Proxy/ownKeys/return-duplicate-symbol-entries-throws.js: > default: 'Test262Error: Expected a TypeError to be thrown but no exception was thrown at all' > strict mode: 'Test262Error: Expected a TypeError to be thrown but no exception was thrown at all' >-test/built-ins/RegExp/S15.10.2.12_A2_T1.js: >- default: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff' >- strict mode: 'Test262Error: #6: Error matching character class S between character 1800 and 1bff' > test/built-ins/RegExp/named-groups/groups-object-subclass-sans.js: > default: 'Test262Error: Expected SameValue(ëbû, ë$<a>û) to be true' > strict mode: 'Test262Error: Expected SameValue(ëbû, ë$<a>û) to be true'
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=354248">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Formatted Diff
|
Diff
Attachments on
bug 191415
:
354209
|
354211
|
354215
|
354216
|
354223
|
354224
|
354225
|
354232
| 354248