refactor(core): don't use Result in ExtensionBuilder::state (#18066)

There's no point for this API to expect result. If something fails it should
result in a panic during build time to signal to embedder that setup is
wrong.

2 files changed, 0 insertions, 2 deletions

diff --git a/ext/net/lib.rs b/ext/net/lib.rs
index 0536f3233..5dc7823bc 100644
--- a/ext/net/lib.rs
+++ b/ext/net/lib.rs
@@ -96,7 +96,6 @@ pub fn init<P: NetPermissions + 'static>(
       state.put(UnsafelyIgnoreCertificateErrors(
         unsafely_ignore_certificate_errors.clone(),
       ));
-      Ok(())
     })
     .build()
 }
diff --git a/ext/net/ops.rs b/ext/net/ops.rs
index 737a46d8c..8ac08119a 100644
--- a/ext/net/ops.rs
+++ b/ext/net/ops.rs
@@ -910,7 +910,6 @@ mod tests {
       .state(move |state| {
         state.put(TestPermission {});
         state.put(UnstableChecker { unstable: true });
-        Ok(())
       })
       .build();