Add desktop engagement metrics.

chrome/browser/android/metrics/uma_session_stats.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/android/metrics/uma_session_stats.h"
 
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 43 matching lines @@
   ++active_session_count_;
 }
 
 void UmaSessionStats::UmaEndSession(JNIEnv* env,
                                     const JavaParamRef<jobject>& obj) {
   --active_session_count_;
   DCHECK_GE(active_session_count_, 0);
 
   if (active_session_count_ == 0) {
     base::TimeDelta duration = base::TimeTicks::Now() - session_start_time_;
+
+    // This metric is recorded separately for desktops in

[Alexei Svitkine (slow), 2016/07/28 18:37:22]

Nit: "for desktops" -> "on desktop"

Add a "Note: " at the front of the comment.

[gayane -on leave until 09-2017, 2016/07/28 21:21:34]

Done.

+    // DesktopEngagementService::EndSession.
     UMA_HISTOGRAM_LONG_TIMES("Session.TotalDuration", duration);
 
     DCHECK(g_browser_process);
     // Tell the metrics service it was cleanly shutdown.
     metrics::MetricsService* metrics = g_browser_process->metrics_service();
     if (metrics) {
       metrics->OnAppEnterBackground();
     }
   }
 }
@@ skipping 120 matching lines @@
   // We should have only one UmaSessionStats instance.
   DCHECK(!g_uma_session_stats);
   g_uma_session_stats = new UmaSessionStats();
   return reinterpret_cast<intptr_t>(g_uma_session_stats);
 }
 
 // Register native methods
 bool RegisterUmaSessionStats(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }