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

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: 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/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..a30d145762e4dc50141d72619cc81b7e5ba2c9e5 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"
@@ -238,6 +239,20 @@ VTVideoDecodeAccelerator::VTVideoDecodeAccelerator(
VTVideoDecodeAccelerator::~VTVideoDecodeAccelerator() {
}
+// static
+media::VideoDecodeAccelerator::SupportedResolution
+VTVideoDecodeAccelerator::GetSupportedResolution() {
+ media::VideoDecodeAccelerator::SupportedResolution resolution;
+ resolution.min.SetSize(16, 16);
+ base::CPU cpu;
+ // Ivy Bridge+ platforms can support more than 1920x1080.
+ if ((cpu.vendor_name() == "GenuineIntel") && cpu.model() >= 55)
wuchengli 2014/12/26 09:37:07 Mac only uses Intel and no ARM?
henryhsu 2014/12/26 10:25:18 keep this first and wait sandersd to confirm maxim
+ resolution.max.SetSize(4096, 2160);
+ else
+ resolution.max.SetSize(1920, 1088);
+ return resolution;
+}
+
bool VTVideoDecodeAccelerator::Initialize(
media::VideoCodecProfile profile,
Client* client) {

Powered by Google App Engine
This is Rietveld 408576698