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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: LayoutTests/fast/css/all-shorthand-css-text.html

Issue 341033003: StylePropertySerializer should expand all property if needed. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 3 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 | Annotate | Revision Log
« no previous file with comments | « LayoutTests/editing/style/script-tests/push-down-inline-styles.js ('k') | LayoutTests/fast/css/all-shorthand-css-text-expected.txt » ('j') | Source/core/css/StylePropertySerializer.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <script src="../../resources/js-test.js"></script> 2 <script src="../../resources/js-test.js"></script>
3 <style> 3 <style>
4 div { all: initial; } 4 div { all: initial; }
5 div { all: initial; color: red; } 5 div { all: initial; color: red; }
6 div { color: red; all: initial; } 6 div { color: red; all: initial; }
7 div { all: initial !important; color: red; } 7 div { all: initial !important; color: red; }
8 div { all: initial; color: red !important; } 8 div { all: initial; color: red !important; }
9 9
10 div { all: inherit; } 10 div { all: inherit; }
11 div { all: inherit; color: red; } 11 div { all: inherit; color: red; }
12 div { color: red; all: inherit; } 12 div { color: red; all: inherit; }
13 div { all: inherit !important; color: red; } 13 div { all: inherit !important; color: red; }
14 div { all: inherit; color: red !important; } 14 div { all: inherit; color: red !important; }
15 15
16 div { all: red; all: none; all: 10px; all: auto; all: url(about:blank); } 16 div { all: red; all: none; all: 10px; all: auto; all: url(about:blank); }
17 17
18 div { direction: ltr; all: initial; direction: rtl; } 18 div { direction: ltr; all: initial; direction: rtl; }
19 div { direction: ltr; unicode-bidi: bidi-override; all: initial !important; } 19 div { direction: ltr; unicode-bidi: bidi-override; all: initial !important; }
20 div { direction: ltr; all: initial; color: red; } 20 div { direction: ltr; all: initial; color: red; }
21 21
22 div { all: initial; font-size: 10px; } 22 div { all: initial; font-size: 10px; }
23 23
24 div { all: initial; width: inherit; } 24 div { all: initial; width: inherit; }
25
26 div { all: unset; }
27 div { all: unset; color: red; }
25 </style> 28 </style>
26 <script> 29 <script>
27 description('Test for crbug.com/172051: all shorthand property.'); 30 description('Test for crbug.com/172051: all shorthand property.');
28 31
29 function contains(cssText, property) { 32 function contains(cssText, property) {
30 return cssText.indexOf(property) == 0 || cssText.indexOf(' ' + property) != - 1; 33 return cssText.indexOf(property) == 0 || cssText.indexOf(' ' + property) != - 1;
31 } 34 }
32 35
33 var cssRules = document.styleSheets[1].cssRules; 36 var cssRules = document.styleSheets[1].cssRules;
34 37
35 shouldBe('cssRules.length', '16'); 38 shouldBe('cssRules.length', '18');
36 shouldBe('cssRules[0].cssText', '"div { all: initial; }"'); 39 shouldBe('cssRules[0].cssText', '"div { all: initial; }"');
37 shouldNotBe('cssRules[1].cssText', '"div { all: initial; color: red; }"'); 40 shouldNotBe('cssRules[1].cssText', '"div { all: initial; color: red; }"');
38 shouldBeTrue('contains(cssRules[1].cssText, "color: red;")'); 41 shouldBeTrue('contains(cssRules[1].cssText, "color: red;")');
39 shouldBeTrue('contains(cssRules[1].cssText, "background: initial;")'); 42 shouldBeTrue('contains(cssRules[1].cssText, "background: initial;")');
40 shouldBeFalse('contains(cssRules[1].cssText, "direction: ")'); 43 shouldBeFalse('contains(cssRules[1].cssText, "direction: ")');
41 shouldBeFalse('contains(cssRules[1].cssText, "unicode-bidi: ")'); 44 shouldBeFalse('contains(cssRules[1].cssText, "unicode-bidi: ")');
42 shouldBe('cssRules[2].cssText', '"div { all: initial; }"'); 45 shouldBe('cssRules[2].cssText', '"div { all: initial; }"');
43 shouldBe('cssRules[3].cssText', '"div { all: initial !important; }"'); 46 shouldBe('cssRules[3].cssText', '"div { all: initial !important; }"');
44 shouldNotBe('cssRules[4].cssText', '"div { all: initial; color: red !important; }"'); 47 shouldNotBe('cssRules[4].cssText', '"div { all: initial; color: red !important; }"');
45 shouldBeTrue('contains(cssRules[4].cssText, "color: red !important;")'); 48 shouldBeTrue('contains(cssRules[4].cssText, "color: red !important;")');
(...skipping 14 matching lines...) Expand all
60 shouldBeFalse('contains(cssRules[9].cssText, "direction: ")'); 63 shouldBeFalse('contains(cssRules[9].cssText, "direction: ")');
61 shouldBeFalse('contains(cssRules[9].cssText, "unicode-bidi: ")'); 64 shouldBeFalse('contains(cssRules[9].cssText, "unicode-bidi: ")');
62 65
63 shouldBe('cssRules[10].cssText', '"div { }"'); 66 shouldBe('cssRules[10].cssText', '"div { }"');
64 67
65 shouldBe('cssRules[11].cssText', '"div { all: initial; direction: rtl; }"'); 68 shouldBe('cssRules[11].cssText', '"div { all: initial; direction: rtl; }"');
66 69
67 shouldBe('cssRules[12].cssText', '"div { direction: ltr; unicode-bidi: bidi-over ride; all: initial !important; }"'); 70 shouldBe('cssRules[12].cssText', '"div { direction: ltr; unicode-bidi: bidi-over ride; all: initial !important; }"');
68 71
69 shouldBeFalse('contains(cssRules[13].cssText, "all: initial;")'); 72 shouldBeFalse('contains(cssRules[13].cssText, "all: initial;")');
73 shouldBeFalse('contains(cssRules[13].cssText, "-internal-")');
70 shouldBeTrue('contains(cssRules[13].cssText, "direction: ltr;")'); 74 shouldBeTrue('contains(cssRules[13].cssText, "direction: ltr;")');
71 shouldBeTrue('contains(cssRules[13].cssText, "color: red;")'); 75 shouldBeTrue('contains(cssRules[13].cssText, "color: red;")');
72 76
73 shouldBeFalse('contains(cssRules[14].cssText, "all: initial;")'); 77 shouldBeFalse('contains(cssRules[14].cssText, "all: initial;")');
78 shouldBeFalse('contains(cssRules[14].cssText, "-internal-")');
74 shouldBeFalse('contains(cssRules[14].cssText, "font: initial;")'); 79 shouldBeFalse('contains(cssRules[14].cssText, "font: initial;")');
75 shouldBeTrue('contains(cssRules[14].cssText, "font-family: initial;")'); 80 shouldBeTrue('contains(cssRules[14].cssText, "font-family: initial;")');
76 81
77 shouldBeFalse('contains(cssRules[15].cssText, "all: initial;")'); 82 shouldBeFalse('contains(cssRules[15].cssText, "all: initial;")');
83 shouldBeFalse('contains(cssRules[15].cssText, "-internal-")');
78 shouldBeFalse('contains(cssRules[14].cssText, "font: initial;")'); 84 shouldBeFalse('contains(cssRules[14].cssText, "font: initial;")');
79 shouldBeFalse('contains(cssRules[14].cssText, "marker: initial;")'); 85 shouldBeFalse('contains(cssRules[14].cssText, "marker: initial;")');
80 shouldBeTrue('contains(cssRules[15].cssText, "width: inherit;")'); 86 shouldBeTrue('contains(cssRules[15].cssText, "width: inherit;")');
81 shouldBeTrue('contains(cssRules[15].cssText, "min-width: initial;")'); 87 shouldBeTrue('contains(cssRules[15].cssText, "min-width: initial;")');
82 shouldBeTrue('contains(cssRules[15].cssText, "max-width: initial;")'); 88 shouldBeTrue('contains(cssRules[15].cssText, "max-width: initial;")');
83 shouldBeTrue('contains(cssRules[15].cssText, "marker-start: initial;")'); 89 shouldBeTrue('contains(cssRules[15].cssText, "marker-start: initial;")');
84 shouldBeTrue('contains(cssRules[15].cssText, "marker-mid: initial;")'); 90 shouldBeTrue('contains(cssRules[15].cssText, "marker-mid: initial;")');
85 shouldBeTrue('contains(cssRules[15].cssText, "marker-end: initial;")'); 91 shouldBeTrue('contains(cssRules[15].cssText, "marker-end: initial;")');
92
93 shouldBe('cssRules[16].cssText', '"div { all: unset; }"');
94 shouldBeFalse('contains(cssRules[17].cssText, "all: unset;")');
95 shouldBeTrue('contains(cssRules[17].cssText, "color: red;")');
96 <!-- FIXME: when using "all: unset", the all should be expanded to be -->
97 <!-- "display: unset; ..." -->
98 shouldBeTrue('contains(cssRules[17].cssText, "display: unset;")');
86 </script> 99 </script>
OLDNEW
« no previous file with comments | « LayoutTests/editing/style/script-tests/push-down-inline-styles.js ('k') | LayoutTests/fast/css/all-shorthand-css-text-expected.txt » ('j') | Source/core/css/StylePropertySerializer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698