Bug 1637922. Disables netinfo in some tests. r=annevk,webidl,peterv

Adds comment on keeping the code instead of removing it.

Differential Revision: https://phabricator.services.mozilla.com/D126447

dom/serviceworkers/test/test_serviceworker_interfaces.js

@@ -224,7 +224,7 @@ var interfaceNamesInGlobalScope = [
   // IMPORTANT: Do not change this list without review from a DOM peer!
   "MessagePort",
   // IMPORTANT: Do not change this list without review from a DOM peer!
-  { name: "NetworkInformation", android: true },
+  { name: "NetworkInformation", disabled: true },
   // IMPORTANT: Do not change this list without review from a DOM peer!
   "NavigationPreloadManager",
   // IMPORTANT: Do not change this list without review from a DOM peer!

modules/libpref/init/StaticPrefList.yaml

@@ -2827,6 +2827,11 @@
   mirror: always
 
 # Network Information API
+# This feature is not available on Firefox desktop. It exposes too much
+# user information. Let's be consistent and disable it on Android.
+# But let's keep it around in case it becomes necessary for webcompat
+# reasons
+# https://bugzilla.mozilla.org/show_bug.cgi?id=1637922
 - name: dom.netinfo.enabled
   type: RelaxedAtomicBool
   value: false