Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 } |
| OLD | NEW |