Indexed DB: Remove transaction throttle
A transaction throttle was introduced in https://crrev.com/1511973004
to mitigate behavior where a renderer was issuing transactions faster
than they could be processed, leading to eventual OOM. Unfortunately,
it looks like some apps are relying on behavior that trips this
throttle, so we need to rethink it. Remove the throttle, but keep
recording UMA stats.
R=cmumford@chromium.org
BUG=338800,584768
Review URL: https://codereview.chromium.org/1671193003
Cr-Commit-Position: refs/heads/master@{#374443}
(cherry picked from commit 8b348ae7675a7db880b2dee87a582a79377d8fa8)
Review URL: https://codereview.chromium.org/1690093003 .
Cr-Commit-Position: refs/branch-heads/2623@{#380}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
diff --git a/content/browser/indexed_db/indexed_db_database.cc b/content/browser/indexed_db/indexed_db_database.cc
index 3196cd1..d90af34 100644
--- a/content/browser/indexed_db/indexed_db_database.cc
+++ b/content/browser/indexed_db/indexed_db_database.cc
@@ -1672,18 +1672,6 @@
"WebCore.IndexedDB.Database.OutstandingTransactionCount",
transactions_.size());
- // Throttle transaction creation so that a renderer in a tight loop can't
- // cause browser memory to grow unbounded by creating transactions faster
- // than they can be processed.
- const size_t kMaxTransactionCount = 256;
- if (transactions_.size() >= kMaxTransactionCount) {
- connection->callbacks()->OnAbort(
- transaction_id, IndexedDBDatabaseError(
- blink::WebIDBDatabaseExceptionUnknownError,
- "Internal error: Too many transactions queued."));
- return;
- }
-
// The transaction will add itself to this database's coordinator, which
// manages the lifetime of the object.
TransactionCreated(IndexedDBClassFactory::Get()->CreateIndexedDBTransaction(
