Make the LayoutRect->FloatRect constructor explicit.

Source/core/paint/LayerClipRecorderTest.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/LayerClipRecorder.h"
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/compositing/DeprecatedPaintLayerCompositor.h"
@@ skipping 30 matching lines @@
     }
     void TearDown() override
     {
         RuntimeEnabledFeatures::setSlimmingPaintEnabled(m_originalSlimmingPaintEnabled);
     }
 
     LayoutView* m_layoutView;
     bool m_originalSlimmingPaintEnabled;
 };
 
-void drawEmptyClip(GraphicsContext& context, LayoutView& layoutView, PaintPhase phase, const FloatRect& bound)
+void drawEmptyClip(GraphicsContext& context, LayoutView& layoutView, PaintPhase phase)
 {
     LayoutRect rect(1, 1, 9, 9);
     ClipRect clipRect(rect);
     LayerClipRecorder LayerClipRecorder(context, *layoutView.compositor()->rootLayer()->layoutObject(), DisplayItem::ClipLayerForeground, clipRect, 0, LayoutPoint(), PaintLayerFlags());
 }
 
-void drawRectInClip(GraphicsContext& context, LayoutView& layoutView, PaintPhase phase, const FloatRect& bound)
+void drawRectInClip(GraphicsContext& context, LayoutView& layoutView, PaintPhase phase, const LayoutRect& bound)

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

Unsure, but maybe I think should stay FloatRect; why would you like to change
it?

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

Same reply as for other test.

 {
     IntRect rect(1, 1, 9, 9);
     ClipRect clipRect((LayoutRect(rect)));
     LayerClipRecorder LayerClipRecorder(context, *layoutView.compositor()->rootLayer()->layoutObject(), DisplayItem::ClipLayerForeground, clipRect, 0, LayoutPoint(), PaintLayerFlags());
     if (!LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(context, layoutView, phase)) {
         LayoutObjectDrawingRecorder drawingRecorder(context, layoutView, phase, bound);
         context.drawRect(rect);
     }
 }
 
 TEST_F(LayerClipRecorderTest, Single)
 {
     GraphicsContext context(&rootDisplayItemList());
-    FloatRect bound = layoutView().viewRect();
+    LayoutRect bound = layoutView().viewRect();
     EXPECT_EQ((size_t)0, rootDisplayItemList().displayItems().size());
 
     drawRectInClip(context, layoutView(), PaintPhaseForeground, bound);
     rootDisplayItemList().commitNewDisplayItems();
     EXPECT_EQ((size_t)3, rootDisplayItemList().displayItems().size());
     EXPECT_TRUE(DisplayItem::isClipType(rootDisplayItemList().displayItems()[0].type()));
     EXPECT_TRUE(DisplayItem::isDrawingType(rootDisplayItemList().displayItems()[1].type()));
     EXPECT_TRUE(DisplayItem::isEndClipType(rootDisplayItemList().displayItems()[2].type()));
 }
 
 TEST_F(LayerClipRecorderTest, Empty)
 {
     GraphicsContext context(&rootDisplayItemList());
-    FloatRect bound = layoutView().viewRect();
     EXPECT_EQ((size_t)0, rootDisplayItemList().displayItems().size());
 
-    drawEmptyClip(context, layoutView(), PaintPhaseForeground, bound);
+    drawEmptyClip(context, layoutView(), PaintPhaseForeground);
     rootDisplayItemList().commitNewDisplayItems();
     EXPECT_EQ((size_t)0, rootDisplayItemList().displayItems().size());
 }
 
 }
 }