Bug 1432531 - Move FlushAsyncPaints so it is unconditionally run in CompositorBridgeChild::Destroy. r=nical

--HG--
extra : rebase_source : a6d6739d363ee144b8c721bccf62bcb139339880

gfx/layers/ipc/CompositorBridgeChild.cpp

@@ -156,6 +156,9 @@ CompositorBridgeChild::Destroy()
     mLayerManager = nullptr;
   }
 
+  // Flush async paints before we destroy texture data.
+  FlushAsyncPaints();
+
   if (!mCanSend) {
     // We may have already called destroy but still have lingering references
     // or CompositorBridgeChild::ActorDestroy was called. Ensure that we do our
@@ -182,9 +185,6 @@ CompositorBridgeChild::Destroy()
     wrBridge->Destroy(/* aIsSync */ false);
   }
 
-  // Flush async paints before we destroy texture data.
-  FlushAsyncPaints();
-
   const ManagedContainer<PTextureChild>& textures = ManagedPTextureChild();
   for (auto iter = textures.ConstIter(); !iter.Done(); iter.Next()) {
     RefPtr<TextureClient> texture = TextureClient::AsTextureClient(iter.Get()->GetKey());