[Remoting Host] Add more codec spec information for VP9

remoting/protocol/webrtc_dummy_video_encoder.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/webrtc_dummy_video_encoder.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 115 matching lines @@
   webrtc::CodecSpecificInfo codec_specific_info;
   memset(&codec_specific_info, 0, sizeof(codec_specific_info));
   codec_specific_info.codecType = codec_type_;
 
   if (codec_type_ == webrtc::kVideoCodecVP8) {
     codec_specific_info.codecSpecific.VP8.simulcastIdx = 0;
     codec_specific_info.codecSpecific.VP8.temporalIdx = webrtc::kNoTemporalIdx;
     codec_specific_info.codecSpecific.VP8.tl0PicIdx = webrtc::kNoTl0PicIdx;
     codec_specific_info.codecSpecific.VP8.pictureId = webrtc::kNoPictureId;
   } else if (codec_type_ == webrtc::kVideoCodecVP9) {
-    codec_specific_info.codecSpecific.generic.simulcast_idx = 0;
-    codec_specific_info.codecSpecific.VP9.gof_idx = webrtc::kNoGofIdx;
-    codec_specific_info.codecSpecific.VP9.temporal_idx = webrtc::kNoTemporalIdx;
-    codec_specific_info.codecSpecific.VP9.spatial_idx = webrtc::kNoSpatialIdx;
-    codec_specific_info.codecSpecific.VP9.tl0_pic_idx = webrtc::kNoTl0PicIdx;
-    codec_specific_info.codecSpecific.VP9.picture_id = webrtc::kNoPictureId;
+    webrtc::CodecSpecificInfoVP9* vp9_info =

[Sergey Ulanov, 2017/04/24 17:11:35]

maybe add vp8_info for VP8 above, for consistency.

[Yuwei, 2017/04/24 19:13:27]

Done.

+        &codec_specific_info.codecSpecific.VP9;
+    vp9_info->inter_pic_predicted = !frame.key_frame;
+    vp9_info->ss_data_available = frame.key_frame;
+    vp9_info->spatial_layer_resolution_present = frame.key_frame;
+    if (frame.key_frame) {
+      vp9_info->width[0] = frame.size.width();
+      vp9_info->height[0] = frame.size.height();
+    }
+    vp9_info->num_spatial_layers = 1;
+    vp9_info->gof_idx = webrtc::kNoGofIdx;
+    vp9_info->temporal_idx = webrtc::kNoTemporalIdx;
+    vp9_info->spatial_idx = webrtc::kNoSpatialIdx;
+    vp9_info->tl0_pic_idx = webrtc::kNoTl0PicIdx;
+    vp9_info->picture_id = webrtc::kNoPictureId;
   } else {
     NOTREACHED();
   }
 
   webrtc::RTPFragmentationHeader header;
   memset(&header, 0, sizeof(header));
 
   header.VerifyAndAllocateFragmentationHeader(1);
   header.fragmentationOffset[0] = 0;
   header.fragmentationLength[0] = buffer_size;
@@ skipping 80 matching lines @@
 void WebrtcDummyVideoEncoderFactory::SetVideoChannelStateObserver(
     base::WeakPtr<VideoChannelStateObserver> video_channel_state_observer) {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
   DCHECK(encoders_.empty());
   base::AutoLock lock(lock_);
   video_channel_state_observer_ = video_channel_state_observer;
 }
 
 }  // namespace protocol
 }  // namespace remoting