Public nodes needing paint or layout #166148

emerssso · 2025-03-28T17:56:49Z

This PR adds 2 new @protected fields to PipelineOwner which list the RenderObjects currently needing paint or layout for the next frame.

This PR addresses #166147, which has further discussion of the motivation.

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

…le guide.

jonahwilliams · 2025-03-28T18:10:03Z

packages/flutter/lib/src/rendering/object.dart

+  /// when they are marked for layout. Subclasses of [PipelineOwner] may use this list
+  /// to invalidate caches or otherwise make performance optimizations.
+  @protected
+  List<RenderObject> get nodesNeedingLayout => _nodesNeedingLayout;


I would expose these as Iterables so that its more obvious they are intended to be read only. I guess in theory adding nodes to this list shouldn't cause bugs but I wouldn't want to test that.

Great idea, thanks! Done.

emerssso · 2025-03-28T18:26:07Z

@chunhtai FYI, here's the PR we discussed in flutter.dev/go/ondirtycallbacks. Looks like I don't have perms to add you as a reviewer as requested.

jonahwilliams

LGTM

chunhtai

LGTM, I am glad we can reach a solution without touching the rendering code

LongCatIsLooong · 2025-03-28T20:31:38Z

packages/flutter/lib/src/rendering/object.dart

+  /// when they are marked for layout. Subclasses of [PipelineOwner] may use them
+  /// to invalidate caches or otherwise make performance optimizations.
+  @protected
+  Iterable<RenderObject> get nodesNeedingLayout => _nodesNeedingLayout;


FWIW, it will be tricky to pick the correct timing to safety read the dirty list, as new dirty nodes can be added in transient callbacks / microtask flushes, or as a result of widget re-build. The dirty list can even be mutated during a flushLayout call (with RenderObject.invokeLayoutCallback).

Yes, agreed. So far, I've not had problems in practice, I think possibly because for my use-case a lot of repaint boundaries are already dirtied when they're marked in these cases (in particular, I think anything marked during invokeLayoutCallback should already be invalidated based on the docs). I am trying to add more edge cases to test though.

uhm sorry I didn't notice the @protected annotation. So it looks like these getters will only be accessed in subclasses' flushLayout, which works for your use case (but it would be nice if it's documented as such). Also nodesNeedingLayout does not include the dirty nodes from child / descendant PipelineOwners (although I don't think those are actually used in the framework atm).

Good point, docs added on timing and descendant PipelineOwners.

I have a few workarounds in mind for descendant PipelineOwners (either wrapping children for instrumentation as they are adopted, or enforcing that only instrumenting PipelineOwners can be adopted or similar). Haven't quite settled on an approach yet.

I don't think you can "wrap" any children IIUC, PiplelineOwner is a base class so composition is probably not an option, you'll have to extend the child's class in case it's a different PipelineOwner subclass and has custom logic.

Hmm, I think this works for my mental model, but please double check me (I'm also going to start tackling this handling ASAP to make sure it's OK!)

I was thinking the adoption flow would look like one of the following cases:

RootPipelineOwner case: ChildPipelineOwner passed to adoptChild is wrapped with DelegatePipelineOwner, which is adopted, but passes all public API calls to ChildPipelineOwner.

ChildPipelineOwner case: GrandchildPipelineOwner passed to ChildPipelineOwner.adoptChild, wrapped with DelegatePipeline owner, which is adopted, but passes all public API calls to GrandchildPipelineOwner.

So either way, a LayoutBuilder PipelineOwner would be wrapped in DelegatePipelineOwner when it was adopted into the pipeline tree, but would otherwise be unaffected.

I think this should work in release mode, but the _debugParent field would get messed up by this. It doesn't look like that field is used too much, but it definitely will be a little odd.

I generally think any custom logic in a descendant PipelineOwner should still be executed by this setup (since any calls to Delegate will be passed through unchanged). In fact, I'm a little more worried that I'd miss registering children with my child tracking delegates.

I took a closer look at #166148 (comment), I doubt that it would work:

void flushPaint() { // This shouldn't upset the check for @protected // since it's within an extender of PipelineOwner doStuffWith(child.nodesNeedingPaint); child.flushPaint(); }

child.flushPaint() wouldn't do anything here, since there's no public API for adding dirty nodes to child: in markNeedsPaint render objects would add dirty nodes to this._nodsNeedingPaint instead of child._nodsNeedingPaint. That was why PipelineOwner was made a base class

Ah, I see now. Yes, you're right, RenderObject ownership with the delegate is broken. Further, your LayoutBuilder example makes me think that mandating a subclass for all PipelineOwners also isn't practical.

I guess a more reliable solution would be a way to iterate over PipelineOwner's children. I could imagine adding either an @protected children or a visitChildren(void Function(PipelineOwner) similar to RenderObject and Element (altho I think I prefer the latter). Thoughts?

OK, I've had my coffee now and realized that visitChildren already exists. I'm gonna try out how this would work using that in the root's onLayout/onPaint to check each child's dirty nodes.

Using visitChildren seems to work pretty well with a child depth of 1-2. I'm gonna run with this for now, thanks again for thinking through this with me :)

justinmc

LGTM once the checks are passing 👍

justinmc · 2025-03-28T23:51:53Z

packages/flutter/lib/src/rendering/object.dart

+  /// Since nodes may be marked for layout at any time, they are best checked during
+  /// [flushLayout].
+  ///
+  /// Note that this does not include marked children of child [PipelineOwner]s.


See that style guide on "note that": https://github.com/flutter/flutter/blob/master/docs/contributing/Style-guide-for-Flutter-repo.md#avoid-empty-prose

+1, also maybe add that in the iterable relayout boundaries appear in arbitrary order and the same RO may appear multiple times.

Fixed and added. Thanks!

justinmc · 2025-03-28T23:52:05Z

packages/flutter/lib/src/rendering/object.dart

+  /// Since nodes may be marked for layout at any time, they are best checked during
+  /// [flushPaint].
+  ///
+  /// Note that this does not include marked children of child [PipelineOwner]s.


Another "note that" that should be rephrased.

LongCatIsLooong · 2025-03-29T00:38:44Z

packages/flutter/lib/src/rendering/object.dart

+  /// [flushLayout].
+  ///
+  /// Note that this does not include marked children of child [PipelineOwner]s.
+  @protected


@nonVirtual too, if you're not overriding it?

LongCatIsLooong · 2025-03-29T00:46:05Z

packages/flutter/lib/src/rendering/object.dart

+  /// when they are marked for layout. Subclasses of [PipelineOwner] may use them
+  /// to invalidate caches or otherwise make performance optimizations.
+  @protected
+  Iterable<RenderObject> get nodesNeedingLayout => _nodesNeedingLayout;


I don't think you can "wrap" any children IIUC, PiplelineOwner is a base class so composition is probably not an option, you'll have to extend the child's class in case it's a different PipelineOwner subclass and has custom logic.

LongCatIsLooong · 2025-03-29T00:51:12Z

packages/flutter/lib/src/rendering/object.dart

@@ -1088,6 +1088,18 @@ base class PipelineOwner with DiagnosticableTreeMixin {
  bool _shouldMergeDirtyNodes = false;
  List<RenderObject> _nodesNeedingLayout = <RenderObject>[];

+  /// The [RenderObject]s which need to be laid out in the next [flushLayout] pass.


nit: I'd say relayout boundaries since this is not a comprehensive list of dirty nodes.

LongCatIsLooong · 2025-03-29T00:52:08Z

packages/flutter/lib/src/rendering/object.dart

@@ -1088,6 +1088,18 @@ base class PipelineOwner with DiagnosticableTreeMixin {
  bool _shouldMergeDirtyNodes = false;
  List<RenderObject> _nodesNeedingLayout = <RenderObject>[];

+  /// The [RenderObject]s which need to be laid out in the next [flushLayout] pass.
+  ///
+  /// [RenderObject]s with [RenderObject.isRepaintBoundary] are added


This doesn't sound right. For layout we collect relayout boundaries instead of repaint boundaries.

Woops! Fixed

LongCatIsLooong · 2025-03-29T00:56:33Z

packages/flutter/lib/src/rendering/object.dart

+  /// Since nodes may be marked for layout at any time, they are best checked during
+  /// [flushLayout].
+  ///
+  /// Note that this does not include marked children of child [PipelineOwner]s.


+1, also maybe add that in the iterable relayout boundaries appear in arbitrary order and the same RO may appear multiple times.

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/flutter@05b5e79...a0b1b32 2025-04-01 [email protected] Roll Dart SDK from b4d374ec59ec to 4e1f02bc704f (2 revisions) (flutter/flutter#166342) 2025-04-01 [email protected] Trim any text before osascript JSON response (flutter/flutter#166296) 2025-04-01 [email protected] [Gen-l10n] Add `Message.resourceId` and `locale` to all `L10nException` error messages (flutter/flutter#163654) 2025-04-01 [email protected] Add `--ignore-timeouts` flag for `flutter test` command (flutter/flutter#164437) 2025-04-01 [email protected] Update TESTOWNERS username (flutter/flutter#166191) 2025-04-01 [email protected] Roll Skia from 4b07443e6071 to 52cbb917fffd (4 revisions) (flutter/flutter#166329) 2025-04-01 [email protected] Roll Dart SDK from 6b07a09cbd2d to b4d374ec59ec (2 revisions) (flutter/flutter#166321) 2025-04-01 [email protected] [tool] Improve using project files in build targets (flutter/flutter#166211) 2025-04-01 [email protected] Rename FlRenderer to FlCompositorOpenGL (flutter/flutter#166037) 2025-04-01 [email protected] [engine, web_ui] Fix instances of library_private_types_in_public_api (flutter/flutter#166156) 2025-04-01 [email protected] Roll Dart SDK from 509faa921c95 to 6b07a09cbd2d (1 revision) (flutter/flutter#166301) 2025-04-01 [email protected] [Impeller] small cpu perf for text contents. (flutter/flutter#166199) 2025-04-01 [email protected] [android_engine_test] disable old HC mode tests. (flutter/flutter#166293) 2025-04-01 [email protected] [impeller] fixes diagonal antialiased lines (flutter/flutter#166298) 2025-04-01 [email protected] Roll Skia from 5f262bd2cbb4 to 4b07443e6071 (10 revisions) (flutter/flutter#166299) 2025-03-31 [email protected] [Impeller] Directly tessellate conics to linear path segments (flutter/flutter#166165) 2025-03-31 [email protected] [tool] Don't write the .flutter-plugins-dependencies file if it is unchanged (flutter/flutter#166164) 2025-03-31 [email protected] Move `.cxx` directory out of `android/app` (flutter/flutter#166277) 2025-03-31 [email protected] Fix typo in carousel.dart (flutter/flutter#164727) 2025-03-31 [email protected] Roll Dart SDK from c5fa06710bb6 to 509faa921c95 (1 revision) (flutter/flutter#166283) 2025-03-31 [email protected] Public nodes needing paint or layout (flutter/flutter#166148) 2025-03-31 [email protected] [Gen-l10n] Infer placeholder types on both templates and localizations (flutter/flutter#163690) 2025-03-31 [email protected] [Engine][iOS] Cancel animation when recieved `UIKeyboardWillHideNotification` with duration 0.0 (flutter/flutter#164884) 2025-03-31 [email protected] [fuchsia] Remove explicit LogSink and InspectSink routing and use dictionaries instead (flutter/flutter#162780) 2025-03-31 [email protected] Updated to latest AVD to Support Android 16 (API 36) (flutter/flutter#165926) 2025-03-31 [email protected] Feat: Add brightnessOf method for theme (flutter/flutter#163733) 2025-03-31 [email protected] Marks Linux_mokey new_gallery__crane_perf to be flaky (flutter/flutter#165964) 2025-03-31 [email protected] [ Tool ] Correctly select entrypoint target for web build from positional argument list (flutter/flutter#166260) 2025-03-31 [email protected] [Impeller] remove validation warning ignores. (flutter/flutter#166205) 2025-03-31 [email protected] [Impeller] handle shader ordering bug on macOS. (flutter/flutter#165937) 2025-03-31 [email protected] Fix CODEOWNERS for the iOS review team (flutter/flutter#166178) 2025-03-31 [email protected] Remove `<meta content="IE=Edge" http-equiv="X-UA-Compatible">` (flutter/flutter#166252) 2025-03-31 [email protected] Roll Dart SDK from b9c35e05feb5 to c5fa06710bb6 (1 revision) (flutter/flutter#166251) 2025-03-31 [email protected] Roll Skia from 418c68ea5ccb to 5f262bd2cbb4 (2 revisions) (flutter/flutter#166244) 2025-03-31 [email protected] Roll Skia from b6a3bbd1d153 to 418c68ea5ccb (1 revision) (flutter/flutter#166236) 2025-03-31 [email protected] [Impeller] fix min filter for GL external textures. (flutter/flutter#166224) 2025-03-31 [email protected] Roll Skia from 10f4cf9a817d to b6a3bbd1d153 (13 revisions) (flutter/flutter#166231) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose ...

…r#8985) Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/flutter@05b5e79...a0b1b32 2025-04-01 [email protected] Roll Dart SDK from b4d374ec59ec to 4e1f02bc704f (2 revisions) (flutter/flutter#166342) 2025-04-01 [email protected] Trim any text before osascript JSON response (flutter/flutter#166296) 2025-04-01 [email protected] [Gen-l10n] Add `Message.resourceId` and `locale` to all `L10nException` error messages (flutter/flutter#163654) 2025-04-01 [email protected] Add `--ignore-timeouts` flag for `flutter test` command (flutter/flutter#164437) 2025-04-01 [email protected] Update TESTOWNERS username (flutter/flutter#166191) 2025-04-01 [email protected] Roll Skia from 4b07443e6071 to 52cbb917fffd (4 revisions) (flutter/flutter#166329) 2025-04-01 [email protected] Roll Dart SDK from 6b07a09cbd2d to b4d374ec59ec (2 revisions) (flutter/flutter#166321) 2025-04-01 [email protected] [tool] Improve using project files in build targets (flutter/flutter#166211) 2025-04-01 [email protected] Rename FlRenderer to FlCompositorOpenGL (flutter/flutter#166037) 2025-04-01 [email protected] [engine, web_ui] Fix instances of library_private_types_in_public_api (flutter/flutter#166156) 2025-04-01 [email protected] Roll Dart SDK from 509faa921c95 to 6b07a09cbd2d (1 revision) (flutter/flutter#166301) 2025-04-01 [email protected] [Impeller] small cpu perf for text contents. (flutter/flutter#166199) 2025-04-01 [email protected] [android_engine_test] disable old HC mode tests. (flutter/flutter#166293) 2025-04-01 [email protected] [impeller] fixes diagonal antialiased lines (flutter/flutter#166298) 2025-04-01 [email protected] Roll Skia from 5f262bd2cbb4 to 4b07443e6071 (10 revisions) (flutter/flutter#166299) 2025-03-31 [email protected] [Impeller] Directly tessellate conics to linear path segments (flutter/flutter#166165) 2025-03-31 [email protected] [tool] Don't write the .flutter-plugins-dependencies file if it is unchanged (flutter/flutter#166164) 2025-03-31 [email protected] Move `.cxx` directory out of `android/app` (flutter/flutter#166277) 2025-03-31 [email protected] Fix typo in carousel.dart (flutter/flutter#164727) 2025-03-31 [email protected] Roll Dart SDK from c5fa06710bb6 to 509faa921c95 (1 revision) (flutter/flutter#166283) 2025-03-31 [email protected] Public nodes needing paint or layout (flutter/flutter#166148) 2025-03-31 [email protected] [Gen-l10n] Infer placeholder types on both templates and localizations (flutter/flutter#163690) 2025-03-31 [email protected] [Engine][iOS] Cancel animation when recieved `UIKeyboardWillHideNotification` with duration 0.0 (flutter/flutter#164884) 2025-03-31 [email protected] [fuchsia] Remove explicit LogSink and InspectSink routing and use dictionaries instead (flutter/flutter#162780) 2025-03-31 [email protected] Updated to latest AVD to Support Android 16 (API 36) (flutter/flutter#165926) 2025-03-31 [email protected] Feat: Add brightnessOf method for theme (flutter/flutter#163733) 2025-03-31 [email protected] Marks Linux_mokey new_gallery__crane_perf to be flaky (flutter/flutter#165964) 2025-03-31 [email protected] [ Tool ] Correctly select entrypoint target for web build from positional argument list (flutter/flutter#166260) 2025-03-31 [email protected] [Impeller] remove validation warning ignores. (flutter/flutter#166205) 2025-03-31 [email protected] [Impeller] handle shader ordering bug on macOS. (flutter/flutter#165937) 2025-03-31 [email protected] Fix CODEOWNERS for the iOS review team (flutter/flutter#166178) 2025-03-31 [email protected] Remove `<meta content="IE=Edge" http-equiv="X-UA-Compatible">` (flutter/flutter#166252) 2025-03-31 [email protected] Roll Dart SDK from b9c35e05feb5 to c5fa06710bb6 (1 revision) (flutter/flutter#166251) 2025-03-31 [email protected] Roll Skia from 418c68ea5ccb to 5f262bd2cbb4 (2 revisions) (flutter/flutter#166244) 2025-03-31 [email protected] Roll Skia from b6a3bbd1d153 to 418c68ea5ccb (1 revision) (flutter/flutter#166236) 2025-03-31 [email protected] [Impeller] fix min filter for GL external textures. (flutter/flutter#166224) 2025-03-31 [email protected] Roll Skia from 10f4cf9a817d to b6a3bbd1d153 (13 revisions) (flutter/flutter#166231) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose ...

emerssso added 3 commits March 28, 2025 13:12

Add unmodifiable public getters for nodesNeedingLayout and Paint.

54fbbd4

Make new fields modifiable but protected to better conform to the sty…

d12af17

…le guide.

Update dartdoc to reflect switch to modifiable protected member.

8acb2fb

github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Mar 28, 2025

jonahwilliams reviewed Mar 28, 2025

View reviewed changes

Switch to exposing iterables to make read-only intention clearer.

30d0c8e

jonahwilliams requested a review from justinmc March 28, 2025 18:26

jonahwilliams approved these changes Mar 28, 2025

View reviewed changes

chunhtai approved these changes Mar 28, 2025

View reviewed changes

Use _TestPipelineOwner to avoid invalid access to protected fields.

a949732

LongCatIsLooong reviewed Mar 28, 2025

View reviewed changes

emerssso added 2 commits March 28, 2025 17:08

Run dartfmt

f608dd6

Add dartdoc on best time to use and limitations on child PipelineOwners.

a1edbf2

justinmc approved these changes Mar 28, 2025

View reviewed changes

LongCatIsLooong reviewed Mar 29, 2025

View reviewed changes

emerssso added 3 commits March 31, 2025 10:45

Address documentation feedback and add @nonvirtual

6ec3a39

Also address for layout.

efa21f0

Fix so that nonVirtual methods are not overridden in tests.

8b50657

LongCatIsLooong added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 31, 2025

auto-submit bot added this pull request to the merge queue Mar 31, 2025

Merged via the queue into flutter:master with commit a13f771 Mar 31, 2025
74 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 31, 2025

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to 70cd54c3fd13 (259 revisions) flutter/packages#8973

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

c5c65ec

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to 0befaaafc952 (260 revisions) flutter/packages#8974

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

8dba3dc

chunhtai mentioned this pull request Apr 1, 2025

Expose nodes needing layout or paint from PipelineOwner #166147

Closed

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to dbef1314b186 (261 revisions) flutter/packages#8976

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

9a21fac

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to a0b1b3253416 (262 revisions) flutter/packages#8977

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

ffc7fa7

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 2, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

02684a3

engine-flutter-autoroll mentioned this pull request Apr 2, 2025

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/packages#8983

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 2, 2025

a13f7717b Public nodes needing paint or layout (flutter/flutter#166148)

73e0360

engine-flutter-autoroll mentioned this pull request Apr 2, 2025

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/packages#8985

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Public nodes needing paint or layout #166148

Public nodes needing paint or layout #166148

emerssso commented Mar 28, 2025

jonahwilliams Mar 28, 2025

emerssso Mar 28, 2025

emerssso commented Mar 28, 2025

jonahwilliams left a comment

chunhtai left a comment

LongCatIsLooong Mar 28, 2025 •

edited

Loading

emerssso Mar 28, 2025

LongCatIsLooong Mar 28, 2025

emerssso Mar 28, 2025

LongCatIsLooong Mar 29, 2025

emerssso Mar 31, 2025

LongCatIsLooong Mar 31, 2025

emerssso Apr 1, 2025

emerssso Apr 1, 2025

emerssso Apr 1, 2025

justinmc left a comment

justinmc Mar 28, 2025

LongCatIsLooong Mar 29, 2025

emerssso Mar 31, 2025

justinmc Mar 28, 2025

emerssso Mar 31, 2025

LongCatIsLooong Mar 29, 2025

emerssso Mar 31, 2025

LongCatIsLooong Mar 29, 2025

LongCatIsLooong Mar 29, 2025

emerssso Mar 31, 2025

LongCatIsLooong Mar 29, 2025

emerssso Mar 31, 2025

LongCatIsLooong Mar 29, 2025

Public nodes needing paint or layout #166148

Public nodes needing paint or layout #166148

Conversation

emerssso commented Mar 28, 2025

Pre-launch Checklist

Choose a reason for hiding this comment

Choose a reason for hiding this comment

emerssso commented Mar 28, 2025

jonahwilliams left a comment

Choose a reason for hiding this comment

chunhtai left a comment

Choose a reason for hiding this comment

LongCatIsLooong Mar 28, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

justinmc left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LongCatIsLooong Mar 28, 2025 •

edited

Loading