Fixes bad code in OnViewVisibilityChanged

services/view_manager/view_manager_client_apptest.cc

 // Copyright 2014 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 "mojo/services/view_manager/public/cpp/view_manager.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_vector.h"
 #include "base/run_loop.h"
 #include "mojo/public/cpp/application/application_connection.h"
@@ skipping 407 matching lines @@
     QuitRunLoop();
   }
 
   View* view_;
 
   MOJO_DISALLOW_COPY_AND_ASSIGN(VisibilityChangeObserver);
 };
 
 }  // namespace
 
-// TODO(alhaad): Re-enable this once
-// ViewManagerClientImpl::OnViewVisibilityChanged() does not cause a race
-// condition when called from two clients.
-TEST_F(ViewManagerTest, DISABLED_Visible) {
+TEST_F(ViewManagerTest, Visible) {
   View* view1 = window_manager()->CreateView();
   view1->SetVisible(true);
   window_manager()->GetRoot()->AddChild(view1);
 
   // Embed another app and verify initial state.
   ViewManager* embedded = Embed(window_manager(), view1);
   ASSERT_NE(nullptr, embedded->GetRoot());
   View* embedded_root = embedded->GetRoot();
   EXPECT_TRUE(embedded_root->visible());
   EXPECT_TRUE(embedded_root->IsDrawn());
@@ skipping 135 matching lines @@
     view11->SetFocus();
     DoRunLoop();
     ASSERT_NE(nullptr, observer.last_gained_focus());
     ASSERT_NE(nullptr, observer.last_lost_focus());
     EXPECT_EQ(view11->id(), observer.last_gained_focus()->id());
     EXPECT_EQ(embedded->GetRoot()->id(), observer.last_lost_focus()->id());
   }
 }
 
 }  // namespace mojo