Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Unified Diff: content/common/gpu/media/dxva_video_decode_accelerator.cc

Issue 795633005: Add VDA supported profile to GPUInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: dispatch GetSupportedResolution to each vda Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/gpu/media/dxva_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/dxva_video_decode_accelerator.cc b/content/common/gpu/media/dxva_video_decode_accelerator.cc
index 348d8cc440119a4f31651070200a8b80ffce7d29..9a50660781d2fcc25973a2a12a7e3bd619ffba5a 100644
--- a/content/common/gpu/media/dxva_video_decode_accelerator.cc
+++ b/content/common/gpu/media/dxva_video_decode_accelerator.cc
@@ -518,6 +518,19 @@ DXVAVideoDecodeAccelerator::~DXVAVideoDecodeAccelerator() {
client_ = NULL;
}
+// static
+media::VideoDecodeAccelerator::SupportedResolution
+DXVAVideoDecodeAccelerator::GetSupportedResolution() {
wuchengli 2014/12/26 09:37:07 Be consistent with the order in the header file. M
henryhsu 2014/12/26 10:25:18 Done.
+ media::VideoDecodeAccelerator::SupportedResolution resolution;
+ // Windows Media Foundation H.264 decoding does not support decoding videos
+ // with any dimension smaller than 48 pixels:
+ // http://msdn.microsoft.com/en-us/library/windows/desktop/dd797815
+ resolution.min.SetSize(48, 48);
+ // Use 1088 to account for 16x16 macroblocks.
+ resolution.max.SetSize(1920, 1088);
+ return resolution;
+}
+
bool DXVAVideoDecodeAccelerator::Initialize(media::VideoCodecProfile profile,
Client* client) {
client_ = client;

Powered by Google App Engine
This is Rietveld 408576698