1876533 - ServiceWorker target not created when file URL contains port

Status: RESOLVED FIXED

(DevTools :: Framework, defect)

Description

[Nicolas Chevobbe [:nchevobbe]]

I was testing service worker with a simple local server, serving the files on localhost:8000 and couldn't get the service worker target to be created (devtools.debugger.features.windowless-service-workers was set to true)

This is because in https://searchfox.org/mozilla-central/rev/a2709c2f0e31b2ad1a7b6447117626a34063fe57/devtools/server/connectors/process-actor/DevToolsServiceWorkerChild.sys.mjs#363,378-379

_shouldHandleWorker(sessionData, dbg) {
...
  const workerHost = dbg.principal.host;
  return workerHost == sessionData["browser-element-host"][0];

dbg.principal.host was "localhost", butsessionData["browser-element-host"][0]was"localhost:8000"`.

The following change

-    const workerHost = dbg.principal.host;
+    const workerHost = dbg.principal.hostPort;
     return workerHost == sessionData["browser-element-host"][0];
   }

does fix the issue, as hostPort does include the port

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1876533 - [devtools] Use principal hostPort in DevToolsServiceWorkerChild _shouldHandleWorker. r=#devtools-reviewers.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ba2007531f3
[devtools] Use principal hostPort in DevToolsServiceWorkerChild _shouldHandleWorker. r=devtools-reviewers,ochameau.

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/6ba2007531f3