Stabilize empty rect handling in EnclosingRect.

ui/gfx/geometry/size_f.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/size_f.h"
 
 #include "base/strings/stringprintf.h"
 
 namespace gfx {
 
 float SizeF::GetArea() const {
   return width() * height();
 }
 
 void SizeF::Enlarge(float grow_width, float grow_height) {
   SetSize(width() + grow_width, height() + grow_height);
 }
 
 void SizeF::SetToMin(const SizeF& other) {
-  width_ = width() <= other.width() ? width() : other.width();
-  height_ = height() <= other.height() ? height() : other.height();
+  set_width(width() <= other.width() ? width() : other.width());

[danakj, 2017/03/23 16:10:42]

These change doesn't seem needed. Both width() and other.width() will be valid
widths since they are coming from SizeF.

[Peter Mayo, 2017/03/23 19:29:23]

Done.

+  set_height(height() <= other.height() ? height() : other.height());
 }
 
 void SizeF::SetToMax(const SizeF& other) {
-  width_ = width() >= other.width() ? width() : other.width();
-  height_ = height() >= other.height() ? height() : other.height();
+  set_width(width() >= other.width() ? width() : other.width());
+  set_height(height() >= other.height() ? height() : other.height());
 }
 
 std::string SizeF::ToString() const {
   return base::StringPrintf("%fx%f", width(), height());
 }
 
 SizeF ScaleSize(const SizeF& s, float x_scale, float y_scale) {
   SizeF scaled_s(s);
   scaled_s.Scale(x_scale, y_scale);
   return scaled_s;
 }
 
 }  // namespace gfx