Closed Bug 1578561 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 18817 - Add a relaxedDurability option to IDBDatabase transaction

Categories

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

Product:
Core
Component:
Storage: IndexedDB
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla71
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Adrienne Walker <enne@chromium.org> wrote:

Add a relaxedDurability option to IDBDatabase transaction

This plumbs a new optional relaxedDurability flag from the IDBDatabase
transaction web api in the renderer to the IndexedDBBackingStore
in the browser.

Flushing after every file write is very expensive and burns power,
and so it's desirable to eventually experiment with turning this
option off by default. It is currently implicitly on. This patch
doesn't change the default, but only exposes the option.

Results from my local Linux machine on the issue 785305 test case:

num  |  master   |  patch

------+-----------+---------
1000 | 41386 ms | 1843 ms
5100 | 214892 ms | 14283 ms
10000 | 423271 ms | 34271 ms

This test case had extremely variable output times, but this seems
to be about a 10x performance improvement on that test case.

Bug: 965883, 785305
Change-Id: Id54b65bd462f0983f8af44abc26575135d35f535
Reviewed-on: https://chromium-review.googlesource.com/1775557
WPT-Export-Revision: 0c6c4b19971e4c3678cf3121c374fa20d418cdc6

Component: web-platform-tests → DOM: IndexedDB
Product: Testing → Core
Ran 2 tests and 8 subtests
OK     : 2
PASS   : 8
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f0869c0cc3f3
[wpt PR 18817] - Add a relaxedDurability option to IDBDatabase transaction, a=testonly

https://hg.mozilla.org/mozilla-central/rev/f0869c0cc3f3

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