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

Side by Side 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 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/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
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
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698