Fixes DesktopWindowTreeHostMus::SetSize and IsVisible

ui/views/mus/desktop_window_tree_host_mus.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 "ui/views/mus/desktop_window_tree_host_mus.h"
 
 #include "base/memory/ptr_util.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/client/drag_drop_client.h"
 #include "ui/aura/client/focus_client.h"
@@ skipping 104 matching lines @@
 }
 
 void DesktopWindowTreeHostMus::ShowMaximizedWithBounds(
     const gfx::Rect& restored_bounds) {
   window()->SetProperty(aura::client::kRestoreBoundsKey,
                         new gfx::Rect(restored_bounds));
   ShowWindowWithState(ui::SHOW_STATE_MAXIMIZED);
 }
 
 bool DesktopWindowTreeHostMus::IsVisible() const {
-  return window()->IsVisible();
+  // Go through the DesktopNativeWidgetAura::IsVisible() for checking
+  // visibility of the parent as it has additional checks beyond checking the
+  // aura::Window.
+  return window()->IsVisible() &&
+         (!parent_ ||
+          static_cast<const internal::NativeWidgetPrivate*>(
+              parent_->desktop_native_widget_aura_)
+              ->IsVisible());
 }
 
 void DesktopWindowTreeHostMus::SetSize(const gfx::Size& size) {
-  // TODO: handle device scale, http://crbug.com/663524.
-  SetBounds(gfx::Rect(window()->bounds().origin(), size));
+  // Use GetBounds() as the origin of window() is always at 0, 0.
+  gfx::Rect screen_bounds = GetBounds();
+  // TODO: handle device scale, http://crbug.com/663524. Also, |screen_bounds|
+  // is in pixels and should be dip.
+  screen_bounds.set_size(size);
+  SetBounds(screen_bounds);
 }
 
 void DesktopWindowTreeHostMus::StackAbove(aura::Window* window) {
   // TODO: implement window stacking, http://crbug.com/663617.
   NOTIMPLEMENTED();
 }
 
 void DesktopWindowTreeHostMus::StackAtTop() {
   // TODO: implement window stacking, http://crbug.com/663617.
   NOTIMPLEMENTED();
@@ skipping 263 matching lines @@
   if (window == this->window()) {
     is_active_ = true;
     desktop_native_widget_aura_->HandleActivationChanged(true);
   } else if (is_active_) {
     is_active_ = false;
     desktop_native_widget_aura_->HandleActivationChanged(false);
   }
 }
 
 }  // namespace views