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

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 push function to match old behavior... 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 407 matching lines...) Expand 10 before | Expand all | Expand 10 after
418 LayoutRepainter repainter(*this, checkForPaintInvalidationDuringLayout()); 418 LayoutRepainter repainter(*this, checkForPaintInvalidationDuringLayout());
419 SubtreeLayoutScope layouter(*this); 419 SubtreeLayoutScope layouter(*this);
420 420
421 421
422 // If any table section moved vertically, we will just repaint everything fr om that 422 // If any table section moved vertically, we will just repaint everything fr om that
423 // section down (it is quite unlikely that any of the following sections 423 // section down (it is quite unlikely that any of the following sections
424 // did not shift). 424 // did not shift).
425 bool sectionMoved = false; 425 bool sectionMoved = false;
426 LayoutUnit movedSectionLogicalTop = 0; 426 LayoutUnit movedSectionLogicalTop = 0;
427 { 427 {
428 LayoutStateMaintainer statePusher(*this, locationOffset()); 428 LayoutState state(*this, locationOffset());
429 429
430 setLogicalHeight(0); 430 setLogicalHeight(0);
431 431
432 LayoutUnit oldLogicalWidth = logicalWidth(); 432 LayoutUnit oldLogicalWidth = logicalWidth();
433 updateLogicalWidth(); 433 updateLogicalWidth();
434 434
435 if (logicalWidth() != oldLogicalWidth) { 435 if (logicalWidth() != oldLogicalWidth) {
436 for (unsigned i = 0; i < m_captions.size(); i++) 436 for (unsigned i = 0; i < m_captions.size(); i++)
437 layouter.setNeedsLayout(m_captions[i]); 437 layouter.setNeedsLayout(m_captions[i]);
438 } 438 }
(...skipping 1004 matching lines...) Expand 10 before | Expand all | Expand 10 after
1443 const BorderValue& RenderTable::tableEndBorderAdjoiningCell(const RenderTableCel l* cell) const 1443 const BorderValue& RenderTable::tableEndBorderAdjoiningCell(const RenderTableCel l* cell) const
1444 { 1444 {
1445 ASSERT(cell->isFirstOrLastCellInRow()); 1445 ASSERT(cell->isFirstOrLastCellInRow());
1446 if (hasSameDirectionAs(cell->row())) 1446 if (hasSameDirectionAs(cell->row()))
1447 return style()->borderEnd(); 1447 return style()->borderEnd();
1448 1448
1449 return style()->borderStart(); 1449 return style()->borderStart();
1450 } 1450 }
1451 1451
1452 } 1452 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698