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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp

Issue 2400863005: Reformat comments in core/layout up until LayoutTableRow (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
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());

Powered by Google App Engine
This is Rietveld 408576698