Skip to content

Commit aab316c

Browse files
authored
Remove buggy assertion in EmbedderTest::CanPostTaskToAllNativeThreads. (flutter#16071)
This was introduced in flutter/engine@c5329ef. The assertion was originally written to check that more than 4 threads were managed by the engine (UI, Platform, GPU, IO + ConcurrentWQWorkers). However, the assertion actually only checked the count of workers in the ConcurrentWQ. As written, this assertion would fail on all hosts with a hardware concurrency of less than 4. Remove the assertion. The engine threads count and its assertions already check callbacks on workers. So this check was incorrect and redundant.
1 parent da1a5d5 commit aab316c

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

shell/platform/embedder/tests/embedder_unittests.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3936,7 +3936,6 @@ TEST_F(EmbedderTest, CanPostTaskToAllNativeThreads) {
39363936

39373937
sync_latch.Wait();
39383938

3939-
ASSERT_GT(worker_count, 4u /* three base threads plus workers */);
39403939
const auto engine_threads_count = worker_count + 4u;
39413940

39423941
struct Captures {

0 commit comments

Comments
 (0)