Clean up obsolete LayoutUnit.rawValue() FIXME comments.

third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp

 /*
  * (C) 1999 Lars Knoll (knoll@kde.org)
  * (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  *
  * 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 62 matching lines @@
     // FIXME: Is it actually possible to try and paint a dirty InlineTextBox?
     InlineTextBoxPainter::removeFromTextBlobCache(*this);
 
     m_len = 0;
     m_start = 0;
     InlineBox::markDirty();
 }
 
 LayoutRect InlineTextBox::logicalOverflowRect() const
 {
     if (knownToHaveNoOverflow() || !gTextBoxesWithOverflow) {

[leviw_travelin_and_unemployed, 2016/05/11 00:49:24]

And remove those braces! :)

[wkorman, 2016/05/15 22:01:03]

Done.

-        // FIXME: the call to rawValue() below is temporary and should be removed once the transition
-        // to LayoutUnit-based types is complete (crbug.com/321237). The call to enclosingIntRect()
-        // should also likely be switched to LayoutUnit pixel-snapping.
-        return LayoutRect(enclosingIntRect(logicalFrameRect()));
+        return logicalFrameRect();
     }
 
     return gTextBoxesWithOverflow->get(this);
 }
 
 void InlineTextBox::setLogicalOverflowRect(const LayoutRect& rect)
 {
     ASSERT(!knownToHaveNoOverflow());
     if (!gTextBoxesWithOverflow)
         gTextBoxesWithOverflow = new InlineTextBoxOverflowMap;
@@ skipping 132 matching lines @@
 
     StringBuilder charactersWithHyphen;
     bool respectHyphen = ePos == m_len && hasHyphen();
     TextRun textRun = constructTextRun(styleToUse, font, respectHyphen ? &charactersWithHyphen : 0);
 
     LayoutPoint startingPoint = LayoutPoint(logicalLeft(), selTop);
     LayoutRect r;
     if (sPos || ePos != static_cast<int>(m_len)) {
         r = LayoutRect(enclosingIntRect(font.selectionRectForText(textRun, FloatPoint(startingPoint), selHeight, sPos, ePos)));
     } else { // Avoid computing the font width when the entire line box is selected as an optimization.
-        // FIXME: the call to rawValue() below is temporary and should be removed once the transition
-        // to LayoutUnit-based types is complete (crbug.com/321237)
         r = LayoutRect(enclosingIntRect(LayoutRect(startingPoint, LayoutSize(m_logicalWidth, selHeight))));
     }
 
     LayoutUnit logicalWidth = r.width();
     if (r.x() > logicalRight())
         logicalWidth = LayoutUnit();
     else if (r.maxX() > logicalRight())
         logicalWidth = logicalRight() - r.x();
 
     LayoutPoint topPoint;
@@ skipping 374 matching lines @@
     printedCharacters = fprintf(stderr, "\t%s %p", obj.name(), obj.debugPointer());
     const int layoutObjectCharacterOffset = 75;
     for (; printedCharacters < layoutObjectCharacterOffset; printedCharacters++)
         fputc(' ', stderr);
     fprintf(stderr, "(%d,%d) \"%s\"\n", start(), start() + len(), value.utf8().data());
 }
 
 #endif
 
 } // namespace blink