Vaapi VEA: always generate IDR when keyframe is requested.

content/common/gpu/media/vaapi_video_encode_accelerator.cc

Index: content/common/gpu/media/vaapi_video_encode_accelerator.cc
diff --git a/content/common/gpu/media/vaapi_video_encode_accelerator.cc b/content/common/gpu/media/vaapi_video_encode_accelerator.cc
index 14ffe34355012f74e4ce4e72c13f24ae8b915b7a..39c9ee9e6e09c51a57a7be71505258d79808e2ad 100644
--- a/content/common/gpu/media/vaapi_video_encode_accelerator.cc
+++ b/content/common/gpu/media/vaapi_video_encode_accelerator.cc
@@ -286,15 +286,13 @@ void VaapiVideoEncodeAccelerator::BeginFrame(bool force_keyframe) {
   current_pic_.frame_num = frame_num_++;
   frame_num_ %= idr_period_;
 
-  if (current_pic_.frame_num % i_period_ == 0 || force_keyframe)
+  if (current_pic_.frame_num % i_period_ == 0 || force_keyframe) {
     current_pic_.type = media::H264SliceHeader::kISlice;
-  else
-    current_pic_.type = media::H264SliceHeader::kPSlice;
-
-  if (current_pic_.frame_num % idr_period_ == 0) {

[Pawel Osciak, 2014/10/30 00:09:46]

This is generating IDR on each i_period, which is not what we want. You want to
generate IDR only on force_keyframe or idr_period, but only an I frame on each
i_period.

[hshi1, 2014/10/30 00:18:01]

Done.

     current_pic_.idr = true;
     last_idr_frame_num_ = current_pic_.frame_num;
     ref_pic_list0_.clear();
+  } else {
+    current_pic_.type = media::H264SliceHeader::kPSlice;
   }
 
   if (current_pic_.type != media::H264SliceHeader::kBSlice)