NativeWidgetViews: Implement Maximize.

NativeWidgetViews: Implement Maximize.

BUG=none
TEST=manually

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=102330

[sky, 2011-09-19 20:32:47]

http://codereview.chromium.org/7925006/diff/1/views/widget/native_widget_view...
File views/widget/native_widget_views.cc (right):

http://codereview.chromium.org/7925006/diff/1/views/widget/native_widget_view...
views/widget/native_widget_views.cc:428: minimized_ = true;
Could we use an enum to avoid having to update two separate fields?

[sadrul, 2011-09-22 17:00:24]

http://codereview.chromium.org/7925006/diff/1/views/widget/native_widget_view...
File views/widget/native_widget_views.cc (right):

http://codereview.chromium.org/7925006/diff/1/views/widget/native_widget_view...
views/widget/native_widget_views.cc:428: minimized_ = true;
On 2011/09/19 20:32:47, sky wrote:
> Could we use an enum to avoid having to update two separate fields?

Good idea. I updated to use ui::WindowShowState instead.

[sky, 2011-09-22 17:59:10]

http://codereview.chromium.org/7925006/diff/5001/views/widget/native_widget_v...
File views/widget/native_widget_views.cc (right):

http://codereview.chromium.org/7925006/diff/5001/views/widget/native_widget_v...
views/widget/native_widget_views.cc:416: restored_bounds_ = view_->bounds();
Should we only do this if we're not minimized?

[sadrul, 2011-09-22 18:24:45]

http://codereview.chromium.org/7925006/diff/5001/views/widget/native_widget_v...
File views/widget/native_widget_views.cc (right):

http://codereview.chromium.org/7925006/diff/5001/views/widget/native_widget_v...
views/widget/native_widget_views.cc:416: restored_bounds_ = view_->bounds();
On 2011/09/22 17:59:10, sky wrote:
> Should we only do this if we're not minimized?

Yep. Good catch. Made this change here and also in ::Minimize.

[sky, 2011-09-22 19:18:48]

LGTM