Read container_instance_id.

chromeos/dbus/session_manager_client.h

 // 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.
 
 #ifndef CHROMEOS_DBUS_SESSION_MANAGER_CLIENT_H_
 #define CHROMEOS_DBUS_SESSION_MANAGER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 54 matching lines @@
     // unlocked successfully (i.e. after NotifyLockScreenDismissed() has
     // been called).
     virtual void ScreenIsUnlocked() {}
 
     // Called after EmitLoginPromptVisible is called.
     virtual void EmitLoginPromptVisibleCalled() {}
 
     // Called when the ARC instance is stopped after it had already started.
     // |clean| is true if the instance was stopped as a result of an explicit
     // request, false if it died unexpectedly.
-    virtual void ArcInstanceStopped(bool clean) {}
+    // |container_instance_id| is the identifier of the container instance.
+    // See details for StartArcInstance().

[Luis Héctor Chávez, 2017/05/19 15:23:03]

maybe it's better to refer to the details of StartArcInstanceCallback, since
it's the one that has the details.

[hidehiko, 2017/05/22 08:38:06]

Done.

+    virtual void ArcInstanceStopped(bool clean,
+                                    const std::string& container_instance_id) {}
   };
 
   // Interface for performing actions on behalf of the stub implementation.
   class StubDelegate {
    public:
     virtual ~StubDelegate() {}
 
     // Locks the screen. Invoked by the stub when RequestLockScreen() is called.
     // In the real implementation of SessionManagerClient::RequestLockScreen(),
     // a lock request is forwarded to the session manager; in the stub, this is
@@ skipping 198 matching lines @@
   // blocks Android ACTION_BOOT_COMPLETED broadcast for 3rd party applications.
   // Upon completion, invokes |callback| with the result.
   // Running ARC requires some amount of disk space. LOW_FREE_DISK_SPACE will
   // be returned when there is not enough free disk space for ARC.
   // UNKNOWN_ERROR is returned for any other errors.
   enum class StartArcInstanceResult {
     SUCCESS,
     UNKNOWN_ERROR,
     LOW_FREE_DISK_SPACE,
   };
-  using StartArcInstanceCallback = base::Callback<void(StartArcInstanceResult)>;
+  // In case of success, container_instance_id will be passed as its second

[Luis Héctor Chávez, 2017/05/19 15:23:03]

nit: |container_instance_id|, for consistency.

[hidehiko, 2017/05/22 08:38:06]

Done.

+  // param. The ID is passed to ArcInstanceStopped() to identify which instance
+  // is stopped.
+  using StartArcInstanceCallback =
+      base::Callback<void(StartArcInstanceResult result,
+                          const std::string& container_instance_id)>;
   virtual void StartArcInstance(const cryptohome::Identification& cryptohome_id,
                                 bool disable_boot_completed_broadcast,
                                 bool enable_vendor_privileged,
                                 const StartArcInstanceCallback& callback) = 0;
 
   // Asynchronously stops the ARC instance.  Upon completion, invokes
   // |callback| with the result; true on success, false on failure (either
   // session manager failed to stop an instance or session manager can not be
   // reached).
   virtual void StopArcInstance(const ArcCallback& callback) = 0;
@@ skipping 33 matching lines @@
   // Create() should be used instead.
   SessionManagerClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SessionManagerClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_SESSION_MANAGER_CLIENT_H_