New issue
Advanced search Search tips

Issue 660420 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Remove SafeBrowsingAndroid experiment

Project Member Reported by nparker@chromium.org, Oct 28 2016

Issue description

The XLB setting is on-by-default in the code. The only thing this will affect is the list of "types_to_check" -- the experiment overrides the code still, and the only thing that changes is that types 16 and 17 aren't checked due to the experiment. Those are "CSP_REPORT" and "PLUGIN_RESOURCE" which according to UMA are never used on Android.

I'll keep the .cc code that looks for the finch trial so we can still re-enable checking types that the .cc code disables by default, in case we discover a campaign using (say) only blacklisting CSS or images.

https://cs.chromium.org/chromium/src/components/safe_browsing_db/remote_database_manager.cc?l=111
 
Status: Fixed (was: Started)
I sunset the experiment config for vers > 55.*.  http://cl/137735476

Sign in to add a comment