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/style/ComputedStyle.h" | 8 #include "core/style/ComputedStyle.h" |
9 | 9 |
10 namespace blink { | 10 namespace blink { |
(...skipping 13 matching lines...) Expand all Loading... |
24 void LayoutTableBoxComponent::imageChanged(WrappedImagePtr, const IntRect*) { | 24 void LayoutTableBoxComponent::imageChanged(WrappedImagePtr, const IntRect*) { |
25 setShouldDoFullPaintInvalidation(); | 25 setShouldDoFullPaintInvalidation(); |
26 m_backgroundChangedSinceLastPaintInvalidation = true; | 26 m_backgroundChangedSinceLastPaintInvalidation = true; |
27 } | 27 } |
28 | 28 |
29 bool LayoutTableBoxComponent::doCellsHaveDirtyWidth( | 29 bool LayoutTableBoxComponent::doCellsHaveDirtyWidth( |
30 const LayoutObject& tablePart, | 30 const LayoutObject& tablePart, |
31 const LayoutTable& table, | 31 const LayoutTable& table, |
32 const StyleDifference& diff, | 32 const StyleDifference& diff, |
33 const ComputedStyle& oldStyle) { | 33 const ComputedStyle& oldStyle) { |
34 // ComputedStyle::diffNeedsFullLayoutAndPaintInvalidation sets needsFullLayout
when border sizes | 34 // ComputedStyle::diffNeedsFullLayoutAndPaintInvalidation sets needsFullLayout |
35 // change: checking diff.needsFullLayout() is an optimization, not required fo
r correctness. | 35 // when border sizes change: checking diff.needsFullLayout() is an |
36 // TODO(dgrogan): Remove tablePart.needsLayout()? Perhaps it was an old optimi
zation but now it | 36 // optimization, not required for correctness. |
37 // seems that diff.needsFullLayout() implies tablePart.needsLayout(). | 37 // TODO(dgrogan): Remove tablePart.needsLayout()? Perhaps it was an old |
| 38 // optimization but now it seems that diff.needsFullLayout() implies |
| 39 // tablePart.needsLayout(). |
38 return diff.needsFullLayout() && tablePart.needsLayout() && | 40 return diff.needsFullLayout() && tablePart.needsLayout() && |
39 table.collapseBorders() && | 41 table.collapseBorders() && |
40 !oldStyle.border().sizeEquals(tablePart.style()->border()); | 42 !oldStyle.border().sizeEquals(tablePart.style()->border()); |
41 } | 43 } |
42 | 44 |
43 } // namespace blink | 45 } // namespace blink |
OLD | NEW |