arc: add enterprise_reporting.mojom interface.

chrome/browser/chromeos/arc/arc_user_data_service.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 "components/arc/user_data/arc_user_data_service.h"
+#include "chrome/browser/chromeos/arc/arc_user_data_service.h"
 
 #include <utility>
 
+#include "base/bind.h"
 #include "base/command_line.h"
+#include "base/logging.h"
+#include "base/threading/thread_task_runner_handle.h"
+#include "chrome/browser/chromeos/arc/arc_auth_service.h"
 #include "chromeos/chromeos_switches.h"
 #include "chromeos/cryptohome/cryptohome_parameters.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/session_manager_client.h"
 #include "components/prefs/pref_member.h"
 #include "components/signin/core/account_id/account_id.h"
 #include "components/user_manager/user_manager.h"
 
 namespace arc {
 
 ArcUserDataService::ArcUserDataService(
     ArcBridgeService* bridge_service,
     std::unique_ptr<BooleanPrefMember> arc_enabled_pref,
     const AccountId& account_id)
     : ArcService(bridge_service),
       arc_enabled_pref_(std::move(arc_enabled_pref)),
       primary_user_account_id_(account_id),
+      binding_(this),
       weak_ptr_factory_(this) {
   arc_bridge_service()->AddObserver(this);
+  arc_bridge_service()->enterprise_reporting()->AddObserver(this);
   pref_change_registrar_.Init(arc_enabled_pref_->prefs());
   pref_change_registrar_.Add(
       arc_enabled_pref_->GetPrefName(),
       base::Bind(&ArcUserDataService::OnOptInPreferenceChanged,
                  weak_ptr_factory_.GetWeakPtr()));
-  ClearIfDisabled();
+  ClearIfRequired();
 }
 
 ArcUserDataService::~ArcUserDataService() {
   DCHECK(thread_checker_.CalledOnValidThread());
   arc_bridge_service()->RemoveObserver(this);
+  arc_bridge_service()->enterprise_reporting()->RemoveObserver(this);
 }
 
 void ArcUserDataService::OnBridgeStopped(ArcBridgeService::StopReason reason) {
   DCHECK(thread_checker_.CalledOnValidThread());
   const AccountId& account_id =
       user_manager::UserManager::Get()->GetPrimaryUser()->GetAccountId();
   if (account_id != primary_user_account_id_) {
     LOG(ERROR) << "User preferences not loaded for "
                << primary_user_account_id_.GetUserEmail()
                << ", but current primary user is " << account_id.GetUserEmail();
     primary_user_account_id_ = EmptyAccountId();
     return;
   }
-  ClearIfDisabled();
+  ClearIfRequired();
+
+  if (restart_required_) {
+    restart_required_ = false;
+    base::ThreadTaskRunnerHandle::Get()->PostTask(
+        FROM_HERE, base::Bind(&ArcUserDataService::RestartArc,

[Luis Héctor Chávez, 2016/07/22 17:00:15]

IIUC this is racy. What needs to happen is that the
session_manager_client->RemoveArcData should have a callback that gets invoked
once it finishes removing the data. Otherwise you can get into a state where the
invocation to restart arc times out, and it wouldn't attempt to restart since
the initial connection was not ready.

[Polina Bondarenko, 2016/07/25 16:17:59]

Done.

+                              weak_ptr_factory_.GetWeakPtr()));
+  }
 }
 
-void ArcUserDataService::ClearIfDisabled() {
+void ArcUserDataService::OnInstanceReady() {
+  arc_bridge_service()->enterprise_reporting()->instance()->Init(
+      binding_.CreateInterfacePtrAndBind());
+}
+
+void ArcUserDataService::ReportManagementState(mojom::ManagementState state) {
+  VLOG(1) << "ReportManagementState state=" << state;
+  if (state == mojom::ManagementState::MANAGED_DO_LOST) {
+    clean_required_ = true;
+    restart_required_ = true;
+    ArcAuthService::Get()->StopArc();
+  }
+}
+
+void ArcUserDataService::ClearIfRequired() {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (!arc_bridge_service()->stopped()) {
     LOG(ERROR) << "ARC instance not stopped, user data can't be cleared";
     return;
   }
-  if ((arc_enabled_pref_->GetValue() && !arc_disabled_) ||
+  if ((arc_enabled_pref_->GetValue() && !clean_required_) ||
       base::CommandLine::ForCurrentProcess()->HasSwitch(
           chromeos::switches::kDisableArcDataWipe)) {
     return;
   }
-  arc_disabled_ = false;
+  VLOG(1) << "Wipe ARC user data.";
+  clean_required_ = false;
   const cryptohome::Identification cryptohome_id(primary_user_account_id_);
   chromeos::SessionManagerClient* session_manager_client =
       chromeos::DBusThreadManager::Get()->GetSessionManagerClient();
   session_manager_client->RemoveArcData(cryptohome_id);
 }
 
 void ArcUserDataService::OnOptInPreferenceChanged() {
   if (!arc_enabled_pref_->GetValue())
-    arc_disabled_ = true;
+    clean_required_ = true;
+}
+
+void ArcUserDataService::RestartArc() {

[Luis Héctor Chávez, 2016/07/22 17:00:15]

This method does not access any members, so it can be in the anonymous
namespace.

[Polina Bondarenko, 2016/07/25 16:17:59]

Moved this method to ArcEnterpriseReportingService and changed.

+  VLOG(1) << "Restart ARC";
+  ArcAuthService::Get()->EnableArc();
 }
 
 }  // namespace arc