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 0e09e8210dfe3bf321a65f4cdcffc2a507c297c8..d28a407c93988c32dcc1d37a6fa489b9142c7f71 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -9975,6 +9975,37 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Cryptohome.DircryptoMigrationInitialFreeSpaceInMb" units="MB">
+  <owner>dspaid@chromium.org</owner>
+  <summary>
+    The total mount of free disk space on the system when the user started the

[hashimoto, 2017/06/12 06:52:51]

nit: s/mount/amount/?

[dspaid, 2017/06/12 07:00:19]

Done.

+    migration from ecryptfs to ext4-crypto.  This is only reported when the user
+    encounters an insufficient space error.
+  </summary>
+</histogram>
+
+<histogram name="Cryptohome.DircryptoMigrationNoSpaceFailureFreeSpaceInMb"
+    units="MB">
+  <owner>dspaid@chromium.org</owner>
+  <summary>
+    The total amount of free disk space on the system when the user received an
+    insufficient space error during the migration from ecryptfs to ext4-crypto.
+    This is only collected when the user encounters an insufficient space error.
+  </summary>
+</histogram>
+
+<histogram name="Cryptohome.DircryptoMigrationNoSpaceXattrSizeInBytes"
+    units="bytes">
+  <owner>dspaid@chromium.org</owner>
+  <summary>
+    The total number of bytse of xattr data that we are trying to store on a
+    file when we receive an insufficient space error.  This includes both xattr
+    data already present on the file as well as the new xattr we are trying to
+    set.  Recorder during ecryptfs to ext4-crypto migration if we receive an

[hashimoto, 2017/06/12 06:52:51]

nit: s/Recorder/Recorded/?

The descriptions of existing Cryptohome.Dircrypto* histograms say "logged", but
in this CL you are using "reported", "collected", and "recorded".
Could you stick to one verb to be consistent?

[dspaid, 2017/06/12 07:00:19]

Done.

+    insufficient space error when setting a new extended attribute.
+  </summary>
+</histogram>
+
 <histogram name="Cryptohome.DircryptoMigrationStartStatus"
     enum="DircryptoMigrationStartStatus">
   <owner>dspaid@chromium.org</owner>