services/shell: Fix running the connection-lost callback.

content/public/common/mojo_shell_connection.h

 // Copyright 2015 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.
 
 #ifndef CONTENT_PUBLIC_COMMON_MOJO_SHELL_CONNECTION_H_
 #define CONTENT_PUBLIC_COMMON_MOJO_SHELL_CONNECTION_H_
 
 #include <memory>
 
 #include "base/callback_forward.h"
@@ skipping 46 matching lines @@
                      bool is_external);
 
   virtual shell::Connector* GetConnector() = 0;
 
   virtual const shell::Identity& GetIdentity() const = 0;
 
   // Indicates whether the shell connection is to an external shell (true) or
   // a shell embedded in the browser process (false).
   virtual bool UsingExternalShell() const = 0;
 
-  // Sets a closure that is called when the connection is lost.
+  // Sets a closure that is called when the connection is lost. Note that
+  // connection may already have been closed, in which case |closure| will be
+  // run immediately before returning from this function.
   virtual void SetConnectionLostClosure(const base::Closure& closure) = 0;
 
   // [De]Register an impl of Listener that will be consulted when the wrapped
   // ShellConnection exposes services to inbound connections.
   // Registered listeners are owned by this MojoShellConnection. If a listener
   // is removed, then the ownership is transferred back to the caller.
   virtual void AddListener(std::unique_ptr<Listener> listener) = 0;
   virtual std::unique_ptr<Listener> RemoveListener(Listener* listener) = 0;
 
  protected:
   virtual ~MojoShellConnection();
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_COMMON_MOJO_SHELL_CONNECTION_H_