IndexedDB: Measure the total database open time.

content/browser/indexed_db/indexed_db_dispatcher_host.cc

Index: content/browser/indexed_db/indexed_db_dispatcher_host.cc
diff --git a/content/browser/indexed_db/indexed_db_dispatcher_host.cc b/content/browser/indexed_db/indexed_db_dispatcher_host.cc
index 0cdde9d016e8fd3520056202f5adef2cddf770b8..8de50ba8f13230c22a83ab20ec718b6f0bd88b70 100644
--- a/content/browser/indexed_db/indexed_db_dispatcher_host.cc
+++ b/content/browser/indexed_db/indexed_db_dispatcher_host.cc
@@ -291,6 +291,7 @@ void IndexedDBDispatcherHost::OnIDBFactoryGetDatabaseNames(
 void IndexedDBDispatcherHost::OnIDBFactoryOpen(
     const IndexedDBHostMsg_FactoryOpen_Params& params) {
   DCHECK(indexed_db_context_->TaskRunner()->RunsTasksOnCurrentThread());
+  base::TimeTicks begin_time = base::TimeTicks::Now();

[jsbell, 2014/08/14 22:58:06]

Why a temporary here rather than just passing in Now() below?

(it doesn't look like we do a measurable amount of work in between)

[cmumford, 2014/08/14 23:53:03]

Just being cautious. The stack value is super small and I wanted this to be a
bit more "future proof" so that if somebody puts in something that does take
time then it'll get measured.

If you want it removed let me know - no problem.

[jsbell, 2014/08/15 00:05:16]

It's fine as is, and I agree with your logic. :)

   base::FilePath indexed_db_path = indexed_db_context_->data_path();
 
   GURL origin_url =
@@ -307,6 +308,7 @@ void IndexedDBDispatcherHost::OnIDBFactoryOpen(
                              params.ipc_database_callbacks_id,
                              host_transaction_id,
                              origin_url);
+  callbacks->SetDatbaseOpenStartTime(begin_time);
   scoped_refptr<IndexedDBDatabaseCallbacks> database_callbacks =
       new IndexedDBDatabaseCallbacks(
           this, params.ipc_thread_id, params.ipc_database_callbacks_id);