chromium/src
0
Fork 0
Code Activity

blink: Enable paint holding by default without VT dependency.

Browse Source 
Enable paint holding for OOPIFs by default. This was implicitly done by
the ViewTransition for iframes flag. This change enables it explicitly
for all platforms except WebView.

Fixed: 330920526
Change-Id: I6e04665f3b25c5b3a3af4362d5276432939a2b8f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5739279
Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
Auto-Submit: Khushal Sagar <khushalsagar@chromium.org>
Commit-Queue: Khushal Sagar <khushalsagar@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1334395}
This commit is contained in:
Khushal Sagar
2024-07-29 19:37:53 +00:00
committed by Chromium LUCI CQ
parent d89d09ea7b
commit d29beb9ad8
2 changed files with 6 additions and 3 deletions
android_webview/browser
aw_field_trials.cc
third_party/blink/renderer/platform
runtime_enabled_features.json5

5
android_webview/browser/aw_field_trials.cc

@ -226,6 +226,11 @@ void AwFieldTrials::RegisterFeatureOverrides(base::FeatureList* feature_list) {
aw_feature_overrides.DisableFeature(blink::features::kDevicePosture);
aw_feature_overrides.DisableFeature(blink::features::kViewportSegments);
// PaintHolding for OOPIFs. This should be a no-op since WebView doesn't use
// site isolation but field trial testing doesn't indicate that. Revisit when
// enabling site isolation. See crbug.com/356170748.
aw_feature_overrides.DisableFeature(blink::features::kPaintHoldingForIframes);
if (base::CommandLine::ForCurrentProcess()->HasSwitch(switches::kDebugBsa)) {
// Feature parameters can only be set via a field trial.
const char kTrialName[] = "StudyDebugBsa";

4
third_party/blink/renderer/platform/runtime_enabled_features.json5 vendored

@ -3060,13 +3060,11 @@
},
{
name: "PaintHoldingForIframes",
status: "test",
implied_by: ["ViewTransitionOnNavigationForIframes"],
status: "stable",
},
{
name: "PaintHoldingForLocalIframes",
status: "stable",
implied_by: ["ViewTransitionOnNavigationForIframes"],
},
{
name: "PaintUnderInvalidationChecking",