<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>102933</bug_id>
          
          <creation_ts>2012-11-21 06:22:36 -0800</creation_ts>
          <short_desc>GetStats RTCStatsElement should have an enumerator for available stat names</short_desc>
          <delta_ts>2012-11-22 03:00:37 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Harald Alvestrand">hta</reporter>
          <assigned_to name="Harald Alvestrand">hta</assigned_to>
          <cc>abarth</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>feature-media-reviews</cc>
    
    <cc>ojan</cc>
    
    <cc>tommyw</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>772872</commentid>
    <comment_count>0</comment_count>
    <who name="Harald Alvestrand">hta</who>
    <bug_when>2012-11-21 06:22:36 -0800</bug_when>
    <thetext>For debugging, it is extremely useful to see which stats names are being returned at any given time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>772962</commentid>
    <comment_count>1</comment_count>
      <attachid>175443</attachid>
    <who name="Harald Alvestrand">hta</who>
    <bug_when>2012-11-21 07:41:50 -0800</bug_when>
    <thetext>Created attachment 175443
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773050</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-11-21 09:56:23 -0800</bug_when>
    <thetext>Is this part of the spec?  It&apos;s not clear from Comment #0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773059</commentid>
    <comment_count>3</comment_count>
    <who name="Harald Alvestrand">hta</who>
    <bug_when>2012-11-21 10:00:50 -0800</bug_when>
    <thetext>I&apos;ve asked for it to be added; it&apos;s been suggested by others before this.

http://lists.w3.org/Archives/Public/public-webrtc/2012Nov/0132.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773063</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-11-21 10:05:01 -0800</bug_when>
    <thetext>Looks like you just send that message today.  You don&apos;t want to wait for feedback before implementing it?  I haven&apos;t been following the working group, so I don&apos;t have much context for the discussion.

I&apos;m going to mark this r+, but in general, we don&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773778</commentid>
    <comment_count>5</comment_count>
      <attachid>175443</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-22 03:00:33 -0800</bug_when>
    <thetext>Comment on attachment 175443
Patch

Clearing flags on attachment: 175443

Committed r135500: &lt;http://trac.webkit.org/changeset/135500&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773779</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-22 03:00:37 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>175443</attachid>
            <date>2012-11-21 07:41:50 -0800</date>
            <delta_ts>2012-11-22 03:00:33 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-102933-20121121163936.patch</filename>
            <type>text/plain</type>
            <size>5125</size>
            <attacher name="Harald Alvestrand">hta</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTM1MzkzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzhiMTUxZmYxOTc4ZmU5
ZmU4M2Q3Yjc0YzQzMTg2OTEwZDJjZDgzMy4uYWY0Y2Q4ZDg2NjhkNjY3NzVhNDU2MWIzZTFlOWRk
MTNkZmE5Y2I3MyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDEyLTExLTIxICBIYXJh
bGQgQWx2ZXN0cmFuZCAgPGh0YUBnb29nbGUuY29tPgorCisgICAgICAgIE5hbWUgZW51bWVyYXRv
ciBmdW5jdGlvbiBmb3IgR2V0U3RhdHMgUlRDU3RhdHNFbGVtZW50CisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDI5MzMKKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUZXN0ZWQgYnkgZXh0ZW5zaW9ucyB0byB0
aGUgUlRDUGVlckNvbm5lY3Rpb24tc3RhdHMgdGVzdC4KKworICAgICAgICAqIE1vZHVsZXMvbWVk
aWFzdHJlYW0vUlRDU3RhdHNFbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlJUQ1N0YXRz
RWxlbWVudDo6bmFtZXMpOgorICAgICAgICAoV2ViQ29yZSk6CisgICAgICAgICogTW9kdWxlcy9t
ZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuaDoKKyAgICAgICAgKFJUQ1N0YXRzRWxlbWVudCk6
CisgICAgICAgICogTW9kdWxlcy9tZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuaWRsOgorCiAy
MDEyLTExLTIxICBGbG9yaW4gTWFsaXRhICA8Zm1hbGl0YUBjaHJvbWl1bS5vcmc+CiAKICAgICAg
ICAgW0Nocm9taXVtXSBVbnJldmlld2VkIE1hYyBidWlsZCBmaXggYWZ0ZXIgaHR0cDovL3RyYWMu
d2Via2l0Lm9yZy9jaGFuZ2VzZXQvMTM1MzkwLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUv
TW9kdWxlcy9tZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuY3BwIGIvU291cmNlL1dlYkNvcmUv
TW9kdWxlcy9tZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuY3BwCmluZGV4IDlkNTFlNTQwNTkz
Y2M1MTkwNGUxZTc3NDJjZGZjZWQ5NWU5NWY0ZmUuLjYyNDE2NzNiZTg1ZTAwNzNjNzI5NDZkODBm
YjhlNWFiNzQ1YWVkOTcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvbWVkaWFz
dHJlYW0vUlRDU3RhdHNFbGVtZW50LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21l
ZGlhc3RyZWFtL1JUQ1N0YXRzRWxlbWVudC5jcHAKQEAgLTQ3LDYgKzQ3LDE2IEBAIFN0cmluZyBS
VENTdGF0c0VsZW1lbnQ6OnN0YXQoY29uc3QgU3RyaW5nJiBuYW1lKSBjb25zdAogICAgIHJldHVy
biBtX3N0YXRzLmdldChuYW1lKTsKIH0KIAorVmVjdG9yPFN0cmluZz4gUlRDU3RhdHNFbGVtZW50
OjpuYW1lcygpIGNvbnN0Cit7CisgICAgVmVjdG9yPFN0cmluZz4gcmVzdWx0OworICAgIGZvciAo
SGFzaE1hcDxTdHJpbmcsIFN0cmluZz46OmNvbnN0X2l0ZXJhdG9yIGl0ID0gbV9zdGF0cy5iZWdp
bigpOworICAgICAgICBpdCAhPSBtX3N0YXRzLmVuZCgpOyArK2l0KSB7CisgICAgICAgIHJlc3Vs
dC5hcHBlbmQoaXQtPmtleSk7CisgICAgfQorICAgIHJldHVybiByZXN1bHQ7Cit9CisKIHZvaWQg
UlRDU3RhdHNFbGVtZW50OjphZGRTdGF0aXN0aWMoY29uc3QgU3RyaW5nJiBuYW1lLCBjb25zdCBT
dHJpbmcmIHZhbHVlKQogewogICAgIG1fc3RhdHMuYWRkKG5hbWUsIHZhbHVlKTsKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvbWVkaWFzdHJlYW0vUlRDU3RhdHNFbGVtZW50Lmgg
Yi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3RyZWFtL1JUQ1N0YXRzRWxlbWVudC5oCmlu
ZGV4IDBhZmFiNWYyYjUxMTVhYjEzMDkxNDZjZDJlODg5OWUxYjQ4NzI2NGUuLjIyZDRhY2YwYmIy
MTBlOGE5ZjEyODUwNDY0OWQwZjY3ZGY4ZDk5NjggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3Jl
L01vZHVsZXMvbWVkaWFzdHJlYW0vUlRDU3RhdHNFbGVtZW50LmgKKysrIGIvU291cmNlL1dlYkNv
cmUvTW9kdWxlcy9tZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuaApAQCAtMzcsNiArMzcsNyBA
QCBwdWJsaWM6CiAgICAgc3RhdGljIFBhc3NSZWZQdHI8UlRDU3RhdHNFbGVtZW50PiBjcmVhdGUo
ZG91YmxlIHRpbWVzdGFtcCk7CiAgICAgZG91YmxlIHRpbWVzdGFtcCgpIGNvbnN0IHsgcmV0dXJu
IG1fdGltZXN0YW1wOyB9CiAgICAgU3RyaW5nIHN0YXQoY29uc3QgU3RyaW5nJiBuYW1lKSBjb25z
dDsKKyAgICBWZWN0b3I8U3RyaW5nPiBuYW1lcygpIGNvbnN0OwogICAgIHZvaWQgYWRkU3RhdGlz
dGljKGNvbnN0IFN0cmluZyYgbmFtZSwgY29uc3QgU3RyaW5nJiB2YWx1ZSk7CiAKIHByaXZhdGU6
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3RyZWFtL1JUQ1N0YXRz
RWxlbWVudC5pZGwgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3RyZWFtL1JUQ1N0YXRz
RWxlbWVudC5pZGwKaW5kZXggYWE0ZTc5ZmZjY2QwNDlkZmViZGUxZmY0NWNiMDk1N2E0ZTU0NTRl
Mi4uNmE4MzBkZTg3NzQ2ZWFkNDY0NGRlNTVlODBiNGNiOTZiMjQ4Nzk3MiAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYkNvcmUvTW9kdWxlcy9tZWRpYXN0cmVhbS9SVENTdGF0c0VsZW1lbnQuaWRsCisr
KyBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvbWVkaWFzdHJlYW0vUlRDU3RhdHNFbGVtZW50Lmlk
bApAQCAtMjcsNCArMjcsNSBAQAogXSBpbnRlcmZhY2UgUlRDU3RhdHNFbGVtZW50IHsKICAgICBy
ZWFkb25seSBhdHRyaWJ1dGUgRGF0ZSB0aW1lc3RhbXA7CiAgICAgRE9NU3RyaW5nIHN0YXQoaW4g
RE9NU3RyaW5nIG5hbWUpOworICAgIHNlcXVlbmNlPFN0cmluZz4gbmFtZXMoKTsKIH07CmRpZmYg
LS1naXQgYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5k
ZXggMjBlNWFiYjdhMTAxMTkyNThkMWE4ZjYwMmQyYzlmMjk0MWRmNDBlZS4uNjk4ODE3NTMzNGI1
N2I4YjhhYzJhNzM0NTU4ODMyNTZkNjYyYmQxMCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hh
bmdlTG9nCisrKyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEy
LTExLTIxICBIYXJhbGQgQWx2ZXN0cmFuZCAgPGh0YUBnb29nbGUuY29tPgorCisgICAgICAgIE5h
bWUgZW51bWVyYXRvciBmdW5jdGlvbiBmb3IgR2V0U3RhdHMgUlRDU3RhdHNFbGVtZW50CisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDI5MzMKKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGZhc3QvbWVkaWFz
dHJlYW0vUlRDUGVlckNvbm5lY3Rpb24tc3RhdHMtZXhwZWN0ZWQudHh0OgorICAgICAgICAqIGZh
c3QvbWVkaWFzdHJlYW0vUlRDUGVlckNvbm5lY3Rpb24tc3RhdHMuaHRtbDoKKwogMjAxMi0xMS0y
MSAgSnVzc2kgS3Vra29uZW4gIDxqdXNzaS5rdWtrb25lbkBpbnRlbC5jb20+CiAKICAgICAgICAg
W0VGTF1bV0syXSBHYXJkZW5pbmc6IGNzczMvZGV2aWNlLWFkYXB0L29wZXJhL2NvbnN0cmFpbi0w
MDYgYW5kIC0wMDcgZmFpbApkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9tZWRpYXN0cmVh
bS9SVENQZWVyQ29ubmVjdGlvbi1zdGF0cy1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9mYXN0
L21lZGlhc3RyZWFtL1JUQ1BlZXJDb25uZWN0aW9uLXN0YXRzLWV4cGVjdGVkLnR4dAppbmRleCA4
OTliYmQ4ZTA2ZjA0YmYzNDBmNzc5N2M2NGU0OGU3ZjY0ODI0YTM5Li5hNmQ3NTIyN2UwNDE4YTI4
M2QwZjdmYmU2MGYzNzM3YWZkNjk2MTZlIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L21l
ZGlhc3RyZWFtL1JUQ1BlZXJDb25uZWN0aW9uLXN0YXRzLWV4cGVjdGVkLnR4dAorKysgYi9MYXlv
dXRUZXN0cy9mYXN0L21lZGlhc3RyZWFtL1JUQ1BlZXJDb25uZWN0aW9uLXN0YXRzLWV4cGVjdGVk
LnR4dApAQCAtMTQsNiArMTQsOCBAQCBQQVNTIHBjLmdldFN0YXRzKHN0YXRzSGFuZGxlcjIpIGRp
ZCBub3QgdGhyb3cgZXhjZXB0aW9uLgogUEFTUyBzdGF0c0hhbmRsZXIyIHdhcyBjYWxsZWQKIFBB
U1MgcmVzdWx0Lmxlbmd0aCBpcyA+PSAyCiBQQVNTIHRpbWVzdGFtcCBpcyA+PSBzdGFydFRpbWUK
K1BBU1MgbG9jYWwubmFtZXMoKS5sZW5ndGggaXMgPj0gMQorUEFTUyBsb2NhbC5uYW1lcygpLmlu
ZGV4T2YoInR5cGUiKSBpcyA+PSAwCiBQQVNTIGxvY2FsLnN0YXQoInR5cGUiKSBpcyAiYXVkaW8i
CiBQQVNTIHN1Y2Nlc3NmdWxseVBhcnNlZCBpcyB0cnVlCiAKZGlmZiAtLWdpdCBhL0xheW91dFRl
c3RzL2Zhc3QvbWVkaWFzdHJlYW0vUlRDUGVlckNvbm5lY3Rpb24tc3RhdHMuaHRtbCBiL0xheW91
dFRlc3RzL2Zhc3QvbWVkaWFzdHJlYW0vUlRDUGVlckNvbm5lY3Rpb24tc3RhdHMuaHRtbAppbmRl
eCBkMjVmZmE0ZDZiYWQ4ZTFkNGQxNDMzMzUwMzc2NDU5MTU0YzVlNjYzLi4yOWQ1ODE0NGI1Yzlk
NzFmZGYxZmVmMWMzYzE3MzdlNWUyNjBlMWQxIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0
L21lZGlhc3RyZWFtL1JUQ1BlZXJDb25uZWN0aW9uLXN0YXRzLmh0bWwKKysrIGIvTGF5b3V0VGVz
dHMvZmFzdC9tZWRpYXN0cmVhbS9SVENQZWVyQ29ubmVjdGlvbi1zdGF0cy5odG1sCkBAIC0xLDMg
KzEsNCBAQAorCiA8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL0lFVEYvL0RURCBIVE1MLy9FTiI+
CiA8aHRtbD4KIDxoZWFkPgpAQCAtNTEsNiArNTIsOCBAQCBmdW5jdGlvbiBzdGF0c0hhbmRsZXIy
KHN0YXR1cykKICAgICBsb2NhbCA9IHJlc3VsdFswXS5sb2NhbDsKICAgICB0aW1lc3RhbXAgPSBs
b2NhbC50aW1lc3RhbXA7CiAgICAgc2hvdWxkQmVHcmVhdGVyVGhhbk9yRXF1YWwoJ3RpbWVzdGFt
cCcsICdzdGFydFRpbWUnKTsKKyAgICBzaG91bGRCZUdyZWF0ZXJUaGFuT3JFcXVhbCgnbG9jYWwu
bmFtZXMoKS5sZW5ndGgnLCAnMScpOworICAgIHNob3VsZEJlR3JlYXRlclRoYW5PckVxdWFsKCds
b2NhbC5uYW1lcygpLmluZGV4T2YoInR5cGUiKScsICcwJyk7CiAgICAgc2hvdWxkQmUoJ2xvY2Fs
LnN0YXQoInR5cGUiKScsICciYXVkaW8iJyk7CiAgICAgZmluaXNoSlNUZXN0KCk7CiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>