Skip to content

Conversation

@robert-ancell
Copy link
Contributor

This simplifies the OpenGL/software case and allows us more control over the buffering.

Compositing is now done on the Flutter rendering thread and we block the
GTK rendering until the frame is generated. This no longer requires the task
runner to be involved.
This allows Flutter to render without any GTK windows present (required for
multi-window)
@robert-ancell robert-ancell requested a review from mattkae July 18, 2025 02:58
@github-actions github-actions bot added engine flutter/engine related. See also e: labels. platform-linux Building on or for Linux specifically a: desktop Running on desktop labels Jul 18, 2025
@robert-ancell
Copy link
Contributor Author

Builds on #172330

This simplifies the OpenGL/software case and allows us more control over the buffering.
@robert-ancell robert-ancell force-pushed the linux-remove-gtkglarea4 branch from b48db2a to 562262f Compare July 18, 2025 03:01
Copy link
Contributor

@mattkae mattkae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor thing, but LGTM!

@robert-ancell robert-ancell enabled auto-merge July 22, 2025 22:15
@robert-ancell robert-ancell added this pull request to the merge queue Jul 23, 2025
Merged via the queue into flutter:master with commit afba7d7 Jul 23, 2025
178 checks passed
@robert-ancell robert-ancell deleted the linux-remove-gtkglarea4 branch July 23, 2025 02:34
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jul 23, 2025
flutter/flutter@ee0cc66...afba7d7

2025-07-23 [email protected] Remove GtkGLArea and render directly into GtkDrawingArea (flutter/flutter#172343)
2025-07-23 [email protected] Prefix generated Dart plugin imports for `registerWith` (flutter/flutter#172511)
2025-07-23 [email protected] Remove stale references to `.packages` in tool tests (flutter/flutter#172582)
2025-07-23 [email protected] Wraps all `git` executions in a `Git(...).*`, use `*=noglob` on Windows (flutter/flutter#172495)
2025-07-22 [email protected] Omit instruction to `cd .` after `flutter create` (flutter/flutter#172513)
2025-07-22 [email protected] Improve assertion message in `AlignmentDirectional.resolve` (flutter/flutter#172096)
2025-07-22 [email protected] Update warnGradleVersion to `8.7.0` (flutter/flutter#172576)
2025-07-22 [email protected] Use a fake representation of `cache/artifacts/gradle_wrapper` (flutter/flutter#172503)
2025-07-22 [email protected] Revert #160653 Fix view removal process for AutofillContextAction.cancel (flutter/flutter#172490)
2025-07-22 [email protected] Bump meta to 0.17.0 (flutter/flutter#172541)

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] 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

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
azatech pushed a commit to azatech/flutter that referenced this pull request Jul 28, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
vashworth pushed a commit to vashworth/packages that referenced this pull request Jul 30, 2025
…r#9665)

flutter/flutter@ee0cc66...afba7d7

2025-07-23 [email protected] Remove GtkGLArea and render directly into GtkDrawingArea (flutter/flutter#172343)
2025-07-23 [email protected] Prefix generated Dart plugin imports for `registerWith` (flutter/flutter#172511)
2025-07-23 [email protected] Remove stale references to `.packages` in tool tests (flutter/flutter#172582)
2025-07-23 [email protected] Wraps all `git` executions in a `Git(...).*`, use `*=noglob` on Windows (flutter/flutter#172495)
2025-07-22 [email protected] Omit instruction to `cd .` after `flutter create` (flutter/flutter#172513)
2025-07-22 [email protected] Improve assertion message in `AlignmentDirectional.resolve` (flutter/flutter#172096)
2025-07-22 [email protected] Update warnGradleVersion to `8.7.0` (flutter/flutter#172576)
2025-07-22 [email protected] Use a fake representation of `cache/artifacts/gradle_wrapper` (flutter/flutter#172503)
2025-07-22 [email protected] Revert #160653 Fix view removal process for AutofillContextAction.cancel (flutter/flutter#172490)
2025-07-22 [email protected] Bump meta to 0.17.0 (flutter/flutter#172541)

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] 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

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
ksokolovskyi pushed a commit to ksokolovskyi/flutter that referenced this pull request Aug 19, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
mboetger pushed a commit to mboetger/flutter that referenced this pull request Sep 18, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
robert-ancell added a commit to robert-ancell/flutter that referenced this pull request Sep 19, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
korca0220 pushed a commit to korca0220/flutter that referenced this pull request Sep 22, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
lucaantonelli pushed a commit to lucaantonelli/flutter that referenced this pull request Nov 21, 2025
)

This simplifies the OpenGL/software case and allows us more control over
the buffering.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: desktop Running on desktop engine flutter/engine related. See also e: labels. platform-linux Building on or for Linux specifically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants