-
-
Notifications
You must be signed in to change notification settings - Fork 344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No adblock when a tab is reopened from "Recent tabs"? #681
Comments
It could be related to the subresource filter not being added in such situation.
The ad blocker is in fact present also in Chromium, so this could be tested also there - although it uses a very specific default set of ads to block. |
I thought about that after I posed the issue. If you have any idea how to find a site to test with, I will. I poked around the Chromium source, but everything just said things like what is blocked are ads that violate the "Better Ads Standard", without any block list that I could locate. |
I found that bug is related to the fact that the subframe (in In fact, I found that the web_contents created in https://source.chromium.org/chromium/chromium/src/+/master:chrome/browser/sessions/session_restore_android.cc;l=35 did not have connect This - I think - is given by it happens because Now, the definitive solution would be to try to generate a suitable web_contents_android, but I would have to replicate I think that the bugs does not exist in upstream because some parts of code are called by the safe_browsing component, in bromite doesn't exits. |
Very nice analysis, thanks @uazo. Is it possible that there are other bugs upstream because of this asymmetry? If you find a security bug - present upstream even with the safe browsing component - then you could submit it, claim a bounty and see it fixed relatively quickly. We could also check if GrapheneOS/Vanadium is affected. For the time being the PR you made should suffice. |
from what I've seen, all calls to |
Bromite version
Version:
84.0.4147.121
Arch:
arm
Android version:
10
Device model:
Nokia 1
Is this bug about the SystemWebView?
No
Is the bug reproducible with latest version?
Yes
Can the bug be reproduced with corresponding Chromium version?
N/A: Bromite adblock issue
Is the bug a crash?
No
Describe the bug
When I reopen tabs from the "Recent tabs" menu option, those tabs show ads that are blocked if I navigate to the same page in a different way or just reload the page. I've reproduced on a few sites:
Steps to reproduce the bug
Steps to reproduce the bug:
Expected behavior
Tabs reopened from "Recent tabs" should have ads blocked like a page opened any other way.
The text was updated successfully, but these errors were encountered: