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

Unified Diff: content/common/gpu/media/v4l2_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/v4l2_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/v4l2_video_decode_accelerator.cc b/content/common/gpu/media/v4l2_video_decode_accelerator.cc
index 243fee58d2e4fcfccdb336c68fbc8ae54a066352..66d05b1e246cb7f9055f6e07c39df4c586122abd 100644
--- a/content/common/gpu/media/v4l2_video_decode_accelerator.cc
+++ b/content/common/gpu/media/v4l2_video_decode_accelerator.cc
@@ -205,6 +205,22 @@ V4L2VideoDecodeAccelerator::~V4L2VideoDecodeAccelerator() {
DCHECK(output_buffer_map_.empty());
}
+// static
+media::VideoDecodeAccelerator::SupportedResolution
+V4L2VideoDecodeAccelerator::GetSupportedResolution() {
wuchengli 2014/12/26 09:37:07 Use the same order in the header file.
henryhsu 2014/12/26 10:25:18 Done.
+ // NOTE: additional autodetection logic may require updating input buffer size
+ // selection in platform-specific implementations, such as
+ // V4L2VideoDecodeAccelerator.
wuchengli 2014/12/26 09:37:07 Update the comment. This is already V4L2VDA.
henryhsu 2014/12/26 10:25:18 Done.
+ media::VideoDecodeAccelerator::SupportedResolution resolution;
+ resolution.min.SetSize(16, 16);
+ if (CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kIgnoreResolutionLimitsForAcceleratedVideoDecode))
+ resolution.max.SetSize(4096, 2160);
+ else
+ resolution.max.SetSize(1920, 1088);
+ return resolution;
+}
+
bool V4L2VideoDecodeAccelerator::Initialize(media::VideoCodecProfile profile,
Client* client) {
DVLOG(3) << "Initialize()";

Powered by Google App Engine
This is Rietveld 408576698