MediaStreamTrackProcessor Constructor

[nicopowa]

I've been compiling a project using MediaStreamTrackProcessor instances for a long time, it was not included in the compiler so I was using a custom extern file with the following content :

/**
 * @constructor 
 * @param {MediaStreamTrack} track : 
 */
var MediaStreamTrackProcessor = function(track) {};

The latest version returned a redeclared variable error, so I removed my extern file and got a new error : MediaStreamTrackProcessor is not constructor.

externs.zip//w3c_rtc.js:3645:9

I added two suppress annotations and re-included my externs file.

/**
 * @suppress {duplicate}
 * @suppress {checkTypes}
 * @constructor 
 * @param {MediaStreamTrack} track : 
 */
var MediaStreamTrackProcessor = function(track) {};

My project is compiling properly but I still get warnings about MediaStreamTrackProcessor not being a constructor.

[niloc132]

According to the linked externs, the MediaStreamTrackProcessor is not a real class, but an interface that describes structure only. This doesn't seem to be correct according to mdn's docs (edit: and the spec too), which looks somewhat similar to what you've shared, except that the first arg should not be MediaStreamTrack track, but an options object (a @struct/@record in closure's terms) which includes that track.

The linked externs conflict with yours, but both appear to be technically wrong. Perhaps make a PR to propose fixes to the linked file? Since commits come from inside of Google, it isn't clear who wrote or reviewed 70bd4d0 to ask about this discrepancy, maybe a maintainer can take a look?

[h-joo]

Would you mind sending a PR to update what correctly reflects the spec?

[nicopowa]

It would be my first pull request :-D
According to the docs, MediaStreamTrackProcessor should be a constructor.
Closure is one of the best programs I ever used.
Maybe I should share the code I built on top of it to compile all my vanilla ice cream projects with one click :)

[niloc132]

I'd be happy to work with you to get the PR ready to go.

Sharing how you run the compiler would be great too - maybe a good post to discussions?

[nicopowa]

That would be nice, maybe this week-end ? Should I clone the repository ?

I opened a discussion !