<?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>139471</bug_id>
          
          <creation_ts>2014-12-09 23:42:29 -0800</creation_ts>
          <short_desc>[GStreamer] SourceBufferPrivateGstreamer should emit &apos;updateend&apos;</short_desc>
          <delta_ts>2014-12-10 03:46:42 -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>Media</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>139441</dup_id>
          
          <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="Bartlomiej Gajda">b.gajda</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>pnormand</cc>
    
    <cc>slomo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1054058</commentid>
    <comment_count>0</comment_count>
    <who name="Bartlomiej Gajda">b.gajda</who>
    <bug_when>2014-12-09 23:42:29 -0800</bug_when>
    <thetext>Currently it does not.

While of course this is not ready for complete specification (mostly due to buffered missing), 
enabling this would allow testing for other independent issues.

//btw:

What should be proper way to tag such issues? both [GStreamer] and [MSE] ? 
I don&apos;t want people unrelated to platform specific issues to be on auto CC, but it would separate MSE specific issues from general media player related etc.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1054061</commentid>
    <comment_count>1</comment_count>
      <attachid>242990</attachid>
    <who name="Bartlomiej Gajda">b.gajda</who>
    <bug_when>2014-12-09 23:47:39 -0800</bug_when>
    <thetext>Created attachment 242990
call appendComplete so it will emit updateend per spec

Initial version of this.

Check for &apos;updateend&apos; is part of other tests, but we won&apos;t pass them completely, so shall I add new one which tests only this? Or edit something? Or no test is needed since this is yet unstable and/or unsupported port feature?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1054074</commentid>
    <comment_count>2</comment_count>
    <who name="Sebastian Dröge (slomo)">slomo</who>
    <bug_when>2014-12-10 00:41:21 -0800</bug_when>
    <thetext>That&apos;s fixed already in my patch in bug #139441, which also solves many other problems. I&apos;m currently changing quite a bit on top of that to make even more tests pass.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1054102</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2014-12-10 03:44:36 -0800</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 139441 ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>242990</attachid>
            <date>2014-12-09 23:47:39 -0800</date>
            <delta_ts>2014-12-10 03:46:42 -0800</delta_ts>
            <desc>call appendComplete so it will emit updateend per spec</desc>
            <filename>patch_gst_no_updateend</filename>
            <type>text/plain</type>
            <size>3774</size>
            <attacher name="Bartlomiej Gajda">b.gajda</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAyMDRiYzk4Li4xYzQxNjhlIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAg
QEAKKzIwMTQtMTItMDkgIEJhcnRsb21pZWogR2FqZGEgIDxiLmdhamRhQHNhbXN1bmcuY29tPgor
CisgICAgICAgIFtHU3RyZWFtZXJdIFNvdXJjZUJ1ZmZlclByaXZhdGVHc3RyZWFtZXIgc2hvdWxk
IGVtaXQgJ3VwZGF0ZWVuZCcKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTEzOTQ3MQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIFNvdXJjZUJ1ZmZlclByaXZhdGVHU3RyZWFtZXIgbmVlZCBjYWxsIGFwcGVuZENv
bXBsZXRlLCBzbyBldmVudHMgd2lsbCBiZSBjb3JyZWN0bHkKKyAgICAgICAgZW1pdHRlZC4KKwor
ICAgICAgICBObyBuZXcgdGVzdHMgbmVlZGVkLgorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhp
Y3MvZ3N0cmVhbWVyL1NvdXJjZUJ1ZmZlclByaXZhdGVHU3RyZWFtZXIuY3BwOgorICAgICAgICAo
V2ViQ29yZTo6U291cmNlQnVmZmVyUHJpdmF0ZUdTdHJlYW1lcjo6U291cmNlQnVmZmVyUHJpdmF0
ZUdTdHJlYW1lcik6CisgICAgICAgIChXZWJDb3JlOjpTb3VyY2VCdWZmZXJQcml2YXRlR1N0cmVh
bWVyOjphcHBlbmQpOgorICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL2dzdHJlYW1lci9Tb3Vy
Y2VCdWZmZXJQcml2YXRlR1N0cmVhbWVyLmg6CisKIDIwMTQtMTItMDkgIEFsYmVydG8gR2FyY2lh
ICA8YmVydG9AaWdhbGlhLmNvbT4KIAogICAgICAgICBbTWVkaWFdW0dUS11bRUZMXSBSZWR1Y2Ug
c3R5bGUgdXBkYXRlcyAocGFpbnRpbmcpIGluIGNvbnRyb2xzCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvU291cmNlQnVmZmVyUHJpdmF0ZUdT
dHJlYW1lci5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIv
U291cmNlQnVmZmVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKaW5kZXggNDVjNWU5MS4uMDE1Nzg3ZCAx
MDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL1Nv
dXJjZUJ1ZmZlclByaXZhdGVHU3RyZWFtZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRm
b3JtL2dyYXBoaWNzL2dzdHJlYW1lci9Tb3VyY2VCdWZmZXJQcml2YXRlR1N0cmVhbWVyLmNwcApA
QCAtMzEsNiArMzEsNyBAQAogCiAjaW5jbHVkZSAiY29uZmlnLmgiCiAjaW5jbHVkZSAiU291cmNl
QnVmZmVyUHJpdmF0ZUdTdHJlYW1lci5oIgorI2luY2x1ZGUgIlNvdXJjZUJ1ZmZlclByaXZhdGVD
bGllbnQuaCIKIAogI2lmIEVOQUJMRShNRURJQV9TT1VSQ0UpICYmIFVTRShHU1RSRUFNRVIpCiAK
QEAgLTM5LDE5ICs0MCwyMyBAQAogCiBuYW1lc3BhY2UgV2ViQ29yZSB7CiAKLVNvdXJjZUJ1ZmZl
clByaXZhdGVHU3RyZWFtZXI6OlNvdXJjZUJ1ZmZlclByaXZhdGVHU3RyZWFtZXIoUGFzc1JlZlB0
cjxNZWRpYVNvdXJjZUNsaWVudEdzdHJlYW1lcj4gY2xpZW50LCBjb25zdCBDb250ZW50VHlwZSYg
Y29udGVudFR5cGUpCitTb3VyY2VCdWZmZXJQcml2YXRlR1N0cmVhbWVyOjpTb3VyY2VCdWZmZXJQ
cml2YXRlR1N0cmVhbWVyKFBhc3NSZWZQdHI8TWVkaWFTb3VyY2VDbGllbnRHc3RyZWFtZXI+IG1l
ZGlhU291cmNlLCBjb25zdCBDb250ZW50VHlwZSYgY29udGVudFR5cGUpCiAgICAgOiBtX3JlYWR5
U3RhdGUoTWVkaWFQbGF5ZXI6OkhhdmVOb3RoaW5nKQogewotICAgIG1fY2xpZW50ID0gY2xpZW50
OworICAgIG1fbWVkaWFTb3VyY2VDbGllbnQgPSBtZWRpYVNvdXJjZTsKICAgICBtX3R5cGUgPSBj
b250ZW50VHlwZS50eXBlKCk7CiB9CiAKIHZvaWQgU291cmNlQnVmZmVyUHJpdmF0ZUdTdHJlYW1l
cjo6YXBwZW5kKGNvbnN0IHVuc2lnbmVkIGNoYXIqIGRhdGEsIHVuc2lnbmVkIGxlbmd0aCkKIHsK
LSAgICBBU1NFUlQobV9jbGllbnQpOwotICAgIG1fY2xpZW50LT5kaWRSZWNlaXZlRGF0YShyZWlu
dGVycHJldF9jYXN0X3B0cjxjb25zdCBjaGFyKj4oZGF0YSksIGxlbmd0aCwgbV90eXBlKTsKKyAg
ICBTb3VyY2VCdWZmZXJQcml2YXRlQ2xpZW50OjpBcHBlbmRSZXN1bHQgcmVzdWx0ID0gU291cmNl
QnVmZmVyUHJpdmF0ZUNsaWVudDo6QXBwZW5kU3VjY2VlZGVkOworICAgIEFTU0VSVChtX21lZGlh
U291cmNlQ2xpZW50KTsKKyAgICBtX21lZGlhU291cmNlQ2xpZW50LT5kaWRSZWNlaXZlRGF0YShy
ZWludGVycHJldF9jYXN0X3B0cjxjb25zdCBjaGFyKj4oZGF0YSksIGxlbmd0aCwgbV90eXBlKTsK
IAotICAgIC8vIEZJWE1FOiBjYWxsIFNvdXJjZUJ1ZmZlclByaXZhdGVDbGllbnQ6OnNvdXJjZUJ1
ZmZlclByaXZhdGVBcHBlbmRDb21wbGV0ZSgpLgorICAgIC8vIEZJWE1FOiBvbmNlIHdlIGhhdmUg
c29tZSBtb3JlIGNvbXBsaWNhdGVkIGxvZ2ljLCB0aGVuIHRocm93IGV2ZXJ5dGhpbmcgdG8gZ3N0
LCByZXN1bHQgc2hhbGwgYmUKKyAgICAvLyBjaGFuZ2VkIGRlcGVuZGluZyBvbiBhY3R1YWwgcmVz
dWx0CisgICAgaWYgKG1fY2xpZW50KQorICAgICAgICBtX2NsaWVudC0+c291cmNlQnVmZmVyUHJp
dmF0ZUFwcGVuZENvbXBsZXRlKHRoaXMsIHJlc3VsdCk7CiB9CiAKIHZvaWQgU291cmNlQnVmZmVy
UHJpdmF0ZUdTdHJlYW1lcjo6YWJvcnQoKQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxh
dGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL1NvdXJjZUJ1ZmZlclByaXZhdGVHU3RyZWFtZXIuaCBi
L1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2dzdHJlYW1lci9Tb3VyY2VCdWZmZXJQ
cml2YXRlR1N0cmVhbWVyLmgKaW5kZXggNTc0ZTIxYS4uYjBiYmNhNCAxMDA2NDQKLS0tIGEvU291
cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL1NvdXJjZUJ1ZmZlclByaXZh
dGVHU3RyZWFtZXIuaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3Ry
ZWFtZXIvU291cmNlQnVmZmVyUHJpdmF0ZUdTdHJlYW1lci5oCkBAIC00NCw3ICs0NCw3IEBAIHB1
YmxpYzoKICAgICBTb3VyY2VCdWZmZXJQcml2YXRlR1N0cmVhbWVyKFBhc3NSZWZQdHI8TWVkaWFT
b3VyY2VDbGllbnRHc3RyZWFtZXI+LCBjb25zdCBDb250ZW50VHlwZSYpOwogICAgIH5Tb3VyY2VC
dWZmZXJQcml2YXRlR1N0cmVhbWVyKCkgeyB9CiAKLSAgICB2b2lkIHNldENsaWVudChTb3VyY2VC
dWZmZXJQcml2YXRlQ2xpZW50KikgeyB9CisgICAgdm9pZCBzZXRDbGllbnQoU291cmNlQnVmZmVy
UHJpdmF0ZUNsaWVudCogY2xpZW50KSB7IG1fY2xpZW50ID0gY2xpZW50OyB9CiAgICAgdm9pZCBh
cHBlbmQoY29uc3QgdW5zaWduZWQgY2hhciosIHVuc2lnbmVkKTsKICAgICB2b2lkIGFib3J0KCk7
CiAgICAgdm9pZCByZW1vdmVkRnJvbU1lZGlhU291cmNlKCk7CkBAIC01NSw3ICs1NSw4IEBAIHB1
YmxpYzoKIAogcHJpdmF0ZToKICAgICBTdHJpbmcgbV90eXBlOwotICAgIFJlZlB0cjxNZWRpYVNv
dXJjZUNsaWVudEdzdHJlYW1lcj4gbV9jbGllbnQ7CisgICAgUmVmUHRyPE1lZGlhU291cmNlQ2xp
ZW50R3N0cmVhbWVyPiBtX21lZGlhU291cmNlQ2xpZW50OworICAgIFNvdXJjZUJ1ZmZlclByaXZh
dGVDbGllbnQqIG1fY2xpZW50OwogICAgIE1lZGlhUGxheWVyOjpSZWFkeVN0YXRlIG1fcmVhZHlT
dGF0ZTsKIH07CiAK
</data>
<flag name="review"
          id="267850"
          type_id="1"
          status="-"
          setter="pnormand"
    />
    <flag name="commit-queue"
          id="267851"
          type_id="3"
          status="-"
          setter="pnormand"
    />
          </attachment>
      

    </bug>

</bugzilla>