[IndexedDB] Refactoring to remove ref ptrs and host transaction ids.

content/browser/indexed_db/indexed_db_connection.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_
 #define CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_
 
 #include <memory>
+#include <unordered_map>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "content/browser/indexed_db/indexed_db_database.h"
-#include "content/browser/indexed_db/indexed_db_database_callbacks.h"
-#include "content/browser/indexed_db/indexed_db_observer.h"
 
 namespace content {
+class IndexedDBDatabaseCallbacks;
 class IndexedDBDatabaseError;
+class IndexedDBObserver;
+class IndexedDBTransaction;
 
 class CONTENT_EXPORT IndexedDBConnection {
  public:
-  IndexedDBConnection(scoped_refptr<IndexedDBDatabase> db,
+  IndexedDBConnection(int child_process_id,
+                      scoped_refptr<IndexedDBDatabase> db,
                       scoped_refptr<IndexedDBDatabaseCallbacks> callbacks);
   virtual ~IndexedDBConnection();
 
   // These methods are virtual to allow subclassing in unit tests.
   virtual void ForceClose();
   virtual void Close();
   virtual bool IsConnected();
 
   void VersionChangeIgnored();
 
   virtual void ActivatePendingObservers(
       std::vector<std::unique_ptr<IndexedDBObserver>> pending_observers);
   // Removes observer listed in |remove_observer_ids| from active_observer of
   // connection or pending_observer of transactions associated with this
   // connection.
   virtual void RemoveObservers(const std::vector<int32_t>& remove_observer_ids);
 
   int32_t id() const { return id_; }
+  int child_process_id() const { return child_process_id_; }
 
   IndexedDBDatabase* database() const { return database_.get(); }
   IndexedDBDatabaseCallbacks* callbacks() const { return callbacks_.get(); }
   const std::vector<std::unique_ptr<IndexedDBObserver>>& active_observers()
       const {
     return active_observers_;
   }
   base::WeakPtr<IndexedDBConnection> GetWeakPtr() {
     return weak_factory_.GetWeakPtr();
   }
 
+  // Creates a transaction for this connection.
+  IndexedDBTransaction* CreateTransaction(
+      int64_t id,
+      const std::set<int64_t>& scope,
+      blink::WebIDBTransactionMode mode,
+      IndexedDBBackingStore::Transaction* backing_store_transaction);
+
+  IndexedDBTransaction* GetTransaction(int64_t id);
+
+  IndexedDBTransaction* StoreTransactionForTesting(
+      std::unique_ptr<IndexedDBTransaction> transaction);
+
+  // We ignore calls where the id doesn't exist to facilitate our AbortAll call
+  // below.
+  // TODO(dmurph): Change this to make aborting more robust.
+  void EraseTransaction(int64_t id);
+
+  void AbortAllTransactions(const IndexedDBDatabaseError& error);
+
+  const std::unordered_map<int64_t, std::unique_ptr<IndexedDBTransaction>>&
+  transactions() {

[cmumford, 2016/11/04 23:33:12]

make method const.

[dmurph, 2016/11/07 20:05:22]

Done.

+    return transactions_;
+  }
+
  private:
   const int32_t id_;
+  const int child_process_id_;

[jsbell, 2016/11/04 17:48:19]

Can this be base::ProcessId rather than int?

[jsbell, 2016/11/04 17:48:19]

Can you add a comment about why we track the process id?

[dmurph, 2016/11/04 22:52:24]

Well, we're given an 'int' down the whole stack, so I was just using that.
(starting from the constructor of IndexedDBDispatcherHost, where the
RenderProcessHostImpl only gives an int.

[dmurph, 2016/11/04 22:52:24]

Done.

[jsbell, 2016/11/05 00:01:48]

If it's given to us as an `int` that's fine (I didn't trace it back that far).
It's just odd to see a plain `int` these days. :)

 
   // NULL in some unit tests, and after the connection is closed.
   scoped_refptr<IndexedDBDatabase> database_;
+  std::unordered_map<int64_t, std::unique_ptr<IndexedDBTransaction>>

[jsbell, 2016/11/04 17:48:19]

Is there a good reason to use unordered_map vs. map? This means the order of
aborts will be non-deterministic; does that matter?
 

[jsbell, 2016/11/04 17:48:19]

Can you add comments to members as you touch them?

[dmurph, 2016/11/04 22:52:24]

Done.

[dmurph, 2016/11/04 22:52:24]

Huh. I don't know. I don't think so.

+      transactions_;
 
   // The callbacks_ member is cleared when the connection is closed.
   // May be NULL in unit tests.
   scoped_refptr<IndexedDBDatabaseCallbacks> callbacks_;
   std::vector<std::unique_ptr<IndexedDBObserver>> active_observers_;
   base::WeakPtrFactory<IndexedDBConnection> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(IndexedDBConnection);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_CONNECTION_H_