<?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>208120</bug_id>
          
          <creation_ts>2020-02-24 00:01:09 -0800</creation_ts>
          <short_desc>[GTK] fast/mediastream/media-element-current-time.html is failing since added in r256346</short_desc>
          <delta_ts>2022-08-08 10:48:24 -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>New Bugs</component>
          <version>WebKit 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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Diego Pino">dpino</reporter>
          <assigned_to name="Philippe Normand">philn</assigned_to>
          <cc>eric.carlson</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>glenn</cc>
    
    <cc>hta</cc>
    
    <cc>jer.noble</cc>
    
    <cc>philipj</cc>
    
    <cc>philn</cc>
    
    <cc>pnormand</cc>
    
    <cc>sergio</cc>
    
    <cc>tommyw</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1622063</commentid>
    <comment_count>0</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-02-24 00:01:09 -0800</bug_when>
    <thetext>* fast/mediastream/media-element-current-time.html [ Failure ]

--- /home/igalia/dpino/workspace/webkit-gtk/layout-test-results/retries/fast/mediastream/media-element-current-time-expected.txt
+++ /home/igalia/dpino/workspace/webkit-gtk/layout-test-results/retries/fast/mediastream/media-element-current-time-actual.txt
@@ -1,4 +1,4 @@


-PASS Check video.currentTime behavior 
+FAIL Check video.currentTime behavior assert_not_equals: Playback has started, currentTime must not be zero got disallowed value 0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1770671</commentid>
    <comment_count>1</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2021-06-17 09:15:28 -0700</bug_when>
    <thetext>I think the way this test was written kind of makes it prone to flakiness. Changing the playback state and waiting 10ms until performing a test is not great for the gst engine.

It would be better to rely on events.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1826289</commentid>
    <comment_count>2</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2021-12-22 06:50:18 -0800</bug_when>
    <thetext>Current failure is:


-PASS Check video.currentTime behavior
+FAIL Check video.currentTime behavior assert_greater_than: video is playing, time should advance expected a number greater than 0.284381 but got 0.284381
 

Which happens because the position is queried immediately after the play() call. Waiting 10ms gives time to the engine to have the pipeline position going forward.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1827021</commentid>
    <comment_count>3</comment_count>
      <attachid>448027</attachid>
    <who name="Diego Pino">dpino</who>
    <bug_when>2021-12-27 20:27:08 -0800</bug_when>
    <thetext>Created attachment 448027
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1827139</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-12-28 17:17:14 -0800</bug_when>
    <thetext>Committed r287477 (245612@main): &lt;https://commits.webkit.org/245612@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 448027.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1827140</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-12-28 17:18:20 -0800</bug_when>
    <thetext>&lt;rdar://problem/86967699&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1889494</commentid>
    <comment_count>6</comment_count>
    <who name="Philippe Normand">philn</who>
    <bug_when>2022-08-07 01:57:22 -0700</bug_when>
    <thetext>This is still very flaky.

(In reply to Philippe Normand from comment #1)
&gt; I think the way this test was written kind of makes it prone to flakiness.
&gt; Changing the playback state and waiting 10ms until performing a test is not
&gt; great for the gst engine.
&gt; 
&gt; It would be better to rely on events.

Patch incoming...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1889495</commentid>
    <comment_count>7</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2022-08-07 01:59:53 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3085</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1889711</commentid>
    <comment_count>8</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-08-08 10:48:22 -0700</bug_when>
    <thetext>Committed 253220@main (3e994a2e3142): &lt;https://commits.webkit.org/253220@main&gt;

Reviewed commits have been landed. Closing PR #3085 and removing active labels.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>448027</attachid>
            <date>2021-12-27 20:27:08 -0800</date>
            <delta_ts>2021-12-28 17:17:15 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-208120-20211228122705.patch</filename>
            <type>text/plain</type>
            <size>2717</size>
            <attacher name="Diego Pino">dpino</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg3NDYxCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYTM3ZjJhYjFkM2Y2Yjk1NTE5YWEx
M2ZkMWIxOTMwNzlmZWZmYWRjNi4uNDMwODBiNjhlYTgyY2RlYTY1OTQzZDlkYmEwNmJmN2IyZjY4
MTU0OCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDIxLTEyLTI3ICBEaWVnbyBQaW5vIEdhcmNp
YSAgPGRwaW5vQGlnYWxpYS5jb20+CisKKyAgICAgICAgW0dUS10gZmFzdC9tZWRpYXN0cmVhbS9t
ZWRpYS1lbGVtZW50LWN1cnJlbnQtdGltZS5odG1sIGlzIGZhaWxpbmcgc2luY2UgYWRkZWQgaW4g
cjI1NjM0NgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
MjA4MTIwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
V2FpdCAxMG1zIGFmdGVyIHBsYXkoKSB0byBnaXZlIHRpbWUgdG8gdGhlIGVuZ2luZSB0byBoYXZl
IHRoZSBwaXBlbGluZSBwb3NpdGlvbiBnb2luZyBmb3J3YXJkLgorCisgICAgICAgICogZmFzdC9t
ZWRpYXN0cmVhbS9tZWRpYS1lbGVtZW50LWN1cnJlbnQtdGltZS5odG1sOgorICAgICAgICAqIHBs
YXRmb3JtL2dsaWIvVGVzdEV4cGVjdGF0aW9uczoKKwogMjAyMS0xMi0yNyAgVGltIE5ndXllbiAg
PG50aW1AYXBwbGUuY29tPgogCiAgICAgICAgIENvbnNvbGlkYXRlIGNvbXB1dGVkLXN0eWxlLSou
dHh0IGFuZCBnZXRDb21wdXRlZFN0eWxlLWJhc2ljLWV4cGVjdGVkLnR4dCB0ZXN0IHJlc3VsdHMK
ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvbWVkaWFzdHJlYW0vbWVkaWEtZWxlbWVudC1j
dXJyZW50LXRpbWUuaHRtbCBiL0xheW91dFRlc3RzL2Zhc3QvbWVkaWFzdHJlYW0vbWVkaWEtZWxl
bWVudC1jdXJyZW50LXRpbWUuaHRtbAppbmRleCAzZGI3NGRjYzYxNzA5YzZhYWNkZDdjNmE1MDhi
ZGU3NGI3OTA5NzY5Li5mMTI1YzJkNGZhMzZkNWZkMjBkZjIyYzQyYmZlN2JhZjllNzNjMDE4IDEw
MDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L21lZGlhc3RyZWFtL21lZGlhLWVsZW1lbnQtY3Vy
cmVudC10aW1lLmh0bWwKKysrIGIvTGF5b3V0VGVzdHMvZmFzdC9tZWRpYXN0cmVhbS9tZWRpYS1l
bGVtZW50LWN1cnJlbnQtdGltZS5odG1sCkBAIC0yNCw2ICsyNCw3IEBAIHByb21pc2VfdGVzdChh
c3luYygpID0+IHsKICAgICBhc3NlcnRfZXF1YWxzKHZpZGVvLmN1cnJlbnRUaW1lLCBjdXJyZW50
VGltZSwgInZpZGVvIGlzIHBhdXNlZCwgY3VycmVudFRpbWUgbXVzdCBub3QgYWR2YW5jZSIpOwog
CiAgICAgYXdhaXQgdmlkZW8ucGxheSgpOworICAgIGF3YWl0IG5ldyBQcm9taXNlKHJlc29sdmUg
PT4gc2V0VGltZW91dChyZXNvbHZlLCAxMCkpOwogICAgIGFzc2VydF9ncmVhdGVyX3RoYW4odmlk
ZW8uY3VycmVudFRpbWUsIGN1cnJlbnRUaW1lLCAiUGxheWJhY2sgaGFzIHN0YXJ0ZWQsIGN1cnJl
bnRUaW1lIHNob3VsZCBhZHZhbmNlIik7CiAKIH0sICJDaGVjayB2aWRlby5jdXJyZW50VGltZSBi
ZWhhdmlvciIpOwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ2xpYi9UZXN0RXhw
ZWN0YXRpb25zIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ2xpYi9UZXN0RXhwZWN0YXRpb25zCmlu
ZGV4IGI1MGQzNzZjYjlhZDNmN2I2MjdmMjliYmU2MTU4ZDRiZTZkMTlmNzQuLjk1YjQ2N2RjYzM1
OWQyN2FiNWJiZDQ1ZmY1MzhmZDYwZGU5YzMxYTUgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3Bs
YXRmb3JtL2dsaWIvVGVzdEV4cGVjdGF0aW9ucworKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9n
bGliL1Rlc3RFeHBlY3RhdGlvbnMKQEAgLTIyODQsNyArMjI4NCw2IEBAIHdlYmtpdC5vcmcvYi8y
MDc2NzggaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy91cmwvYS1lbGVtZW50LW9yaWdp
bi14aHRtbC54CiB3ZWJraXQub3JnL2IvMjA3Njc4IGltcG9ydGVkL3czYy93ZWItcGxhdGZvcm0t
dGVzdHMvdXJsL2EtZWxlbWVudC14aHRtbC54aHRtbCBbIEZhaWx1cmUgXQogd2Via2l0Lm9yZy9i
LzIwNzcxMSBbIERlYnVnIF0gZmFzdC9zZWxlY3RvcnMvc2xvdy1zdHlsZS1zaGFyaW5nLXdpdGgt
bG9uZy1jb3VzaW4tbGlzdC5odG1sIFsgVGltZW91dCBdCiB3ZWJraXQub3JnL2IvMjA3NzExIFsg
RGVidWcgXSBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL2Nzcy9tZWRpYXF1ZXJpZXMv
dGVzdF9tZWRpYV9xdWVyaWVzLmh0bWwgWyBUaW1lb3V0IF0KLXdlYmtpdC5vcmcvYi8yMDgxMjAg
ZmFzdC9tZWRpYXN0cmVhbS9tZWRpYS1lbGVtZW50LWN1cnJlbnQtdGltZS5odG1sIFsgRmFpbHVy
ZSBdCiB3ZWJraXQub3JnL2IvMjA4MTIxIGh0dHAvdGVzdHMvY2FjaGUvZGlzay1jYWNoZS9kaXNr
LWNhY2hlLXZhcnktY29va2llLmh0bWwgWyBGYWlsdXJlIF0KIHdlYmtpdC5vcmcvYi8yMDgxMzkg
aHR0cC90ZXN0cy9jYWNoZS9kaXNrLWNhY2hlL2Rpc2stY2FjaGUtdmFyeS1jb29raWUtcHJpdmF0
ZS5odG1sIFsgRmFpbHVyZSBdCiB3ZWJraXQub3JnL2IvMjA4MzY4IGltcG9ydGVkL3czYy93ZWIt
cGxhdGZvcm0tdGVzdHMvd2FzbS9qc2FwaS9jb25zdHJ1Y3Rvci9pbnN0YW50aWF0ZS1iYWQtaW1w
b3J0cy5hbnkud29ya2VyLmh0bWwgWyBGYWlsdXJlIFBhc3MgXQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>