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

Side by Side Diff: third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5

Issue 2889353002: Generate diffs for properties that are generated in ComputedStyle (Closed)
Patch Set: Moved GetPosition() to ScrollAnchorDisablingPropertyChanged() Created 3 years, 7 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
OLDNEW
1 { 1 {
2 // This file specifies the fields we want to diff in the various diff function s 2 // This file specifies the fields we want to diff in the various diff function s
3 // in ComputedStyle. 3 // in ComputedStyle.
4 4
5 parameters: { 5 parameters: {
6 // The name parameter represents the name of the function to be created. 6 // The name parameter represents the name of the function to be created.
7 7
8 fields_to_diff: { 8 fields_to_diff: {
9 default: [],
9 }, 10 },
10 11
11 // A list of methods to diff (these can be public getters of fields or 12 // A list of methods to diff (these can be public getters of fields or
12 // functions that use fields to determine a value). 13 // functions that use fields to determine a value).
13 // Each entry is expressed as a dict of two fields. 14 // Each entry is expressed as a dict of two fields.
14 // The fields are method - the method to be diffed and field_dependencies 15 // The fields are method - the method to be diffed and field_dependencies
15 // which lists the properties this method depends on. 16 // which lists the properties this method depends on.
16 methods_to_diff: { 17 methods_to_diff: {
17 default: [], 18 default: [],
18 }, 19 },
19 }, 20 },
20 data: [ 21 data: [
21 { 22 {
22 name: "ScrollAnchorDisablingPropertyChanged", 23 name: "ScrollAnchorDisablingPropertyChanged",
23 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "margin-top", "margin-left", "margin-right", "margin-bottom", 24 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "margin-top", "margin-left", "margin-right", "margin-bottom",
24 "left", "right", "top", "bottom", "padding-top", 25 "left", "right", "top", "bottom", "padding-top",
25 "padding-left", "padding-right", "padding-bottom"] 26 "padding-left", "padding-right", "padding-bottom"],
27 methods_to_diff: [
28 {
29 method: "GetPosition()",
30 field_dependencies: ["position"]
31 },
32 ]
26 }, 33 },
27 { 34 {
28 name: "DiffNeedsFullLayoutAndPaintInvalidation", 35 name: "DiffNeedsFullLayoutAndPaintInvalidation",
29 fields_to_diff: ["padding-top", "padding-left", "padding-right", 36 fields_to_diff: ["padding-top", "padding-left", "padding-right",
30 "padding-bottom", "line-height", "font", "-webkit-border-horizon tal-spacing", "-webkit-border-vertical-spacing"], 37 "padding-bottom", "line-height", "font", "-webkit-border-horizon tal-spacing", "-webkit-border-vertical-spacing", "TextAutosizingMultiplier"],
31 methods_to_diff: [ 38 methods_to_diff: [
32 { 39 {
33 method: "BorderLeftWidth()", 40 method: "BorderLeftWidth()",
34 field_dependencies: ["border-left-width"] 41 field_dependencies: ["border-left-width"]
35 }, 42 },
36 { 43 {
37 method: "BorderTopWidth()", 44 method: "BorderTopWidth()",
38 field_dependencies: ["border-top-width"] 45 field_dependencies: ["border-top-width"]
39 }, 46 },
40 { 47 {
41 method: "BorderBottomWidth()", 48 method: "BorderBottomWidth()",
42 field_dependencies: ["border-bottom-width"] 49 field_dependencies: ["border-bottom-width"]
43 }, 50 },
44 { 51 {
45 method: "BorderRightWidth()", 52 method: "BorderRightWidth()",
46 field_dependencies: ["border-right-width"] 53 field_dependencies: ["border-right-width"]
47 } 54 },
55 {
56 method: "FontInternal().LoadingCustomFonts()",
57 field_dependencies: ["font"]
58 },
59 {
60 method: "HasPseudoStyle(kPseudoIdScrollbar)",
61 field_dependencies: ["StyleType"]
62 },
63 {
64 method: "BoxDirection()",
65 field_dependencies: ["-webkit-box-direction"]
66 },
67 {
68 method: "RtlOrdering()",
69 field_dependencies: ["-webkit-rtl-ordering"]
70 },
71 {
72 method: "GetTextAlign()",
73 field_dependencies: ["text-align"]
74 },
75 {
76 method: "TextTransform()",
77 field_dependencies: ["text-transform"]
78 },
79 {
80 method: "Direction()",
81 field_dependencies: ["direction"]
82 },
83 {
84 method: "WhiteSpace()",
85 field_dependencies: ["white-space"]
86 },
87 {
88 method: "GetWritingMode()",
89 field_dependencies: ["writing-mode"]
90 },
91 {
92 method: "OverflowX()",
93 field_dependencies: ["overflow-x"]
94 },
95 {
96 method: "OverflowY()",
97 field_dependencies: ["overflow-y"]
98 },
99 {
100 method: "Clear()",
101 field_dependencies: ["clear"]
102 },
103 {
104 method: "GetUnicodeBidi()",
105 field_dependencies: ["unicode-bidi"]
106 },
107 {
108 method: "Floating()",
109 field_dependencies: ["float"]
110 },
111 {
112 method: "OriginalDisplay()",
113 field_dependencies: ["OriginalDisplay"]
114 },
alancutter (OOO until 2018) 2017/05/24 03:31:17 Let's sort these alphabetically.
nainar 2017/05/24 06:07:21 We want them sorted in order in which they appeare
alancutter (OOO until 2018) 2017/05/24 07:25:36 Fair point, that's probably important.
48 ] 115 ]
49 }, 116 },
50 { 117 {
51 name: "DiffNeedsFullLayout", 118 name: "DiffNeedsFullLayout",
52 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "box-sizing"] 119 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "box-sizing"]
120 },
121 {
122 name: "DiffNeedsPaintInvalidationObject",
123 methods_to_diff: [
124 {
125 method: "Visibility()",
126 field_dependencies: ["visibility"]
127 },
128 {
129 method: "PrintColorAdjust()",
130 field_dependencies: ["-webkit-print-color-adjust"]
131 },
132 {
133 method: "InsideLink()",
134 field_dependencies: ["InsideLink"]
135 }
136 ]
137 },
138 {
139 name: "DiffNeedsVisualRectUpdate",
140 methods_to_diff: [
141 {
142 method: "Visibility()",
143 field_dependencies: ["visibility"]
144 }
145 ]
53 } 146 }
54 ], 147 ],
55 } 148 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698