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

Unified Diff: content/common/gpu/media/gpu_video_accelerator_util.h

Issue 795633005: Add VDA supported profile to GPUInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address patch set 13 review comments Created 5 years, 9 months 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/gpu_video_accelerator_util.h
diff --git a/content/common/gpu/media/gpu_video_accelerator_util.h b/content/common/gpu/media/gpu_video_accelerator_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..f5cd34b78798b8db47b343ac4bafb8f88c1da707
--- /dev/null
+++ b/content/common/gpu/media/gpu_video_accelerator_util.h
@@ -0,0 +1,41 @@
+// Copyright (c) 2015 The Chromium Authors. All rights reserved.
wuchengli 2015/03/18 08:02:37 remove (c)
henryhsu 2015/03/18 11:06:09 Done.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_ACCELERATOR_UTIL_H_
+#define CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_ACCELERATOR_UTIL_H_
+
+#include <vector>
+
+#include "gpu/config/gpu_info.h"
+#include "media/video/video_decode_accelerator.h"
+#include "media/video/video_encode_accelerator.h"
+
+namespace content {
+
+class GpuVideoAcceleratorUtil {
+ public:
+ static std::vector<media::VideoDecodeAccelerator::SupportedProfile>
wuchengli 2015/03/18 08:02:37 Are all four functions used?
henryhsu 2015/03/18 11:06:09 Yes. ConvertGpuToMediaDecodeProfiles is used in re
+ ConvertGpuToMediaDecodeProfiles(const std::vector<
+ gpu::VideoDecodeAcceleratorSupportedProfile>& gpu_profiles);
+
+ // Convert media profile to gpu profile and eliminate duplicate profile
wuchengli 2015/03/18 08:02:37 s/Convert media/Convert decoder media/ s/profile/p
henryhsu 2015/03/18 11:06:09 Done.
+ // element for decoder.
wuchengli 2015/03/18 08:02:37 s/element//. I don't think using "element" is more
henryhsu 2015/03/18 11:06:09 Done.
+ static std::vector<gpu::VideoDecodeAcceleratorSupportedProfile>
+ ConvertMediaToGpuDecodeProfiles(const std::vector<
+ media::VideoDecodeAccelerator::SupportedProfile>& media_profiles);
+
+ static std::vector<media::VideoEncodeAccelerator::SupportedProfile>
+ ConvertGpuToMediaEncodeProfiles(const std::vector<
wuchengli 2015/03/18 08:02:37 Add function comment.
henryhsu 2015/03/18 11:06:09 Done.
+ gpu::VideoEncodeAcceleratorSupportedProfile>& gpu_profiles);
+
+ // Convert media profile to gpu profile and eliminate duplicate profile
+ // element for encoder.
+ static std::vector<gpu::VideoEncodeAcceleratorSupportedProfile>
+ ConvertMediaToGpuEncodeProfiles(const std::vector<
+ media::VideoEncodeAccelerator::SupportedProfile>& media_profiles);
+};
+
+} // namespace content
+
+#endif // CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_ACCELERATOR_UTIL_H_

Powered by Google App Engine
This is Rietveld 408576698