Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(96)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 472573003: IndexedDB: Added timing histograms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed WebCore.IndexedDB.LevelDB.Remove Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « content/browser/indexed_db/leveldb/leveldb_transaction.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index d51e0645a245845bb7e73e3b994bdfcbdbbf0277..a302e36887f0a5536ceb1fecbede2d95d5504e89 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -35746,6 +35746,37 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="WebCore.IndexedDB.LevelDB.OpenTime.Disk" units="milliseconds">
Ilya Sherman 2014/08/14 20:52:11 nit: Why does this histogram have a ".Disk" suffix
cmumford 2014/08/14 21:58:37 Good question. We also open up IDB's in memory - f
+ <owner>cmumford@chromium.org</owner>
+ <summary>
+ The time that it takes to open IndexedDB's LevelDB backing store (on disk).
+ </summary>
+</histogram>
+
+<histogram name="WebCore.IndexedDB.LevelDB.PutTime" units="milliseconds">
+ <owner>cmumford@chromium.org</owner>
+ <summary>
+ The time that it takes to write the data IndexedDB to the LevelDB backing
+ store for a put operation.
+ </summary>
+</histogram>
+
+<histogram name="WebCore.IndexedDB.LevelDB.Transaction.CommitTime"
+ units="milliseconds">
+ <owner>cmumford@chromium.org</owner>
+ <summary>
+ The time that it takes to commit an IndexedDB transaction to it's LevelDB
Ilya Sherman 2014/08/14 20:52:11 nit: "it's" -> "its"
cmumford 2014/08/14 21:58:38 Done.
+ backing store.
+ </summary>
+</histogram>
+
+<histogram name="WebCore.IndexedDB.LevelDB.WriteTime" units="milliseconds">
+ <owner>cmumford@chromium.org</owner>
+ <summary>
+ The time that it takes to write data to a IndexedDB LevelDB backing store.
Ilya Sherman 2014/08/14 20:52:12 nit: "IndexedDB" -> "IndexedDB's"?
Ilya Sherman 2014/08/14 20:52:12 nit: "a" -> "an".
cmumford 2014/08/14 21:58:37 Done.
cmumford 2014/08/14 21:58:37 Done.
+ </summary>
+</histogram>
+
<histogram name="WebCore.IndexedDB.LevelDBOpenErrors" enum="LevelDBErrorTypes">
<owner>dgrogan@chromium.org</owner>
<summary>
« no previous file with comments | « content/browser/indexed_db/leveldb/leveldb_transaction.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698