Do not convert from RectF to Rect by flooring.

ui/gfx/size_base.h

Index: ui/gfx/size_base.h
diff --git a/ui/gfx/size_base.h b/ui/gfx/size_base.h
index ef687c697d9dd15bd21837b2b4e10c6ab5b9dbbe..2f061ec70967fa0c6ecb13cb5ea5f16cf14a9a12 100644
--- a/ui/gfx/size_base.h
+++ b/ui/gfx/size_base.h
@@ -41,8 +41,8 @@ class UI_EXPORT SizeBase {
                  static_cast<Type>(height_ * y_scale));
   }
 
-  void set_width(Type width);
-  void set_height(Type height);
+  void set_width(Type width) { width_ = width; }
+  void set_height(Type height) { height_ = height; }
 
   bool operator==(const Class& s) const {
     return width_ == s.width_ && height_ == s.height_;
@@ -53,15 +53,25 @@ class UI_EXPORT SizeBase {
   }
 
   bool IsEmpty() const {
-    // Size doesn't allow negative dimensions, so testing for 0 is enough.
-    return (width_ == 0) || (height_ == 0);
+    return (width_ <= 0) || (height_ <= 0);
+  }
+
+  void ClampToNonNegative() {
+    if (width_ < 0)
+      width_ = 0;
+    if (height_ < 0)
+      height_ = 0;
   }
 
  protected:
-  SizeBase(Type width, Type height);
+  SizeBase(Type width, Type height)
+      : width_(width),
+        height_(height)
+  { }

[sky, 2012/09/27 20:33:55]

{} on previous line

+
   // Destructor is intentionally made non virtual and protected.
   // Do not make this public.
-  ~SizeBase();
+  ~SizeBase() { }

[sky, 2012/09/27 20:33:55]

{}

 
  private:
   Type width_;