Tweaks to ToggleButton details.

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 "ui/gfx/geometry/insets_f.h"
 #include "ui/gfx/gfx_export.h"
 
 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.
 //
 // This can be used to represent a space within a rectangle, by "shrinking" the
 // rectangle by the inset amount on all four sides. Alternatively, it can
 // represent a border that has a different thickness on each side.
 class GFX_EXPORT Insets {
  public:
   constexpr Insets() : top_(0), left_(0), bottom_(0), right_(0) {}
@@ skipping 86 matching lines @@
 inline Insets operator+(Insets lhs, const Insets& rhs) {
   lhs += rhs;
   return lhs;
 }
 
 inline Insets operator-(Insets lhs, const Insets& rhs) {
   lhs -= 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_