1973799 - [wpt-sync] Sync PR 53362 - IDB: handle double deletion in SQLite store.

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 53362 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/53362
Details from upstream follow.

Evan Stade <[e-mail]> wrote:

IDB: handle double deletion in SQLite store.

calling deleteDatabase() on a non-existent database normally doesn't
even create a Database object, but if it does exist and
deleteDatabase is called on it twice, the first call will delete the
DatabaseConnection and the second call (enqueued behind the first)
needs to be able to handle this state.

Bug: 40253999
Change-Id: I64279e940e79af5435447cda7cf9304d5f61a288
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6658503
Reviewed-by: Abhishek Shanthkumar \<[e-mail]>
Commit-Queue: Evan Stade \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1478097}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=099286aa0826985aa1b248d559e175638e82403c

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 4 tests and 4 subtests

Status Summary

Firefox

OK : 4
PASS: 16

Chrome

OK : 4
PASS: 16

Safari

OK : 4
PASS: 16

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/dfed61b6554b
https://hg.mozilla.org/integration/autoland/rev/305b8cef8f0a
[wpt PR 53362] - IDB: handle double deletion in SQLite store., a=testonly

Comment 4

[amarc]

https://hg.mozilla.org/mozilla-central/rev/305b8cef8f0a