Fix QuerySet ownership of ComputePass

[Wumpf]

Connections

• Follow-up & based on Remove lifetime dependency of ComputePass to its parent command encoder #5620
• Puzzle piece of Lifetimes on RenderPass make it difficult to use.  #1453

Part of a series towards lifetime removal on compute passes:

• Remove exposed C symbols from renderpass/computepass recording #5409
• Separate out ComputeCommand id->arc resolve (a step towards no lifetimes on wgpu::ComputePass) #5432
• Remove lifetime constraints from wgpu::ComputePass methods #5570
• Make compute pass end consume the pass #5575
• Remove lifetime dependency of ComputePass to its parent command encoder #5620
• ➡️ Fix QuerySet ownership of ComputePass #5671

last piece of solving lifetime constraints for ComputePass

Description
Technically this PR is a bugfix: Previously, when destroying a QuerySet that is used in any way (either write_timestamp, ComputePassTimestampWrites or begin_pipeline_statistics_query) before executing the pass, we would crash.
This solves this as in the PRs before by taking ownership of the query the moment it is provided.

Testing
Adds two new tests that test for gracefully handling destroyed query set before the compute pass is executed.
Both tests failed before the fix.

Checklist

• Run cargo fmt.
• Run cargo clippy. If applicable, add:
  • --target wasm32-unknown-unknown
  • --target wasm32-unknown-emscripten
• Run cargo xtask test to run tests.
• Add change to CHANGELOG.md. See simple instructions inside file.

[ErichDonGubler]

praise: This was easier to audit than I thought it would be, based on the LoC. 😀 Whew! Sorry for the delay in getting to this.