Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(155)

Side by Side Diff: tests/corelib/string_trimlr_test.dart

Issue 2748003002: Revert "Remove U+180E from whitespace in js_runtime and tests" (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tests/corelib/string_trim2_test.dart ('k') | tests/corelib_strong/double_parse_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import "package:expect/expect.dart"; 5 import "package:expect/expect.dart";
6 6
7 // Characters with Whitespace property (Unicode 6.3). 7 // Characters with Whitespace property (Unicode 6.2).
8 // 0009..000D ; White_Space # Cc <control-0009>..<control-000D> 8 // 0009..000D ; White_Space # Cc <control-0009>..<control-000D>
9 // 0020 ; White_Space # Zs SPACE 9 // 0020 ; White_Space # Zs SPACE
10 // 0085 ; White_Space # Cc <control-0085> 10 // 0085 ; White_Space # Cc <control-0085>
11 // 00A0 ; White_Space # Zs NO-BREAK SPACE 11 // 00A0 ; White_Space # Zs NO-BREAK SPACE
12 // 1680 ; White_Space # Zs OGHAM SPACE MARK 12 // 1680 ; White_Space # Zs OGHAM SPACE MARK
13 // 180E ; White_Space # Zs MONGOLIAN VOWEL SEPARATOR
13 // 2000..200A ; White_Space # Zs EN QUAD..HAIR SPACE 14 // 2000..200A ; White_Space # Zs EN QUAD..HAIR SPACE
14 // 2028 ; White_Space # Zl LINE SEPARATOR 15 // 2028 ; White_Space # Zl LINE SEPARATOR
15 // 2029 ; White_Space # Zp PARAGRAPH SEPARATOR 16 // 2029 ; White_Space # Zp PARAGRAPH SEPARATOR
16 // 202F ; White_Space # Zs NARROW NO-BREAK SPACE 17 // 202F ; White_Space # Zs NARROW NO-BREAK SPACE
17 // 205F ; White_Space # Zs MEDIUM MATHEMATICAL SPACE 18 // 205F ; White_Space # Zs MEDIUM MATHEMATICAL SPACE
18 // 3000 ; White_Space # Zs IDEOGRAPHIC SPACE 19 // 3000 ; White_Space # Zs IDEOGRAPHIC SPACE
19 // And BOM: 20 // And BOM:
20 // FEFF ; Byte order mark. 21 // FEFF ; Byte order mark.
21 const WHITESPACE = const [ 22 const WHITESPACE = const [
22 0x09, 23 0x09,
23 0x0A, 24 0x0A,
24 0x0B, 25 0x0B,
25 0x0C, 26 0x0C,
26 0x0D, 27 0x0D,
27 0x20, 28 0x20,
28 0x85, 29 0x85,
29 0xA0, 30 0xA0,
30 0x1680, 31 0x1680,
32 0x180E,
31 0x2000, 33 0x2000,
32 0x2001, 34 0x2001,
33 0x2002, 35 0x2002,
34 0x2003, 36 0x2003,
35 0x2004, 37 0x2004,
36 0x2005, 38 0x2005,
37 0x2006, 39 0x2006,
38 0x2007, 40 0x2007,
39 0x2008, 41 0x2008,
40 0x2009, 42 0x2009,
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 // Empty strings. 82 // Empty strings.
81 Expect.identical("", "".trimLeft()); 83 Expect.identical("", "".trimLeft());
82 Expect.identical("", "".trimRight()); 84 Expect.identical("", "".trimRight());
83 85
84 // Test all BMP chars and one surrogate pair. 86 // Test all BMP chars and one surrogate pair.
85 for (int i = 0, j = 0; i <= 0x10000; i++) { 87 for (int i = 0, j = 0; i <= 0x10000; i++) {
86 if (j < WHITESPACE.length && i == WHITESPACE[j]) { 88 if (j < WHITESPACE.length && i == WHITESPACE[j]) {
87 j++; 89 j++;
88 continue; 90 continue;
89 } 91 }
90 // See below for these exceptions. 92 // U+200b is currently being treated as whitespace by some JS engines.
91 if (i == 0x180E) continue; 93 // Should be fixed in tip-of-tree V8 per 2014-02-10.
92 if (i == 0x200B) continue; 94 // This line makes string_trimlr_test/none fail but /01 succeede where
95 // this bug is in the JS. Both succeede on the VM and where the bug is
96 // not. Remove this line and comment if all JS engines fix it.
97 if (i == 0x200b) continue; /// 01: ok
93 98
94 var s = new String.fromCharCode(i); 99 var s = new String.fromCharCode(i);
95 Expect.identical(s, s.trimLeft()); 100 Expect.identical(s, s.trimLeft());
96 Expect.identical(s, s.trimRight()); 101 Expect.identical(s, s.trimRight());
97 } 102 }
98
99 // U+200b is currently being treated as whitespace by some JS engines.
100 // string_trimlr_test/01 fails on these engines.
101 // Should be fixed in tip-of-tree V8 per 2014-02-10.
102 var s200B = new String.fromCharCode(0x200B);
103 Expect.identical(s200B, s200B.trimLeft()); /// 01: ok
104 Expect.identical(s200B, s200B.trimRight()); /// 01: ok
105
106 // U+180E ceased to be whitespace in Unicode version 6.3.0
107 // string_trimlr_test/02 fails on implementations using earlier versions.
108 var s180E = new String.fromCharCode(0x180E);
109 Expect.identical(s180E, s180E.trimLeft()); /// 02: ok
110 Expect.identical(s180E, s180E.trimRight()); /// 02: ok
111 } 103 }
OLDNEW
« no previous file with comments | « tests/corelib/string_trim2_test.dart ('k') | tests/corelib_strong/double_parse_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698