Commit 3f9e427f authored by Yao Xiao's avatar Yao Xiao Committed by Commit Bot

Enable the feature BlockingDownloadsInAdFrameWithoutUserActivation

blink LGTMs:
https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/tXKnmZD3P9o/x5YFru_YBQAJ

Bug: 929911
Change-Id: I0d5ebe43b8c4bca13393035dd3b356ab602ea5b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1625781Reviewed-by: default avatarChris Harrelson <chrishtr@chromium.org>
Reviewed-by: default avatarCharlie Harrison <csharrison@chromium.org>
Commit-Queue: Yao Xiao <yaoxia@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662642}
parent 4ded38ed
...@@ -70,8 +70,8 @@ void PossiblyWarnFeatureDeprecation(content::RenderFrameHost* rfh, ...@@ -70,8 +70,8 @@ void PossiblyWarnFeatureDeprecation(content::RenderFrameHost* rfh,
case WebFeature::kDownloadInAdFrameWithoutUserGesture: case WebFeature::kDownloadInAdFrameWithoutUserGesture:
rfh->AddMessageToConsole( rfh->AddMessageToConsole(
blink::mojom::ConsoleMessageLevel::kWarning, blink::mojom::ConsoleMessageLevel::kWarning,
"[Deprecation] Download in ad frame without user activation is " "[Intervention] Download in ad frame without user activation is "
"deprecated and will be removed in M76, around July 2019. See " "not allowed. See "
"https://www.chromestatus.com/feature/6311883621531648 for more " "https://www.chromestatus.com/feature/6311883621531648 for more "
"details."); "details.");
return; return;
......
...@@ -14,7 +14,7 @@ namespace features { ...@@ -14,7 +14,7 @@ namespace features {
// frame without user activation. // frame without user activation.
const base::Feature kBlockingDownloadsInAdFrameWithoutUserActivation{ const base::Feature kBlockingDownloadsInAdFrameWithoutUserActivation{
"BlockingDownloadsInAdFrameWithoutUserActivation", "BlockingDownloadsInAdFrameWithoutUserActivation",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_ENABLED_BY_DEFAULT};
// Enable defer commits a bit to avoid flash. // Enable defer commits a bit to avoid flash.
const base::Feature kAvoidFlashBetweenNavigation{ const base::Feature kAvoidFlashBetweenNavigation{
......
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