Revert "Fix flake by making thread ID tracking in CanPostTaskToAllNativeThreads thread safe. (flutter#16081)"

This reverts commit 27e0001.

Author: NoamDev

shell/platform/embedder/tests/embedder_unittests.cc

@@ -3943,7 +3943,6 @@ TEST_F(EmbedderTest, CanPostTaskToAllNativeThreads) {
     fml::CountDownLatch latch;
 
     // Ensures that the expect number of distinct threads were serviced.
-    std::mutex thread_ids_mutex;
     std::set<std::thread::id> thread_ids;
 
     size_t platform_threads_count = 0;
@@ -3961,6 +3960,7 @@ TEST_F(EmbedderTest, CanPostTaskToAllNativeThreads) {
                   engine.get(),
                   [](FlutterNativeThreadType type, void* baton) {
                     auto captures = reinterpret_cast<Captures*>(baton);
+
                     switch (type) {
                       case kFlutterNativeThreadTypeRender:
                         captures->render_threads_count++;
@@ -3975,10 +3975,8 @@ TEST_F(EmbedderTest, CanPostTaskToAllNativeThreads) {
                         captures->platform_threads_count++;
                         break;
                     }
-                    {
-                      std::scoped_lock lock(captures->thread_ids_mutex);
-                      captures->thread_ids.insert(std::this_thread::get_id());
-                    }
+
+                    captures->thread_ids.insert(std::this_thread::get_id());
                     captures->latch.CountDown();
                   },
                   &captures),