Bug 102933

Summary: GetStats RTCStatsElement should have an enumerator for available stat names
Product: WebKit Reporter: Harald Alvestrand <hta>
Component: WebKit Misc.Assignee: Harald Alvestrand <hta>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric.carlson, feature-media-reviews, ojan, tommyw, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Harald Alvestrand
Reported 2012-11-21 06:22:36 PST
For debugging, it is extremely useful to see which stats names are being returned at any given time.
Attachments
Patch (5.00 KB, patch)
2012-11-21 07:41 PST, Harald Alvestrand 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Harald Alvestrand
Comment 1 2012-11-21 07:41:50 PST
Created attachment 175443 [details] Patch
Adam Barth
Comment 2 2012-11-21 09:56:23 PST
Is this part of the spec? It's not clear from Comment #0.
Harald Alvestrand
Comment 3 2012-11-21 10:00:50 PST
I've asked for it to be added; it's been suggested by others before this. http://lists.w3.org/Archives/Public/public-webrtc/2012Nov/0132.html
Adam Barth
Comment 4 2012-11-21 10:05:01 PST
Looks like you just send that message today. You don't want to wait for feedback before implementing it? I haven't been following the working group, so I don't have much context for the discussion. I'm going to mark this r+, but in general, we don't want to get too far ahead of the spec. We want to end up with an interoperable implementation. If we implement things too far ahead of the spec, we run the risk of making other implementors sad.
WebKit Review Bot
Comment 5 2012-11-22 03:00:33 PST
Comment on attachment 175443 [details] Patch Clearing flags on attachment: 175443 Committed r135500: <http://trac.webkit.org/changeset/135500>
WebKit Review Bot
Comment 6 2012-11-22 03:00:37 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.