Opened 3 years ago

Last modified 2 years ago

#6255 new change

Use full request frame tree

Reported by: asmirnov Assignee:
Priority: P3 Milestone:
Module: Adblock-Plus-for-Chromium Keywords:
Cc: Blocked By:
Blocking: Platform: Android
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



We're using only last request referrer as known frame parent:

However for proper adblocking we'd better use full frame tree, flattened and passed as documentUrls:

What to change

Use full request frame tree

Change History (6)

comment:1 Changed 2 years ago by asmirnov

  • Priority changed from P2 to P3

comment:2 Changed 2 years ago by asmirnov

It looks like we can't get RenderFrameHost instance ptr instantly, we have to use callback that should be called on UI thread:

Since we try to access it from it seems to be problematic: we will have to wait until we get response in UI thread and return back here (so it will lock this background thread and increase loading time significantly). A user can also do smth in UI thread (like scrolling) so it looks even more problematic.

comment:5 Changed 2 years ago by asmirnov

Note: See TracTickets for help on using tickets.