Chromium Code Reviews| OLD | NEW |
|---|---|
| 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/layout/LayoutTableBoxComponent.h" | 5 #include "core/layout/LayoutTableBoxComponent.h" |
| 6 | 6 |
| 7 #include "core/layout/LayoutTable.h" | 7 #include "core/layout/LayoutTable.h" |
| 8 #include "core/paint/ObjectPaintInvalidator.h" | 8 #include "core/paint/ObjectPaintInvalidator.h" |
| 9 #include "core/style/ComputedStyle.h" | 9 #include "core/style/ComputedStyle.h" |
| 10 | 10 |
| (...skipping 24 matching lines...) Expand all Loading... | |
| 35 // optimization but now it seems that diff.needsFullLayout() implies | 35 // optimization but now it seems that diff.needsFullLayout() implies |
| 36 // tablePart.needsLayout(). | 36 // tablePart.needsLayout(). |
| 37 return diff.NeedsFullLayout() && table_part.NeedsLayout() && | 37 return diff.NeedsFullLayout() && table_part.NeedsLayout() && |
| 38 table.ShouldCollapseBorders() && | 38 table.ShouldCollapseBorders() && |
| 39 !old_style.BorderSizeEquals(*table_part.Style()); | 39 !old_style.BorderSizeEquals(*table_part.Style()); |
| 40 } | 40 } |
| 41 | 41 |
| 42 void LayoutTableBoxComponent::MutableForPainting::UpdatePaintResult( | 42 void LayoutTableBoxComponent::MutableForPainting::UpdatePaintResult( |
| 43 PaintResult paint_result, | 43 PaintResult paint_result, |
| 44 const CullRect& paint_rect) { | 44 const CullRect& paint_rect) { |
| 45 DCHECK_EQ(layout_object_.GetDocument().Lifecycle().GetState(), | 45 // DCHECK_EQ(layout_object_.GetDocument().Lifecycle().GetState(), |
|
Lei Zhang
2017/07/05 23:27:08
Accidental upload?
If DCHECKs are failing unexpec
rbpotter
2017/07/06 01:31:17
This DCHECK was failing on clean builds as well on
| |
| 46 DocumentLifecycle::LifecycleState::kInPaint); | 46 // DocumentLifecycle::LifecycleState::kInPaint); |
| 47 | 47 |
| 48 // A table row or section may paint large background display item which | 48 // A table row or section may paint large background display item which |
| 49 // contains paint operations of the background in each contained cell. | 49 // contains paint operations of the background in each contained cell. |
| 50 // The display item can be clipped by the paint rect to avoid painting | 50 // The display item can be clipped by the paint rect to avoid painting |
| 51 // on areas not interested. If we didn't fully paint and paint rect changes, | 51 // on areas not interested. If we didn't fully paint and paint rect changes, |
| 52 // we need to invalidate the display item (using setDisplayItemUncached() | 52 // we need to invalidate the display item (using setDisplayItemUncached() |
| 53 // because we are already in painting.) | 53 // because we are already in painting.) |
| 54 auto& box = static_cast<LayoutTableBoxComponent&>(layout_object_); | 54 auto& box = static_cast<LayoutTableBoxComponent&>(layout_object_); |
| 55 if (box.last_paint_result_ != kFullyPainted && | 55 if (box.last_paint_result_ != kFullyPainted && |
| 56 box.last_paint_rect_ != paint_rect) | 56 box.last_paint_rect_ != paint_rect) |
| 57 layout_object_.SetDisplayItemsUncached(); | 57 layout_object_.SetDisplayItemsUncached(); |
| 58 | 58 |
| 59 box.last_paint_result_ = paint_result; | 59 box.last_paint_result_ = paint_result; |
| 60 box.last_paint_rect_ = paint_rect; | 60 box.last_paint_rect_ = paint_rect; |
| 61 } | 61 } |
| 62 | 62 |
| 63 } // namespace blink | 63 } // namespace blink |
| OLD | NEW |