arc: Add cryptohome_id to EmitArcBooted.

chromeos/dbus/fake_session_manager_client.cc

Index: chromeos/dbus/fake_session_manager_client.cc
diff --git a/chromeos/dbus/fake_session_manager_client.cc b/chromeos/dbus/fake_session_manager_client.cc
index 253609d26168a3ac2b4fc64d745c91a57b62e83c..ee57772b83b230f7628e302e8806a4d7da9bddd3 100644
--- a/chromeos/dbus/fake_session_manager_client.cc
+++ b/chromeos/dbus/fake_session_manager_client.cc
@@ -189,7 +189,14 @@ void FakeSessionManagerClient::SetArcCpuRestriction(
       FROM_HERE, base::Bind(callback, arc_available_));
 }
 
-void FakeSessionManagerClient::EmitArcBooted() {}
+void FakeSessionManagerClient::EmitArcBooted(
+    const cryptohome::Identification& cryptohome_id,
+    const ArcCallback& callback) {
+  if (!callback.is_null()) {

[Luis Héctor Chávez, 2017/02/15 19:02:11]

use guard clause pattern:

if (callback.is_null())
  return;
// rest.

then again, why would |callback| be null? Why not DCHECK(!callback.is_null()); ?

[xzhou, 2017/02/16 18:34:50]

I am not sure why we check it here. I see RemoveArcData checks it. Is that
related the life cycle of ArcSessionManager?

[Yusuke Sato, 2017/02/16 18:42:57]

RemoveArcData's check seems just redundant. I don't think you need it here.

[xzhou, 2017/02/16 20:17:31]

Done.

+    base::ThreadTaskRunnerHandle::Get()->PostTask(
+        FROM_HERE, base::Bind(callback, arc_available_));
+  }
+}
 
 void FakeSessionManagerClient::GetArcStartTime(
     const GetArcStartTimeCallback& callback) {