Update present_with_transaction

[xiaopengli89]

Checklist

• Run cargo clippy.
• Run RUSTFLAGS=--cfg=web_sys_unstable_apis cargo clippy --target wasm32-unknown-unknown if applicable.
• Add change to CHANGELOG.md. See simple instructions inside file.

Description
Set present_with_transaction according to PresentMode.

[codecov-commenter]

Codecov Report

Merging #3111 (c24d237) into master (ce08179) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3111   +/-   ##
=======================================
  Coverage   64.75%   64.75%           
=======================================
  Files          81       81           
  Lines       38711    38711           
=======================================
+ Hits        25066    25069    +3     
+ Misses      13645    13642    -3     

Impacted Files	Coverage Δ
wgpu-hal/src/dx12/descriptor.rs	87.56% <0.00%> (-0.50%)	⬇️
wgpu-hal/src/gles/device.rs	80.12% <0.00%> (+0.42%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[cwfitzgerald]

I don't think this is what we actually want. Presenting with transaction is a much more intense form of synchronization which synchronizes with CoreAnimation.

[xiaopengli89]

Is there currently a way to configure wgpu to use CoreAnimation? Because I found that there is a memory leak problem here:

wgpu/wgpu-hal/src/metal/mod.rs

Line 381 in ce08179

command_buffer.present_drawable(&texture.drawable);

it is likely to be a metal driver's bug. When there is no better solution, I hope I can somehow avoid using this api.

[grovesNL]

@xiaopengli89 does this leak happen in regular Metal Objective-C or Swift examples? It would be great to verify if this was a actually a Metal bug or if we're accidentally holding an extra reference somewhere (i.e., we have an auto release pool missing somewhere other than present_drawable)