Bug 102933 - GetStats RTCStatsElement should have an enumerator for available stat names
Summary: GetStats RTCStatsElement should have an enumerator for available stat names
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Harald Alvestrand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-21 06:22 PST by Harald Alvestrand
Modified: 2012-11-22 03:00 PST (History)
6 users (show)

See Also:

Attachments
Patch (5.00 KB, patch)
2012-11-21 07:41 PST, Harald Alvestrand 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Alvestrand 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.
Comment 1 Harald Alvestrand 2012-11-21 07:41:50 PST 
Created attachment 175443 [details]
Patch
Comment 2 Adam Barth 2012-11-21 09:56:23 PST 
Is this part of the spec?  It's not clear from Comment #0.
Comment 3 Harald Alvestrand 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
Comment 4 Adam Barth 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.
Comment 5 WebKit Review Bot 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>
Comment 6 WebKit Review Bot 2012-11-22 03:00:37 PST 
All reviewed patches have been landed.  Closing bug.