Commit 6412f40b authored by Dominique Fauteux-Chapleau's avatar Dominique Fauteux-Chapleau Committed by Commit Bot

Refactor OnPerformDrop tests to use Connector policies

This CL refactors chrome_web_contents_view_handle_drop_unittest.cc to
use Connector policies directly instead of using the indirect helpers
that map to legacies policies.

Bug: 1109242
Change-Id: I343f5e2270ebc7c77606fcb23b168cf76c1173d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518035Reviewed-by: default avatarRoger Tawa <rogerta@chromium.org>
Reviewed-by: default avatarJohn Abd-El-Malek <jam@chromium.org>
Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824077}
parent 17857f75
......@@ -60,8 +60,28 @@ class ChromeWebContentsViewDelegateHandleOnPerformDrop : public testing::Test {
}
void EnableDeepScanning(bool enable, bool scan_succeeds) {
SetScanPolicies(enable ? safe_browsing::CHECK_UPLOADS
: safe_browsing::CHECK_NONE);
if (enable) {
static constexpr char kEnabled[] = R"(
{
"service_provider": "google",
"enable": [
{
"url_list": ["*"],
"tags": ["dlp"]
}
],
"block_until_verdict": 1
})";
safe_browsing::SetAnalysisConnector(enterprise_connectors::FILE_ATTACHED,
kEnabled);
safe_browsing::SetAnalysisConnector(
enterprise_connectors::BULK_DATA_ENTRY, kEnabled);
} else {
safe_browsing::ClearAnalysisConnector(
enterprise_connectors::FILE_ATTACHED);
safe_browsing::ClearAnalysisConnector(
enterprise_connectors::BULK_DATA_ENTRY);
}
run_loop_.reset(new base::RunLoop());
......@@ -132,12 +152,6 @@ class ChromeWebContentsViewDelegateHandleOnPerformDrop : public testing::Test {
std::string small_text() const { return "random small text"; }
private:
void SetScanPolicies(safe_browsing::CheckContentComplianceValues state) {
safe_browsing::SetDlpPolicyForConnectors(state);
safe_browsing::SetDelayDeliveryUntilVerdictPolicyForConnectors(
safe_browsing::DELAY_UPLOADS);
}
content::BrowserTaskEnvironment task_environment_;
base::test::ScopedFeatureList scoped_feature_list_;
TestingProfileManager profile_manager_{TestingBrowserProcess::GetGlobal()};
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment