<?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>134625</bug_id>
          
          <creation_ts>2014-07-04 02:25:54 -0700</creation_ts>
          <short_desc>[MediaStream] MediaStream.removeTrack should not check for active state.</short_desc>
          <delta_ts>2014-07-04 02:57:43 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></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="Seongjun Kim">isair</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bunhere</cc>
    
    <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>glenn</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>hta</cc>
    
    <cc>isair</cc>
    
    <cc>jer.noble</cc>
    
    <cc>philipj</cc>
    
    <cc>sergio</cc>
    
    <cc>tommyw</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1020458</commentid>
    <comment_count>0</comment_count>
    <who name="Seongjun Kim">isair</who>
    <bug_when>2014-07-04 02:25:54 -0700</bug_when>
    <thetext>W3C removed MediaStream state check from the removeTrack() algorithm, since February 18, 2014


For example, following code should not throw INVALID_STATE_ERR

// video: MediaStream that has 1 video track.
// audeo: MediaStream that has 1 audio track.
 video.removeTrack(video.getVideoTracks()[0]);
 video.removeTrack(audio.getAudioTracks()[0]); // should not throw


Relative issue: https://www.w3.org/Bugs/Public/show_bug.cgi?id=24930
Full Test: http://w3c-test.org/mediacapture-streams/stream-api/mediastream/mediastream-removetrack.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1020463</commentid>
    <comment_count>1</comment_count>
      <attachid>234397</attachid>
    <who name="Seongjun Kim">isair</who>
    <bug_when>2014-07-04 02:56:02 -0700</bug_when>
    <thetext>Created attachment 234397
PATCH

First Patch.

I will add a new test for this cass.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1020464</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-07-04 02:57:43 -0700</bug_when>
    <thetext>Attachment 234397 did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the &apos;No new tests&apos; and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 1 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>234397</attachid>
            <date>2014-07-04 02:56:02 -0700</date>
            <delta_ts>2014-07-04 02:56:02 -0700</delta_ts>
            <desc>PATCH</desc>
            <filename>0001-MediaStream-MediaStream.removeTrack-should-not-check.patch</filename>
            <type>text/plain</type>
            <size>1904</size>
            <attacher name="Seongjun Kim">isair</attacher>
            
              <data encoding="base64">RnJvbSA3NGVjMzQ2MjU2YWI4Mjg4MDY3NDBlZTI5ZWM2NTg5MTM5OTkyMzdjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBTZW9uZ2p1biBLaW0gPGlzQWlyQGNvbXBhbnkxMDAuY29tPgpE
YXRlOiBGcmksIDQgSnVsIDIwMTQgMTg6NDE6MTEgKzA5MDAKU3ViamVjdDogPT9VVEYtOD9xP1tN
ZWRpYVN0cmVhbV09MjBNZWRpYVN0cmVhbS5yZW1vdmVUcmFjaz0yMHNob3VsZD0yMG5vdD89CiA9
P1VURi04P3E/PTIwY2hlY2s9MjBmb3I9MjBhY3RpdmU9MjBzdGF0ZS49MEFodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93Pz0KID0/VVRGLTg/cT89NUZidWcuY2dpPTNGaWQ9M0QxMzQ2MjU/PQoK
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgpObyBuZXcgdGVzdHMgKE9PUFMhKS4KCiogTW9k
dWxlcy9tZWRpYXN0cmVhbS9NZWRpYVN0cmVhbS5jcHA6CihXZWJDb3JlOjpNZWRpYVN0cmVhbTo6
cmVtb3ZlVHJhY2spOgotLS0KIFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyAgICAgICAgICAgICAg
ICAgICAgICAgICAgIHwgMTIgKysrKysrKysrKysrCiBTb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21l
ZGlhc3RyZWFtL01lZGlhU3RyZWFtLmNwcCB8ICA1IC0tLS0tCiAyIGZpbGVzIGNoYW5nZWQsIDEy
IGluc2VydGlvbnMoKyksIDUgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv
cmUvQ2hhbmdlTG9nIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IDk5MjdkNmMuLjJj
NTUwY2MgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2Uv
V2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAxNC0wNy0wNCAgU2VvbmdqdW4g
S2ltICA8aXNBaXJAY29tcGFueTEwMC5jb20+CisKKyAgICAgICAgW01lZGlhU3RyZWFtXSBNZWRp
YVN0cmVhbS5yZW1vdmVUcmFjayBzaG91bGQgbm90IGNoZWNrIGZvciBhY3RpdmUgc3RhdGUuCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMzQ2MjUKKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBObyBuZXcgdGVz
dHMgKE9PUFMhKS4KKworICAgICAgICAqIE1vZHVsZXMvbWVkaWFzdHJlYW0vTWVkaWFTdHJlYW0u
Y3BwOgorICAgICAgICAoV2ViQ29yZTo6TWVkaWFTdHJlYW06OnJlbW92ZVRyYWNrKToKKwogMjAx
NC0wNy0wNCAgWmFuIERvYmVyc2VrICA8emRvYmVyc2VrQGlnYWxpYS5jb20+CiAKICAgICAgICAg
VW5yZXZpZXdlZC4gRml4aW5nIHRoZSBidWlsZCBhZnRlciByMTcwNzk1LgpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvTW9kdWxlcy9tZWRpYXN0cmVhbS9NZWRpYVN0cmVhbS5jcHAgYi9Tb3Vy
Y2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3RyZWFtL01lZGlhU3RyZWFtLmNwcAppbmRleCA1NTk4
OTVmLi43NGE4ZDhmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3Ry
ZWFtL01lZGlhU3RyZWFtLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL21lZGlhc3Ry
ZWFtL01lZGlhU3RyZWFtLmNwcApAQCAtMTczLDExICsxNzMsNiBAQCBib29sIE1lZGlhU3RyZWFt
OjphZGRUcmFjayhQYXNzUmVmUHRyPE1lZGlhU3RyZWFtVHJhY2s+IHBycFRyYWNrKQogCiB2b2lk
IE1lZGlhU3RyZWFtOjpyZW1vdmVUcmFjayhQYXNzUmVmUHRyPE1lZGlhU3RyZWFtVHJhY2s+IHBy
cFRyYWNrLCBFeGNlcHRpb25Db2RlJiBlYykKIHsKLSAgICBpZiAoIWFjdGl2ZSgpKSB7Ci0gICAg
ICAgIGVjID0gSU5WQUxJRF9TVEFURV9FUlI7Ci0gICAgICAgIHJldHVybjsKLSAgICB9Ci0KICAg
ICBpZiAoIXBycFRyYWNrKSB7CiAgICAgICAgIGVjID0gVFlQRV9NSVNNQVRDSF9FUlI7CiAgICAg
ICAgIHJldHVybjsKLS0gCjEuOS4xCgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>