Index: third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp |
diff --git a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp |
index c4cfa941714f96c5d7629638f28ceb0438da46cd..f9d1dbdd61f35a056a406fe239ef331d5fe30163 100644 |
--- a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp |
+++ b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp |
@@ -31,10 +31,12 @@ bool LayoutTableBoxComponent::doCellsHaveDirtyWidth( |
const LayoutTable& table, |
const StyleDifference& diff, |
const ComputedStyle& oldStyle) { |
- // ComputedStyle::diffNeedsFullLayoutAndPaintInvalidation sets needsFullLayout when border sizes |
- // change: checking diff.needsFullLayout() is an optimization, not required for correctness. |
- // TODO(dgrogan): Remove tablePart.needsLayout()? Perhaps it was an old optimization but now it |
- // seems that diff.needsFullLayout() implies tablePart.needsLayout(). |
+ // ComputedStyle::diffNeedsFullLayoutAndPaintInvalidation sets needsFullLayout |
+ // when border sizes change: checking diff.needsFullLayout() is an |
+ // optimization, not required for correctness. |
+ // TODO(dgrogan): Remove tablePart.needsLayout()? Perhaps it was an old |
+ // optimization but now it seems that diff.needsFullLayout() implies |
+ // tablePart.needsLayout(). |
return diff.needsFullLayout() && tablePart.needsLayout() && |
table.collapseBorders() && |
!oldStyle.border().sizeEquals(tablePart.style()->border()); |