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

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

Issue 2528573002: Changed EBorderCollapse to an enum class and renamed its members (Closed)
Patch Set: Rebase Created 4 years 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, 2009, 2010 Apple Inc. 7 * Copyright (C) 2003, 2004, 2005, 2006, 2009, 2010 Apple Inc.
8 * All rights reserved. 8 * All rights reserved.
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 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 explicit LayoutTable(Element*); 137 explicit LayoutTable(Element*);
138 ~LayoutTable() override; 138 ~LayoutTable() override;
139 139
140 // Per CSS 3 writing-mode: "The first and second values of the 140 // Per CSS 3 writing-mode: "The first and second values of the
141 // 'border-spacing' property represent spacing between columns and rows 141 // 'border-spacing' property represent spacing between columns and rows
142 // respectively, not necessarily the horizontal and vertical spacing 142 // respectively, not necessarily the horizontal and vertical spacing
143 // respectively". 143 // respectively".
144 int hBorderSpacing() const { return m_hSpacing; } 144 int hBorderSpacing() const { return m_hSpacing; }
145 int vBorderSpacing() const { return m_vSpacing; } 145 int vBorderSpacing() const { return m_vSpacing; }
146 146
147 bool collapseBorders() const { return style()->borderCollapse(); } 147 bool collapseBorders() const {
148 return style()->borderCollapse() == EBorderCollapse::Collapse;
149 }
148 150
149 int borderStart() const override { return m_borderStart; } 151 int borderStart() const override { return m_borderStart; }
150 int borderEnd() const override { return m_borderEnd; } 152 int borderEnd() const override { return m_borderEnd; }
151 int borderBefore() const override; 153 int borderBefore() const override;
152 int borderAfter() const override; 154 int borderAfter() const override;
153 155
154 int borderLeft() const override { 156 int borderLeft() const override {
155 if (style()->isHorizontalWritingMode()) 157 if (style()->isHorizontalWritingMode())
156 return style()->isLeftToRightDirection() ? borderStart() : borderEnd(); 158 return style()->isLeftToRightDirection() ? borderStart() : borderEnd();
157 return style()->isFlippedBlocksWritingMode() ? borderAfter() 159 return style()->isFlippedBlocksWritingMode() ? borderAfter()
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
582 if (m_firstBody) 584 if (m_firstBody)
583 return m_firstBody; 585 return m_firstBody;
584 return m_foot; 586 return m_foot;
585 } 587 }
586 588
587 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTable, isTable()); 589 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTable, isTable());
588 590
589 } // namespace blink 591 } // namespace blink
590 592
591 #endif // LayoutTable_h 593 #endif // LayoutTable_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698