Revise comments on various Shell interfaces. Make some methods private on ShellConnection.

ui/views/mus/platform_test_helper_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/test/platform_test_helper.h"
 
 #include "base/command_line.h"
 #include "mojo/shell/background/background_shell.h"
 #include "mojo/shell/public/cpp/shell_client.h"
 #include "mojo/shell/public/cpp/shell_connection.h"
@@ skipping 20 matching lines @@
 
     background_shell_.reset(new mojo::shell::BackgroundShell);
     background_shell_->Init();
     shell_client_.reset(new DefaultShellClient);
     shell_connection_.reset(new mojo::ShellConnection(
         shell_client_.get(),
         background_shell_->CreateShellClientRequest(GURL("mojo://test-app"))));
     shell_connection_->WaitForInitialize();
     // ui/views/mus requires a WindowManager running, for now use the desktop
     // one.
-    shell_connection_->Connect("mojo:desktop_wm");
+    mojo::Shell* shell = shell_connection_.get();
+    shell->Connect("mojo:desktop_wm");
     WindowManagerConnection::Create(shell_connection_.get());
   }
 
   ~PlatformTestHelperMus() override {
     WindowManagerConnection::Reset();
     // |app_| has a reference to us, destroy it while we are still valid.
     shell_connection_.reset();
   }
 
  private:
   scoped_ptr<mojo::shell::BackgroundShell> background_shell_;
   scoped_ptr<mojo::ShellConnection> shell_connection_;
   scoped_ptr<DefaultShellClient> shell_client_;
 
   DISALLOW_COPY_AND_ASSIGN(PlatformTestHelperMus);
 };
 
 }  // namespace
 
 // static
 scoped_ptr<PlatformTestHelper> PlatformTestHelper::Create() {
   return make_scoped_ptr(new PlatformTestHelperMus);
 }
 
 }  // namespace views