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

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

Issue 2884573002: Replace LayoutTableCell::AbsoluteColumnIndex() with EffectiveColumnIndex()
Patch Set: - 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/PaintInvalidator.h" 5 #include "core/paint/PaintInvalidator.h"
6 6
7 #include "core/editing/FrameSelection.h" 7 #include "core/editing/FrameSelection.h"
8 #include "core/frame/FrameView.h" 8 #include "core/frame/FrameView.h"
9 #include "core/frame/LocalFrame.h" 9 #include "core/frame/LocalFrame.h"
10 #include "core/frame/Settings.h" 10 #include "core/frame/Settings.h"
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 if (table.ShouldCollapseBorders() && !table.CollapsedBorders().IsEmpty()) 228 if (table.ShouldCollapseBorders() && !table.CollapsedBorders().IsEmpty())
229 context.painting_layer->SetNeedsPaintPhaseDescendantBlockBackgrounds(); 229 context.painting_layer->SetNeedsPaintPhaseDescendantBlockBackgrounds();
230 } 230 }
231 231
232 // The following flags are for descendants of the layer object only. 232 // The following flags are for descendants of the layer object only.
233 if (object == context.painting_layer->GetLayoutObject()) 233 if (object == context.painting_layer->GetLayoutObject())
234 return; 234 return;
235 235
236 if (object.IsTableSection()) { 236 if (object.IsTableSection()) {
237 const auto& section = ToLayoutTableSection(object); 237 const auto& section = ToLayoutTableSection(object);
238 if (section.Table()->HasColElements()) 238 if (section.Table()->HasColOrColGroups())
239 context.painting_layer->SetNeedsPaintPhaseDescendantBlockBackgrounds(); 239 context.painting_layer->SetNeedsPaintPhaseDescendantBlockBackgrounds();
240 } 240 }
241 241
242 if (object.StyleRef().HasOutline()) 242 if (object.StyleRef().HasOutline())
243 context.painting_layer->SetNeedsPaintPhaseDescendantOutlines(); 243 context.painting_layer->SetNeedsPaintPhaseDescendantOutlines();
244 244
245 if (object.HasBoxDecorationBackground() 245 if (object.HasBoxDecorationBackground()
246 // We also paint overflow controls in background phase. 246 // We also paint overflow controls in background phase.
247 || (object.HasOverflowClip() && 247 || (object.HasOverflowClip() &&
248 ToLayoutBox(object).GetScrollableArea()->HasOverflowControls())) { 248 ToLayoutBox(object).GetScrollableArea()->HasOverflowControls())) {
(...skipping 275 matching lines...) Expand 10 before | Expand all | Expand 10 after
524 524
525 void PaintInvalidator::ProcessPendingDelayedPaintInvalidations() { 525 void PaintInvalidator::ProcessPendingDelayedPaintInvalidations() {
526 for (auto target : pending_delayed_paint_invalidations_) { 526 for (auto target : pending_delayed_paint_invalidations_) {
527 target->GetMutableForPainting() 527 target->GetMutableForPainting()
528 .SetShouldDoFullPaintInvalidationWithoutGeometryChange( 528 .SetShouldDoFullPaintInvalidationWithoutGeometryChange(
529 PaintInvalidationReason::kDelayedFull); 529 PaintInvalidationReason::kDelayedFull);
530 } 530 }
531 } 531 }
532 532
533 } // namespace blink 533 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698