Browser extension not detecting offscreen canvas passed to webworker?

[kevzettler]

The browser extension doesn't seem to be detecting my a canvas that I have passed to a webworker

const offscreen = canvas.transferControlToOffscreen();
const worker = new Worker();
worker.postMessage(
  {
    type: "main/canvas-init",
    payload: {
      canvas:  offscreen,
      width: window.innerWidth,
      height: window.innerHeight,
      player: fetchLocalStoragePlayerData(),
    }
  },
  [offscreen]
);

Is this supported? Should I manually use the spector.js lib instead?

[sebavan]

Yup unfortunately it is not supported and even the lib would not be working due to the way we capture texture.

As this is getting more and more common, I really need to work on it but I won t have time before a month I guess.

[kevzettler]

Thanks @sebavan Any notes on the work needed? Maybe I or other community could hack on it.

[sebavan]

Feel free that would be perfect, I would say at first we would need to detect the offscreen and be able to record the calls to them by simply disabling the texture captures (who relies on canvas2D internally).

This would be a great start :-)

[XiaoDiDiDa]

is there any progress of this ticket?

[sebavan]

Not at the moment, would would like to give it a try ?

[Jeggery]

It has been three years and I really hope to see this feature implemented

[sebavan]

I wish I had more time, but I will not be able to work on it unfortunately. As it is open source, do you want to give it a try ?

[Jeggery]

I wish I had more time, but I will not be able to work on it unfortunately. As it is open source, do you want to give it a try ?

This is a lot of difficulty for me, and I still need to delve deeper into the Spector source code and webgl principles

[junyuecao]

@sebavan I created a PR which might support this feature.
#298

@Jeggery Maybe you can have a try.

BTW, the extension is based on Chrome extension manifest versoin 2 which is deprecated and can't be update anymore without upgrading to version 3

[pmhpereira]

Hey @junyuecao.
I wanted to test your PR, but I haven't been able to compile your fork locally.
I ran npm install and npm run build and I'm getting a lot of typescript errors.

[junyuecao]

@pmhpereira
Could you try with npm run start? It works on my Ubuntu 20.04

[sebavan]

I completely missed all of this !!! I am so sorry @junyuecao I will have a deep look ASAP but it sounds really good !!!