Commit ebb1e89e authored by Dale Curtis's avatar Dale Curtis Committed by Commit Bot

Enable preload=metadata lazy load for macOS now that bugs are fixed.

All known issues should be fixed, so lets try this again.

BUG=879406
TEST=bots
TBR=liberato

Change-Id: I88e703e65d767d5a501b5f4ae4f7334572481672
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1637606Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#664959}
parent fe46f16b
...@@ -199,16 +199,10 @@ const base::Feature kPictureInPicture { ...@@ -199,16 +199,10 @@ const base::Feature kPictureInPicture {
#endif #endif
}; };
// Only decode preload=metadata elements upon visibility. macOS still has some // Only decode preload=metadata elements upon visibility.
// undiagnosed performance and test issues, so isn't enabled yet. // TODO(crbug.com/879406): Remove this after M76 ships to stable
const base::Feature kPreloadMetadataLazyLoad { const base::Feature kPreloadMetadataLazyLoad{"PreloadMetadataLazyLoad",
"PreloadMetadataLazyLoad", base::FEATURE_ENABLED_BY_DEFAULT};
#if defined(OS_MACOSX)
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