Update wgpu-hal to 0.16.1 to fix mobile Safari (#2296)

The release contains gfx-rs/wgpu#3780

Closes #2032

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)

<!-- This line will get updated when the PR build summary job finishes.
-->
PR Build Summary: https://build.rerun.io/pr/2296

<!-- pr-link-docs:start -->
Docs preview: https://rerun.io/preview/9ebf96a/docs
<!-- pr-link-docs:end -->

crates/re_build_web_viewer/Cargo.toml

@@ -17,6 +17,8 @@ all-features = true
 
 
 [dependencies]
+re_error.workspace = true
+
 anyhow.workspace = true
 cargo_metadata = "0.15"
 wasm-bindgen-cli-support = "0.2.86"

crates/re_build_web_viewer/src/lib.rs

@@ -108,13 +108,27 @@ pub fn build(release: bool, webgpu: bool) -> anyhow::Result<()> {
  .join(format!("{crate_name}.wasm"));
 
  // wasm-bindgen --target web target_wasm_path --no-typescript --out-name target_name --out-dir build_dir
- wasm_bindgen_cli_support::Bindgen::new()
+ if let Err(err) = wasm_bindgen_cli_support::Bindgen::new()
  .no_modules(true)?
  .input_path(target_wasm_path.as_str())
  .typescript(false)
  .out_name(target_name.as_str())
  .generate(build_dir.as_str())
- .context("Failed to run wasm-bindgen")?;
+ {
+ if err
+ .to_string()
+ .starts_with("cannot import from modules (`env`")
+ {
+ // Very common error: "cannot import from modules (`env`) with `--no-modules`"
+ anyhow::bail!(
+ "Failed to run wasm-bindgen: {err}. This is often because some dependency is calling `std::time::Instant::now()` or similar. You can try diagnosing this with:\n\
+ wasm2wat {target_wasm_path} | rg '\"env\"'\n\
+ wasm2wat {target_wasm_path} | rg 'call .now\\b' -B 20"
+ );
+ } else {
+ return Err(err.context("Failed to run wasm-bindgen"));
+ }
+ }
 
  // --------------------------------------------------------------------------------
 

crates/re_build_web_viewer/src/main.rs

@@ -34,7 +34,7 @@ fn main() -> ExitCode {
  };
 
  if let Err(err) = re_build_web_viewer::build(release, webgpu) {
- eprintln!("Failed to build web viewer: {err}");
+ eprintln!("Failed to build web viewer: {}", re_error::format(err));
  ExitCode::FAILURE
  } else {
  ExitCode::SUCCESS

crates/re_error/src/lib.rs

@@ -2,7 +2,7 @@
 
 /// Format an error, including its chain of sources.
 ///
-/// Always use this when displaying an error.
+/// Always use this when displaying an error, especially `anyhow::Error`.
 pub fn format(error: impl AsRef<dyn std::error::Error>) -> String {
  fn format_impl(error: &dyn std::error::Error) -> String {
  let mut string = error.to_string();

crates/re_renderer/Cargo.toml

@@ -67,6 +67,7 @@ static_assertions = "1.1"
 thiserror.workspace = true
 type-map = "0.5"
 wgpu.workspace = true
+wgpu-hal = "0.16.1" # wgpu-hal 0.16.1 contains a critical fix for mobile devices: https://github.com/gfx-rs/wgpu/pull/3780
 
 # optional
 arrow2 = { workspace = true, optional = true }

crates/re_web_viewer_server/Cargo.toml

@@ -66,3 +66,4 @@ re_analytics = { workspace = true, optional = true }
 [build-dependencies]
 re_build_build_info.workspace = true
 re_build_web_viewer.workspace = true
+re_error.workspace = true

crates/re_web_viewer_server/build.rs

@@ -45,6 +45,10 @@ fn main() {
  eprintln!("__ci feature detected: Skipping building of web viewer wasm.");
  } else {
  let release = std::env::var("PROFILE").unwrap() == "release";
- re_build_web_viewer::build(release, is_tracked_env_var_set("RERUN_BUILD_WEBGPU")).unwrap();
+ if let Err(err) =
+ re_build_web_viewer::build(release, is_tracked_env_var_set("RERUN_BUILD_WEBGPU"))
+ {
+ panic!("Failed to build web viewer: {}", re_error::format(err));
+ }
  }
 }