Type conversion fixes, ui/gfx/ edition.

ui/gfx/geometry/insets.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_INSETS_H_
 #define UI_GFX_GEOMETRY_INSETS_H_
 
 #include <string>
 
 #include "build/build_config.h"
@@ skipping 16 matching lines @@
   }
 
   Insets Scale(float x_scale, float y_scale) const {
     return Insets(static_cast<int>(top() * y_scale),
                   static_cast<int>(left() * x_scale),
                   static_cast<int>(bottom() * y_scale),
                   static_cast<int>(right() * x_scale));
   }
 
   operator InsetsF() const {
-    return InsetsF(top(), left(), bottom(), right());
+    return InsetsF(static_cast<float>(top()), static_cast<float>(left()),

[danakj, 2014/10/18 18:40:06]

i really wanna remove these int->float operators, I think they're evil. after i
argued to keep them before :/

[Peter Kasting, 2014/10/20 23:38:56]

I don't have an opinion, other than that I'd rather not do it in this patch
unless none have any callers.

[danakj, 2014/10/23 15:35:44]

Sorry ya I didn't mean in this patch, just griping aloud about the existence of
these :)

+                   static_cast<float>(bottom()), static_cast<float>(right()));
   }
 
   // Returns a string representation of the insets.
   std::string ToString() const;
 };
 
 #if !defined(COMPILER_MSVC) && !defined(__native_client__)
 extern template class InsetsBase<Insets, int>;
 #endif
 
 }  // namespace gfx
 
 #endif  // UI_GFX_GEOMETRY_INSETS_H_