arc: Handle metrics for Auth operations.

chrome/browser/chromeos/arc/arc_auth_service.cc

 // Copyright 2015 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/arc_auth_service.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
+#include "base/time/time.h"
 #include "chrome/browser/chromeos/arc/arc_optin_uma.h"
 #include "chrome/browser/chromeos/arc/arc_session_manager.h"
 #include "chrome/browser/chromeos/arc/auth/arc_active_directory_enrollment_token_fetcher.h"
 #include "chrome/browser/chromeos/arc/auth/arc_auth_info_fetcher.h"
 #include "chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.h"
 #include "chrome/browser/chromeos/arc/auth/arc_manual_auth_code_fetcher.h"
 #include "chrome/browser/chromeos/arc/auth/arc_robot_auth_code_fetcher.h"
 #include "chrome/browser/chromeos/arc/policy/arc_policy_util.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
@@ skipping 148 matching lines @@
       ConvertArcSignInFailureReasonToProvisioningResult(reason));
 }
 
 void ArcAuthService::RequestAccountInfo() {
   RequestAccountInfoInternal(
       base::MakeUnique<ArcAuthService::AccountInfoNotifier>(
           base::Bind(&ArcAuthService::OnAccountInfoReady,
                      weak_ptr_factory_.GetWeakPtr())));
 }
 
+void ArcAuthService::ReportMetrics(mojom::MetricsType metrics_type,
+                                   int32_t value) {
+  switch (metrics_type) {
+    case mojom::MetricsType::NETWORK_WAITING_TIME_MILLISECONDS:
+      UpdateAuthTiming("NetworkWaitTime",

[dcheng, 2017/02/23 20:12:36]

I don't see Arc.Auth. prefix here, maybe wrong PS got uploaded?

[khmel, 2017/02/23 20:14:53]

ArcAuth. is added at UpdateAuthTiming/UpdateAuthAttempts. NetworkWaitTime is
used to construct ArcAuth.NetworkWaitTime. Idea of this CL is to define set of 4
UMAs started with ArcAuth.

+                       base::TimeDelta::FromMilliseconds(value));
+      break;
+    case mojom::MetricsType::CHECKIN_ATTEMPTS:
+      UpdateAuthAttempts("CheckinAttempts", value);
+      break;
+    case mojom::MetricsType::CHECKIN_TIME_MILLISECONDS:
+      UpdateAuthTiming("CheckinTime", base::TimeDelta::FromMilliseconds(value));
+      break;
+    case mojom::MetricsType::SIGNIN_TIME_MILLISECONDS:
+      UpdateAuthTiming("SignInTime", base::TimeDelta::FromMilliseconds(value));
+      break;
+  }
+}
+
 void ArcAuthService::OnAccountInfoReady(mojom::AccountInfoPtr account_info) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   auto* instance = ARC_GET_INSTANCE_FOR_METHOD(arc_bridge_service()->auth(),
                                                OnAccountInfoReady);
   DCHECK(instance);
   instance->OnAccountInfoReady(std::move(account_info));
 }
 
 void ArcAuthService::GetAuthCodeDeprecated0(
     const GetAuthCodeDeprecated0Callback& callback) {
@@ skipping 103 matching lines @@
     return;
   }
 
   notifier_->Notify(
       !IsArcOptInVerificationDisabled(), auth_code, GetAccountType(),
       policy_util::IsAccountManaged(ArcSessionManager::Get()->profile()));
   notifier_.reset();
 }
 
 }  // namespace arc