Fixes bad code in OnViewVisibilityChanged

mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.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 "view_manager/public/cpp/lib/view_manager_client_impl.h"
 
 #include "mojo/public/cpp/application/application_impl.h"
 #include "mojo/public/cpp/application/connect.h"
 #include "mojo/public/cpp/application/service_provider_impl.h"
 #include "mojo/public/interfaces/application/service_provider.mojom.h"
@@ skipping 361 matching lines @@
   if (view)
     ViewPrivate(view).LocalDestroy();
 }
 
 void ViewManagerClientImpl::OnViewVisibilityChanged(Id view_id, bool visible) {
   // TODO(sky): there is a race condition here. If this client and another
   // client change the visibility at the same time the wrong value may be set.
   // Deal with this some how.
   View* view = GetViewById(view_id);
   if (view)
-    view->SetVisible(visible);
+    ViewPrivate(view).LocalSetVisible(visible);
 }
 
 void ViewManagerClientImpl::OnViewDrawnStateChanged(Id view_id, bool drawn) {
   View* view = GetViewById(view_id);
   if (view)
     ViewPrivate(view).LocalSetDrawn(drawn);
 }
 
 void ViewManagerClientImpl::OnViewSharedPropertyChanged(
     Id view_id,
@@ skipping 94 matching lines @@
 void ViewManagerClientImpl::OnActionCompleted(bool success) {
   if (!change_acked_callback_.is_null())
     change_acked_callback_.Run();
 }
 
 Callback<void(bool)> ViewManagerClientImpl::ActionCompletedCallback() {
   return [this](bool success) { OnActionCompleted(success); };
 }
 
 }  // namespace mojo