Return Result from ProtocolRegistry::register() (#36688)

Instead of returning true / false it's better to return a Result (even
if we continue ignoring possible error).

Testing: Doesn't change any behavior

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>

components/net/protocols/mod.rs

@@ -54,27 +54,45 @@ pub struct ProtocolRegistry {
     pub(crate) handlers: HashMap<String, Box<dyn ProtocolHandler>>, // Maps scheme -> handler
 }
 
+#[derive(Clone, Copy, Debug)]
+pub enum ProtocolRegisterError {
+    ForbiddenScheme,
+    SchemeAlreadyRegistered,
+}
+
 impl ProtocolRegistry {
     pub fn with_internal_protocols() -> Self {
         let mut registry = Self::default();
-        registry.register("data", DataProtocolHander::default());
+        // We just created a new registry, and know that we aren't using
-        registry.register("blob", BlobProtocolHander::default());
+        // any forbidden schemes, so this should never panic.
-        registry.register("file", FileProtocolHander::default());
+        registry
+            .register("data", DataProtocolHander::default())
+            .expect("Infallible");
+        registry
+            .register("blob", BlobProtocolHander::default())
+            .expect("Infallible");
+        registry
+            .register("file", FileProtocolHander::default())
+            .expect("Infallible");
         registry
     }
 
-    pub fn register(&mut self, scheme: &str, handler: impl ProtocolHandler + 'static) -> bool {
+    pub fn register(
+        &mut self,
+        scheme: &str,
+        handler: impl ProtocolHandler + 'static,
+    ) -> Result<(), ProtocolRegisterError> {
         if FORBIDDEN_SCHEMES.contains(&scheme) {
             error!("Protocol handler for '{scheme}' is not allowed to be registered.");
-            return false;
+            return Err(ProtocolRegisterError::ForbiddenScheme);
         }
 
         if let Entry::Vacant(entry) = self.handlers.entry(scheme.into()) {
             entry.insert(Box::new(handler));
-            true
+            Ok(())
         } else {
             error!("Protocol handler for '{scheme}' is already registered.");
-            false
+            Err(ProtocolRegisterError::SchemeAlreadyRegistered)
         }
     }
 

ports/servoshell/desktop/app.rs

@@ -122,12 +122,13 @@ impl App {
         }
 
         let mut protocol_registry = ProtocolRegistry::default();
-        protocol_registry.register(
+        let _ = protocol_registry.register(
             "urlinfo",
             protocols::urlinfo::UrlInfoProtocolHander::default(),
         );
-        protocol_registry.register("servo", protocols::servo::ServoProtocolHandler::default());
+        let _ =
-        protocol_registry.register(
+            protocol_registry.register("servo", protocols::servo::ServoProtocolHandler::default());
+        let _ = protocol_registry.register(
             "resource",
             protocols::resource::ResourceProtocolHandler::default(),
         );