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 2b5b538ee0273722f0267734e23d4ad4a412c30f..a0398eb1f04e00a2d3a3ec6717dc445aade4e215 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -4791,6 +4791,36 @@ other types of suffix sets.
   </summary>
 </histogram>
 
+<histogram name="FileBrowser.SuggestApps.Close"
+    enum="SuggestAppsDialogCloseReason">
+  <summary>
+    Chrome OS File Browser: the reason why the suggest apps dialog is closed.

[Ilya Sherman, 2013/10/28 23:00:56]

nit: "is" -> "was"

[yoshiki, 2013/10/30 18:48:53]

Done.

+  </summary>
+</histogram>
+
+<histogram name="FileBrowser.SuggestApps.Install"
+    enum="SuggestAppsDialogInstall">
+  <summary>
+    Chrome OS File Browser: if the Webstore item user selected is successfully

[Ilya Sherman, 2013/10/28 23:00:56]

nit: "if" -> "whether"; "is" -> "was"

[yoshiki, 2013/10/30 18:48:53]

Done.

+    installed or not.
+  </summary>
+</histogram>
+
+<histogram name="FileBrowser.SuggestApps.Load" enum="SuggestAppsDialogLoad">
+  <summary>
+    Chrome OS File Browser: if the initialization of the dialog is successed or

[Ilya Sherman, 2013/10/28 23:00:56]

nit: "if" -> "whether"; "is" -> "was"

[Ilya Sherman, 2013/10/28 23:00:56]

nit: "is successed" -> "succeeded"

[yoshiki, 2013/10/30 18:48:53]

Done.

[yoshiki, 2013/10/30 18:48:53]

Done.

+    not.
+  </summary>
+</histogram>
+
+<histogram name="FileBrowser.SuggestApps.LoadTime" units="milliseconds">
+  <summary>
+    Chrome OS File Browser: time to load the suggest apps dialog. Measured
+    between the moment window appears and the moment it is all available after

[Ilya Sherman, 2013/10/28 23:00:56]

I'm not sure what "it is all available" means.  Could you please try rephrasing
that to be a little clearer?

[yoshiki, 2013/10/30 18:48:53]

Done.

+    ChromeWebstore widget is ready.
+  </summary>
+</histogram>
+
 <histogram name="FileBrowser.ViewingFileType" enum="ViewFileType">
   <summary>
     File types that were tried to be viewed through browser. This is recorded
@@ -27070,6 +27100,25 @@ other types of suffix sets.
   <int value="15" label="PROCEED_NEW_SITE"/>
 </enum>
 
+<enum name="SuggestAppsDialogCloseReason" type="int">
+  <int value="0" label="Unknown error"/>
+  <int value="1" label="Item installed"/>
+  <int value="2" label="User cancelled"/>
+  <int value="3" label="Webstore link clicked"/>
+</enum>
+
+<enum name="SuggestAppsDialogInstall" type="int">
+  <int value="0" label="Install succeeded"/>
+  <int value="1" label="Install cancelled"/>
+  <int value="2" label="Install failed"/>
+</enum>
+
+<enum name="SuggestAppsDialogLoad" type="int">
+  <int value="0" label="Load succeeded"/>
+  <int value="1" label="Load cancelled"/>
+  <int value="2" label="Load failed"/>
+</enum>
+
 <enum name="SuspendStatus" type="int">
   <int value="0" label="Success"/>
   <int value="1" label="Failure"/>