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

Conversation

@flar
Copy link
Contributor

@flar flar commented Nov 25, 2024

A new source code/header structure was introduced when the DlColorSource and DlImageFilter objects were migrated to Impeller geometry classes. Even though the DlColorFilter objects did not depend on Skia geometry objects, they need to be updated to the new source layout for consistency.

@flar flar force-pushed the dl-color-filter-use-impeller-geometry branch from a994c34 to 6d00232 Compare November 26, 2024 00:40
@flar flar requested a review from jonahwilliams November 26, 2024 02:15
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@flar flar added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 26, 2024
@auto-submit auto-submit bot merged commit 9975c85 into flutter:main Nov 26, 2024
36 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 26, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 26, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 27, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 27, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 27, 2024
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Nov 27, 2024
…159523)

flutter/engine@bd165dc...da71729

2024-11-27 skia-flutter-autoroll@skia.org Roll Dart SDK from
72b594775c83 to 0740ded7b9bf (1 revision) (flutter/engine#56826)
2024-11-27 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from
50xtjbMWWrqay_7m_... to u0agumUB4Ag6cTiKk... (flutter/engine#56823)
2024-11-27 flar@google.com [DisplayList] Delete (publicly) unused
DlColorColorSource (flutter/engine#56825)
2024-11-26 jonahwilliams@google.com [engine] reland: more consistently
flush message loops tasks (flutter/engine#56815)
2024-11-26 jonahwilliams@google.com [Impeller] cache even more text
frame data to skip lookups. (flutter/engine#56798)
2024-11-26 flar@google.com [DisplayList] migrate DlColorFilter objects
to new source layout (flutter/engine#56785)
2024-11-26 skia-flutter-autoroll@skia.org Roll Dart SDK from
ca02d403f1a8 to 72b594775c83 (1 revision) (flutter/engine#56819)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from 50xtjbMWWrqa to u0agumUB4Ag6

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter:
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
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…tter/engine#56785)

A new source code/header structure was introduced when the DlColorSource and DlImageFilter objects were migrated to Impeller geometry classes. Even though the DlColorFilter objects did not depend on Skia geometry objects, they need to be updated to the new source layout for consistency.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App e: impeller

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants