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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutTableRow.h

Issue 2805103003: Optimize collapsed border calculation (step 2) (Closed)
Patch Set: - Created 3 years, 7 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 /* 1 /*
2 * Copyright (C) 1997 Martin Jones (mjones@kde.org) 2 * Copyright (C) 1997 Martin Jones (mjones@kde.org)
3 * (C) 1997 Torben Weis (weis@kde.org) 3 * (C) 1997 Torben Weis (weis@kde.org)
4 * (C) 1998 Waldo Bastian (bastian@kde.org) 4 * (C) 1998 Waldo Bastian (bastian@kde.org)
5 * (C) 1999 Lars Knoll (knoll@kde.org) 5 * (C) 1999 Lars Knoll (knoll@kde.org)
6 * (C) 1999 Antti Koivisto (koivisto@kde.org) 6 * (C) 1999 Antti Koivisto (koivisto@kde.org)
7 * Copyright (C) 2003, 2004, 2005, 2006, 2009, 2013 Apple Inc. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2009, 2013 Apple Inc.
8 * All rights reserved. 8 * All rights reserved.
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 return kNoPaintLayer; 168 return kNoPaintLayer;
169 } 169 }
170 170
171 void Paint(const PaintInfo&, const LayoutPoint&) const override; 171 void Paint(const PaintInfo&, const LayoutPoint&) const override;
172 172
173 void StyleDidChange(StyleDifference, const ComputedStyle* old_style) override; 173 void StyleDidChange(StyleDifference, const ComputedStyle* old_style) override;
174 174
175 void NextSibling() const = delete; 175 void NextSibling() const = delete;
176 void PreviousSibling() const = delete; 176 void PreviousSibling() const = delete;
177 177
178 void InvalidateCollapsedBordersOfAffectedCells();
wkorman 2017/04/27 22:05:36 Affected by what? Document method, consider rename
179
178 // This field should never be read directly. It should be read through 180 // This field should never be read directly. It should be read through
179 // rowIndex() above instead. This is to ensure that we never read this 181 // rowIndex() above instead. This is to ensure that we never read this
180 // value before it is set. 182 // value before it is set.
181 unsigned row_index_ : 31; 183 unsigned row_index_ : 31;
182 }; 184 };
183 185
184 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTableRow, IsTableRow()); 186 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTableRow, IsTableRow());
185 187
186 inline LayoutTableRow* LayoutTableRow::PreviousRow() const { 188 inline LayoutTableRow* LayoutTableRow::PreviousRow() const {
187 return ToLayoutTableRow(LayoutObject::PreviousSibling()); 189 return ToLayoutTableRow(LayoutObject::PreviousSibling());
188 } 190 }
189 191
190 inline LayoutTableRow* LayoutTableRow::NextRow() const { 192 inline LayoutTableRow* LayoutTableRow::NextRow() const {
191 return ToLayoutTableRow(LayoutObject::NextSibling()); 193 return ToLayoutTableRow(LayoutObject::NextSibling());
192 } 194 }
193 195
194 inline LayoutTableRow* LayoutTableSection::FirstRow() const { 196 inline LayoutTableRow* LayoutTableSection::FirstRow() const {
195 return ToLayoutTableRow(FirstChild()); 197 return ToLayoutTableRow(FirstChild());
196 } 198 }
197 199
198 inline LayoutTableRow* LayoutTableSection::LastRow() const { 200 inline LayoutTableRow* LayoutTableSection::LastRow() const {
199 return ToLayoutTableRow(LastChild()); 201 return ToLayoutTableRow(LastChild());
200 } 202 }
201 203
202 } // namespace blink 204 } // namespace blink
203 205
204 #endif // LayoutTableRow_h 206 #endif // LayoutTableRow_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698