Fixes visibility bugs in DesktopWindowTreeHostMus

ui/views/view_unittest.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/view.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <memory>
@@ skipping 4640 matching lines @@
   widget->Init(params);
   widget->SetBounds(gfx::Rect(0, 0, 100, 100));
   widget->GetContentsView()->AddChildView(&view);
   widget->Show();
 
   EXPECT_TRUE(view.layer());
   EXPECT_TRUE(view.GetWidget());
   EXPECT_FALSE(view.was_hidden());
 
   widget->Close();
-  if (IsAuraMusClient()) {
-    // Mus on Ozone doesn't send the visibility change during Close().
-    // See http://crbug.com/674003.
-    EXPECT_TRUE(view.layer());
-    EXPECT_FALSE(view.was_hidden());
-  } else {
-    EXPECT_FALSE(view.layer());
-    // Ensure the layer went away via VisibilityChanged().
-    EXPECT_TRUE(view.was_hidden());
-  }
+  EXPECT_FALSE(view.layer());
+  // Ensure the layer went away via VisibilityChanged().
+  EXPECT_TRUE(view.was_hidden());
 
   // Not removed from Widget until Close() completes.
   EXPECT_TRUE(view.GetWidget());
   base::RunLoop().RunUntilIdle();  // Let the Close() complete.
   EXPECT_FALSE(view.GetWidget());
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Observer tests.
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 146 matching lines @@
   std::unique_ptr<View> view = NewView();
   std::unique_ptr<View> child_view = NewView();
   std::unique_ptr<View> child_view2 = NewView();
   view->AddChildView(child_view.get());
   view->AddChildView(child_view2.get());
   view->ReorderChildView(child_view2.get(), 0);
   EXPECT_EQ(child_view2.get(), view_reordered());
 }
 
 }  // namespace views