-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Not enough arguments to mozRTCPeerConnection.getStats #1
Comments
That's because in the Firefox implementation the getStats function need to receive a MediaStreamTrack, like this: peer.getStats(
mediaStreamTrack,
function (res) {
var items = [];
res.forEach(function (result) {
items.push(result);
});
cb(items);
},
cb
); |
+1 |
@muaz-khan you should close this issue and pull request #2 since you already pushed the change to the master branch. |
@ulifigueroa How did this work out for you? From my understanding |
@simkesd I'm sorry, I don't know, I posted this fix in 2015 so probably it's not up to date |
I'm working on an RTC stats UI for an application and came across this project, and this issue. For others, and contributors who want to look at updating this project: The current Chrome implementation is Mozilla uses the |
Hi, in firefox it does not work. Got error "Not enough arguments to mozRTCPeerConnection.getStats". Error fires on line 155
The text was updated successfully, but these errors were encountered: