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

Unified Diff: third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5

Issue 2889353002: Generate diffs for properties that are generated in ComputedStyle (Closed)
Patch Set: Only check not inherited fields and fix fonts 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5
diff --git a/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5 b/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5
index 0c84d05f0b9dd5f64b16de7bf6cdcb81b736a00e..70ba195c8cfef8c4503059f1de9ba690857d94bb 100644
--- a/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5
+++ b/third_party/WebKit/Source/core/css/ComputedStyleDiffFunctions.json5
@@ -6,6 +6,7 @@
// The name parameter represents the name of the function to be created.
fields_to_diff: {
+ default: [],
},
// A list of methods to diff (these can be public getters of fields or
@@ -27,10 +28,14 @@
{
name: "DiffNeedsFullLayoutAndPaintInvalidation",
fields_to_diff: ["padding-top", "padding-left", "padding-right",
- "padding-bottom", "line-height", "font", "-webkit-border-horizontal-spacing", "-webkit-border-vertical-spacing"],
+ "padding-bottom", "line-height", "font", "-webkit-border-horizontal-spacing", "-webkit-border-vertical-spacing", "TextAutosizingMultiplier"],
methods_to_diff: [
{
- method: "BorderLeftWidth()",
+ method: "GetPosition()",
+ field_dependencies: ["position"]
+ },
+ {
+ method: "BorderLeftWidth()",
field_dependencies: ["border-left-width"]
},
{
@@ -44,12 +49,98 @@
{
method: "BorderRightWidth()",
field_dependencies: ["border-right-width"]
- }
+ },
+ {
shend 2017/05/22 02:48:24 These should be in DiffNeedsFullLayoutAndPaintInva
nainar 2017/05/22 03:12:35 Moved GetPosition() to ScrollAnchorDisablingProper
+ method: "FontInternal().LoadingCustomFonts()",
+ field_dependencies: ["font"]
+ },
+ {
+ method: "HasPseudoStyle(kPseudoIdScrollbar)",
+ field_dependencies: ["StyleType"]
+ },
+ {
+ method: "BoxDirection()",
+ field_dependencies: ["-webkit-box-direction"]
+ },
+ {
+ method: "RtlOrdering()",
+ field_dependencies: ["-webkit-rtl-ordering"]
+ },
+ {
+ method: "GetTextAlign()",
+ field_dependencies: ["text-align"]
+ },
+ {
+ method: "TextTransform()",
+ field_dependencies: ["text-transform"]
+ },
+ {
+ method: "Direction()",
+ field_dependencies: ["direction"]
+ },
+ {
+ method: "WhiteSpace()",
+ field_dependencies: ["white-space"]
+ },
+ {
+ method: "GetWritingMode()",
+ field_dependencies: ["writing-mode"]
+ },
+ {
+ method: "OverflowX()",
+ field_dependencies: ["overflow-x"]
+ },
+ {
+ method: "OverflowY()",
+ field_dependencies: ["overflow-y"]
+ },
+ {
+ method: "Clear()",
+ field_dependencies: ["clear"]
+ },
+ {
+ method: "GetUnicodeBidi()",
+ field_dependencies: ["unicode-bidi"]
+ },
+ {
+ method: "Floating()",
+ field_dependencies: ["float"]
+ },
+ {
+ method: "OriginalDisplay()",
+ field_dependencies: ["OriginalDisplay"]
+ },
]
},
{
name: "DiffNeedsFullLayout",
fields_to_diff: ["width", "min-width", "max-width", "height", "min-height", "max-height", "box-sizing"]
+ },
+ {
+ name: "DiffNeedsPaintInvalidationObject",
+ methods_to_diff: [
+ {
+ method: "Visibility()",
+ field_dependencies: ["visibility"]
+ },
+ {
+ method: "PrintColorAdjust()",
+ field_dependencies: ["-webkit-print-color-adjust"]
+ },
+ {
+ method: "InsideLink()",
+ field_dependencies: ["InsideLink"]
+ }
+ ]
+ },
+ {
+ name: "DiffNeedsVisualRectUpdate",
+ methods_to_diff: [
+ {
+ method: "Visibility()",
+ field_dependencies: ["visibility"]
+ }
+ ]
}
],
}

Powered by Google App Engine
This is Rietveld 408576698