Enable deep scanning for ESB users by default [chromium/src : main]
Daniel Rubery (Gerrit)
Attention is currently required from: Xinghui Lu.
Patch set 1:Auto-Submit +1Commit-Queue +1
To view, visit change 2983244. To unsubscribe, or for help writing mail filters, visit settings.
Xinghui Lu (Gerrit)
Attention is currently required from: Daniel Rubery.
1 comment:
Patchset:
Looks like some tests failed. You may want to disable the feature flag explicitly in some tests.
To view, visit change 2983244. To unsubscribe, or for help writing mail filters, visit settings.
Daniel Rubery (Gerrit)
1 comment:
Patchset:
Alright, looks like tests are passing now. Can you take another look?
To view, visit change 2983244. To unsubscribe, or for help writing mail filters, visit settings.
Xinghui Lu (Gerrit)
Attention is currently required from: Daniel Rubery.
Patch set 4:Code-Review +1Commit-Queue +2
Chromium LUCI CQ (Gerrit)
Chromium LUCI CQ submitted this change.
Enable deep scanning for ESB users by default
This experiment is ready to launch to 100% Stable, so enable it by
default at HEAD. This CL also removes some tests that verified the
proper functioning of the feature flag by enforcing the old behavior.
Bug: 1186736
Change-Id: Ie8075f5ab726cadeda269a27b8aada2d87ec0ed8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2983244
Auto-Submit: Daniel Rubery <dru...@chromium.org>
Commit-Queue: Xinghui Lu <xing...@chromium.org>
Reviewed-by: Xinghui Lu <xing...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#896293}
---
M chrome/browser/safe_browsing/cloud_content_scanning/binary_upload_service_unittest.cc
M chrome/browser/safe_browsing/download_protection/download_protection_service_unittest.cc
M components/safe_browsing/core/features.cc
3 files changed, 5 insertions(+), 134 deletions(-)