| Index: third_party/WebKit/Source/core/layout/LayoutListMarker.cpp
|
| diff --git a/third_party/WebKit/Source/core/layout/LayoutListMarker.cpp b/third_party/WebKit/Source/core/layout/LayoutListMarker.cpp
|
| index 73f4bbb8e0d2d7e920249a15704fe15036bbf5b6..61881ffab858d3153c4e31b2ce3dcd77a2334784 100644
|
| --- a/third_party/WebKit/Source/core/layout/LayoutListMarker.cpp
|
| +++ b/third_party/WebKit/Source/core/layout/LayoutListMarker.cpp
|
| @@ -140,18 +140,18 @@ void LayoutListMarker::layout()
|
| setHeight(imageSize.height());
|
| } else {
|
| setLogicalWidth(minPreferredLogicalWidth());
|
| - setLogicalHeight(style()->fontMetrics().height());
|
| + setLogicalHeight(LayoutUnit(style()->fontMetrics().height()));
|
| }
|
|
|
| - setMarginStart(0);
|
| - setMarginEnd(0);
|
| + setMarginStart(LayoutUnit());
|
| + setMarginEnd(LayoutUnit());
|
|
|
| Length startMargin = style()->marginStart();
|
| Length endMargin = style()->marginEnd();
|
| if (startMargin.isFixed())
|
| - setMarginStart(startMargin.value());
|
| + setMarginStart(LayoutUnit(startMargin.value()));
|
| if (endMargin.isFixed())
|
| - setMarginEnd(endMargin.value());
|
| + setMarginEnd(LayoutUnit(endMargin.value()));
|
|
|
| clearNeedsLayout();
|
| }
|
| @@ -202,14 +202,14 @@ void LayoutListMarker::updateContent()
|
| LayoutUnit LayoutListMarker::getWidthOfTextWithSuffix() const
|
| {
|
| if (m_text.isEmpty())
|
| - return 0;
|
| + return LayoutUnit();
|
| const Font& font = style()->font();
|
| - LayoutUnit itemWidth = font.width(m_text);
|
| + LayoutUnit itemWidth = LayoutUnit(font.width(m_text));
|
| // TODO(wkorman): Look into constructing a text run for both text and suffix
|
| // and painting them together.
|
| UChar suffix[2] = { ListMarkerText::suffix(style()->listStyleType(), m_listItem->value()), ' ' };
|
| TextRun run = constructTextRun(font, suffix, 2, styleRef(), style()->direction());
|
| - LayoutUnit suffixSpaceWidth = font.width(run);
|
| + LayoutUnit suffixSpaceWidth = LayoutUnit(font.width(run));
|
| return itemWidth + suffixSpaceWidth;
|
| }
|
|
|
| @@ -233,7 +233,7 @@ void LayoutListMarker::computePreferredLogicalWidths()
|
| case ListStyleCategory::None:
|
| break;
|
| case ListStyleCategory::Symbol:
|
| - logicalWidth = (font.fontMetrics().ascent() * 2 / 3 + 1) / 2 + 2;
|
| + logicalWidth = LayoutUnit((font.fontMetrics().ascent() * 2 / 3 + 1) / 2 + 2);
|
| break;
|
| case ListStyleCategory::Language:
|
| logicalWidth = getWidthOfTextWithSuffix();
|
| @@ -257,11 +257,11 @@ void LayoutListMarker::updateMargins()
|
|
|
| if (isInside()) {
|
| if (isImage()) {
|
| - marginEnd = cMarkerPaddingPx;
|
| + marginEnd = LayoutUnit(cMarkerPaddingPx);
|
| } else {
|
| switch (listStyleCategory()) {
|
| case ListStyleCategory::Symbol:
|
| - marginStart = -1;
|
| + marginStart = LayoutUnit(-1);
|
| marginEnd = fontMetrics.ascent() - minPreferredLogicalWidth() + 1;
|
| break;
|
| default:
|
| @@ -278,7 +278,7 @@ void LayoutListMarker::updateMargins()
|
| case ListStyleCategory::None:
|
| break;
|
| case ListStyleCategory::Symbol:
|
| - marginStart = -offset - cMarkerPaddingPx - 1;
|
| + marginStart = LayoutUnit(-offset - cMarkerPaddingPx - 1);
|
| break;
|
| default:
|
| marginStart = m_text.isEmpty() ? LayoutUnit() : -minPreferredLogicalWidth();
|
| @@ -287,7 +287,7 @@ void LayoutListMarker::updateMargins()
|
| marginEnd = -marginStart - minPreferredLogicalWidth();
|
| } else {
|
| if (isImage()) {
|
| - marginEnd = cMarkerPaddingPx;
|
| + marginEnd = LayoutUnit(cMarkerPaddingPx);
|
| } else {
|
| int offset = fontMetrics.ascent() * 2 / 3;
|
| switch (listStyleCategory()) {
|
| @@ -297,7 +297,7 @@ void LayoutListMarker::updateMargins()
|
| marginEnd = offset + cMarkerPaddingPx + 1 - minPreferredLogicalWidth();
|
| break;
|
| default:
|
| - marginEnd = 0;
|
| + marginEnd = LayoutUnit();
|
| }
|
| }
|
| marginStart = -marginEnd - minPreferredLogicalWidth();
|
|
|