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

Unified Diff: content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc

Issue 2472213003: [IndexedDB] Refactoring to remove ref ptrs and host transaction ids. (Closed)
Patch Set: comments & rebase Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc
diff --git a/content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc b/content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc
index 4740a98bf6d9abdd6e976114513e6eba12439f5c..f1b82cd23e6ebcd7ee7049e7cd7a3d42aabab24a 100644
--- a/content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc
+++ b/content/browser/indexed_db/mock_browsertest_indexed_db_class_factory.cc
@@ -65,12 +65,12 @@ class IndexedDBTestTransaction : public IndexedDBTransaction {
public:
IndexedDBTestTransaction(
int64_t id,
- base::WeakPtr<IndexedDBConnection> connection,
+ IndexedDBConnection* connection,
const std::set<int64_t>& scope,
blink::WebIDBTransactionMode mode,
IndexedDBBackingStore::Transaction* backing_store_transaction)
: IndexedDBTransaction(id,
- std::move(connection),
+ connection,
scope,
mode,
backing_store_transaction) {}
@@ -263,14 +263,14 @@ MockBrowserTestIndexedDBClassFactory::CreateIndexedDBDatabase(
unique_identifier);
}
-IndexedDBTransaction*
+std::unique_ptr<IndexedDBTransaction>
MockBrowserTestIndexedDBClassFactory::CreateIndexedDBTransaction(
int64_t id,
- base::WeakPtr<IndexedDBConnection> connection,
+ IndexedDBConnection* connection,
const std::set<int64_t>& scope,
blink::WebIDBTransactionMode mode,
IndexedDBBackingStore::Transaction* backing_store_transaction) {
- return new IndexedDBTestTransaction(id, std::move(connection), scope, mode,
+ return new IndexedDBTestTransaction(id, connection, scope, mode,
backing_store_transaction);
}

Powered by Google App Engine
This is Rietveld 408576698