arc: Add cryptohome_id to EmitArcBooted.

chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.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 "chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.h"
 
+#include "base/bind.h"
 #include "base/logging.h"
 #include "chrome/browser/chromeos/arc/boot_phase_monitor/arc_instance_throttle.h"
+#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
+#include "chromeos/cryptohome/cryptohome_parameters.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/session_manager_client.h"
 #include "components/arc/arc_bridge_service.h"
 
 namespace arc {
 
 ArcBootPhaseMonitorBridge::ArcBootPhaseMonitorBridge(
-    ArcBridgeService* bridge_service)
-    : ArcService(bridge_service), binding_(this) {
+    ArcBridgeService* bridge_service,
+    Profile* profile)
+    : ArcService(bridge_service),
+      profile(profile),
+      binding_(this),
+      weak_ptr_factory_(this) {
   DCHECK(thread_checker_.CalledOnValidThread());
   arc_bridge_service()->boot_phase_monitor()->AddObserver(this);
 }
 
 ArcBootPhaseMonitorBridge::~ArcBootPhaseMonitorBridge() {
   DCHECK(thread_checker_.CalledOnValidThread());
   arc_bridge_service()->boot_phase_monitor()->RemoveObserver(this);
 }
 
 void ArcBootPhaseMonitorBridge::OnInstanceReady() {
   DCHECK(thread_checker_.CalledOnValidThread());
   auto* instance = ARC_GET_INSTANCE_FOR_METHOD(
       arc_bridge_service()->boot_phase_monitor(), Init);
   DCHECK(instance);
   instance->Init(binding_.CreateInterfacePtrAndBind());
 }
 
 void ArcBootPhaseMonitorBridge::OnInstanceClosed() {
   DCHECK(thread_checker_.CalledOnValidThread());
 }
 
 void ArcBootPhaseMonitorBridge::OnBootCompleted() {
   DCHECK(thread_checker_.CalledOnValidThread());
   VLOG(2) << "OnBootCompleted";
 
   chromeos::SessionManagerClient* session_manager_client =
       chromeos::DBusThreadManager::Get()->GetSessionManagerClient();
-  session_manager_client->EmitArcBooted();
+  session_manager_client->EmitArcBooted(
+      cryptohome::Identification(
+          multi_user_util::GetAccountIdFromProfile(profile)),
+      base::Bind(&ArcBootPhaseMonitorBridge::OnEmitArcBooted,
+                 weak_ptr_factory_.GetWeakPtr()));
 
   // Start monitoring window activation changes to prioritize/throttle the
   // container when needed.
   throttle_ = base::MakeUnique<ArcInstanceThrottle>();
 }
 
+void ArcBootPhaseMonitorBridge::OnEmitArcBooted(bool success) {
+  LOG_IF(ERROR, !success) << "Failed to emit arc booted signal.";

[hidehiko, 2017/02/15 18:48:09]

Optional: how about VLOG on success?

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

This does not touch any members, so please remove it from the class and put in
the anonymous namespace. That way you don't need the |weak_ptr_factory_|.

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

Done.

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

Moved to Line 18. Done

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

Done.

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

Moved to Line 18. Done

+}
+
 }  // namespace arc