Address comments

Author: starwarfan

scripts/installWebrtc.sh

@@ -71,6 +71,9 @@ download_and_build(){
     echo "src already exists."
   else
     git clone -b 88-sdk https://github.com/open-webrtc-toolkit/owt-deps-webrtc.git src
+    pushd src >/dev/null
+    git reset --hard 0d230afe9c7a968c0f2d966ef9d4d396fee489bf
+    popd >/dev/null
     mkdir -p src/build_overrides/ssl
     echo "" > src/build_overrides/ssl/ssl.gni
     echo $GNI_APPEND >> src/build_overrides/build.gni
@@ -84,7 +87,7 @@ download_and_build(){
 
   export PATH="$PATH:$DEPOT_TOOLS"
   gclient sync  --no-history
-  pushd src >/dev/null  
+  pushd src >/dev/null
   gn gen out --args="$GN_ARGS"
   ninja -C out call default_task_queue_factory
   all=`find ./out/obj/ -name "*.o"`

source/core/owt_base/SsrcGenerator.h

@@ -7,9 +7,9 @@
 
 #include <set>
 
+#include "rtc_base/random.h"
 #include "rtc_base/synchronization/mutex.h"
 #include "rtc_base/thread_annotations.h"
-#include "rtc_base/random.h"
 
 namespace owt_base {
 

source/core/rtc_adapter/VideoSendAdapter.cc

@@ -180,6 +180,10 @@ bool VideoSendAdapterImpl::init()
     configuration.retransmission_rate_limiter = m_retransmissionRateLimiter.get();
     configuration.local_media_ssrc = m_ssrc; //rtp_config.ssrcs[i];
     // TODO: enable UlpfecGenerator
+    // should follow similar logic as
+    // webrtc/call/rtp_video_sender.cc:MaybeCreateFecGenerator()
+    // for creating the fec genetator,
+    // instead of hardcoding it to ulpfec_generator.
     /*
     std::make_unique<UlpfecGenerator>(
         rtp.ulpfec.red_payload_type, rtp.ulpfec.ulpfec_payload_type, clock);