Convert some call sites to use explicit LayoutRect->FloatRect conversion.

Source/core/editing/markers/DocumentMarkerControllerTest.cpp

Index: Source/core/editing/markers/DocumentMarkerControllerTest.cpp
diff --git a/Source/core/editing/markers/DocumentMarkerControllerTest.cpp b/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
index 79fcfd7786feba7c2bc766fc5fe4ac5155f11bc9..3c5fdee4e95e32034675efaad0b63b4520ad8d54 100644
--- a/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
+++ b/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
@@ -201,7 +201,7 @@ TEST_F(DocumentMarkerControllerTest, NodeWillBeRemovedBySetInnerHTML)
 
 TEST_F(DocumentMarkerControllerTest, UpdateRenderedRects)
 {
-    LayoutRect invalidRect = RenderedDocumentMarker::create(DocumentMarker(0, 0, false))->renderedRect();
+    IntRect invalidRect(RenderedDocumentMarker::create(DocumentMarker(0, 0, false))->renderedRect());

[eae, 2015/08/25 17:47:16]

renderRect() returns a LayoutRect, this implicitly down converts that to an
IntRect which doesn't seem right. 

Explicitly calling pixelSnappedIntRect or enclosingIntRect would be better.

[chrishtr, 2015/08/25 18:30:43]

It's implicit in the same way that renderedRectsForMarkers is implicit, because
it builds a vector of IntRects that are constructed by calling
results.append() with a LayoutRect:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

Should there be a followup task to remove the IntRect constructor that takes in
a LayoutRect?

 
     setBodyInnerHTML("<div style='margin: 100px'>foo</div>");
     RefPtrWillBeRawPtr<Element> div = toElement(document().body()->firstChild());