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 {
+  // Go through the Widget api for checking visibility as it has additional

[msw, 2016/11/10 01:33:56]

Hmm, Widget::IsVisible() just returns |native_widget_->IsVisible();|, so should
this just call parent_->desktop_native_widget_aura_->IsVisible()? or do
something like:
bool DesktopWindowTreeHostMus::IsVisible() const {
  return (!parent_ || parent->Visible()) &&
         desktop_native_widget_aura_->IsVisible() &&
         window()->IsVisible();
}

[sky, 2016/11/10 05:44:46]

Good call. fixed!

+  // checks beyond checking the aura::Window.
+  if (parent_ && !parent_->native_widget_delegate_->AsWidget()->IsVisible())
+    return false;
   return window()->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