[Win Aero] Correctly set the minimum window size.

chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc

Index: chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc
diff --git a/chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc b/chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc
index ee2fe5a95cdaa283d55862aa499e25ab6e7e30c6..625049db141280a16f520abc868a49682a5eda94 100644
--- a/chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc
+++ b/chrome/browser/ui/views/apps/glass_app_window_frame_view_win.cc
@@ -115,7 +115,8 @@ const char* GlassAppWindowFrameViewWin::GetClassName() const {
 gfx::Size GlassAppWindowFrameViewWin::GetMinimumSize() const {
   gfx::Size min_size = widget_->client_view()->GetMinimumSize();
   gfx::Rect client_bounds = GetBoundsForClientView();
-  min_size.Enlarge(0, client_bounds.y());
+  min_size.Enlarge(width() - client_bounds.width(),
+                   height() - client_bounds.height());

[benwells1, 2014/07/22 05:37:52]

This looks right. Are there tests which should have caught this when I
introduced the glass app window frame?

[jackhou1, 2014/07/23 03:36:04]

The API is covered in platform_apps/window_api/test.js, but the bug is that the
size we pass to WM_GETMINMAXINFO is wrong. So we'd have to have a platform
specific test in order to cover this.

   return min_size;
 }