Explain unrooted_must_root on ServiceWorkerContainer::Register

1 files changed, 4 insertions, 3 deletions

diff --git a/components/script/dom/serviceworkercontainer.rs b/components/script/dom/serviceworkercontainer.rs
index db6cfa896cf..c44d140d88b 100644
--- a/components/script/dom/serviceworkercontainer.rs
+++ b/components/script/dom/serviceworkercontainer.rs
@@ -51,9 +51,9 @@ impl ServiceWorkerContainerMethods for ServiceWorkerContainer {
         self.client.get_controller()
     }
 
-    #[allow(unrooted_must_root)]
-    // https://w3c.github.io/ServiceWorker/#service-worker-container-register-method and - A
-    // https://w3c.github.io/ServiceWorker/#start-register-algorithm - B
+    /// https://w3c.github.io/ServiceWorker/#service-worker-container-register-method and - A
+    /// https://w3c.github.io/ServiceWorker/#start-register-algorithm - B
+    #[allow(unrooted_must_root)] // Job is unrooted
     fn Register(&self, script_url: USVString, options: &RegistrationOptions) -> Rc<Promise> {
         // A: Step 1
         let promise = Promise::new(&*self.global());
@@ -124,6 +124,7 @@ impl ServiceWorkerContainerMethods for ServiceWorkerContainer {
             promise.clone(),
             &*self.client,
         );
+        // Job is unrooted here, do not do anything other than immediately scheduling
         ScriptThread::schedule_job(job);
         promise
     }