cc: Stop converting update rect from int to float to int.

Layer::SetNeedsDisplay() would turn the int bounds() into a float RectF
just for PictureLayer to turn it back into an int Rect. This is silly.

If PictureLayer is just going to use ints, we might as well use ints
throughout. So convert update_rect_ from a RectF to a Rect.

R=enne
BUG=342848

Review URL: https://codereview.chromium.org/647253002

Cr-Commit-Position: refs/heads/master@{#299233}

1 files changed, 4 insertions, 5 deletions

diff --git a/cc/layers/picture_layer.cc b/cc/layers/picture_layer.cc
index befef49..7573471 100644
--- a/cc/layers/picture_layer.cc
+++ b/cc/layers/picture_layer.cc
@@ -68,12 +68,11 @@ void PictureLayer::SetLayerTreeHost(LayerTreeHost* host) {
   }
 }
 
-void PictureLayer::SetNeedsDisplayRect(const gfx::RectF& layer_rect) {
-  gfx::Rect rect = gfx::ToEnclosedRect(layer_rect);
-  if (!rect.IsEmpty()) {
+void PictureLayer::SetNeedsDisplayRect(const gfx::Rect& layer_rect) {
+  if (!layer_rect.IsEmpty()) {
     // Clamp invalidation to the layer bounds.
-    rect.Intersect(gfx::Rect(bounds()));
-    pending_invalidation_.Union(rect);
+    pending_invalidation_.Union(
+        gfx::IntersectRects(layer_rect, gfx::Rect(bounds())));
   }
   Layer::SetNeedsDisplayRect(layer_rect);
 }