Add UMA stats for Chromoting connection details.

remoting/webapp/base/js/client_session.js

 // Copyright (c) 2012 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.
 
 /**
  * @fileoverview
  * Class handling creation and teardown of a remoting client session.
  *
  * The ClientSession class controls lifetime of the client plugin
  * object and provides the plugin with the functionality it needs to
@@ skipping 563 matching lines @@
  * @param {remoting.ClientSession.State} oldState The new state for the session.
  * @param {remoting.ClientSession.State} newState The new state for the session.
  * @private
  */
 remoting.ClientSession.prototype.notifyStateChanges_ =
     function(oldState, newState) {
   /** @type {remoting.Error} */
   var error;
   switch (this.state_) {
     case remoting.ClientSession.State.CONNECTED:
+      chrome.metricsPrivate.recordUserAction(
+          'Chromoting.Connections.Connected');

[Sergey Ulanov, 2015/08/20 21:05:19]

Do we need to register the actions somewhere? I think we do.

base/metrics/user_metrics.h says that you need to run
tools/metrics/actions/extract_actions.py which is supposed to extract the list
from sources. But for JavaScrip it has hardcoded list of metrics:
https://code.google.com/p/chromium/codesearch#chromium/src/tools/metrics/acti...

I think you want to add the new metrics into that list.

[anandc, 2015/08/20 23:12:01]

Thanks, added.
I still don't see the actions metrics show up in chrome://histograms/, but
that's probably because only histograms show up there?

       console.log('Connection established.');
       var connectionInfo = new remoting.ConnectionInfo(
           this.host_, this.credentialsProvider_, this, this.plugin_);
       this.listener_.onConnected(connectionInfo);
       break;
 
     case remoting.ClientSession.State.CONNECTING:
+      chrome.metricsPrivate.recordUserAction(
+          'Chromoting.Connections.Connecting');
       remoting.identity.getEmail().then(function(/** string */ email) {
         console.log('Connecting as ' + email);
       });
       break;
 
     case remoting.ClientSession.State.AUTHENTICATED:
+      chrome.metricsPrivate.recordUserAction(
+          'Chromoting.Connections.Authenticated');
       console.log('Connection authenticated.');
       break;
 
     case remoting.ClientSession.State.INITIALIZING:
       console.log('Connection initializing .');
       break;
 
     case remoting.ClientSession.State.CLOSED:
+      chrome.metricsPrivate.recordUserAction('Chromoting.Connections.Closed');
       console.log('Connection closed.');
       this.listener_.onDisconnected(remoting.Error.none());
       break;
 
     case remoting.ClientSession.State.CONNECTION_CANCELED:
     case remoting.ClientSession.State.FAILED:
       error = this.getError();
       if (!error.isNone()) {
+        chrome.metricsPrivate.recordUserAction('Chromoting.Connections.Failed');
         console.error('Connection failed: ' + error.toString());
       }
       this.listener_.onConnectionFailed(error);
       break;
 
     case remoting.ClientSession.State.CONNECTION_DROPPED:
       error = this.getError();
       console.error('Connection dropped: ' + error.toString());
       this.listener_.onDisconnected(error);
+      chrome.metricsPrivate.recordUserAction('Chromoting.Connections.Dropped');
       break;
 
     default:
       console.error('Unexpected client plugin state: ' + newState);
   }
 };
 
 /**
  * @param {remoting.ClientSession.State} previous
  * @param {remoting.ClientSession.State} current
@@ skipping 28 matching lines @@
  * Enable or disable logging of connection errors due to a host being offline.
  * For example, if attempting a connection using a cached JID, host-offline
  * errors should not be logged because the JID will be refreshed and the
  * connection retried.
  *
  * @param {boolean} enable True to log host-offline errors; false to suppress.
  */
 remoting.ClientSession.prototype.logHostOfflineErrors = function(enable) {
   this.logHostOfflineErrors_ = enable;
 };