commit | efecf28e7df18a85150040d756a0d3e3e3d5c567 | [log] [tgz] |
---|---|---|
author | Yao Xiao <yaoxia@chromium.org> | Tue May 20 19:52:33 2025 |
committer | Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue May 20 19:52:33 2025 |
tree | 9ba160c1783d5eae703df6a9c72b38be8255b6c9 | |
parent | 6bea080bf49b175d9019c4d6e765c31f0d35a0a0 [diff] |
[Ad Tracker] Add tests to demonstrate unexpected behavior in redirect scenarios Problem: When a redirecting script is about to execute, `AdTracker::WillExecuteScript` gets only the *initial* URL. The tagging and ancestor linking decision is based on this URL, leading to unexpected behavior (e.g., If a script's initial URL is not on filterlist, but it redirects to a URL that is, its subsequently created frames/scripts are not tagged as ads). This CL adds tests to demonstrate the current behavior, and adds comments to make it clear that this is unwanted and should be fixed. Bug: 417756984 Change-Id: I9f21af9ca495247e648c1af695b3822f60c18d76 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6548216 Reviewed-by: Josh Karlin <jkarlin@chromium.org> Reviewed-by: David Bokan <bokan@chromium.org> Commit-Queue: Yao Xiao <yaoxia@chromium.org> Cr-Commit-Position: refs/heads/main@{#1462992}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
To check out the source code locally, don't use git clone
! Instead, follow the instructions on how to get the code.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure.
For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.
If you found a bug, please file it at https://crbug.com/new.