Handle large rects better.

ui/gfx/geometry/rect_conversions.cc

 // Copyright (c) 2012 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 "ui/gfx/geometry/rect_conversions.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/logging.h"
 #include "ui/gfx/geometry/safe_integer_conversions.h"
 
 namespace gfx {
 
-Rect ToEnclosingRect(const RectF& rect) {
-  int min_x = ToFlooredInt(rect.x());
-  int min_y = ToFlooredInt(rect.y());
-  float max_x = rect.right();
-  float max_y = rect.bottom();
-  int width = rect.size().HasWidth()
-                  ? ToCeiledInt(static_cast<double>(ToCeiledInt(max_x)) - min_x)
-                  : 0;
-  int height =
-      rect.size().HasHeight()
-          ? ToCeiledInt(static_cast<double>(ToCeiledInt(max_y)) - min_y)
-          : 0;
-  return Rect(min_x, min_y, width, height);
+// This is the minimum size of a float rect dimension for use to include it
+// in the enclosing int rect.

[Peter Mayo, 2017/03/23 00:01:31]

Remove comment - Not relevant any more.

+Rect ToEnclosingRect(const RectF& r) {
+  int left = ToFlooredInt(r.x());
+  int right = r.size().HasWidth() ? ToCeiledInt(r.right()) : left;
+  int top = ToFlooredInt(r.y());
+  int bottom = r.size().HasHeight() ? ToCeiledInt(r.bottom()) : top;
+
+  Rect result;
+  result.SetByBounds(left, right, top, bottom);
+  return result;
 }
 
 Rect ToEnclosedRect(const RectF& rect) {
-  int min_x = ToCeiledInt(rect.x());
-  int min_y = ToCeiledInt(rect.y());
-  float max_x = rect.right();
-  float max_y = rect.bottom();
-  int width = std::max(
-      ToFlooredInt(static_cast<float>(ToFlooredInt(max_x)) - min_x), 0);
-  int height = std::max(
-      ToFlooredInt(static_cast<float>(ToFlooredInt(max_y)) - min_y), 0);
-  return Rect(min_x, min_y, width, height);
+  Rect result;
+  result.SetByBounds(ToCeiledInt(rect.x()), ToFlooredInt(rect.right()),
+                     ToCeiledInt(rect.y()), ToFlooredInt(rect.bottom()));
+  return result;
 }
 
 Rect ToNearestRect(const RectF& rect) {
   float float_min_x = rect.x();
   float float_min_y = rect.y();
   float float_max_x = rect.right();
   float float_max_y = rect.bottom();
 
   int min_x = ToRoundedInt(float_min_x);
   int min_y = ToRoundedInt(float_min_y);
   int max_x = ToRoundedInt(float_max_x);
   int max_y = ToRoundedInt(float_max_y);
 
   // If these DCHECKs fail, you're using the wrong method, consider using
   // ToEnclosingRect or ToEnclosedRect instead.
   DCHECK(std::abs(min_x - float_min_x) < 0.01f);
   DCHECK(std::abs(min_y - float_min_y) < 0.01f);
   DCHECK(std::abs(max_x - float_max_x) < 0.01f);
   DCHECK(std::abs(max_y - float_max_y) < 0.01f);
 
-  return Rect(min_x, min_y, max_x - min_x, max_y - min_y);
+  Rect result;
+  result.SetByBounds(min_x, max_x, min_y, max_y);
+
+  return result;
 }
 
 bool IsNearestRectWithinDistance(const gfx::RectF& rect, float distance) {
   float float_min_x = rect.x();
   float float_min_y = rect.y();
   float float_max_x = rect.right();
   float float_max_y = rect.bottom();
 
   int min_x = ToRoundedInt(float_min_x);
   int min_y = ToRoundedInt(float_min_y);
   int max_x = ToRoundedInt(float_max_x);
   int max_y = ToRoundedInt(float_max_y);
 
   return
       (std::abs(min_x - float_min_x) < distance) &&
       (std::abs(min_y - float_min_y) < distance) &&
       (std::abs(max_x - float_max_x) < distance) &&
       (std::abs(max_y - float_max_y) < distance);
 }
 
 Rect ToFlooredRectDeprecated(const RectF& rect) {
   return Rect(ToFlooredInt(rect.x()),
               ToFlooredInt(rect.y()),
               ToFlooredInt(rect.width()),
               ToFlooredInt(rect.height()));

[Peter Mayo, 2017/03/23 00:01:31]

This should use the improved heuristic too I suppose.

 }
 
 }  // namespace gfx