1562669 - Expose quota-manager APIs in PrivateBrowsing

Status: ASSIGNED

(Core :: Storage: Quota Manager, enhancement, P2)

Description

[Andrea Marchesini [:baku]]

This issue has been discussed during the latest all-hands.
I have a prototype that enables IDB, Cache, SW APIs to the private-browsing mode.
In order to avoid to expose the PB origins, they are replaced with null-principals. The private-browsing data usage is not exposed to getUsage().

I want to collect feedback. In particular I want to know if:

1. the current approach is reasonable.
2. how to improve the current code.
3. what else must be done in order to land this feature.

Comment 1

[Andrea Marchesini [:baku]]

Attachment: Bug 1562669 - Expose quota-manager APIs in PrivateBrowsing - part 1 - IndexedDB, r?janv

Comment 2

[Andrea Marchesini [:baku]]

Attachment: Bug 1562669 - Expose quota-manager APIs in PrivateBrowsing - part 2 - DOM Cache, r?janv

Depends on D36496

Comment 3

[Andrea Marchesini [:baku]]

Attachment: Bug 1562669 - Expose quota-manager APIs in PrivateBrowsing - part 3 - ServiceWorkers, r?janv

Depends on D36497

Comment 4

[Andrea Marchesini [:baku]]

Attachment: Bug 1562669 - Expose quota-manager APIs in PrivateBrowsing - part 4 - Use NullPrincipal in QM, r?janv

Depends on D36498

Comment 5

[Tom Tung [:tt, :ttung]]

Andrea has already provided patches, so I guess he is the assignee. Please feel free to correct that if I misunderstood. :)

Comment 6

[Andrea Marchesini [:baku]]

Can you please take a look at these patches?

Comment 7

[Jan Varga [:janv]]

Yes, I'm looking.

Comment 8

[Jan Varga [:janv]]

We need to decide if we just want to wait for UUID based directories in QuotaManager, bug 1593365.

Comment 9

[Jens Stutte [:jstutte]]

(In reply to Jan Varga [:janv] from comment #8)

We need to decide if we just want to wait for UUID based directories in QuotaManager, bug 1593365.

Do we have a decision here?

Comment 10

[Jan Varga [:janv]]

I'm really thankful for the ideas here, it helped us to design QM v4, but it would be better to just wait for QM v4.

Comment 11

[Jan Varga [:janv]]

Besides QM v4, our team will be also working on encryption support for IndexedDB, initiallly done for private browsing.