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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp

Issue 2640143005: Support subpixel layout for borders. (Closed)
Patch Set: Rebaselined tests. Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
index 0dbfa0fcf6ddce106a4d1a3d2b5c07f5b89e327d..fae872606e1fa77fd199bc3e5d4e34111cf34944 100644
--- a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
@@ -638,45 +638,45 @@ LayoutFlexibleBox::getTransformedWritingMode() const {
LayoutUnit LayoutFlexibleBox::flowAwareBorderStart() const {
if (isHorizontalFlow())
- return LayoutUnit(isLeftToRightFlow() ? borderLeft() : borderRight());
- return LayoutUnit(isLeftToRightFlow() ? borderTop() : borderBottom());
+ return isLeftToRightFlow() ? borderLeft() : borderRight();
+ return isLeftToRightFlow() ? borderTop() : borderBottom();
}
LayoutUnit LayoutFlexibleBox::flowAwareBorderEnd() const {
if (isHorizontalFlow())
- return LayoutUnit(isLeftToRightFlow() ? borderRight() : borderLeft());
- return LayoutUnit(isLeftToRightFlow() ? borderBottom() : borderTop());
+ return isLeftToRightFlow() ? borderRight() : borderLeft();
+ return isLeftToRightFlow() ? borderBottom() : borderTop();
}
LayoutUnit LayoutFlexibleBox::flowAwareBorderBefore() const {
switch (getTransformedWritingMode()) {
case TransformedWritingMode::TopToBottomWritingMode:
- return LayoutUnit(borderTop());
+ return borderTop();
case TransformedWritingMode::BottomToTopWritingMode:
- return LayoutUnit(borderBottom());
+ return borderBottom();
case TransformedWritingMode::LeftToRightWritingMode:
- return LayoutUnit(borderLeft());
+ return borderLeft();
case TransformedWritingMode::RightToLeftWritingMode:
- return LayoutUnit(borderRight());
+ return borderRight();
}
NOTREACHED();
- return LayoutUnit(borderTop());
+ return borderTop();
}
DISABLE_CFI_PERF
LayoutUnit LayoutFlexibleBox::flowAwareBorderAfter() const {
switch (getTransformedWritingMode()) {
case TransformedWritingMode::TopToBottomWritingMode:
- return LayoutUnit(borderBottom());
+ return borderBottom();
case TransformedWritingMode::BottomToTopWritingMode:
- return LayoutUnit(borderTop());
+ return borderTop();
case TransformedWritingMode::LeftToRightWritingMode:
- return LayoutUnit(borderRight());
+ return borderRight();
case TransformedWritingMode::RightToLeftWritingMode:
- return LayoutUnit(borderLeft());
+ return borderLeft();
}
NOTREACHED();
- return LayoutUnit(borderTop());
+ return borderTop();
}
LayoutUnit LayoutFlexibleBox::flowAwarePaddingStart() const {

Powered by Google App Engine
This is Rietveld 408576698