commit e519f38eaa1beefece12fe0ffb56c971f643dec4
author Danil Chapovalov <danilchap@webrtc.org> Thu Aug 11 12:26:09 2022 +0200
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Aug 11 12:55:32 2022 +0000
tree 62162146e24cc6703e75b5137555c03b0648968d
parent e2d829cf7731ab327435cd8e2b754dec98f7a6e9

Remove rtc::Location from SendTask test helper

that rtc::Location parameter was used only as extra information for the
RTC_CHECKs directly in the function, thus call stack of the crash should
provide all the information about the caller.

Bug: webrtc:11318
Change-Id: Iec6dd2c5de547f3e1601647a614be7ce57a55734
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/270920
Reviewed-by: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37748}

api/sequence_checker_unittest.cc

diff --git a/api/sequence_checker_unittest.cc b/api/sequence_checker_unittest.cc
index 21a0894..9594662 100644
--- a/api/sequence_checker_unittest.cc
+++ b/api/sequence_checker_unittest.cc
@@ -76,20 +76,16 @@
   SequenceChecker sequence_checker;
   sequence_checker.Detach();
   TaskQueueForTest queue;
-  queue.SendTask([&] { EXPECT_TRUE(sequence_checker.IsCurrent()); },
-                 RTC_FROM_HERE);
+  queue.SendTask([&] { EXPECT_TRUE(sequence_checker.IsCurrent()); });
 }
 
 TEST(SequenceCheckerTest, DetachFromTaskQueueAndUseOnThread) {
   TaskQueueForTest queue;
-  queue.SendTask(
-      [] {
-        SequenceChecker sequence_checker;
-        sequence_checker.Detach();
-        RunOnDifferentThread(
-            [&] { EXPECT_TRUE(sequence_checker.IsCurrent()); });
-      },
-      RTC_FROM_HERE);
+  queue.SendTask([] {
+    SequenceChecker sequence_checker;
+    sequence_checker.Detach();
+    RunOnDifferentThread([&] { EXPECT_TRUE(sequence_checker.IsCurrent()); });
+  });
 }
 
 TEST(SequenceCheckerTest, MethodNotAllowedOnDifferentThreadInDebug) {
@@ -102,8 +98,7 @@
   SequenceChecker sequence_checker;
   TaskQueueForTest queue;
   queue.SendTask(
-      [&] { EXPECT_EQ(sequence_checker.IsCurrent(), !RTC_DCHECK_IS_ON); },
-      RTC_FROM_HERE);
+      [&] { EXPECT_EQ(sequence_checker.IsCurrent(), !RTC_DCHECK_IS_ON); });
 }
 
 TEST(SequenceCheckerTest, DetachFromTaskQueueInDebug) {
@@ -111,15 +106,13 @@
   sequence_checker.Detach();
 
   TaskQueueForTest queue1;
-  queue1.SendTask([&] { EXPECT_TRUE(sequence_checker.IsCurrent()); },
-                  RTC_FROM_HERE);
+  queue1.SendTask([&] { EXPECT_TRUE(sequence_checker.IsCurrent()); });
 
   // IsCurrent should return false in debug builds after moving to
   // another task queue.
   TaskQueueForTest queue2;
   queue2.SendTask(
-      [&] { EXPECT_EQ(sequence_checker.IsCurrent(), !RTC_DCHECK_IS_ON); },
-      RTC_FROM_HERE);
+      [&] { EXPECT_EQ(sequence_checker.IsCurrent(), !RTC_DCHECK_IS_ON); });
 }
 
 class TestAnnotations {
@@ -146,7 +139,7 @@
 void TestAnnotationsOnWrongQueue() {
   TestAnnotations annotations;
   TaskQueueForTest queue;
-  queue.SendTask([&] { annotations.ModifyTestVar(); }, RTC_FROM_HERE);
+  queue.SendTask([&] { annotations.ModifyTestVar(); });
 }
 
 #if RTC_DCHECK_IS_ON