Commit aab316c
authored
Remove buggy assertion in EmbedderTest::CanPostTaskToAllNativeThreads. (flutter-team-archive#16071)
This was introduced in
flutter-team-archive@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
1 file changed
Lines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3936 | 3936 | | |
3937 | 3937 | | |
3938 | 3938 | | |
3939 | | - | |
3940 | 3939 | | |
3941 | 3940 | | |
3942 | 3941 | | |
| |||
0 commit comments