tree cff7cba2f306a52fba2c339554fa0242fe341b33
parent 60ce382651f022aba80eb9d8af6d388aa5d7fdc1
author auto-submit[bot] <98614782+auto-submit[bot]@users.noreply.github.com> 1705454297 +0000
committer GitHub <noreply@github.com> 1705454297 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJlpyrZCRC1aQ7uu5UhlAAARmkQADWGS+d9bewDGsIkwo4drIl1
 TZBfISPAWA+4x8oAGvwglxpJMdHho/NOVQmKSwrXdp54SpzlNo3QRbHT1/E9tD+Y
 rxxw7/542JVtPR2XPPTAMS879gQFX/0ubWAoKrC/Zd2CKSkjSD0lGDtV6Lx95s7L
 pnAl8N9tT5lNqVSjb1pbgcAypNY9Syj5abvIuzbHL0lGcrLucdk+pC3aa46bX65z
 xo/SbNn1l3ZcowUx8+Xcn4qK2DWEo5fMQuqfFEDaBGpzLW3TggAWxX0Fe9DVEiD1
 yflC9O8F9pdB/4orQBA1RsjOVxStIbuq4NSgOyxUQ3lZDhUaH5miq4VULhdajlis
 2qC2ZNkzPVjwVE9RoUh962Ytq1OXEp6dXkbD/6ALKx6/6fDAI4zUxR7qMa7OxDBA
 Zp4ECKIc/FFRXHsXmFKQrbcs/cc4YHgITtIwxuEMU7p9/REWrbaqWXgAc5MnbE0s
 7uhwanAwkpjxj8iq/wUCOgf1HuXWV2509EhLtv2XKMEMAG/hHjNJEikwL6dS6GY6
 TRsJylV++nGcJCRscnVfs1FYwDlR4ADipoBfLThosTTT3v0o738k/OYONjLCJ5nJ
 vz8cAxIyCJXByysBYlE8YwaHSmB1FDicg86L/6YZwKxwkkpN8x60mi+SCXlaPzUf
 0HFl1uBuB+K0qNm00Ysa
 =DH8x
 -----END PGP SIGNATURE-----
 

Reverts "[Impeller] Encode directly to command buffer for Vulkan." (#49818)

Reverts flutter/engine#49780
Initiated by: jonahwilliams
This change reverts the following previous change:
Original Description:
Part of https://github.com/flutter/flutter/issues/140804

Rather than using impeller::Command, the impeller::RenderPass records most state directly into the Vulkan command buffer. This should remove allocation/free overhead of the intermediary structures and make further improvements to the backend even easier. This required a number of other changes to the renderer:

1. The render pass holds a strong ptr to the context. This helps avoid locking continually while encoding, which is quite slow.
2. barriers need to be encoded on the _producing_ side, and not the consuming side. This is because we'll actually run the consuming code before the producing code. i.e. we transition to shader read at the end of a render pass instead of when binding.
3. I've updated the binding code to also provide the descriptor type so that we don't need to look it up from the desc. set.
4. I added a test render pass class that records commands.