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"
+#include "components/signin/core/account_id/account_id.h"

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

remove. you already did it in .h (and not using forward decl in .h is fine
because of
https://google.github.io/styleguide/cppguide.html#Forward_Declarations "Try to
avoid forward declarations of entities defined in another project.")

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

Done. Thanks for the link.

+
+namespace {
+void OnEmitArcBooted(bool success) {

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

space between L17/L18

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

Done.

+  if (!success)
+    VLOG(1) << "Failed to emit arc booted signal.";
+}

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

same as above

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

Done.

+}

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

} // namespace

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

Done.

 
 namespace arc {
 
 ArcBootPhaseMonitorBridge::ArcBootPhaseMonitorBridge(
-    ArcBridgeService* bridge_service)
-    : ArcService(bridge_service), binding_(this) {
+    ArcBridgeService* bridge_service,
+    const AccountId& account_id)
+    : ArcService(bridge_service), account_id_(account_id), binding_(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(account_id_),
+                                        base::Bind(&OnEmitArcBooted));
 
   // Start monitoring window activation changes to prioritize/throttle the
   // container when needed.
   throttle_ = base::MakeUnique<ArcInstanceThrottle>();
 }
-
 }  // namespace arc