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

Unified Diff: content/common/gpu/media/vt_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: address patch set 9 review comments 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/vt_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/vt_video_decode_accelerator.cc b/content/common/gpu/media/vt_video_decode_accelerator.cc
index a7618f5d63993e16b285fda44affe9c2c4bd7eb1..d472463d742dbeb8ec45a401c07d8c09e74b0e26 100644
--- a/content/common/gpu/media/vt_video_decode_accelerator.cc
+++ b/content/common/gpu/media/vt_video_decode_accelerator.cc
@@ -10,6 +10,7 @@
#include "base/bind.h"
#include "base/command_line.h"
+#include "base/cpu.h"
#include "base/logging.h"
#include "base/mac/mac_logging.h"
#include "base/metrics/histogram_macros.h"
@@ -918,4 +919,18 @@ bool VTVideoDecodeAccelerator::CanDecodeOnIOThread() {
return false;
}
+// static
+media::VideoDecodeAccelerator::SupportedResolution
+VTVideoDecodeAccelerator::GetSupportedResolution() {
+ media::VideoDecodeAccelerator::SupportedResolution resolution;
+ resolution.min.SetSize(16, 16);
sandersd (OOO until July 31) 2015/01/05 19:12:04 Below 480P, VideoToolbox uses an internal software
henryhsu 2015/01/13 05:40:44 Done.
sandersd (OOO until July 31) 2015/01/13 18:02:15 We have done additional testing, and most hardware
+ base::CPU cpu;
+ // Ivy Bridge+ platforms can support more than 1920x1080.
sandersd (OOO until July 31) 2015/01/05 19:12:04 As far as I know, this has no bearing on the resol
henryhsu 2015/01/13 05:40:44 The code of cpu detection is for chromeos. Could y
sandersd (OOO until July 31) 2015/01/13 18:02:15 I'm not certain, all the hardware we have tested h
+ if ((cpu.vendor_name() == "GenuineIntel") && cpu.model() >= 55)
+ resolution.max.SetSize(4096, 2160);
+ else
+ resolution.max.SetSize(1920, 1088);
+ return resolution;
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698