[Impeller] Remove transient usage from sampled and stored images

fixes: flutter/flutter#121633

Also works around flutter/flutter#121634

Author: iskakaushik

impeller/renderer/backend/vulkan/allocator_vk.cc

@@ -132,12 +132,27 @@ static constexpr vk::ImageUsageFlags ToVKImageUsageFlags(PixelFormat format,
 
   if (usage & static_cast<TextureUsageMask>(TextureUsage::kShaderRead)) {
     vk_usage |= vk::ImageUsageFlagBits::eSampled;
+    // unmark the usage as transient if it is used as a sampled image
+    // See: https://github.com/flutter/flutter/issues/121633
+    if (mode == StorageMode::kDeviceTransient) {
+      vk_usage &= ~vk::ImageUsageFlagBits::eTransientAttachment;
+    }
   }
 
   if (usage & static_cast<TextureUsageMask>(TextureUsage::kShaderWrite)) {
     vk_usage |= vk::ImageUsageFlagBits::eStorage;
+    // unmark the usage as transient if it is used as a storage image
+    // See: https://github.com/flutter/flutter/issues/121633
+    if (mode == StorageMode::kDeviceTransient) {
+      vk_usage &= ~vk::ImageUsageFlagBits::eTransientAttachment;
+    }
   }
 
+  // TODO (https://github.com/flutter/flutter/issues/121634):
+  // Add transfer usage flags to support blit passes
+  vk_usage |= vk::ImageUsageFlagBits::eTransferSrc |
+              vk::ImageUsageFlagBits::eTransferDst;
+
   return vk_usage;
 }
 

impeller/renderer/backend/vulkan/swapchain_impl_vk.cc

@@ -90,6 +90,13 @@ static std::optional<vk::CompositeAlphaFlagBitsKHR> ChooseAlphaCompositionMode(
   if (flags & vk::CompositeAlphaFlagBitsKHR::eOpaque) {
     return vk::CompositeAlphaFlagBitsKHR::eOpaque;
   }
+  if (flags & vk::CompositeAlphaFlagBitsKHR::eInherit) {
+    return vk::CompositeAlphaFlagBitsKHR::eInherit;
+  }
+  if (flags & vk::CompositeAlphaFlagBitsKHR::ePostMultiplied) {
+    return vk::CompositeAlphaFlagBitsKHR::ePostMultiplied;
+  }
+
   return std::nullopt;
 }