Closed Bug 1632076 Opened 3 years ago Closed 3 years ago

Remove unnecessary mHasStoragePermission member variable from the document

Categories

(Core :: Privacy: Anti-Tracking, task, P1)

Product:
Core
Component:
Privacy: Anti-Tracking
Type:
task

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: timhuang, Assigned: timhuang)

References

Details

Attachments

(1 obsolete file)

The mHasStoragePermission flag is merely a temporary holder of the HasStoragePermission flag which is really unnecessary since we can directly get the flag from the channel of the document when we need it.

Summary: Remove unnecessary Document::HasStoragePermission() function and the member variable → Remove unnecessary mHasStoragePermission member variable from the document

We store the 'HasStoragePermission' flag in the Document. Actually, it's
unnecessary. We can get the flag directly from the loadInfo of the
document's channel.

Blocks: 1612376
No longer blocks: etp-fission
Attachment #9142340 - Attachment is obsolete: true

Per discussion with Dimi, this would be done in Bug 1612376.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.