[Win Aero] Correctly set the minimum window size.

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

 // Copyright 2014 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.
 
 #include "chrome/browser/ui/views/apps/glass_app_window_frame_view_win.h"
 
 #include "apps/ui/native_app_window.h"
 #include "ui/base/hit_test.h"
 #include "ui/gfx/win/dpi.h"
 #include "ui/views/widget/widget.h"
@@ skipping 97 matching lines @@
       .size();
 }
 
 const char* GlassAppWindowFrameViewWin::GetClassName() const {
   return kViewClassName;
 }
 
 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;
 }
 
 gfx::Size GlassAppWindowFrameViewWin::GetMaximumSize() const {
   gfx::Size max_size = widget_->client_view()->GetMaximumSize();
 
   // Add to the client maximum size the height of any title bar and borders.
   gfx::Size client_size = GetBoundsForClientView().size();
   if (max_size.width())
     max_size.Enlarge(width() - client_size.width(), 0);
   if (max_size.height())
     max_size.Enlarge(0, height() - client_size.height());
 
   return max_size;
 }