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

Side by Side Diff: Source/core/rendering/RenderTable.cpp

Issue 335963002: Change LayoutState to be stack-allocated (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix assert Created 6 years, 6 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 | Annotate | Revision Log
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, 2008, 2009, 2010, 2013 Apple Inc. All rights reserved. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2013 Apple Inc. All rights reserved.
8 * Copyright (C) 2006 Alexey Proskuryakov (ap@nypop.com) 8 * Copyright (C) 2006 Alexey Proskuryakov (ap@nypop.com)
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 417 matching lines...) Expand 10 before | Expand all | Expand 10 after
428 LayoutRepainter repainter(*this, checkForPaintInvalidationDuringLayout()); 428 LayoutRepainter repainter(*this, checkForPaintInvalidationDuringLayout());
429 SubtreeLayoutScope layouter(*this); 429 SubtreeLayoutScope layouter(*this);
430 430
431 431
432 // If any table section moved vertically, we will just repaint everything fr om that 432 // If any table section moved vertically, we will just repaint everything fr om that
433 // section down (it is quite unlikely that any of the following sections 433 // section down (it is quite unlikely that any of the following sections
434 // did not shift). 434 // did not shift).
435 bool sectionMoved = false; 435 bool sectionMoved = false;
436 LayoutUnit movedSectionLogicalTop = 0; 436 LayoutUnit movedSectionLogicalTop = 0;
437 { 437 {
438 LayoutStateMaintainer statePusher(*this, locationOffset()); 438 LayoutState statePusher(*this, locationOffset());
439 439
440 setLogicalHeight(0); 440 setLogicalHeight(0);
441 441
442 LayoutUnit oldLogicalWidth = logicalWidth(); 442 LayoutUnit oldLogicalWidth = logicalWidth();
443 updateLogicalWidth(); 443 updateLogicalWidth();
444 444
445 if (logicalWidth() != oldLogicalWidth) { 445 if (logicalWidth() != oldLogicalWidth) {
446 for (unsigned i = 0; i < m_captions.size(); i++) 446 for (unsigned i = 0; i < m_captions.size(); i++)
447 layouter.setNeedsLayout(m_captions[i]); 447 layouter.setNeedsLayout(m_captions[i]);
448 } 448 }
(...skipping 1004 matching lines...) Expand 10 before | Expand all | Expand 10 after
1453 const BorderValue& RenderTable::tableEndBorderAdjoiningCell(const RenderTableCel l* cell) const 1453 const BorderValue& RenderTable::tableEndBorderAdjoiningCell(const RenderTableCel l* cell) const
1454 { 1454 {
1455 ASSERT(cell->isFirstOrLastCellInRow()); 1455 ASSERT(cell->isFirstOrLastCellInRow());
1456 if (hasSameDirectionAs(cell->row())) 1456 if (hasSameDirectionAs(cell->row()))
1457 return style()->borderEnd(); 1457 return style()->borderEnd();
1458 1458
1459 return style()->borderStart(); 1459 return style()->borderStart();
1460 } 1460 }
1461 1461
1462 } 1462 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698