Make the LayoutRect->FloatRect constructor explicit.

Source/core/layout/LayoutTextControlSingleLine.cpp

Index: Source/core/layout/LayoutTextControlSingleLine.cpp
diff --git a/Source/core/layout/LayoutTextControlSingleLine.cpp b/Source/core/layout/LayoutTextControlSingleLine.cpp
index e5c9e5e56766beee07abe19e7882681457b35940..1948b2a8188eb7acb815e80ec08d952fa5ab7985 100644
--- a/Source/core/layout/LayoutTextControlSingleLine.cpp
+++ b/Source/core/layout/LayoutTextControlSingleLine.cpp
@@ -89,7 +89,7 @@ void LayoutTextControlSingleLine::paint(const PaintInfo& paintInfo, const Layout
         // Convert the rect into the coords used for painting the content
         contentsRect.moveBy(paintOffset + location());
         IntRect snappedRect = pixelSnappedIntRect(contentsRect);
-        LayoutObjectDrawingRecorder recorder(*paintInfo.context, *this, paintInfo.phase, snappedRect);
+        LayoutObjectDrawingRecorder recorder(*paintInfo.context, *this, paintInfo.phase, LayoutRect(snappedRect));

[jbroman, 2015/08/26 19:06:43]

This is currently going via FloatRect(const IntRect&), not via LayoutRect. And
it happens that FloatRect is what DrawingRecorder ultimately wants. I happen to
think this implicit conversion is probably fine, but if you disagree, suggest
either making this cast to FloatRect instead, or adding an IntRect overload to
the LayoutObjectDrawingRecorder constructor.

[chrishtr, 2015/08/26 20:53:51]

Reverted.

         LayoutTheme::theme().painter().paintCapsLockIndicator(this, paintInfo, snappedRect);
     }
 }