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

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

Issue 2902433002: Generates predicates to test in diff functions in ComputedStyle (Closed)
Patch Set: alancutter@'s suggestions 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 }, 9 },
10 10
11 // A list of methods to diff (these can be public getters of fields or 11 // A list of methods to diff (these can be public getters of fields or
12 // functions that use fields to determine a value). 12 // functions that use fields to determine a value).
13 // Each entry is expressed as a dict of two fields. 13 // Each entry is expressed as a dict of two fields.
14 // The fields are method - the method to be diffed and field_dependencies 14 // The fields are:
15 // which lists the properties this method depends on. 15 // 1. method - the method to be diffed
16 // 2. field_dependencies - lists the properties this method depends on.
16 methods_to_diff: { 17 methods_to_diff: {
17 default: [], 18 default: [],
18 }, 19 },
20
21 // A list of predicates that have the diff logic built into them and are onl y
22 // called by the diff functions in ComputedStyle.
23 // Each entry is expressed as a dict of two fields.
24 // The fields are:
alancutter (OOO until 2018) 2017/05/25 01:59:20 No need for this line, just end the previous line
nainar 2017/05/25 02:04:26 Done.
25 // 1. predicate - the predicate to be tested
26 // 2. field_dependencies -lists the properties this predicate test depends o n.
alancutter (OOO until 2018) 2017/05/25 01:59:20 Missing space. Maybe use : instead of - to match w
nainar 2017/05/25 02:04:26 Done.
27 // TODO: Note that you also have to pass in the arguments for the predicate.
28 // This may be removed in the future if we are always passing the other Comp utedStyle
29 predicates_to_test: {
30 default: [],
31 },
19 }, 32 },
20 data: [ 33 data: [
21 { 34 {
22 name: "ScrollAnchorDisablingPropertyChanged", 35 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", 36 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", 37 "left", "right", "top", "bottom", "padding-top",
25 "padding-left", "padding-right", "padding-bottom"] 38 "padding-left", "padding-right", "padding-bottom"]
26 }, 39 },
27 { 40 {
28 name: "DiffNeedsFullLayoutAndPaintInvalidation", 41 name: "DiffNeedsFullLayoutAndPaintInvalidation",
29 fields_to_diff: ["padding-top", "padding-left", "padding-right", 42 fields_to_diff: ["padding-top", "padding-left", "padding-right",
30 "padding-bottom", "line-height", "font", "-webkit-border-horizon tal-spacing", "-webkit-border-vertical-spacing"], 43 "padding-bottom", "line-height", "font", "-webkit-border-horizon tal-spacing", "-webkit-border-vertical-spacing"],
31 methods_to_diff: [ 44 methods_to_diff: [
32 { 45 {
33 method: "BorderLeftWidth()", 46 method: "BorderLeftWidth()",
34 field_dependencies: ["border-left-width"] 47 field_dependencies: ["border-left-width"]
35 }, 48 },
36 { 49 {
37 method: "BorderTopWidth()", 50 method: "BorderTopWidth()",
38 field_dependencies: ["border-top-width"] 51 field_dependencies: ["border-top-width"]
39 }, 52 },
40 { 53 {
41 method: "BorderBottomWidth()", 54 method: "BorderBottomWidth()",
42 field_dependencies: ["border-bottom-width"] 55 field_dependencies: ["border-bottom-width"]
43 }, 56 },
44 { 57 {
45 method: "BorderRightWidth()", 58 method: "BorderRightWidth()",
46 field_dependencies: ["border-right-width"] 59 field_dependencies: ["border-right-width"]
47 } 60 }
61 ],
62 predicates_to_test: [
63 {
64 predicate: "TextShadowDataEquivalent(other)",
65 field_dependencies: ["text-shadow"]
66 },
67 {
68 predicate: "QuotesDataEquivalent(other)",
69 field_dependencies: ["quotes"]
70 },
48 ] 71 ]
49 }, 72 },
50 { 73 {
51 name: "DiffNeedsFullLayout", 74 name: "DiffNeedsFullLayout",
52 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "box-sizing"] 75 fields_to_diff: ["width", "min-width", "max-width", "height", "min-heigh t", "max-height", "box-sizing"]
53 } 76 }
54 ], 77 ],
55 } 78 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698