Closed Bug 1811993 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 38133 - IDB WPT: add promise to ensure that the setup is done before navigation

Categories

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

Product:
Core
Component:
Storage: IndexedDB
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
111 Branch
Tracking Flags:
Tracking Status
firefox111 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Mingyu Lei <leimy@chromium.org> wrote:

IDB WPT: add promise to ensure that the setup is done before navigation

We didn't wait until the onupgradeneeded to be fired before
registering the pagehide event handler and navigate the page for
BFCache eligibility test. If the navigation happens before the IndexedDB
connection is set up, the transaction won't start and our test becomes
meaningless.

Bug: 1409621
Change-Id: Ifb87026b3ec48e09838f9eeee37c0137d1252ad0

Reviewed-on: https://chromium-review.googlesource.com/4190320
WPT-Export-Revision: 3eb60ea0974c4fdf1d596adb1729f4360c2ec63d

Component: web-platform-tests → Storage: IndexedDB
Product: Testing → Core
Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/576bace0bc84 [wpt PR 38133] - IDB WPT: add promise to ensure that the setup is done before navigation, a=testonly
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/576bace0bc84

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox111: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.