Lazily create the backingstore_producer vulkan context

bdero · bdero · commit eedf60bd5b66 · 2022-02-01T23:23:26.000-08:00
diff --git a/shell/platform/embedder/tests/embedder_test_backingstore_producer.cc b/shell/platform/embedder/tests/embedder_test_backingstore_producer.cc
@@ -25,7 +25,7 @@ EmbedderTestBackingStoreProducer::EmbedderTestBackingStoreProducer(
 #endif
 #ifdef SHELL_ENABLE_VULKAN
       ,
-      test_vulkan_context_(fml::MakeRefCounted<TestVulkanContext>())
+      test_vulkan_context_(nullptr)
 #endif
 {
 }
@@ -241,6 +241,10 @@ bool EmbedderTestBackingStoreProducer::CreateVulkanImage(
     const FlutterBackingStoreConfig* config,
     FlutterBackingStore* backing_store_out) {
 #ifdef SHELL_ENABLE_VULKAN
+  if (!test_vulkan_context_) {
+    test_vulkan_context_ = fml::MakeRefCounted<TestVulkanContext>();
+  }
+
   auto surface_size = SkISize::Make(config->size.width, config->size.height);
   TestVulkanImage* test_image = new TestVulkanImage(
       std::move(test_vulkan_context_->CreateImage(surface_size).value()));

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ EmbedderTestBackingStoreProducer::EmbedderTestBackingStoreProducer(`
`25`	`25`	`#endif`
`26`	`26`	`#ifdef SHELL_ENABLE_VULKAN`
`27`	`27`	`,`
`28`		`- test_vulkan_context_(fml::MakeRefCounted<TestVulkanContext>())`
	`28`	`+ test_vulkan_context_(nullptr)`
`29`	`29`	`#endif`
`30`	`30`	`{`
`31`	`31`	`}`
`@@ -241,6 +241,10 @@ bool EmbedderTestBackingStoreProducer::CreateVulkanImage(`
`241`	`241`	`const FlutterBackingStoreConfig* config,`
`242`	`242`	`FlutterBackingStore* backing_store_out) {`
`243`	`243`	`#ifdef SHELL_ENABLE_VULKAN`
	`244`	`+ if (!test_vulkan_context_) {`
	`245`	`+ test_vulkan_context_ = fml::MakeRefCounted<TestVulkanContext>();`
	`246`	`+ }`
	`247`	`+`
`244`	`248`	`auto surface_size = SkISize::Make(config->size.width, config->size.height);`
`245`	`249`	`TestVulkanImage* test_image = new TestVulkanImage(`
`246`	`250`	`std::move(test_vulkan_context_->CreateImage(surface_size).value()));`