Make WindowState::CanMinimize() take into account WidgetDelegate::CanMinimize()

ui/views/widget/desktop_aura/desktop_native_widget_aura.cc

 // Copyright (c) 2012 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 "ui/views/widget/desktop_aura/desktop_native_widget_aura.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/client/cursor_client.h"
@@ skipping 918 matching lines @@
 }
 
 bool DesktopNativeWidgetAura::IsTranslucentWindowOpacitySupported() const {
   return content_window_ &&
       desktop_window_tree_host_->IsTranslucentWindowOpacitySupported();
 }
 
 void DesktopNativeWidgetAura::OnSizeConstraintsChanged() {
   content_window_->SetProperty(aura::client::kCanMaximizeKey,
                                GetWidget()->widget_delegate()->CanMaximize());
+  content_window_->SetProperty(aura::client::kCanMinimizeKey,
+                               GetWidget()->widget_delegate()->CanMinimize());

[oshima, 2014/09/30 16:43:07]

q: is this used in desktop aura?

[pkotwicz, 2014/10/01 14:45:03]

It is not used in desktop aura, but it makes sense to have this for consistency

[oshima, 2014/10/01 17:26:55]

If it's not used, I'd recommend to add TODO instead of adding dead code.
I'll defer to sky@ tho.

[sky, 2014/10/01 20:00:27]

Depends upon what you mean by used. kCanMinimizeKey will be used to the same
extend kCanMaximizeKey is used. It's nice to propagate the value to the
underlying window in case someone queries it outside of the widget API.

   content_window_->SetProperty(aura::client::kCanResizeKey,
                                GetWidget()->widget_delegate()->CanResize());
   desktop_window_tree_host_->SizeConstraintsChanged();
 }
 
 void DesktopNativeWidgetAura::RepostNativeEvent(gfx::NativeEvent native_event) {
   OnEvent(native_event);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 260 matching lines @@
   if (cursor_reference_count_ == 0) {
     // We are the last DesktopNativeWidgetAura instance, and we are responsible
     // for cleaning up |cursor_manager_|.
     delete cursor_manager_;
     native_cursor_manager_ = NULL;
     cursor_manager_ = NULL;
   }
 }
 
 }  // namespace views