Last set of tweaks to icon positioning code. Clarifies positioning algorithm, and for non-Windows, modifies sizing algorithm to just make the icon size be the same as the titlebar font size, minimum 16 px, since the old crazy code just seemed wrong.

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/807001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41382 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 21 insertions, 32 deletions

diff --git a/views/window/custom_frame_view.cc b/views/window/custom_frame_view.cc
index 1dc19de..10a0559 100644
--- a/views/window/custom_frame_view.cc
+++ b/views/window/custom_frame_view.cc
@@ -48,6 +48,8 @@ const int kCaptionButtonHeightWithPadding = 19;
 const int kTitlebarTopAndBottomEdgeThickness = 2;
 // The icon is inset 2 px from the left frame border.
 const int kIconLeftSpacing = 2;
+// The icon never shrinks below 16 px on a side.
+const int kIconMinimumSize = 16;
 // There is a 4 px gap between the icon and the title text.
 const int kIconTitleSpacing = 4;
 // There is a 5 px gap between the title text and the caption buttons.
@@ -283,44 +285,31 @@ int CustomFrameView::IconSize() const {
   // size are increased.
   return GetSystemMetrics(SM_CYSMICON);
 #else
-  // Calculate the necessary height from the titlebar font size.
-  // The title text has 2 px of padding between it and the frame border on both
-  // top and bottom.
-  const int kTitleBorderSpacing = 2;
-  // The bottom spacing should be the same apparent height as the top spacing.
-  // The top spacing height is FrameBorderThickness() + kTitleBorderSpacing.  We
-  // omit the frame border portion because that's not part of the icon height.
-  // The bottom spacing, then, is kTitleBorderSpacing + kFrameBorderThickness to
-  // the bottom edge of the titlebar.  We omit TitlebarBottomThickness() because
-  // that's also not part of the icon height.
-  return kTitleBorderSpacing + title_font_->height() + kTitleBorderSpacing +
-      (kFrameBorderThickness - TitlebarBottomThickness());
+  return std::max(title_font_->height(), kIconMinimumSize);
 #endif
 }
 
 gfx::Rect CustomFrameView::IconBounds() const {
   int size = IconSize();
   int frame_thickness = FrameBorderThickness();
-  // This next statement handles two things:
-  //   (1) Vertically centering the icon when the icon is shorter than the
-  //       minimum space we reserve for the caption button.  We want to bias
-  //       rounding to put extra space above the icon, since below it is the 2
-  //       px 3D edge, which looks to the eye like additional space; hence the
-  //       + 1 below.
-  //   (2) Our frame border has a different "3D look" than Windows'.  Theirs has
-  //       a more complex gradient on the top that they push their icon/title
-  //       below; then the maximized window cuts this off and the icon/title are
-  //       centered in the remaining space.  Because the apparent shape of our
-  //       border is simpler, using the same positioning makes things look
-  //       slightly uncentered with restored windows, so we come up to
-  //       compensate.  The frame border has a 2 px 3D edge plus some empty
-  //       space, so we adjust by half the width of the empty space to center
-  //       things.
-  return gfx::Rect(frame_thickness + kIconLeftSpacing,
-      (NonClientTopBorderHeight() - size - TitlebarBottomThickness() + 1 +
-          (frame_->IsMaximized() ?
-              frame_thickness : kTitlebarTopAndBottomEdgeThickness)) / 2,
-      size, size);
+  // Our frame border has a different "3D look" than Windows'.  Theirs has a
+  // more complex gradient on the top that they push their icon/title below;
+  // then the maximized window cuts this off and the icon/title are centered
+  // in the remaining space.  Because the apparent shape of our border is
+  // simpler, using the same positioning makes things look slightly uncentered
+  // with restored windows, so when the window is restored, instead of
+  // calculating the remaining space from below the frame border, we calculate
+  // from below the 3D edge.
+  int unavailable_px_at_top = frame_->IsMaximized() ?
+      frame_thickness : kTitlebarTopAndBottomEdgeThickness;
+  // When the icon is shorter than the minimum space we reserve for the caption
+  // button, we vertically center it.  We want to bias rounding to put extra
+  // space above the icon, since the 3D edge (+ client edge, for restored
+  // windows) below looks (to the eye) more like additional space than does the
+  // 3D edge (or nothing at all, for maximized windows) above; hence the +1.
+  int y = unavailable_px_at_top + (NonClientTopBorderHeight() -
+      unavailable_px_at_top - size - TitlebarBottomThickness() + 1) / 2;
+  return gfx::Rect(frame_thickness + kIconLeftSpacing, y, size, size);
 }
 
 void CustomFrameView::PaintRestoredFrameBorder(gfx::Canvas* canvas) {