Skip to content
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

move getStats() to corresponding stats document #37

Closed
wants to merge 1 commit into from

Conversation

vr000m
Copy link

@vr000m vr000m commented Nov 24, 2014

related to webrtc-ACTION-117: Move getstats and associated idl to stats doc
corresponding pull request in webrtc-stats: w3c/webrtc-stats#4

related to ACTION-07 at TPAC14
@fluffy
Copy link
Contributor

fluffy commented Nov 26, 2014

I think we had previously decided to have the model how stats worked here and the actual definition stats in other spec but perhaps we decided something else later.

@vr000m
Copy link
Author

vr000m commented Nov 26, 2014

What would the model describe? I suppose it should be possible to add some text.

At the TPAC, it was decided to move the entire IDL into the stats-api, so that the definition of RTCStatsType enum and corresponding details would remain in one spec.

@fluffy
Copy link
Contributor

fluffy commented Dec 4, 2014

I think the discussion at last face to face (which I was not at) contradicts our previous decision on this. I started thread on list. Should resolve there first then decide what to do here.

@vr000m
Copy link
Author

vr000m commented Dec 15, 2014

From the discussion, it seems the consensus is towards making the ref to pc-stats informative. IIUC that be the only change, and the rest of the PR remains the same.

@fluffy
Copy link
Contributor

fluffy commented Dec 18, 2014

Hey, Voom, just talked to Dan and talk to me about this some time and we can get it sorted out

@alvestrand
Copy link
Contributor

Closing this one and awaiting a new one according to the discussions we had.

@alvestrand alvestrand closed this Feb 12, 2015
dontcallmedom pushed a commit that referenced this pull request Aug 26, 2016
Fixed some HTML conformance problems.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants