Stabilize empty rect handling in EnclosingRect.

ui/gfx/geometry/size_f.h

 // 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.
 
 #ifndef UI_GFX_GEOMETRY_SIZE_F_H_
 #define UI_GFX_GEOMETRY_SIZE_F_H_
 
 #include <iosfwd>
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/gfx_export.h"
 
 namespace gfx {
 
+static constexpr float kTrivial = 8.f * std::numeric_limits<float>::epsilon();
+
 // A floating version of gfx::Size.
 class GFX_EXPORT SizeF {
  public:
   constexpr SizeF() : width_(0.f), height_(0.f) {}
   constexpr SizeF(float width, float height)
-      : width_(width >= 0 ? width : 0), height_(height >= 0 ? height : 0) {}
+      : width_(width > kTrivial ? width : 0.f),
+        height_(height > kTrivial ? height : 0.f) {}
 
   constexpr explicit SizeF(const Size& size)
       : SizeF(static_cast<float>(size.width()),
               static_cast<float>(size.height())) {}
 
   constexpr float width() const { return width_; }
   constexpr float height() const { return height_; }
 
-  void set_width(float width) { width_ = fmaxf(0, width); }
-  void set_height(float height) { height_ = fmaxf(0, height); }
+  void set_width(float width) { width_ = width > kTrivial ? width : 0.f; }
+  void set_height(float height) { height_ = height > kTrivial ? height : 0.f; }
 
   float GetArea() const;
 
   void SetSize(float width, float height) {
     set_width(width);
     set_height(height);
   }
 
   void Enlarge(float grow_width, float grow_height);
 
   void SetToMin(const SizeF& other);
   void SetToMax(const SizeF& other);
 
-  bool IsEmpty() const { return !width() || !height(); }
+  bool IsEmpty() const { return !height_ || !width_; }
 
   void Scale(float scale) {
     Scale(scale, scale);
   }
 
   void Scale(float x_scale, float y_scale) {
     SetSize(width() * x_scale, height() * y_scale);
   }
 
   std::string ToString() const;
@@ skipping 18 matching lines @@
 }
 
 // This is declared here for use in gtest-based unit tests but is defined in
 // the //ui/gfx:test_support target. Depend on that to use this in your unit
 // test. This should not be used in production code - call ToString() instead.
 void PrintTo(const SizeF& size, ::std::ostream* os);
 
 }  // namespace gfx
 
 #endif  // UI_GFX_GEOMETRY_SIZE_F_H_