Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit faaa2ff

Browse files
authored
Fix: Complete Documentation for RasterStatus::kSkipAndRetry (#44880)
## Description This PR completes the documentation for `RasterStatus::kSkipAndRetry` in the `compositor_context.h` file. The previous comment ended abruptly, leaving ambiguity regarding the behavior when the thread merger is disabled. ### Changes: - Added clarification on the behavior when the thread merger is disabled. - Explained the potential performance implications when threads are not merged. ## Related Issues flutter/flutter#131814 ## Tests No tests were added as this PR only involves documentation changes. [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
1 parent b6e329d commit faaa2ff

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

flow/compositor_context.h

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,11 @@ enum class RasterStatus {
4040
// This is currently used to wait for the thread merger to merge
4141
// the raster and platform threads.
4242
//
43-
// Since the thread merger may be disabled,
43+
// Since the thread merger may be disabled, the system will proceed
44+
// with separate threads for rasterization and platform tasks,
45+
// potentially leading to different performance characteristics.
4446
kSkipAndRetry,
45-
// Frame has been successfully rasterized, but "there are additional items in
47+
// Frame has been successfully rasterized, but there are additional items in
4648
// the pipeline waiting to be consumed. This is currently
4749
// only used when thread configuration change occurs.
4850
kEnqueuePipeline,

0 commit comments

Comments
 (0)