Closed
Bug 1724386
Opened 3 years ago
Closed 3 years ago
Cache InternalStorageAllowed check on inner window
Categories
(Core :: Privacy: Anti-Tracking, task, P1)
Core
Privacy: Anti-Tracking
Tracking
()
RESOLVED
FIXED
93 Branch
Tracking | Status | |
---|---|---|
firefox93 | --- | fixed |
People
(Reporter: pbz, Assigned: pbz)
References
Details
Attachments
(1 file)
In order to make repeated calls to InternalStorageAllowedCheck
faster, we should cache the result on the inner window.
Assignee | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Assignee: nobody → pbz
Status: NEW → ASSIGNED
Updated•3 years ago
|
Attachment #9235063 -
Attachment description: Bug 1706292 - Cache InternalStorageAllowedCheck on inner window. r=timhuang! → Bug 1724386 - Cache InternalStorageAllowedCheck on inner window. r=timhuang!
Pushed by pzuhlcke@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/db47c2899e2e Cache InternalStorageAllowedCheck on inner window. r=timhuang,nika
Comment 3•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox93:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
Comment 4•3 years ago
|
||
This bug has caused the following improvement:
== Change summary for alert #30888 (as of Wed, 11 Aug 2021 13:06:42 GMT) ==
Improvements:
Ratio | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|
12% | nytimes FirstVisualChange | windows10-64-shippable-qr | warm webrender | 264.83 -> 234.17 |
11% | nytimes FirstVisualChange | windows10-64-shippable-qr | warm webrender | 263.17 -> 233.92 |
10% | nytimes fcp | windows10-64-shippable-qr | warm webrender | 229.53 -> 206.38 |
10% | nytimes fnbpaint | windows10-64-shippable-qr | warm webrender | 236.75 -> 213.71 |
7% | nytimes fcp | linux1804-64-shippable-qr | warm webrender | 272.67 -> 253.71 |
... | ... | ... | ... | ... |
2% | nytimes loadtime | linux1804-64-shippable-qr | warm webrender | 1,439.71 -> 1,410.33 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=30888
You need to log in
before you can comment on or make changes to this bug.
Description
•