media: Remove VaapiVP8Encoder media_feature
VP8 HW encoding on intel platform is enabled by default from M72.
It has been tested for sufficient time. Now, the media-feature for VP8 HW
encoding becomes unnecessary.
Bug: 794608
Test: None
Change-Id: I18d2537f5ed374a6affcab29c03a1f1c61268c2e
Reviewed-on: https://chromium-review.googlesource.com/c/1462376
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#631117}diff --git a/media/base/media_switches.cc b/media/base/media_switches.cc
index 489aeeb..4205441 100644
--- a/media/base/media_switches.cc
+++ b/media/base/media_switches.cc
@@ -314,10 +314,6 @@
const base::Feature kUseSurfaceLayerForVideoPIP{
"UseSurfaceLayerForVideoPIP", base::FEATURE_ENABLED_BY_DEFAULT};
-// Enable VA-API hardware encode acceleration for VP8.
-const base::Feature kVaapiVP8Encoder{"VaapiVP8Encoder",
- base::FEATURE_ENABLED_BY_DEFAULT};
-
// Enable VA-API hardware encode acceleration for VP9.
const base::Feature kVaapiVP9Encoder{"VaapiVP9Encoder",
base::FEATURE_DISABLED_BY_DEFAULT};
diff --git a/media/base/media_switches.h b/media/base/media_switches.h
index 493ba61..c6d4e0d4 100644
--- a/media/base/media_switches.h
+++ b/media/base/media_switches.h
@@ -138,7 +138,6 @@
MEDIA_EXPORT extern const base::Feature kUseR16Texture;
MEDIA_EXPORT extern const base::Feature kUseSurfaceLayerForVideo;
MEDIA_EXPORT extern const base::Feature kUseSurfaceLayerForVideoPIP;
-MEDIA_EXPORT extern const base::Feature kVaapiVP8Encoder;
MEDIA_EXPORT extern const base::Feature kVaapiVP9Encoder;
MEDIA_EXPORT extern const base::Feature kVideoBlitColorAccuracy;
diff --git a/media/gpu/vaapi/vaapi_wrapper.cc b/media/gpu/vaapi/vaapi_wrapper.cc
index 7382cc49..f5c7da2 100644
--- a/media/gpu/vaapi/vaapi_wrapper.cc
+++ b/media/gpu/vaapi/vaapi_wrapper.cc
@@ -178,13 +178,6 @@
}
}
- // TODO(posciak): Remove once VP8 encoding is to be enabled by default.
- if (mode == VaapiWrapper::CodecMode::kEncode &&
- va_profile == VAProfileVP8Version0_3 &&
- !base::FeatureList::IsEnabled(kVaapiVP8Encoder)) {
- return true;
- }
-
// TODO(crbug.com/811912): Remove once VP9 encoding is to be enabled by
// default.
if (mode == VaapiWrapper::CodecMode::kEncode &&
