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

Side by Side Diff: third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp

Issue 2872423002: Tweak PaintInvalidationReasons (Closed)
Patch Set: Rebaseline-cl 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 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/paint/PaintPropertyTreeBuilder.h" 5 #include "core/paint/PaintPropertyTreeBuilder.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include "core/dom/DOMNodeIds.h" 8 #include "core/dom/DOMNodeIds.h"
9 #include "core/frame/FrameView.h" 9 #include "core/frame/FrameView.h"
10 #include "core/frame/LocalFrame.h" 10 #include "core/frame/LocalFrame.h"
(...skipping 1128 matching lines...) Expand 10 before | Expand all | Expand 10 after
1139 force_subtree_update); 1139 force_subtree_update);
1140 } 1140 }
1141 1141
1142 if (object.PaintOffset() != context.current.paint_offset) { 1142 if (object.PaintOffset() != context.current.paint_offset) {
1143 // Many paint properties depend on paint offset so we force an update of 1143 // Many paint properties depend on paint offset so we force an update of
1144 // the entire subtree on paint offset changes. 1144 // the entire subtree on paint offset changes.
1145 force_subtree_update = true; 1145 force_subtree_update = true;
1146 1146
1147 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) { 1147 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
1148 object.GetMutableForPainting().SetShouldDoFullPaintInvalidation( 1148 object.GetMutableForPainting().SetShouldDoFullPaintInvalidation(
1149 kPaintInvalidationLocationChange); 1149 PaintInvalidationReason::kGeometry);
1150 } 1150 }
1151 object.GetMutableForPainting().SetPaintOffset(context.current.paint_offset); 1151 object.GetMutableForPainting().SetPaintOffset(context.current.paint_offset);
1152 } 1152 }
1153 1153
1154 if (!object.IsBox()) 1154 if (!object.IsBox())
1155 return; 1155 return;
1156 const LayoutBox& box = ToLayoutBox(object); 1156 const LayoutBox& box = ToLayoutBox(object);
1157 if (box.Size() == box.PreviousSize()) 1157 if (box.Size() == box.PreviousSize())
1158 return; 1158 return;
1159 1159
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
1272 context.force_subtree_update); 1272 context.force_subtree_update);
1273 1273
1274 context.force_subtree_update |= object.SubtreeNeedsPaintPropertyUpdate(); 1274 context.force_subtree_update |= object.SubtreeNeedsPaintPropertyUpdate();
1275 } 1275 }
1276 1276
1277 if (object.CanContainAbsolutePositionObjects()) 1277 if (object.CanContainAbsolutePositionObjects())
1278 context.container_for_absolute_position = &object; 1278 context.container_for_absolute_position = &object;
1279 } 1279 }
1280 1280
1281 } // namespace blink 1281 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698