commit e338b3f4a095cc58eb7f9ebfceceb6d7ce44c1d1
author henrika <henrika@webrtc.org> Thu Oct 05 14:44:30 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Oct 05 16:59:19 2023
tree 5007fcbfdf645f0408658ae6a86487d2644b3165
parent d7703d950536473627533ed6eab24c2d50054e70

Adds TRACE_EVENT for OveruseFrameDetector::CheckForOveruse

Bug: webrtc:15539
Change-Id: I1acdbfddf3869c4464d8bf8e85aa88dac4150834
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/322620
Commit-Queue: Henrik Andreassson <henrika@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40875}

video/adaptation/overuse_frame_detector.cc

diff --git a/video/adaptation/overuse_frame_detector.cc b/video/adaptation/overuse_frame_detector.cc
index e5c2c7d..56fe71a 100644
--- a/video/adaptation/overuse_frame_detector.cc
+++ b/video/adaptation/overuse_frame_detector.cc
@@ -25,6 +25,7 @@
 #include "rtc_base/logging.h"
 #include "rtc_base/numerics/exp_filter.h"
 #include "rtc_base/time_utils.h"
+#include "rtc_base/trace_event.h"
 #include "system_wrappers/include/field_trial.h"
 
 #if defined(WEBRTC_MAC) && !defined(WEBRTC_IOS)
@@ -587,6 +588,7 @@
     return;
 
   int64_t now_ms = rtc::TimeMillis();
+  const char* action = "NoAction";
 
   if (IsOverusing(*encode_usage_percent_)) {
     // If the last thing we did was going up, and now have to back down, we need
@@ -612,21 +614,24 @@
     ++num_overuse_detections_;
 
     observer->AdaptDown();
+    action = "AdaptDown";
   } else if (IsUnderusing(*encode_usage_percent_, now_ms)) {
     last_rampup_time_ms_ = now_ms;
     in_quick_rampup_ = true;
 
     observer->AdaptUp();
+    action = "AdaptUp";
   }
+  TRACE_EVENT2("webrtc", "OveruseFrameDetector::CheckForOveruse",
+               "encode_usage_percent", *encode_usage_percent_, "action",
+               TRACE_STR_COPY(action));
 
   int rampup_delay =
       in_quick_rampup_ ? kQuickRampUpDelayMs : current_rampup_delay_ms_;
 
-  RTC_LOG(LS_VERBOSE) << " Frame stats: "
-                         " encode usage "
-                      << *encode_usage_percent_ << " overuse detections "
-                      << num_overuse_detections_ << " rampup delay "
-                      << rampup_delay;
+  RTC_LOG(LS_INFO) << "CheckForOveruse: encode usage " << *encode_usage_percent_
+                   << " overuse detections " << num_overuse_detections_
+                   << " rampup delay " << rampup_delay << " action " << action;
 }
 
 void OveruseFrameDetector::SetOptions(const CpuOveruseOptions& options) {