[Aura] Fix HtmlDialogBrowserTest.SizeWindow.

views/widget/widget.cc

 // Copyright (c) 2011 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 "views/widget/widget.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "ui/base/hit_test.h"
@@ skipping 148 matching lines @@
       widget_delegate_(NULL),
       non_client_view_(NULL),
       dragged_view_(NULL),
       event_stack_(),
       ownership_(InitParams::NATIVE_WIDGET_OWNS_WIDGET),
       is_secondary_widget_(true),
       frame_type_(FRAME_TYPE_DEFAULT),
       disable_inactive_rendering_(false),
       widget_closed_(false),
       saved_show_state_(ui::SHOW_STATE_DEFAULT),
-      minimum_size_(100, 100),
       focus_on_creation_(true),
       is_top_level_(false),
       native_widget_initialized_(false),
       is_mouse_button_pressed_(false),
       last_mouse_event_was_move_(false) {
 }
 
 Widget::~Widget() {
   while (!event_stack_.empty()) {
     event_stack_.top()->reset();
@@ skipping 731 matching lines @@
     non_client_view_->WindowClosing();
   widget_delegate_->WindowClosing();
 }
 
 void Widget::OnNativeWidgetDestroyed() {
   widget_delegate_->DeleteDelegate();
   widget_delegate_ = NULL;
 }
 
 gfx::Size Widget::GetMinimumSize() {
-  return non_client_view_ ? non_client_view_->GetMinimumSize() : minimum_size_;

[Ben Goodger (Google), 2011/11/17 20:50:15]

Isn't this still required for other NativeWidget impls?

[xiyuan, 2011/11/17 22:05:42]

Besides NativeWidgetAura, only NativeWidgetWin uses this in its OnGetMinMaxInfo
handler. That handler is called when user resizes a window. So it only happens
for top-level window that has a frame. And thus, non_client_view_ code path will
be used.

+  return non_client_view_ ? non_client_view_->GetMinimumSize() : gfx::Size();
 }
 
 void Widget::OnNativeWidgetSizeChanged(const gfx::Size& new_size) {
   root_view_->SetSize(new_size);
 
   // Size changed notifications can fire prior to full initialization
   // i.e. during session restore.  Avoid saving session state during these
   // startup procedures.
   if (native_widget_initialized_)
     SaveWindowPlacement();
@@ skipping 278 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views