Background workers are not permanent in Manifest v3

[polcak]

Hello,

it looks like you are using Manifest v3 and it looks like you store state in you background workers.

For example, let us focus on check_git

DotGit/dotgit.js

Line 101 in ab11f45

let check_git;

. The extension writes to that variable in set_options (

DotGit/dotgit.js

Line 101 in ab11f45

let check_git;

) and in the onMessage listener (

DotGit/dotgit.js

Line 555 in ab11f45

check_git = request.value;

). The extension reads the value in onHeaderReceived WebRequest API listener

DotGit/dotgit.js

Line 714 in ab11f45

if (check_git) {

.

First of all, you are using a global variable and the code looks like it can produce race conditions for that variable if the onMessage listener is executed in short time with different content for the check_git variable. I have not checked if this is this can happen.

My main point is that the worker could have been destroyed between you store the value to the variable and onHeaderReceived fires. the code would broke if the variable should hold true. If I am looking right at the code, the variable would be undefined when the service worker respawns.

[davtur19]

For now I'm not using manifestV3 yet, I was waiting for it to be supported by FireFox (I tried a few months ago and it didn't work) and I just did some testing on chrome and it seemed to work fine, until it's mandatory I don't think I'll make any changes unless necessary.

Anyway thanks for pointing out that in manifestV3 this might cause problems.

PR are welcome.