Fix potential null-point references

Author: iskakaushik

impeller/renderer/backend/vulkan/command_buffer_vk.cc

@@ -129,7 +129,8 @@ std::shared_ptr<RenderPass> CommandBufferVK::OnCreateRenderPass(
 }
 
 std::shared_ptr<BlitPass> CommandBufferVK::OnCreateBlitPass() const {
-  FML_UNREACHABLE();
+  // TODO(kaushikiska): https://github.com/flutter/flutter/issues/112649
+  return nullptr;
 }
 
 std::shared_ptr<ComputePass> CommandBufferVK::OnCreateComputePass() const {

lib/ui/painting/image_decoder_impeller.cc

@@ -15,6 +15,7 @@
 #include "flutter/impeller/renderer/context.h"
 #include "flutter/impeller/renderer/texture.h"
 #include "flutter/lib/ui/painting/image_decoder_skia.h"
+#include "fml/logging.h"
 #include "impeller/base/strings.h"
 #include "impeller/geometry/size.h"
 #include "include/core/SkSize.h"
@@ -179,20 +180,19 @@ sk_sp<DlImage> ImageDecoderImpeller::UploadTexture(
 
   {
     auto command_buffer = context->CreateCommandBuffer();
-    command_buffer->SetLabel("Mipmap Command Buffer");
     if (!command_buffer) {
       FML_DLOG(ERROR)
           << "Could not create command buffer for mipmap generation.";
       return nullptr;
     }
+    command_buffer->SetLabel("Mipmap Command Buffer");
 
     auto blit_pass = command_buffer->CreateBlitPass();
-    blit_pass->SetLabel("Mipmap Blit Pass");
     if (!blit_pass) {
       FML_DLOG(ERROR) << "Could not create blit pass for mipmap generation.";
       return nullptr;
     }
-
+    blit_pass->SetLabel("Mipmap Blit Pass");
     blit_pass->GenerateMipmap(texture);
 
     blit_pass->EncodeCommands(context->GetResourceAllocator());
@@ -233,6 +233,7 @@ void ImageDecoderImpeller::Decode(fml::RefPtr<ImageDescriptor> descriptor,
        io_runner = runners_.GetIOTaskRunner(),                    //
        result                                                     //
   ]() {
+        FML_CHECK(context) << "No valid impeller context";
         auto max_size_supported =
             context->GetResourceAllocator()->GetMaxTextureSizeSupported();
 

shell/platform/android/android_surface_vulkan_impeller.cc

@@ -116,4 +116,8 @@ bool AndroidSurfaceVulkanImpeller::SetNativeWindow(
   return false;
 }
 
+std::shared_ptr<impeller::Context> AndroidSurfaceVulkanImpeller::GetImpellerContext() {
+  return impeller_context_;
+}
+
 }  // namespace flutter

shell/platform/android/android_surface_vulkan_impeller.h

@@ -40,6 +40,9 @@ class AndroidSurfaceVulkanImpeller : public AndroidSurface {
   // |AndroidSurface|
   bool ResourceContextClearCurrent() override;
 
+  // |AndroidSurface|
+  std::shared_ptr<impeller::Context> GetImpellerContext() override;
+
   // |AndroidSurface|
   bool SetNativeWindow(fml::RefPtr<AndroidNativeWindow> window) override;