[Impeller Scene] Add ColorSourceContents for drawing a node by bdero · Pull Request #38485 · flutter-team-archive/engine

bdero · 2022-12-23T18:04:38Z

This adds a ColorSourceContents which renders an Impeller Scene to a texture and blits it to the current layer respecting the geometry, blend mode, and clip stack.

I also added commented code in the display list dispatcher that can be uncommented once the DlSceneColorSource has been implemented.

chinmaygarde · 2022-12-23T19:36:50Z

+  scene.Render(pass.GetRenderTarget(), camera_transform_);
+
+  // Render the texture to the pass.
+  TiledTextureContents final;


I thought final was a keyword. Odd that this works.

Hah, apparently it's fine. I ended up renaming this since it was a weird spur of the moment airport name.

bdero · 2022-12-24T19:17:23Z

Added a Aiks playground and fixed some bugs. We've still got some corruption going on not being caught by Metal validation that I need to work out. My guess is attachment load/store ops.

Screen.Recording.2022-12-24.at.11.00.36.AM.mov

chinmaygarde · 2022-12-24T19:21:05Z

The load/store ops being off seems like a good guess. You could patch ToMTLStoreAction and the ToMTLLoadAction in formats_mtl to always load and store the dont-care cases to see if that fixes your issue. Then you know for sure your hunch is correct.

bdero · 2022-12-24T19:25:02Z

The fact that it was mostly fuchsia was throwing me off. Just forgot to initialize/clear the color attachment on load. :)

Screen.Recording.2022-12-24.at.11.22.16.AM.mov

bdero · 2022-12-28T21:15:06Z

~~This PR is failing to run only on the simulator. I suspect I'm not packaging or loading the scene shaders correctly.~~ I forgot to include the scene shaders when the Impeller contexts are being initialized for the ios/android embedder surfaces.

2022-12-26 09:20:57.656745-0800 Scenarios[48446:293119] [VERBOSE-2:FlutterDarwinContextMetalImpeller.mm(29)] Using the Impeller rendering backend.
2022-12-26 09:20:57.658137-0800 Scenarios[48446:293203] [VERBOSE-3:validation.cc(31)] Could not resolve pipeline entrypoint(s) 'unskinned_vertex_main' and 'unlit_fragment_main' for pipeline named 'Unskinned'.

…lutter-team-archive/engine#38485)

…ng a node (flutter-team-archive/engine#38485) (#117769) Commit: aa70994f60d2c0e33b0a82364ba87d0ac95f9fad

…lutter-team-archive/engine#38485) (#117769)

…g a node (flutter-team-archive/engine#38485) (flutter/flutter#117769)

* 9bb4ffe Roll Flutter Engine from 9e63c1ae1480 to 190f743a8506 (2 revisions) (flutter/flutter#117744) * babeb61 cc8055d5f Roll Skia from 4b578d72dc2e to e4c86c2fed9a (1 revision) (flutter-team-archive/engine#38535) (flutter/flutter#117745) * 9afaf6b f3cc581d9 Roll Dart SDK from 6e07d9b025bd to f7b36d5e50a5 (1 revision) (flutter-team-archive/engine#38536) (flutter/flutter#117749) * 0e83ada Update M3 IconButton unselected focused opacity (flutter/flutter#117321) * 2783d31 Roll Flutter Engine from f3cc581d97e1 to 3655bf981d4c (2 revisions) (flutter/flutter#117751) * 4a6ab96 Roll Flutter Engine from 3655bf981d4c to 34f75ed27c9b (3 revisions) (flutter/flutter#117760) * 2ffa65c e9e79180e Roll Skia from 2923399853d9 to 0027eb334691 (6 revisions) (flutter-team-archive/engine#38545) (flutter/flutter#117763) * cee9ec5 606e77f35 Roll Skia from 0027eb334691 to 668260c85e9d (1 revision) (flutter-team-archive/engine#38547) (flutter/flutter#117766) * c089c19 Revert "[reland] Add Material 3 support for `ListTile` - Part 1 (#116963)" (flutter/flutter#117756) * bdefebc 9b534a5f8 Roll Skia from 668260c85e9d to 25ffa2b757e9 (1 revision) (flutter-team-archive/engine#38548) (flutter/flutter#117768) * aa70994 8655ec022 [Impeller Scene] Add ColorSourceContents for drawing a node (flutter-team-archive/engine#38485) (flutter/flutter#117769) * cbc184d Roll Flutter Engine from 8655ec022fe1 to ecd47e0b256a (2 revisions) (flutter/flutter#117772) * 0b69719 Roll Flutter Engine from ecd47e0b256a to 45e689b15d1a (2 revisions) (flutter/flutter#117778) * d2127ad e44a0de4c Roll Fuchsia Mac SDK from JLTTlcNPJeScjSO2B... to FeFYsNPy64-PEXPer... (flutter-team-archive/engine#38558) (flutter/flutter#117779)

…team-archive#38485)

…lutter-team-archive/engine#38485) (flutter#117769)

…hive#6892) * 9bb4ffe Roll Flutter Engine from 9e63c1ae1480 to 190f743a8506 (2 revisions) (flutter/flutter#117744) * babeb61 cc8055d5f Roll Skia from 4b578d72dc2e to e4c86c2fed9a (1 revision) (flutter-team-archive/engine#38535) (flutter/flutter#117745) * 9afaf6b f3cc581d9 Roll Dart SDK from 6e07d9b025bd to f7b36d5e50a5 (1 revision) (flutter-team-archive/engine#38536) (flutter/flutter#117749) * 0e83ada Update M3 IconButton unselected focused opacity (flutter/flutter#117321) * 2783d31 Roll Flutter Engine from f3cc581d97e1 to 3655bf981d4c (2 revisions) (flutter/flutter#117751) * 4a6ab96 Roll Flutter Engine from 3655bf981d4c to 34f75ed27c9b (3 revisions) (flutter/flutter#117760) * 2ffa65c e9e79180e Roll Skia from 2923399853d9 to 0027eb334691 (6 revisions) (flutter-team-archive/engine#38545) (flutter/flutter#117763) * cee9ec5 606e77f35 Roll Skia from 0027eb334691 to 668260c85e9d (1 revision) (flutter-team-archive/engine#38547) (flutter/flutter#117766) * c089c19 Revert "[reland] Add Material 3 support for `ListTile` - Part 1 (#116963)" (flutter/flutter#117756) * bdefebc 9b534a5f8 Roll Skia from 668260c85e9d to 25ffa2b757e9 (1 revision) (flutter-team-archive/engine#38548) (flutter/flutter#117768) * aa70994 8655ec022 [Impeller Scene] Add ColorSourceContents for drawing a node (flutter-team-archive/engine#38485) (flutter/flutter#117769) * cbc184d Roll Flutter Engine from 8655ec022fe1 to ecd47e0b256a (2 revisions) (flutter/flutter#117772) * 0b69719 Roll Flutter Engine from ecd47e0b256a to 45e689b15d1a (2 revisions) (flutter/flutter#117778) * d2127ad e44a0de4c Roll Fuchsia Mac SDK from JLTTlcNPJeScjSO2B... to FeFYsNPy64-PEXPer... (flutter-team-archive/engine#38558) (flutter/flutter#117779)

bdero added the e: impeller label Dec 23, 2022

bdero requested review from chinmaygarde and zanderso December 23, 2022 18:04

bdero self-assigned this Dec 23, 2022

chinmaygarde approved these changes Dec 23, 2022

View reviewed changes

bdero force-pushed the bdero/scene-contents branch 3 times, most recently from 0064e85 to 7595d3c Compare December 25, 2022 12:44

bdero added 3 commits December 25, 2022 18:00

[Impeller Scene] Add ColorSourceContents for drawing a node

7900cb9

Add Aiks test, fix bugs

4cc1b00

Clear attachments

0a31d25

bdero force-pushed the bdero/scene-contents branch from 7595d3c to 0a31d25 Compare December 26, 2022 02:00

Add scene shaders to impeller surfaces

350be55

bdero merged commit 8655ec0 into flutter-team-archive:main Dec 29, 2022

engine-flutter-autoroll mentioned this pull request Dec 29, 2022

Roll Flutter Engine from 9b534a5f8888 to 8655ec022fe1 (1 revision) flutter/flutter#117769

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 29, 2022

8655ec0 [Impeller Scene] Add ColorSourceContents for drawing a node (f…

6927e7a

…lutter-team-archive/engine#38485)

sourcegraph-bot pushed a commit to sgtest/megarepo that referenced this pull request Dec 29, 2022

flutter: 8655ec022 [Impeller Scene] Add ColorSourceContents for drawi…

62d167b

…ng a node (flutter-team-archive/engine#38485) (#117769) Commit: aa70994f60d2c0e33b0a82364ba87d0ac95f9fad

auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Dec 29, 2022

8655ec0 [Impeller Scene] Add ColorSourceContents for drawing a node (f…

aa70994

…lutter-team-archive/engine#38485) (#117769)

This was referenced Dec 29, 2022

Roll Flutter from 17482fd425ee to d2127ad344e8 (14 revisions) flutter/packages#3009

Merged

Roll Flutter from 17482fd425ee to d2127ad344e8 (14 revisions) flutter-team-archive/plugins#6892

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 29, 2022

aa70994 8655ec022 [Impeller Scene] Add ColorSourceContents for drawin…

408e50f

…g a node (flutter-team-archive/engine#38485) (flutter/flutter#117769)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Dec 29, 2022

aa70994 8655ec022 [Impeller Scene] Add ColorSourceContents for drawin…

3e96c67

…g a node (flutter-team-archive/engine#38485) (flutter/flutter#117769)

loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Jan 3, 2023

[Impeller Scene] Add ColorSourceContents for drawing a node (flutter-…

a22f043

…team-archive#38485)

loic-sharma pushed a commit to fluttergithubbot/flutter that referenced this pull request Jan 6, 2023

8655ec0 [Impeller Scene] Add ColorSourceContents for drawing a node (f…

1c75288

…lutter-team-archive/engine#38485) (flutter#117769)

gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023

8655ec0 [Impeller Scene] Add ColorSourceContents for drawing a node (f…

36a522e

…lutter-team-archive/engine#38485) (flutter#117769)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Impeller Scene] Add ColorSourceContents for drawing a node#38485

[Impeller Scene] Add ColorSourceContents for drawing a node#38485
bdero merged 4 commits into
flutter-team-archive:mainfrom
bdero:bdero/scene-contents

bdero commented Dec 23, 2022

Uh oh!

chinmaygarde Dec 23, 2022

Uh oh!

bdero Dec 24, 2022

Uh oh!

bdero commented Dec 24, 2022

Uh oh!

chinmaygarde commented Dec 24, 2022

Uh oh!

bdero commented Dec 24, 2022

Uh oh!

bdero commented Dec 28, 2022 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

bdero commented Dec 23, 2022

Uh oh!

chinmaygarde Dec 23, 2022

Choose a reason for hiding this comment

Uh oh!

bdero Dec 24, 2022

Choose a reason for hiding this comment

Uh oh!

bdero commented Dec 24, 2022

Uh oh!

chinmaygarde commented Dec 24, 2022

Uh oh!

bdero commented Dec 24, 2022

Uh oh!

bdero commented Dec 28, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

bdero commented Dec 28, 2022 •

edited

Loading