script: Limit public exports. (#34915)

* script: Restrict reexport visibility of DOM types.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* script: Mass pub->pub(crate) conversion.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* script: Hide existing dead code warnings.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Formatting.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Fix clippy warnings.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Formatting.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Fix unit tests.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Fix clippy.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* More formatting.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

---------

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

1 files changed, 7 insertions, 7 deletions

diff --git a/components/script/serviceworker_manager.rs b/components/script/serviceworker_manager.rs
index 3be5d3a9ef2..9f1ad777110 100644
--- a/components/script/serviceworker_manager.rs
+++ b/components/script/serviceworker_manager.rs
@@ -38,13 +38,13 @@ enum Message {
 
 /// <https://w3c.github.io/ServiceWorker/#dfn-service-worker>
 #[derive(Clone)]
-struct ServiceWorker {
+pub(crate) struct ServiceWorker {
     /// A unique identifer.
-    pub id: ServiceWorkerId,
+    pub(crate) id: ServiceWorkerId,
     /// <https://w3c.github.io/ServiceWorker/#dfn-script-url>
-    pub script_url: ServoUrl,
+    pub(crate) script_url: ServoUrl,
     /// A sender to the running service worker scope.
-    pub sender: Sender<ServiceWorkerScriptMsg>,
+    pub(crate) sender: Sender<ServiceWorkerScriptMsg>,
 }
 
 impl ServiceWorker {
@@ -140,7 +140,7 @@ struct ServiceWorkerRegistration {
 }
 
 impl ServiceWorkerRegistration {
-    pub fn new() -> ServiceWorkerRegistration {
+    pub(crate) fn new() -> ServiceWorkerRegistration {
         ServiceWorkerRegistration {
             id: ServiceWorkerRegistrationId::new(),
             active_worker: None,
@@ -241,7 +241,7 @@ impl ServiceWorkerManager {
         }
     }
 
-    pub fn get_matching_scope(&self, load_url: &ServoUrl) -> Option<ServoUrl> {
+    pub(crate) fn get_matching_scope(&self, load_url: &ServoUrl) -> Option<ServoUrl> {
         for scope in self.registrations.keys() {
             if longest_prefix_match(scope, load_url) {
                 return Some(scope.clone());
@@ -516,6 +516,6 @@ impl ServiceWorkerManagerFactory for ServiceWorkerManager {
     }
 }
 
-pub fn serviceworker_enabled() -> bool {
+pub(crate) fn serviceworker_enabled() -> bool {
     pref!(dom.serviceworker.enabled)
 }