Window cycle ui improvements

ash/wm/window_mirror_view.cc

 // Copyright 2016 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 "ash/wm/window_mirror_view.h"
 
 #include "ash/aura/wm_window_aura.h"
 #include "ash/common/wm/forwarding_layer_delegate.h"
 #include "ash/common/wm/window_state.h"
 #include "ash/wm/window_state_aura.h"
@@ skipping 33 matching lines @@
 
   // Some extra work is needed when the target window is minimized.
   if (target_->GetWindowState()->IsMinimized()) {
     GetMirrorLayer()->SetVisible(true);
     GetMirrorLayer()->SetOpacity(1);
     EnsureAllChildrenAreVisible(GetMirrorLayer());
   }
 }
 
 gfx::Size WindowMirrorView::GetPreferredSize() const {
-  const int kMaxWidth = 512;

[Evan Stade, 2016/07/20 17:01:09]

Seems like this logic belongs in the window cycle list, so I moved it.

[sky, 2016/07/20 19:48:45]

Agreed.

-  const int kMaxHeight = 256;
-
-  gfx::Size target_size = target_->GetBounds().size();
-  if (target_size.width() <= kMaxWidth && target_size.height() <= kMaxHeight) {
-    return target_size;
-  }
-
-  float scale = std::min(kMaxWidth / static_cast<float>(target_size.width()),
-                         kMaxHeight / static_cast<float>(target_size.height()));
-  return gfx::ScaleToCeiledSize(target_size, scale, scale);
+  return target_->GetBounds().size();
 }
 
 void WindowMirrorView::Layout() {
   // Position at 0, 0.
   GetMirrorLayer()->SetBounds(gfx::Rect(GetMirrorLayer()->bounds().size()));
 
   // Scale down if necessary.
   gfx::Transform mirror_transform;
   if (size() != target_->GetBounds().size()) {
     const float scale =
@@ skipping 12 matching lines @@
 
   return delegates_.back().get();
 }
 
 ui::Layer* WindowMirrorView::GetMirrorLayer() {
   return layer_owner_->root();
 }
 
 }  // namespace wm
 }  // namespace ash