chore: update chrome version to 143.0.7474.2, update patch file

Author: vpalmisano

chromium/build.sh

@@ -13,7 +13,7 @@ else
 fi
 
 # https://chromium.googlesource.com/chromium/src/+refs
-export VERSION="141.0.7364.1"
+export VERSION="143.0.7474.2"
 export DEFAULT_BRANCH="tags/${VERSION}"
 
 function setup() {
@@ -91,7 +91,7 @@ function apply_patch() {
     local filepath=${DIR}/max-video-decoders_$(echo ${branch} | sed s/'tags\/'//).patch
     if [ ! -f ${filepath} ]; then
         echo "INFO: patch file not found: ${filepath}, using default patch"
-        filepath=${DIR}/max-video-decoders_main.patch
+        filepath=${DIR}/max-video-decoders_latest.patch
     fi
     cd ${CHROMIUM_SRC}/third_party/webrtc
     git apply < ${filepath}
@@ -111,9 +111,9 @@ function update() {
     git pull
     cd ${CHROMIUM_SRC}
     git rebase --abort || true
-    git fetch origin ${branch} --no-tags
-    git checkout ${branch}
-    git pull origin ${branch}
+    #git fetch origin ${branch} --no-tags
+    #git checkout ${branch}
+    #git pull origin ${branch}
     gclient sync -D --force --reset --no-history --revision=${branch}
     apply_patch ${branch}
 }
@@ -123,6 +123,7 @@ function build() {
     if [ "${PLATFORM}" = "Linux" ]; then
         time ionice -c3 nice -n19 autoninja -C out/Default "chrome/installer/linux:unstable_deb"
         mv out/Default/*.deb ${DIR}
+        autoninja -C out/Default video_replay
     else
         time nice -n19 autoninja -C out/Default chrome chrome/installer/mac
         rm -rf out/Default/Chromium

chromium/max-video-decoders_main.patch renamed to chromium/max-video-decoders_141.0.7364.1.patch

@@ -0,0 +1,100 @@
+diff --git a/video/video_receive_stream2.cc b/video/video_receive_stream2.cc
+index ea4b474a..cb11023b 100644
+--- a/video/video_receive_stream2.cc
++++ b/video/video_receive_stream2.cc
+@@ -105,6 +105,31 @@ namespace {
+ constexpr TimeDelta kMinBaseMinimumDelay = TimeDelta::Zero();
+ constexpr TimeDelta kMaxBaseMinimumDelay = TimeDelta::Seconds(10);
+ 
++uint getTotalVideoDecoders() {
++  FileWrapper file_wrapper = FileWrapper::OpenReadOnly("/dev/shm/chromium-video-decoders");
++  if (file_wrapper.is_open()) {
++    char buf[256];
++    file_wrapper.Read(buf, sizeof(buf));
++    file_wrapper.Close();
++    return (uint)atoi(buf);
++  }
++  return 0;
++}
++
++void updateTotalVideoDecoders(int value) {
++  char buf[256];
++  uint total_video_decoders = getTotalVideoDecoders();
++  snprintf(buf, sizeof(buf), "%u", std::max<uint>(0, total_video_decoders + value));
++
++  FileWrapper file_wrapper = FileWrapper::OpenWriteOnly("/dev/shm/chromium-video-decoders");
++  if (!file_wrapper.is_open()) {
++    RTC_LOG(LS_ERROR) << "total video decoders file cannot be opened: /dev/shm/chromium-video-decoders";
++    return;
++  }
++  file_wrapper.Write(buf, strlen(buf));
++  file_wrapper.Close();
++}
++
+ // Concrete instance of RecordableEncodedFrame wrapping needed content
+ // from EncodedFrame.
+ class WebRtcRecordableEncodedFrame : public RecordableEncodedFrame {
+@@ -182,7 +207,7 @@ class NullVideoDecoder : public VideoDecoder {
+ 
+   int32_t Decode(const EncodedImage& input_image,
+                  int64_t render_time_ms) override {
+-    RTC_LOG(LS_ERROR) << "The NullVideoDecoder doesn't support decoding.";
++    RTC_LOG(LS_WARNING) << "The NullVideoDecoder doesn't support decoding.";
+     return WEBRTC_VIDEO_CODEC_OK;
+   }
+ 
+@@ -394,6 +419,20 @@ void VideoReceiveStream2::Start() {
+     renderer = this;
+   }
+ 
++  // Check if we should create video decoders.
++  enable_video_decoders_ = true;
++  std::string max_video_decoders = env_.field_trials().Lookup("WebRTC-MaxVideoDecoders");
++  if (!max_video_decoders.empty()) {
++    uint video_decoders = getTotalVideoDecoders();
++    uint max_decoders = (uint)atoi(max_video_decoders.c_str());
++    RTC_LOG(LS_ERROR) << "WebRTC-MaxVideoDecoders: " << video_decoders << "/" << max_decoders;
++    if (video_decoders >= max_decoders) {
++      enable_video_decoders_ = false;
++    }
++  }
++  if (enable_video_decoders_)
++    updateTotalVideoDecoders(1);
++
+   for (const Decoder& decoder : config_.decoders) {
+     VideoDecoder::Settings settings;
+     settings.set_codec_type(
+@@ -472,6 +511,9 @@ void VideoReceiveStream2::Stop() {
+     stats_proxy_.DecoderThreadStopped();
+ 
+     UpdateHistograms();
++
++    if (enable_video_decoders_)
++      updateTotalVideoDecoders(-1);
+   }
+ 
+   // TODO(bugs.webrtc.org/11993): Make these calls on the network thread.
+@@ -563,8 +605,9 @@ void VideoReceiveStream2::CreateAndRegisterExternalDecoder(
+     const Decoder& decoder) {
+   TRACE_EVENT0("webrtc",
+                "VideoReceiveStream2::CreateAndRegisterExternalDecoder");
+-  std::unique_ptr<VideoDecoder> video_decoder =
+-      config_.decoder_factory->Create(env_, decoder.video_format);
++  std::unique_ptr<VideoDecoder> video_decoder = enable_video_decoders_ ?
++    config_.decoder_factory->Create(env_, decoder.video_format) :
++    nullptr;
+   // If we still have no valid decoder, we have to create a "Null" decoder
+   // that ignores all calls. The reason we can get into this state is that the
+   // old decoder factory interface doesn't have a way to query supported
+diff --git a/video/video_receive_stream2.h b/video/video_receive_stream2.h
+index a07bd08f..0aa6a3f6 100644
+--- a/video/video_receive_stream2.h
++++ b/video/video_receive_stream2.h
+@@ -298,6 +298,7 @@ class VideoReceiveStream2
+   RtpStreamsSynchronizer rtp_stream_sync_;
+ 
+   std::unique_ptr<VideoStreamBufferController> buffer_;
++  bool enable_video_decoders_;
+ 
+   // `receiver_controller_` is valid from when RegisterWithTransport is invoked
+   //  until UnregisterFromTransport.