| OLD | NEW |
| 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 Apple Inc. All rights reserved. | 7 * Copyright (C) 2003, 2004, 2005, 2006, 2009 Apple Inc. All rights reserved. |
| 8 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) | 8 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) |
| 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 42 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 53 | 53 |
| 54 LayoutTable* table = this->table(); | 54 LayoutTable* table = this->table(); |
| 55 if (!table) | 55 if (!table) |
| 56 return; | 56 return; |
| 57 | 57 |
| 58 // TODO(dgrogan): Is the "else" necessary for correctness or just a brittle | 58 // TODO(dgrogan): Is the "else" necessary for correctness or just a brittle |
| 59 // optimization? The optimization would be: if the first branch is taken then | 59 // optimization? The optimization would be: if the first branch is taken then |
| 60 // the next one can't be, so don't even check its condition. | 60 // the next one can't be, so don't even check its condition. |
| 61 if (!table->selfNeedsLayout() && !table->normalChildNeedsLayout() && | 61 if (!table->selfNeedsLayout() && !table->normalChildNeedsLayout() && |
| 62 oldStyle->border() != style()->border()) { | 62 oldStyle->border() != style()->border()) { |
| 63 table->invalidateCollapsedBorders(); | 63 table->invalidateCollapsedBorders(PaintInvalidationStyleChange); |
| 64 } else if ((oldStyle->logicalWidth() != style()->logicalWidth()) || | 64 } else if ((oldStyle->logicalWidth() != style()->logicalWidth()) || |
| 65 LayoutTableBoxComponent::doCellsHaveDirtyWidth(*this, *table, diff, | 65 LayoutTableBoxComponent::doCellsHaveDirtyWidth(*this, *table, diff, |
| 66 *oldStyle)) { | 66 *oldStyle)) { |
| 67 // TODO(dgrogan): Optimization opportunities: | 67 // TODO(dgrogan): Optimization opportunities: |
| 68 // (1) Only mark cells which are affected by this col, not every cell in the | 68 // (1) Only mark cells which are affected by this col, not every cell in the |
| 69 // table. | 69 // table. |
| 70 // (2) If only the col width changes and its border width doesn't, do the | 70 // (2) If only the col width changes and its border width doesn't, do the |
| 71 // cells need to be marked as needing layout or just given dirty | 71 // cells need to be marked as needing layout or just given dirty |
| 72 // widths? | 72 // widths? |
| 73 table->markAllCellsWidthsDirtyAndOrNeedsLayout( | 73 table->markAllCellsWidthsDirtyAndOrNeedsLayout( |
| (...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 193 const BorderValue& LayoutTableCol::borderAdjoiningCellAfter( | 193 const BorderValue& LayoutTableCol::borderAdjoiningCellAfter( |
| 194 const LayoutTableCell* cell) const { | 194 const LayoutTableCell* cell) const { |
| 195 DCHECK_EQ(table() | 195 DCHECK_EQ(table() |
| 196 ->colElementAtAbsoluteColumn(cell->absoluteColumnIndex() - 1) | 196 ->colElementAtAbsoluteColumn(cell->absoluteColumnIndex() - 1) |
| 197 .innermostColOrColGroup(), | 197 .innermostColOrColGroup(), |
| 198 this); | 198 this); |
| 199 return style()->borderEnd(); | 199 return style()->borderEnd(); |
| 200 } | 200 } |
| 201 | 201 |
| 202 } // namespace blink | 202 } // namespace blink |
| OLD | NEW |