Add another set of UMA histograms for tab switching on Android.

tools/metrics/histograms/histograms.xml

 <!--
 Copyright 2013 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.
 -->
 
 <!--
 This file is used to generate a comprehensive list of Chrome histograms along
 with a detailed description for each histogram.
 
@@ skipping 5835 matching lines @@
 </histogram>
 
 <histogram name="Sqlite.Thumbnail.Error" enum="SqliteErrorCode">
   <summary>Error codes returned by sqlite for the thumbnail db.</summary>
 </histogram>
 
 <histogram name="Sqlite.Web.Error" enum="SqliteErrorCode">
   <summary>Error codes returned by sqlite the web db.</summary>
 </histogram>
 
+<histogram name="Tab.RestoreResult" enum="TabRestoreResult">
+  <summary>
+    When the browser restores a tab, whether the load was successful. Loads can
+    fail for instance when there is no connectivity.
+  </summary>
+</histogram>
+
+<histogram name="Tab.RestoreTime" unit="ms">
+  <summary>Load times for successful tab restores.</summary>
+</histogram>
+
+<histogram name="Tab.RestoreUserPersistence" enum="TabRestoreUserAction">
+  <summary>
+    When the browser restores a tab, whether the user waits for completion of
+    the load or if he gives up by switching to another tab or leaving Chrome.

[ppi, 2013/04/30 12:12:27]

supernit: "if they give up"? Does anybody know if we have a policy on
http://en.wikipedia.org/wiki/Singular_they ?

[bulach, 2013/04/30 12:38:11]

how about avoiding the issue altogether and name it something like
RestoreCompletionStatus?

<int value="0" label="RestoredFully"/>
<int value="1" label="SwitchedToAnotherTab"/>
<int value="2" label="SwitchedToAnotherApp"/>

?

[ppi, 2013/04/30 12:42:42]

Please note that strictly speaking "RestoredFully" does not exclude
"SwitchedToAnotherTab". I think I would prefer to keep the first one as
"WaitForCompletion".

Another possible easy fix: s/he/the user/.

On 2013/04/30 12:38:11, bulach wrote:

+  </summary>
+</histogram>
+
 <histogram name="Tab.SwitchedToForegroundAge" units="ms">
   <summary>Age (in ms) when the tab was switched to foreground.</summary>
 </histogram>
 
 <histogram name="Tab.SwitchedToForegroundLaunchedWithURL"
     enum="TabSwitchedToForegroundLaunchedWithURL">
   <summary>
     Each time a tab is brought to the foreground, this histogram indicates if
     chrome was launched without an URL (i.e., from the launcher), or with an URL
     (i.e., from another app).
@@ skipping 3513 matching lines @@
   </int>
   <int value="26" label="SQLITE_NOTADB">
     File opened that is not a database file
   </int>
   <int value="100" label="SQLITE_ROW">sqlite3_step() has another row ready</int>
   <int value="101" label="SQLITE_DONE">
     sqlite3_step() has finished executing
   </int>
 </enum>
 
+<enum name="TabRestoreResult" type="int">
+  <int value="0" label="Failed"/>
+  <int value="1" label="Succeeded"/>
+</enum>
+
+<enum name="TabRestoreUserAction" type="int">
+  <int value="0" label="WaitForCompletion"/>
+  <int value="1" label="SwitchToAnotherTab"/>
+  <int value="2" label="LeaveChrome"/>
+</enum>
+
 <enum name="TabSwitchedToForegroundLaunchedWithURL" type="int">
   <int value="0" label="Launched without an URL"/>
   <int value="1" label="Launched with an URL"/>
 </enum>
 
 <enum name="TabSwitchedToForegroundRevisit" type="int">
   <int value="0" label="First time"/>
   <int value="1" label="Revisit"/>
 </enum>
 
@@ skipping 1318 matching lines @@
 <fieldtrial name="SSLResumption">
   <group name="Resume_Handshake" label="Session Resumption"/>
   <group name="Full_Handshake" label="Full"/>
   <affected-histogram name="Net.SSL_Connection_Latency"/>
   <affected-histogram name="Net.SSL_Connection_Latency_Google"/>
 </fieldtrial>
 
 </fieldtrials>
 
 </histogram-configuration>