Open Bug 1280368 Opened 8 years ago Updated 2 years ago

Implement asynchronous content blocking API


(Core :: DOM: Security, task, P3)




Tracking Status
e10s - ---


(Reporter: ckerschb, Unassigned)



(Keywords: perf, Whiteboard: [domsecurity-backlog1])

Over time the needs changed and we should consider implementing an asynchronous API for content blocking which could replace nsIContentPolicy. Let's gather some thoughts and ideas within this bug as a first measure.
Whiteboard: [domsecurity-backlog]
For reference, this is currently causing performance issues in Adblock Plus with e10s. Our content process code needs to query the parent for content policy decisions because that's where the state is. Currently it has to happen synchronously because nsIContentPolicy requires a synchronous decision - so rather than delaying a single network request slightly we block the entire content process every time. IMHO, that's what is causing the issue described in bug 1192585 comment 17.
Blocks: abp, e10s-abp
It's normal that ADP make terrible slow down some pages when e10s is on and dom.ipc.processCount;8, even when I turn off any filter nad "Count filter hits" option? I load 80 page with "Open all in Tabs" option and get this:

ADP ON (without any filter): 138s, 136s, 139s
ADP OFF: 22s, 24s, 22s

ADP ON (without any filter): 16s, 17s, 19s
ADP OFF: 9s, 9s, 9s

I use I7-2600k, 32GB RAM, HDD 7200rpm, Win7 64bit, FX Nightly 64bit.

When I turn off e10s results are terrible, even when turn off ADB, but it's FX problem (older version was much bether in this area: But for e10s finally there is some progress. Why ADP so much affect the first domain?
Of course not ADP but ABP:) Even when I made "Disable everywhere" (ABP icon becomes gray) slowdown still exist. To make the tests I must  turn off ABP in standard way (through about:addons).
Priority: -- → P3
Whiteboard: [domsecurity-backlog] → [domsecurity-backlog1]
Type: defect → task
QA Whiteboard: qa-not-actionable

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Severity: major → --
You need to log in before you can comment on or make changes to this bug.