1789586 - FS: Connect the internal database to quota management

Status: RESOLVED FIXED

(Core :: DOM: File, task, P2)

Description

[Jan Varga [:janv]]

We need to switch to file URLs and pass directoryLockId like we do here https://searchfox.org/mozilla-central/rev/3aaca0a12a2d1463da54933bdbdae2f06fead06f/dom/indexedDB/ActorsParent.cpp#627

Comment 1

[Jari Jalkanen]

Attachment: Bug 1789586 - Connect internal database to quota management. r=#dom-storage

Comment 2

[Jari Jalkanen]

Attachment: Bug 1789586 - Clean up temporary GetDirectoryForOrigin getter. r=#dom-storage

Depends on D159760

Comment 3

[Phabricator Automation]

Comment on attachment 9299533 [details]
Bug 1789586 - Clean up temporary GetDirectoryForOrigin getter. r=#dom-storage

Revision D159880 was moved to bug 1798015. Setting attachment 9299533 [details] to obsolete.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/951741a7fe6b
Connect internal database to quota management. r=dom-storage-reviewers,janv

Comment 5

[Serban Stanca [:SerbanS]]

Backed out for causing bp-hybrid bustages on FileSystemFileManager.

• Backout link
• Push with failures
• Failure Log
• Failure line: /builds/worker/workspace/obj-build/dist/include/nsCOMPtr.h:436:5: error: static assertion failed due to requirement '1 < sizeof (TestForIID<nsIFileURL>(nullptr))': nsCOMPtr only works for types with IIDs. Either use RefPtr; add an IID to your type with NS_DECLARE_STATIC_IID_ACCESSOR/NS_DEFINE_STATIC_IID_ACCESSOR; or make the nsCOMPtr point to a base class with an IID.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9c22af82cf07
Connect internal database to quota management. r=dom-storage-reviewers,janv

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9c22af82cf07

Comment 8

[Jari Jalkanen]

The patch landed and the bug is now fixed.