Tweaks to ToggleButton details.

ui/gfx/geometry/insets.h

Index: ui/gfx/geometry/insets.h
diff --git a/ui/gfx/geometry/insets.h b/ui/gfx/geometry/insets.h
index bc5e3460ca3bdff536c43c01226566aa11bde4e3..3a202c38eb0da6d8bc945c5b3cc51d437cd610e4 100644
--- a/ui/gfx/geometry/insets.h
+++ b/ui/gfx/geometry/insets.h
@@ -12,6 +12,8 @@
 
 namespace gfx {
 
+class Vector2d;
+
 // Represents the widths of the four borders or margins of an unspecified
 // rectangle. An Insets stores the thickness of the top, left, bottom and right
 // edges, without storing the actual size and position of the rectangle itself.
@@ -118,6 +120,8 @@ inline Insets operator-(Insets lhs, const Insets& rhs) {
   return lhs;
 }
 
+GFX_EXPORT Insets operator+(const Insets& insets, const gfx::Vector2d& vector);

[danakj, 2016/10/28 18:44:59]

So this moves the top/bottom and left/right insets in the same direction based
on the vector? I think the operation of + isn't super obvious for this and I'd
prefer a member method with a name that describes it like OffsetBy() or
something? wdyt?

[Evan Stade, 2016/10/28 19:22:03]

assuming it returns an Inset (unlike gfx::Rect::Offset, which acts in-place
which I usually find less useful), sure.

[danakj, 2016/10/28 19:46:56]

That would match the behaviour of Scale() so sure

[Evan Stade, 2016/10/28 20:12:10]

done (unlike Scale, the impl is in the .cc file to reduce transitive includes)

+
 }  // namespace gfx
 
 #endif  // UI_GFX_GEOMETRY_INSETS_H_