Closed
Bug 1487390
Opened 7 years ago
Closed 7 years ago
Notify when a channel is aborted because of fastblocking
Categories
(Core :: Security, defect, P1)
Core
Security
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: baku, Assigned: baku)
References
Details
Attachments
(1 file)
|
5.17 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
The notification could be nsIWebProgressListener::STATE_FAST_BLOCKED.
| Assignee | ||
Comment 1•7 years ago
|
||
Attachment #9005191 -
Flags: review?(ehsan)
Comment 2•7 years ago
|
||
Comment on attachment 9005191 [details] [diff] [review]
fastBlock.patch
Review of attachment 9005191 [details] [diff] [review]:
-----------------------------------------------------------------
::: uriloader/base/nsIWebProgressListener.idl
@@ +315,5 @@
> const unsigned long STATE_COOKIES_BLOCKED_BY_PERMISSION = 0x10000000;
> const unsigned long STATE_COOKIES_BLOCKED_TRACKER = 0x20000000;
> const unsigned long STATE_COOKIES_BLOCKED_ALL = 0x40000000;
> const unsigned long STATE_COOKIES_BLOCKED_FOREIGN = 0x80000000;
> + const unsigned long STATE_FAST_BLOCKED = 0x00000040;
This name doesn't read too well... How about calling it STATE_BLOCKED_SLOW_TRACKING_CONTENT (similar to STATE_BLOCKED_TRACKING_CONTENT)?
Also can you add a comment right after STATE_LOADED_MIXED_ACTIVE_CONTENT saying 0x00000040 is used further down?
Attachment #9005191 -
Flags: review?(ehsan) → review+
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/37133ece7ce2
Notify when a channel is aborted because of fastblocking, r=ehsan
Updated•7 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Comment 4•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•