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

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

Issue 24667002: EVEA H.264: Disable periodic key frames by default. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disable periodic key frames altogether, as suggested by sheu. Created 7 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/gpu/media/exynos_video_encode_accelerator.cc
diff --git a/content/common/gpu/media/exynos_video_encode_accelerator.cc b/content/common/gpu/media/exynos_video_encode_accelerator.cc
index 00f8303ccb68371938e06c3d3ded9732a9e01047..ab86e6fd3ba067eeb0af39a4522bdc536f68d2e7 100644
--- a/content/common/gpu/media/exynos_video_encode_accelerator.cc
+++ b/content/common/gpu/media/exynos_video_encode_accelerator.cc
@@ -1364,7 +1364,7 @@ bool ExynosVideoEncodeAccelerator::SetMfcFormats() {
}
bool ExynosVideoEncodeAccelerator::InitMfcControls() {
- struct v4l2_ext_control ctrls[8];
+ struct v4l2_ext_control ctrls[9];
struct v4l2_ext_controls control;
memset(&ctrls, 0, sizeof(ctrls));
memset(&control, 0, sizeof(control));
@@ -1394,6 +1394,9 @@ bool ExynosVideoEncodeAccelerator::InitMfcControls() {
// Use H.264 level 4.0 to match the supported max resolution.
ctrls[7].id = V4L2_CID_MPEG_VIDEO_H264_LEVEL;
ctrls[7].value = V4L2_MPEG_VIDEO_H264_LEVEL_4_0;
+ // Disable periodic key frames.
+ ctrls[8].id = V4L2_CID_MPEG_VIDEO_GOP_SIZE;
+ ctrls[8].value = 0;
control.ctrl_class = V4L2_CTRL_CLASS_MPEG;
control.count = arraysize(ctrls);
control.controls = ctrls;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698