Chromium Code Reviews| Index: third_party/WebKit/Source/core/style/ComputedStyle.cpp |
| diff --git a/third_party/WebKit/Source/core/style/ComputedStyle.cpp b/third_party/WebKit/Source/core/style/ComputedStyle.cpp |
| index bfb7efe5293c98400e4b136fd40f1c01062127ca..944699fa8ae51ba26847d3bc00a05ad08ca25887 100644 |
| --- a/third_party/WebKit/Source/core/style/ComputedStyle.cpp |
| +++ b/third_party/WebKit/Source/core/style/ComputedStyle.cpp |
| @@ -647,7 +647,8 @@ bool ComputedStyle::DiffNeedsPaintInvalidationObject( |
| return true; |
| if (!BorderVisuallyEqual(other) || !RadiiEqual(other) || |
|
shend
2017/06/14 03:30:54
Can this be generated as well?
nainar
2017/06/14 04:57:02
Going to do this in a separate CL at a future time
|
| - *background_data_ != *other.background_data_) |
| + (BackgroundInternal() != other.BackgroundInternal() || |
| + BackgroundColorInternal() != other.BackgroundColorInternal())) |
| return true; |
| if (rare_non_inherited_data_->paint_images_) { |
| @@ -735,25 +736,17 @@ void ComputedStyle::UpdatePropertySpecificDifferences( |
| other)) |
| diff.SetBackdropFilterChanged(); |
| - if (!diff.NeedsFullPaintInvalidation()) { |
| - if ((inherited_data_->color_ != other.inherited_data_->color_ || |
| - inherited_data_->visited_link_color_ != |
| - other.inherited_data_->visited_link_color_ || |
| - HasSimpleUnderlineInternal() != other.HasSimpleUnderlineInternal() || |
| - visual_data_->text_decoration_ != |
| - other.visual_data_->text_decoration_) || |
| - ComputedStyleBase:: |
| - UpdatePropertySpecificDifferencesTextDecorationOrColor(*this, |
| - other)) { |
| - diff.SetTextDecorationOrColorChanged(); |
| - } |
| + if (!diff.NeedsFullPaintInvalidation() && |
| + ComputedStyleBase::UpdatePropertySpecificDifferencesTextDecorationOrColor( |
| + *this, other)) { |
| + diff.SetTextDecorationOrColorChanged(); |
| } |
| - bool has_clip = HasOutOfFlowPosition() && !visual_data_->has_auto_clip_; |
| + bool has_clip = HasOutOfFlowPosition() && !HasAutoClipInternal(); |
| bool other_has_clip = |
| - other.HasOutOfFlowPosition() && !other.visual_data_->has_auto_clip_; |
| + other.HasOutOfFlowPosition() && !other.HasAutoClipInternal(); |
| if (has_clip != other_has_clip || |
| - (has_clip && visual_data_->clip_ != other.visual_data_->clip_)) |
| + (has_clip && ClipInternal() != other.ClipInternal())) |
| diff.SetCSSClipChanged(); |
| } |