1679542 - Get rid of *QUOTA_NAMESPACE* macros and stop including QuotaCommon.h everywhere

Status: RESOLVED FIXED

(Core :: Storage: Quota Manager, task)

Description

[Simon Giesecke [:sg] [he/him]]

QuotaCommon.h is included in other exported header files and pulls in a lot of other dependencies, which are only required internally in the quota manager and its clients.

This would call for splitting the header up. Maybe this should still be done for logical reasons, but the only reason for several of these includes are the BEGIN_QUOTA_NAMESPACE and END_QUOTA_NAMESPACE macros, which have become obsolete since we have C++17 nested namespace declarations.

These macros are now removed, and the now unused include directives for QuotaCommon.h are removed along.

Comment 1

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1679542 - Get rid of *QUOTA_NAMESPACE* macros and stop including QuotaCommon.h everywhere. r=#dom-workers-and-storage

Depends on D98076

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2d9303f2d336
Get rid of *QUOTA_NAMESPACE* macros and stop including QuotaCommon.h everywhere. r=dom-workers-and-storage-reviewers,ttung

Comment 3

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/2d9303f2d336