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

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

Issue 2394683005: Remove ASSERT_UNUSED (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 /* 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, 2007, 2009, 2013 Apple Inc. All rights reserved. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2009, 2013 Apple Inc. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 235 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 246
247 const BorderValue& borderAdjoiningTableEnd() const { 247 const BorderValue& borderAdjoiningTableEnd() const {
248 ASSERT(isFirstOrLastCellInRow()); 248 ASSERT(isFirstOrLastCellInRow());
249 if (section()->hasSameDirectionAs(table())) 249 if (section()->hasSameDirectionAs(table()))
250 return style()->borderEnd(); 250 return style()->borderEnd();
251 251
252 return style()->borderStart(); 252 return style()->borderStart();
253 } 253 }
254 254
255 const BorderValue& borderAdjoiningCellBefore(const LayoutTableCell* cell) { 255 const BorderValue& borderAdjoiningCellBefore(const LayoutTableCell* cell) {
256 ASSERT_UNUSED(cell, table()->cellAfter(cell) == this); 256 DCHECK_EQ(table()->cellAfter(cell), this);
257 // FIXME: https://webkit.org/b/79272 - Add support for mixed directionality at the cell level. 257 // FIXME: https://webkit.org/b/79272 - Add support for mixed directionality at the cell level.
258 return style()->borderStart(); 258 return style()->borderStart();
259 } 259 }
260 260
261 const BorderValue& borderAdjoiningCellAfter(const LayoutTableCell* cell) { 261 const BorderValue& borderAdjoiningCellAfter(const LayoutTableCell* cell) {
262 ASSERT_UNUSED(cell, table()->cellBefore(cell) == this); 262 DCHECK_EQ(table()->cellBefore(cell), this);
263 // FIXME: https://webkit.org/b/79272 - Add support for mixed directionality at the cell level. 263 // FIXME: https://webkit.org/b/79272 - Add support for mixed directionality at the cell level.
264 return style()->borderEnd(); 264 return style()->borderEnd();
265 } 265 }
266 266
267 #if ENABLE(ASSERT) 267 #if ENABLE(ASSERT)
268 bool isFirstOrLastCellInRow() const { 268 bool isFirstOrLastCellInRow() const {
269 return !table()->cellAfter(this) || !table()->cellBefore(this); 269 return !table()->cellAfter(this) || !table()->cellBefore(this);
270 } 270 }
271 #endif 271 #endif
272 272
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
409 return toLayoutTableCell(firstChild()); 409 return toLayoutTableCell(firstChild());
410 } 410 }
411 411
412 inline LayoutTableCell* LayoutTableRow::lastCell() const { 412 inline LayoutTableCell* LayoutTableRow::lastCell() const {
413 return toLayoutTableCell(lastChild()); 413 return toLayoutTableCell(lastChild());
414 } 414 }
415 415
416 } // namespace blink 416 } // namespace blink
417 417
418 #endif // LayoutTableCell_h 418 #endif // LayoutTableCell_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698