Commit 35a25733 authored by Dale Curtis's avatar Dale Curtis Committed by Commit Bot

Enable preload=metadata lazy load everywhere.

Turns out I should have pulled the latest 28D data for 74 before
limiting to !CrOS && !macOS. No regressions are shown in that data
anymore, so enable eveywhere.

This leaves the flag in for now until we reach stable.

BUG=879406
TBR=liberato

Change-Id: I48bd5440893ee2c74b8cc54d5575d0347b2b7e78
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1623487Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662017}
parent 8506d539
...@@ -199,19 +199,10 @@ const base::Feature kPictureInPicture { ...@@ -199,19 +199,10 @@ const base::Feature kPictureInPicture {
#endif #endif
}; };
// Only decode preload=metadata elements upon visibility. Experiment results // Only decode preload=metadata elements upon visibility.
// vary across platforms and are still being analyzed on macOS and CrOS. // TODO(crbug.com/879406): Remove this after M76 ships to stable.
// const base::Feature kPreloadMetadataLazyLoad{"PreloadMetadataLazyLoad",
// Android, Windows, and Linux all saw improvements without regressions, so base::FEATURE_ENABLED_BY_DEFAULT};
// enable by default there.
const base::Feature kPreloadMetadataLazyLoad {
"PreloadMetadataLazyLoad",
#if defined(OS_MACOSX) || defined(OS_CHROMEOS)
base::FEATURE_DISABLED_BY_DEFAULT
#else
base::FEATURE_ENABLED_BY_DEFAULT
#endif
};
// Let videos be resumed via remote controls (for example, the notification) // Let videos be resumed via remote controls (for example, the notification)
// when in background. // when in background.
......
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