Commit 9e9517e4 authored by Dale Curtis's avatar Dale Curtis Committed by Commit Bot

Enable preload=metadata lazy load on CrOS, leave off for macOS.

BUG=879406
TBR=liberato

Change-Id: I9a25b69c03edb29e35fcdcb9eb5f69cf7626b461
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627663Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Reviewed-by: default avatarFrank Liberato <liberato@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662806}
parent c1e88cf7
...@@ -199,14 +199,11 @@ const base::Feature kPictureInPicture { ...@@ -199,14 +199,11 @@ const base::Feature kPictureInPicture {
#endif #endif
}; };
// Only decode preload=metadata elements upon visibility. Experiment results // Only decode preload=metadata elements upon visibility. macOS still has some
// vary across platforms and are still being analyzed on macOS and CrOS. // undiagnosed performance and test issues, so isn't enabled yet.
//
// Android, Windows, and Linux all saw improvements without regressions, so
// enable by default there.
const base::Feature kPreloadMetadataLazyLoad { const base::Feature kPreloadMetadataLazyLoad {
"PreloadMetadataLazyLoad", "PreloadMetadataLazyLoad",
#if defined(OS_MACOSX) || defined(OS_CHROMEOS) #if defined(OS_MACOSX)
base::FEATURE_DISABLED_BY_DEFAULT base::FEATURE_DISABLED_BY_DEFAULT
#else #else
base::FEATURE_ENABLED_BY_DEFAULT base::FEATURE_ENABLED_BY_DEFAULT
......
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