Ink drops: set clip rect to the enclosing (instead of nearest) rect

Reading the code and its comments, it seems like calling the
|ToEnclosingRect| method is more suitable than |ToNearestRect| for
setting the ink drop's clip rectangle. |ToNearestRect| isn't the right
method for arbitrary RectFs.

Change-Id: If9d4befcd88be4d7ade4d5df043127a7ec3a9dd0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2071041
Commit-Queue: Manu Cornet <manucornet@chromium.org>
Reviewed-by: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744393}

ui/views/animation/ink_drop_host_view.cc

@@ -186,7 +186,7 @@ bool InkDropHostView::AddInkDropClip(ui::Layer* ink_drop_layer) {
       HighlightPathGenerator::GetRoundRectForView(this);
   if (!clipping_data)
     return false;
-  ink_drop_layer->SetClipRect(gfx::ToNearestRect(clipping_data->bounds));
+  ink_drop_layer->SetClipRect(gfx::ToEnclosingRect(clipping_data->bounds));
   ink_drop_layer->SetRoundedCornerRadius(
       gfx::RoundedCornersF(clipping_data->corner_radius));
   ink_drop_layer->SetIsFastRoundedCorner(true);

ui/views/controls/highlight_path_generator.cc

@@ -42,8 +42,8 @@ SkPath HighlightPathGenerator::GetHighlightPath(const View* view) {
 
   const float corner_radius = round_rect->corner_radius;
   return SkPath().addRoundRect(
-      gfx::RectToSkRect(gfx::ToNearestRect(round_rect->bounds)), corner_radius,
-      corner_radius);
+      gfx::RectToSkRect(gfx::ToEnclosingRect(round_rect->bounds)),
+      corner_radius, corner_radius);
 }
 
 base::Optional<HighlightPathGenerator::RoundRect>