Cleanup of simple TODO(srte) comments.

Just fixing some minor TODOs in my name. Not worth splitting into
separate CLs as the changes are minor.

Bug: webrtc:9883
Change-Id: I05c54b76507a1d51b92cad080ca4e2dfe8546bf1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/155520
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29377}

BUILD.gn

diff --git a/BUILD.gn b/BUILD.gn
index baedb39..32fcfa2 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -538,7 +538,6 @@
   rtc_test("slow_tests") {
     testonly = true
     deps = [
-      "modules/congestion_controller/goog_cc:goog_cc_slow_tests",
       "rtc_base/task_utils:repeating_task_unittests",
       "test:test_main",
     ]

api/transport/BUILD.gn

diff --git a/api/transport/BUILD.gn b/api/transport/BUILD.gn
index 365e5ae..3c11a1a 100644
--- a/api/transport/BUILD.gn
+++ b/api/transport/BUILD.gn
@@ -38,6 +38,7 @@
   deps = [
     ":webrtc_key_value_config",
     "../../rtc_base:deprecation",
+    "../rtc_event_log",
     "../units:data_rate",
     "../units:data_size",
     "../units:time_delta",

api/transport/network_control.h

diff --git a/api/transport/network_control.h b/api/transport/network_control.h
index 8b2958c..1ba65ee 100644
--- a/api/transport/network_control.h
+++ b/api/transport/network_control.h
@@ -14,12 +14,11 @@
 
 #include <memory>
 
+#include "api/rtc_event_log/rtc_event_log.h"
 #include "api/transport/network_types.h"
 #include "api/transport/webrtc_key_value_config.h"
 
 namespace webrtc {
-// TODO(srte): Remove this forward declaration when this is in api.
-class RtcEventLog;
 
 class TargetTransferRateObserver {
  public:

api/transport/network_types.cc

diff --git a/api/transport/network_types.cc b/api/transport/network_types.cc
index d0a0c4a..88b67b3 100644
--- a/api/transport/network_types.cc
+++ b/api/transport/network_types.cc
@@ -13,8 +13,7 @@
 #include <algorithm>
 
 namespace webrtc {
-// TODO(srte): Revert to using default after removing union member.
-StreamsConfig::StreamsConfig() {}
+StreamsConfig::StreamsConfig() = default;
 StreamsConfig::StreamsConfig(const StreamsConfig&) = default;
 StreamsConfig::~StreamsConfig() = default;
 

modules/congestion_controller/goog_cc/BUILD.gn

diff --git a/modules/congestion_controller/goog_cc/BUILD.gn b/modules/congestion_controller/goog_cc/BUILD.gn
index 5782ea2..8e8d2ba 100644
--- a/modules/congestion_controller/goog_cc/BUILD.gn
+++ b/modules/congestion_controller/goog_cc/BUILD.gn
@@ -280,8 +280,4 @@
       "//testing/gmock",
     ]
   }
-
-  # TODO(srte): Remove this target when dependency in root BUILD is gone.
-  rtc_source_set("goog_cc_slow_tests") {
-  }
 }

test/frame_generator.h

diff --git a/test/frame_generator.h b/test/frame_generator.h
index 31543bc..8cc0ac3 100644
--- a/test/frame_generator.h
+++ b/test/frame_generator.h
@@ -60,8 +60,6 @@
 
   enum class OutputType {
     kI420,
-    // TODO(srte): Remove alias when downstream is updated.
-    I420 = kI420,
     kI420A,
     kI010
   };

test/frame_generator_capturer.h

diff --git a/test/frame_generator_capturer.h b/test/frame_generator_capturer.h
index 7a675c6..faf0498 100644
--- a/test/frame_generator_capturer.h
+++ b/test/frame_generator_capturer.h
@@ -38,7 +38,7 @@
 struct FrameGeneratorCapturerConfig {
   struct SquaresVideo {
     int framerate = 30;
-    FrameGenerator::OutputType pixel_format = FrameGenerator::OutputType::I420;
+    FrameGenerator::OutputType pixel_format = FrameGenerator::OutputType::kI420;
     int width = 320;
     int height = 180;
     int num_squares = 10;