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: Source/core/rendering/RenderTable.cpp

Issue 334373002: Clear absolute clip rects when transform changes (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Split updateTransform into two parts 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 545 matching lines...) Expand 10 before | Expand all | Expand 10 after
556 layoutCaption(m_captions[i]); 556 layoutCaption(m_captions[i]);
557 } 557 }
558 558
559 if (isOutOfFlowPositioned()) 559 if (isOutOfFlowPositioned())
560 updateLogicalHeight(); 560 updateLogicalHeight();
561 561
562 // table can be containing block of positioned elements. 562 // table can be containing block of positioned elements.
563 // FIXME: Only pass true if width or height changed. 563 // FIXME: Only pass true if width or height changed.
564 layoutPositionedObjects(true); 564 layoutPositionedObjects(true);
565 565
566 updateLayerTransform(); 566 updateLayerTransformAfterLayout();
567 567
568 // Layout was changed, so probably borders too. 568 // Layout was changed, so probably borders too.
569 invalidateCollapsedBorders(); 569 invalidateCollapsedBorders();
570 570
571 computeOverflow(clientLogicalBottom()); 571 computeOverflow(clientLogicalBottom());
572 } 572 }
573 573
574 // FIXME: This value isn't the intrinsic content logical height, but we need 574 // FIXME: This value isn't the intrinsic content logical height, but we need
575 // to update the value as its used by flexbox layout. crbug.com/367324 575 // to update the value as its used by flexbox layout. crbug.com/367324
576 updateIntrinsicContentLogicalHeight(contentLogicalHeight()); 576 updateIntrinsicContentLogicalHeight(contentLogicalHeight());
(...skipping 876 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