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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: LayoutTests/fast/dom/HTMLInputElement/script-tests/size-attribute.js

Issue 796543003: Stricter parsing for size attribute on HTMLInputElement (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Add some subtests that actually make a difference Created 6 years 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 | « no previous file | LayoutTests/fast/dom/HTMLInputElement/size-attribute-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 description("HTMLInputElement size attribute test"); 1 description("HTMLInputElement size attribute test");
2 2
3 function sizeAttributeEffect(value) 3 function sizeAttributeEffect(value)
4 { 4 {
5 var element = document.createElement("input"); 5 var element = document.createElement("input");
6 element.setAttribute("size", value); 6 element.setAttribute("size", value);
7 return element.size; 7 return element.size;
8 } 8 }
9 9
10 shouldBe('document.createElement("input").size', '20'); 10 shouldBe('document.createElement("input").size', '20');
11 11
12 shouldBe('sizeAttributeEffect("")', '20'); 12 shouldBe('sizeAttributeEffect("")', '20');
13 13
14 shouldBe('sizeAttributeEffect("1")', '1'); 14 shouldBe('sizeAttributeEffect("1")', '1');
15 shouldBe('sizeAttributeEffect("2")', '2'); 15 shouldBe('sizeAttributeEffect("2")', '2');
16 shouldBe('sizeAttributeEffect("10")', '10'); 16 shouldBe('sizeAttributeEffect("10")', '10');
17 17
18 shouldBe('sizeAttributeEffect("0")', '20'); 18 shouldBe('sizeAttributeEffect("0")', '20');
19 19
20 shouldBe('sizeAttributeEffect("-1")', '20'); 20 shouldBe('sizeAttributeEffect("-1")', '20');
21 21
22 shouldBe('sizeAttributeEffect("1x")', '1'); 22 shouldBe('sizeAttributeEffect("1x")', '1');
23 shouldBe('sizeAttributeEffect("1.")', '1'); 23 shouldBe('sizeAttributeEffect("1.")', '1');
24 shouldBe('sizeAttributeEffect("1.9")', '1'); 24 shouldBe('sizeAttributeEffect("1.9")', '1');
25 shouldBe('sizeAttributeEffect("2x")', '2'); 25 shouldBe('sizeAttributeEffect("2x")', '2');
26 shouldBe('sizeAttributeEffect("2.")', '2'); 26 shouldBe('sizeAttributeEffect("2.")', '2');
27 shouldBe('sizeAttributeEffect("2.9")', '2'); 27 shouldBe('sizeAttributeEffect("2.9")', '2');
28 28
29 shouldBe('sizeAttributeEffect("a")', '20'); 29 shouldBe('sizeAttributeEffect("a")', '20');
30 shouldBe('sizeAttributeEffect("\v7")', '20');
31 shouldBe('sizeAttributeEffect(" 7")', '7');
30 32
31 var arabicIndicDigitOne = String.fromCharCode(0x661); 33 var arabicIndicDigitOne = String.fromCharCode(0x661);
32 shouldBe('sizeAttributeEffect(arabicIndicDigitOne)', '20'); 34 shouldBe('sizeAttributeEffect(arabicIndicDigitOne)', '20');
33 shouldBe('sizeAttributeEffect("2" + arabicIndicDigitOne)', '2'); 35 shouldBe('sizeAttributeEffect("2" + arabicIndicDigitOne)', '2');
36
37 shouldBe('sizeAttributeEffect("2147483647")', '2147483647');
38 shouldBe('sizeAttributeEffect("2147483648")', '20');
39 shouldBe('sizeAttributeEffect("4294967295")', '20');
40 shouldBe('sizeAttributeEffect("4294967296")', '20');
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fast/dom/HTMLInputElement/size-attribute-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698