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 4400ab149aff1b051521b1e463b5e4a144f509e0..674687b98f888643e5bdf3dac077928c8a05fdff 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -74494,6 +74494,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="WebCore.IndexedDB.ObjectStoreRecord.KeyType" enum="IDBKeyType">
+  <owner>jsbell@chromium.org</owner>
+  <summary>
+    The type of key (number, string, etc.) used for a record being
+    stored/updated in an Indexed DB object store.
+  </summary>
+</histogram>
+
 <histogram name="WebCore.IndexedDB.OpenTime.Blocked" units="ms">
   <owner>cmumford@chromium.org</owner>
   <summary>
@@ -93700,6 +93708,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="2" label="Array">Key path is an array of strings.</int>
 </enum>
 
+<enum name="IDBKeyType" type="int">
+  <int value="0" label="Invalid">Invalid key.</int>

[cmumford, 2017/02/07 20:10:37]

This will never be logged right? If so how about deleting this enum value, if it
does get logged then UMA will display "0" right?

[jsbell, 2017/02/07 21:58:24]

Correct, it should never be logged.

mpearson@ - what is preferred here?

[Mark P, 2017/02/07 22:59:46]

It could go either way in this case.  It depends in which case you think someone
is more likely to contact you: if someone sees a "0" on the dashboard or if
someone sees an "Invalid key."?

+  <int value="1" label="Array">Key is an array of other keys.</int>
+  <int value="2" label="Binary">Key is a binary buffer.</int>
+  <int value="3" label="String">Key is a string.</int>
+  <int value="4" label="Date">Key is a date.</int>
+  <int value="5" label="Number">Key is a number.</int>
+</enum>
+
 <enum name="IDBLevelDBBackingStoreInternalErrorType" type="int">
   <int value="0" label="IDBLevelDBBackingStoreReadError">
     IndexedDB encountered an error attempting to read or decode a value from the