Implement Default for JS types (#2626)

* Add Default impls for WebIDL types with zero-argument constructors

* Implement Default for js-sys types where it's appropriate

* Implement Default for JsValue

* Remove some `Default` impls to better match Rust

Author: Jules-Bertholet

crates/js-sys/src/lib.rs

@@ -502,6 +502,12 @@ where
     }
 }
 
+impl Default for Array {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 // ArrayBuffer
 #[wasm_bindgen]
 extern "C" {
@@ -817,6 +823,12 @@ impl fmt::Debug for Boolean {
     }
 }
 
+impl Default for Boolean {
+    fn default() -> Self {
+        Self::from(bool::default())
+    }
+}
+
 // DataView
 #[wasm_bindgen]
 extern "C" {
@@ -1253,6 +1265,12 @@ impl Function {
     }
 }
 
+impl Default for Function {
+    fn default() -> Self {
+        Self::new_no_args("")
+    }
+}
+
 // Generator
 #[wasm_bindgen]
 extern "C" {
@@ -1350,6 +1368,12 @@ extern "C" {
     pub fn size(this: &Map) -> u32;
 }
 
+impl Default for Map {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 // Map Iterator
 #[wasm_bindgen]
 extern "C" {
@@ -1982,6 +2006,12 @@ impl fmt::Debug for Number {
     }
 }
 
+impl Default for Number {
+    fn default() -> Self {
+        Self::from(f64::default())
+    }
+}
+
 // Date.
 #[wasm_bindgen]
 extern "C" {
@@ -2692,6 +2722,12 @@ impl PartialEq for Object {
 
 impl Eq for Object {}
 
+impl Default for Object {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 // Proxy
 #[wasm_bindgen]
 extern "C" {
@@ -3162,6 +3198,12 @@ extern "C" {
     pub fn size(this: &Set) -> u32;
 }
 
+impl Default for Set {
+    fn default() -> Self {
+        Self::new(&JsValue::UNDEFINED)
+    }
+}
+
 // SetIterator
 #[wasm_bindgen]
 extern "C" {
@@ -3294,6 +3336,12 @@ extern "C" {
     pub fn delete(this: &WeakMap, key: &Object) -> bool;
 }
 
+impl Default for WeakMap {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 // WeakSet
 #[wasm_bindgen]
 extern "C" {
@@ -3328,6 +3376,12 @@ extern "C" {
     pub fn delete(this: &WeakSet, value: &Object) -> bool;
 }
 
+impl Default for WeakSet {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 #[allow(non_snake_case)]
 pub mod WebAssembly {
     use super::*;
@@ -4502,6 +4556,15 @@ pub mod Intl {
         pub fn supported_locales_of(locales: &Array, options: &Object) -> Array;
     }
 
+    impl Default for Collator {
+        fn default() -> Self {
+            Self::new(
+                &JsValue::UNDEFINED.unchecked_into(),
+                &JsValue::UNDEFINED.unchecked_into(),
+            )
+        }
+    }
+
     // Intl.DateTimeFormat
     #[wasm_bindgen]
     extern "C" {
@@ -4553,6 +4616,15 @@ pub mod Intl {
         pub fn supported_locales_of(locales: &Array, options: &Object) -> Array;
     }
 
+    impl Default for DateTimeFormat {
+        fn default() -> Self {
+            Self::new(
+                &JsValue::UNDEFINED.unchecked_into(),
+                &JsValue::UNDEFINED.unchecked_into(),
+            )
+        }
+    }
+
     // Intl.NumberFormat
     #[wasm_bindgen]
     extern "C" {
@@ -4603,6 +4675,15 @@ pub mod Intl {
         pub fn supported_locales_of(locales: &Array, options: &Object) -> Array;
     }
 
+    impl Default for NumberFormat {
+        fn default() -> Self {
+            Self::new(
+                &JsValue::UNDEFINED.unchecked_into(),
+                &JsValue::UNDEFINED.unchecked_into(),
+            )
+        }
+    }
+
     // Intl.PluralRules
     #[wasm_bindgen]
     extern "C" {
@@ -4644,6 +4725,15 @@ pub mod Intl {
         #[wasm_bindgen(static_method_of = PluralRules, js_namespace = Intl, js_name = supportedLocalesOf)]
         pub fn supported_locales_of(locales: &Array, options: &Object) -> Array;
     }
+
+    impl Default for PluralRules {
+        fn default() -> Self {
+            Self::new(
+                &JsValue::UNDEFINED.unchecked_into(),
+                &JsValue::UNDEFINED.unchecked_into(),
+            )
+        }
+    }
 }
 
 // Promise
@@ -5055,6 +5145,12 @@ macro_rules! arrays {
                 unsafe { $name::new(&$name::view(slice)) }
             }
         }
+
+        impl Default for $name {
+            fn default() -> Self {
+                Self::new(&JsValue::UNDEFINED.unchecked_into())
+            }
+        }
     )*);
 }
 

crates/web-sys/src/features/gen_AddEventListenerOptions.rs

@@ -67,3 +67,8 @@ impl AddEventListenerOptions {
         self
     }
 }
+impl Default for AddEventListenerOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AnalyserOptions.rs

@@ -141,3 +141,8 @@ impl AnalyserOptions {
         self
     }
 }
+impl Default for AnalyserOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AnimationEventInit.rs

@@ -122,3 +122,8 @@ impl AnimationEventInit {
         self
     }
 }
+impl Default for AnimationEventInit {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AnimationPlaybackEventInit.rs

@@ -105,3 +105,8 @@ impl AnimationPlaybackEventInit {
         self
     }
 }
+impl Default for AnimationPlaybackEventInit {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AssignedNodesOptions.rs

@@ -37,3 +37,8 @@ impl AssignedNodesOptions {
         self
     }
 }
+impl Default for AssignedNodesOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AudioBufferSourceOptions.rs

@@ -113,3 +113,8 @@ impl AudioBufferSourceOptions {
         self
     }
 }
+impl Default for AudioBufferSourceOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AudioConfiguration.rs

@@ -88,3 +88,8 @@ impl AudioConfiguration {
         self
     }
 }
+impl Default for AudioConfiguration {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AudioContextOptions.rs

@@ -37,3 +37,8 @@ impl AudioContextOptions {
         self
     }
 }
+impl Default for AudioContextOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AudioNodeOptions.rs

@@ -73,3 +73,8 @@ impl AudioNodeOptions {
         self
     }
 }
+impl Default for AudioNodeOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AudioWorkletNodeOptions.rs

@@ -141,3 +141,8 @@ impl AudioWorkletNodeOptions {
         self
     }
 }
+impl Default for AudioWorkletNodeOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AuthenticationExtensionsClientInputs.rs

@@ -33,3 +33,8 @@ impl AuthenticationExtensionsClientInputs {
         self
     }
 }
+impl Default for AuthenticationExtensionsClientInputs {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AuthenticationExtensionsClientOutputs.rs

@@ -33,3 +33,8 @@ impl AuthenticationExtensionsClientOutputs {
         self
     }
 }
+impl Default for AuthenticationExtensionsClientOutputs {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AuthenticatorSelectionCriteria.rs

@@ -73,3 +73,8 @@ impl AuthenticatorSelectionCriteria {
         self
     }
 }
+impl Default for AuthenticatorSelectionCriteria {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_AutocompleteInfo.rs

@@ -88,3 +88,8 @@ impl AutocompleteInfo {
         self
     }
 }
+impl Default for AutocompleteInfo {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BaseComputedKeyframe.rs

@@ -100,3 +100,8 @@ impl BaseComputedKeyframe {
         self
     }
 }
+impl Default for BaseComputedKeyframe {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BaseKeyframe.rs

@@ -83,3 +83,8 @@ impl BaseKeyframe {
         self
     }
 }
+impl Default for BaseKeyframe {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BasePropertyIndexedKeyframe.rs

@@ -65,3 +65,8 @@ impl BasePropertyIndexedKeyframe {
         self
     }
 }
+impl Default for BasePropertyIndexedKeyframe {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BasicCardRequest.rs

@@ -54,3 +54,8 @@ impl BasicCardRequest {
         self
     }
 }
+impl Default for BasicCardRequest {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BiquadFilterOptions.rs

@@ -144,3 +144,8 @@ impl BiquadFilterOptions {
         self
     }
 }
+impl Default for BiquadFilterOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BlobEventInit.rs

@@ -85,3 +85,8 @@ impl BlobEventInit {
         self
     }
 }
+impl Default for BlobEventInit {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BlobPropertyBag.rs

@@ -51,3 +51,8 @@ impl BlobPropertyBag {
         self
     }
 }
+impl Default for BlobPropertyBag {
+    fn default() -> Self {
+        Self::new()
+    }
+}

crates/web-sys/src/features/gen_BlockParsingOptions.rs

@@ -37,3 +37,8 @@ impl BlockParsingOptions {
         self
     }
 }
+impl Default for BlockParsingOptions {
+    fn default() -> Self {
+        Self::new()
+    }
+}