Use IndentTextOrNot instead of a bool

third_party/WebKit/Source/core/layout/line/BreakingContextInlineHeaders.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All right reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 196 matching lines @@
         return false;
 
     if (!shouldCollapseWhiteSpace(it.object().styleRef(), lineInfo, whitespacePosition) || it.object().isBR())
         return true;
 
     UChar current = it.current();
     bool notJustWhitespace = current != spaceCharacter && current != tabulationCharacter && current != softHyphenCharacter && (current != newlineCharacter || it.object().preservesNewline());
     return notJustWhitespace || isEmptyInline(it.object());
 }
 
-inline void setStaticPositions(LineLayoutBlockFlow block, LineLayoutBox child, bool shouldIndentText)
+inline void setStaticPositions(LineLayoutBlockFlow block, LineLayoutBox child, IndentTextOrNot indentText)
 {
     ASSERT(child.isOutOfFlowPositioned());
     // FIXME: The math here is actually not really right. It's a best-guess approximation that
     // will work for the common cases
     LineLayoutItem containerBlock = child.container();
     LayoutUnit blockHeight = block.logicalHeight();
     if (containerBlock.isLayoutInline()) {
         // A relative positioned inline encloses us. In this case, we also have to determine our
         // position as though we were an inline. Set |staticInlinePosition| and |staticBlockPosition| on the relative positioned
         // inline so that we can obtain the value later.
-        LineLayoutInline(containerBlock).layer()->setStaticInlinePosition(block.startAlignedOffsetForLine(blockHeight, shouldIndentText));
+        LineLayoutInline(containerBlock).layer()->setStaticInlinePosition(block.startAlignedOffsetForLine(blockHeight, indentText));
         LineLayoutInline(containerBlock).layer()->setStaticBlockPosition(blockHeight);
 
         // If |child| is a leading or trailing positioned object this is its only opportunity to ensure it moves with an inline
         // container changing width.
         child.moveWithEdgeOfInlineContainerIfNecessary(child.isHorizontalWritingMode());
     }
-    block.updateStaticInlinePositionForChild(child, blockHeight, shouldIndentText);
+    block.updateStaticInlinePositionForChild(child, blockHeight, indentText);
     child.layer()->setStaticBlockPosition(blockHeight);
 }
 
 // FIXME: The entire concept of the skipTrailingWhitespace function is flawed, since we really need to be building
 // line boxes even for containers that may ultimately collapse away. Otherwise we'll never get positioned
 // elements quite right. In other words, we need to build this function's work into the normal line
 // object iteration process.
 // NB. this function will insert any floating elements that would otherwise
 // be skipped but it will not position them.
 inline void BreakingContext::skipTrailingWhitespace(InlineIterator& iterator, const LineInfo& lineInfo)
 {
     while (!iterator.atEnd() && !requiresLineBox(iterator, lineInfo, TrailingWhitespace)) {
         LineLayoutItem item = iterator.object();
         if (item.isOutOfFlowPositioned())
-            setStaticPositions(m_block, LineLayoutBox(item), false);
+            setStaticPositions(m_block, LineLayoutBox(item), DoNotIndentText);
         else if (item.isFloating())
             m_block.insertFloatingObject(LineLayoutBox(item));
         iterator.increment();
     }
 }
 
 inline void BreakingContext::initializeForCurrentObject()
 {
     m_currentStyle = m_current.object().style();
     m_nextObject = bidiNextSkippingEmptyInlines(m_block, m_current.object());
@@ skipping 706 matching lines @@
         if (m_lastObject.isAtomicInlineLevel() && m_autoWrap && (!m_lastObject.isImage() || m_allowImagesToBreak) && (!m_lastObject.isListMarker() || LineLayoutListMarker(m_lastObject).isInside())
             && !m_lastObject.isRubyRun()) {
             m_width.commit();
             m_lineBreak.moveToStartOf(m_nextObject);
         }
     }
 }
 
 inline IndentTextOrNot requiresIndent(bool isFirstLine, bool isAfterHardLineBreak, const ComputedStyle& style)
 {
-    IndentTextOrNot shouldIndentText = DoNotIndentText;
+    IndentTextOrNot indentText = DoNotIndentText;
     if (isFirstLine || (isAfterHardLineBreak && style.textIndentLine()) == TextIndentEachLine)
-        shouldIndentText = IndentText;
+        indentText = IndentText;
 
     if (style.textIndentType() == TextIndentHanging)
-        shouldIndentText = shouldIndentText == IndentText ? DoNotIndentText : IndentText;
+        indentText = indentText == IndentText ? DoNotIndentText : IndentText;
 
-    return shouldIndentText;
+    return indentText;
 }
 
 }
 
 #endif // BreakingContextInlineHeaders_h