Add UMA stats for Chromoting connection details.

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 3e7c5c170d39e12dc4bfaee1968c489026292535..f60a1806f34bf61f07608ec89f92721451817786 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -3807,6 +3807,47 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Chromoting.Connections.DurationsMinutes.Closed"
+    units="minutes">
+  <owner>anandc@chromium.org</owner>
+  <owner>sergeyu@chromium.org</owner>
+  <summary>
+    The duration of sessions that were closed, normally by a user-initiated
+    action.
+  </summary>
+</histogram>
+
+<histogram name="Chromoting.Connections.DurationsMinutes.Failed"

[Ilya Sherman, 2015/08/20 23:42:46]

I'm not sure what the value of separate "DurationsMinutes" and "TimeMs" names is
-- I think it would be clearer to just have "Duration" or "Time" histograms, and
use the units attribute to distinguish them.  That way all duration histograms
would be grouped together.

[anandc, 2015/08/21 03:15:42]

Yes, I actually first had these simpler, then thought that having the units in
the name would make it easier, especially because the units are different for
connection times and connecition durations.
Now I'm inclined to agree with you again. :-) So changed.

+    units="minutes">
+  <owner>anandc@chromium.org</owner>
+  <owner>sergeyu@chromium.org</owner>
+  <summary>The duration of sessions that failed to connect.</summary>
+</histogram>
+
+<histogram name="Chromoting.Connections.Failed">

[Ilya Sherman, 2015/08/20 23:42:46]

I'd recommend including this metric in the enumerated histogram that I
recommended for the actions.

[anandc, 2015/08/21 03:15:42]

Moved to actions.xml for now.

+  <owner>anandc@chromium.org</owner>
+  <owner>sergeyu@chromium.org</owner>
+  <summary>Count of Chromoting Failed events.</summary>
+</histogram>
+
+<histogram name="Chromoting.Connections.TimesMs.Authenticate"
+    units="milliseconds">
+  <owner>anandc@chromium.org</owner>
+  <owner>sergeyu@chromium.org</owner>
+  <summary>
+    The time taken to authenticate as part of a Chromoting conneciton request.
+  </summary>
+</histogram>
+
+<histogram name="Chromoting.Connections.TimesMs.Connect" units="milliseconds">
+  <owner>anandc@chromium.org</owner>
+  <owner>sergeyu@chromium.org</owner>
+  <summary>
+    The time taken to connect to a remote-host, after a request has been
+    authenticated.
+  </summary>
+</histogram>
+
 <histogram name="Chromoting.Video.Bandwidth" units="Bytes/second">
   <owner>anandc@chromium.org</owner>
   <owner>sergeyu@chromium.org</owner>
@@ -8694,6 +8735,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Enterprise.SystemLogPIILeak" enum="SystemLogPIIType">
+  <owner>pbond@chromium.org</owner>
+  <summary>
+    Events for counting sensitive data occurrences in system logs to upload.
+    (Email addresses, IP addresses, SSIDs, URLs). The lines with this data will
+    be removed before upload.
+  </summary>
+</histogram>

[Ilya Sherman, 2015/08/20 23:42:46]

Why is this part of this CL?  Seems like there are a bunch of spurious diffs.

[anandc, 2015/08/21 03:15:42]

Not sure what happened here. Fixed.

+
 <histogram name="Enterprise.UserCloudPolicyStore.LoadStatus"
     enum="PolicyLoadStatus">
   <owner>atwilson@chromium.org</owner>
@@ -46716,6 +46766,58 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Tabs.StateTransfer.NumberOfOtherTabsActivatedBeforeMadeActive"
+    units="tabs">
+  <owner>kouhei@chromium.org</owner>
+  <owner>tzik@chromium.org</owner>
+  <summary>
+    Recorded once per tab activation. The number of tabs that were activated
+    while the tab was inactive.
+  </summary>
+  <details>
+    This metric is to be removed after M46 hits stable. We don't expect this
+    metric to change over release, so we will remove this once we have the
+    numbers from stable.
+  </details>
+</histogram>
+
+<histogram name="Tabs.StateTransfer.TabDistanceInactiveToActive" units="tabs">
+  <owner>kouhei@chromium.org</owner>
+  <owner>tzik@chromium.org</owner>
+  <summary>
+    Number of tabs between the previously active tab and the new active tab.
+  </summary>
+  <details>
+    This metric is to be removed after M46 hits stable. We don't expect this
+    metric to change over release, so we will remove this once we have the
+    numbers from stable.
+  </details>
+</histogram>
+
+<histogram name="Tabs.StateTransfer.Target" enum="TabStripState">
+  <owner>kouhei@chromium.org</owner>
+  <owner>tzik@chromium.org</owner>
+  <summary>
+    The state to which a tab transitioned. Recorded when a tab transitions from
+    the state in the histogram name to a new state.
+  </summary>
+  <details>
+    kouhei@ and tzik@ will remove some variation of these once M46 hits stable.
+  </details>
+</histogram>
+
+<histogram name="Tabs.StateTransfer.Time" units="milliseconds">
+  <owner>kouhei@chromium.org</owner>
+  <owner>tzik@chromium.org</owner>
+  <summary>
+    Tabs.StateTransfer.TimeA_B measures the time a tab was in state A before it
+    transferred to state B.
+  </summary>
+  <details>
+    kouhei@ and tzik@ will remove some variation of these once M46 hits stable.
+  </details>
+</histogram>
+
 <histogram name="Tabs.SwitchFromCloseLatency" units="ms">
   <owner>simonb@chromium.org</owner>
   <summary>
@@ -62927,7 +63029,9 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="-384589459" label="disable-supervised-user-safesites"/>
   <int value="-378033324" label="disable-win32k-renderer-lockdown"/>
   <int value="-360038744" label="invert-viewport-scroll-order"/>
+  <int value="-351552989" label="disable-hosted-apps-in-windows"/>
   <int value="-349057743" label="extensions-on-chrome-urls"/>
+  <int value="-345838366" label="enable-hosted-apps-in-windows"/>
   <int value="-344343842" label="disable-experimental-app-list"/>
   <int value="-340622848" label="disable-javascript-harmony-shipping"/>
   <int value="-340255045" label="allow-nacl-socket-api"/>
@@ -66655,6 +66759,7 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="1641037564" label="PPB_VideoSource_Private;0.1"/>
   <int value="1645591549" label="PPB_Widget(Dev);0.3"/>
   <int value="1659973365" label="PPB_ImageCapture_Private;0.1"/>
+  <int value="1665257952" label="PPB_VideoDecoder;1.1"/>
   <int value="1677958987" label="PPB_ImageData;1.0"/>
   <int value="1680873803" label="PPB_Console;1.0"/>
   <int value="1681523535" label="PPB_TCPSocket;1.2"/>
@@ -70827,6 +70932,9 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="2" label="SHOW_CAPTIVE_PORTAL_INTERSTITIAL_OVERRIDABLE"/>
   <int value="3" label="SHOW_SSL_INTERSTITIAL_NONOVERRIDABLE"/>
   <int value="4" label="SHOW_SSL_INTERSTITIAL_OVERRIDABLE"/>
+  <int value="5" label="WWW_MISMATCH_FOUND"/>
+  <int value="6" label="WWW_MISMATCH_URL_AVAILABLE"/>
+  <int value="7" label="WWW_MISMATCH_URL_NOT_AVAILABLE"/>
 </enum>
 
 <enum name="SSLErrorTypes" type="int">
@@ -71483,6 +71591,13 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="5" label="Actionable error"/>
 </enum>
 
+<enum name="SystemLogPIIType" type="int">
+  <int value="0" label="Email address"/>
+  <int value="1" label="IP address"/>
+  <int value="2" label="Web URL"/>
+  <int value="3" label="SSID"/>
+</enum>
+
 <enum name="TabBackgroundLoadStatus" type="int">
   <int value="0" label="Loaded on creation and shown"/>
   <int value="1" label="Loaded on creation and lost"/>
@@ -71529,6 +71644,14 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="10" label="Evicted due to page loading when backgrounding"/>
 </enum>
 
+<enum name="TabStripState" type="int">
+  <int value="0" label="Initial state"/>
+  <int value="1" label="Tab is active, shown foreground."/>
+  <int value="2" label="Tab is inactive, in background."/>
+  <int value="3" label="Tab is detached from tab-strip."/>
+  <int value="4" label="Tab is closed."/>
+</enum>
+
 <enum name="TabSwitchedToForegroundLaunchedWithURL" type="int">
   <obsolete>
     Deprecated as of 04/2014.
@@ -73381,6 +73504,19 @@ To add a new entry, add it with any value and run test to compute valid value.
   <affected-histogram name="CrosFirstRun.TimeSpentOnStep"/>
 </histogram_suffixes>
 
+<histogram_suffixes name="CurrentTabState">
+  <suffix name="Initial" label="For a tab that is just being created."/>
+  <suffix name="Active"
+      label="For an active tab which is shown foreground in a browser window."/>
+  <suffix name="Inactive"
+      label="For tabs in background and not shown to user."/>
+  <suffix name="Detached"
+      label="For a tab that is being dragged by user to outside of the
+             browser window."/>
+  <affected-histogram name="Tabs.StateTransfer.Target"/>
+  <affected-histogram name="Tabs.StateTransfer.Time"/>
+</histogram_suffixes>
+
 <histogram_suffixes name="DataReductionProxy">
   <suffix name="DataReductionProxy"
       label="Only page loads through the data reduction proxy are considered."/>
@@ -76036,6 +76172,20 @@ To add a new entry, add it with any value and run test to compute valid value.
   <affected-histogram name="NewTabPage.LoadTime"/>
 </histogram_suffixes>
 
+<histogram_suffixes name="NextTabState">
+  <suffix name="Active"
+      label="For a tab active which is shown foreground in a browser window."/>
+  <suffix name="Inactive"
+      label="For tabs in background and not shown to user."/>
+  <suffix name="Detached"
+      label="For a tab that is being dragged by user to outside of the
+             browser window."/>
+  <suffix name="Closed" label="For a tab that is about to be closed."/>
+  <affected-histogram name="Tabs.StateTransfer.Time_Active"/>
+  <affected-histogram name="Tabs.StateTransfer.Time_Detached"/>
+  <affected-histogram name="Tabs.StateTransfer.Time_Inactive"/>
+</histogram_suffixes>
+
 <histogram_suffixes name="NQE.DifferentPercentiles" separator=".">
   <suffix name="Percentile0" label="0th percentile"/>
   <suffix name="Percentile10" label="10th percentile"/>