[IndexedDB] Adding traces, perf tests

content/browser/indexed_db/leveldb/leveldb_database.cc

Index: content/browser/indexed_db/leveldb/leveldb_database.cc
diff --git a/content/browser/indexed_db/leveldb/leveldb_database.cc b/content/browser/indexed_db/leveldb/leveldb_database.cc
index 2c18f1f5170f34448b4d01f43f28bb82913ed6ab..d35e03f145f4739a4ce4cbf05147ed646a50ad36 100644
--- a/content/browser/indexed_db/leveldb/leveldb_database.cc
+++ b/content/browser/indexed_db/leveldb/leveldb_database.cc
@@ -16,7 +16,9 @@
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/sys_info.h"
+#include "base/trace_event/trace_event.h"
 #include "content/browser/indexed_db/indexed_db_class_factory.h"
+#include "content/browser/indexed_db/indexed_db_tracing.h"
 #include "content/browser/indexed_db/leveldb/leveldb_comparator.h"
 #include "content/browser/indexed_db/leveldb/leveldb_env.h"
 #include "content/browser/indexed_db/leveldb/leveldb_iterator_impl.h"
@@ -106,6 +108,7 @@ static leveldb::Status OpenDB(
     const base::FilePath& path,
     leveldb::DB** db,
     scoped_ptr<const leveldb::FilterPolicy>* filter_policy) {
+  IDB_TRACE("LevelDBDatabase::Open");

[cmumford, 2015/08/03 17:31:33]

What do you think about actually making this a static member function named
LevelDBDatabase::Open?

[dmurph, 2015/08/05 15:30:32]

There already is one, I'll just move this trace there.

   filter_policy->reset(leveldb::NewBloomFilterPolicy(10));
   leveldb::Options options;
   options.comparator = comparator;
@@ -423,6 +426,7 @@ const LevelDBComparator* LevelDBDatabase::Comparator() const {
 
 void LevelDBDatabase::Compact(const base::StringPiece& start,
                               const base::StringPiece& stop) {
+  TRACE_EVENT0("LevelDB", "LevelDBDatabase::Compact");

[cmumford, 2015/08/03 17:31:33]

Why TRACE_EVENT instead of IDB_TRACE?

[dmurph, 2015/08/05 15:30:32]

Whoops :)

   const leveldb::Slice start_slice = MakeSlice(start);
   const leveldb::Slice stop_slice = MakeSlice(stop);
   // NULL batch means just wait for earlier writes to be done