Closed
Bug 1475697
Opened 6 years ago
Closed 6 years ago
Update the security UI when blocking anything based on the new cookie restrictions
Categories
(Firefox :: Protections UI, enhancement, P2)
Firefox
Protections UI
Tracking
()
RESOLVED
FIXED
Firefox 63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(3 files, 3 obsolete files)
|
11.31 KB,
patch
|
Details | Diff | Splinter Review | |
|
5.28 KB,
patch
|
Details | Diff | Splinter Review | |
|
4.05 KB,
patch
|
Details | Diff | Splinter Review |
I have a set of patches which update the security UI about 3rdparty storage restrictions based on a pref (privacy.restrict3rdpartystorage.ui.enabled).
|
||
Updated•6 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
|
Assignee | |
Comment 1•6 years ago
|
||
Attachment #8992112 -
Flags: review?(amarchesini)
|
|
Assignee | |
Comment 2•6 years ago
|
||
Attachment #8992113 -
Flags: review?(amarchesini)
|
|
Assignee | |
Comment 3•6 years ago
|
||
Attachment #8992114 -
Flags: review?(amarchesini)
|
||
Updated•6 years ago
|
Attachment #8992113 -
Flags: review?(amarchesini) → review+
|
|
||
Updated•6 years ago
|
Attachment #8992114 -
Flags: review?(amarchesini) → review+
|
|
||
Updated•6 years ago
|
Attachment #8992112 -
Flags: review?(amarchesini) → review+
|
|
Assignee | |
Updated•6 years ago
|
Keywords: checkin-needed
|
||
Comment 4•6 years ago
|
||
Tried to land your patch.
I got:
applying Bug-1475697---Part-1-Refactor-the-current-way-that.patch
patching file dom/base/nsPIDOMWindow.h
Hunk #1 FAILED at 20
Flags: needinfo?(ehsan)
|
||
Updated•6 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 5•6 years ago
|
||
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8992112 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 6•6 years ago
|
||
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8992113 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 7•6 years ago
|
||
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8992114 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 8•6 years ago
|
||
Rebased the patches.
Flags: needinfo?(ehsan)
Keywords: checkin-needed
Pushed by ccoroiu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/67ceca14e438
Part 1: Refactor the current way that Necko notifies DOM about the existence of blocked content; r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/c51533b97b23
Part 2: Add infrastructure for notifying the DOM about a document containing blocked tracking cookies; r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/ffca257e24be
Part 3: Update the security UI when blocking 3rd party cookies from trackers; r=baku
Keywords: checkin-needed
|
||
Comment 10•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/67ceca14e438
https://hg.mozilla.org/mozilla-central/rev/c51533b97b23
https://hg.mozilla.org/mozilla-central/rev/ffca257e24be
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
|
||
Updated•6 years ago
|
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•