<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GStreamer] usec rounding is wrong during accurate seeking"
   href="https://bugs.webkit.org/show_bug.cgi?id=90734#c26">Comment # 26</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GStreamer] usec rounding is wrong during accurate seeking"
   href="https://bugs.webkit.org/show_bug.cgi?id=90734">bug 90734</a>
              from <span class="vcard"><a class="email" href="mailto:pnormand&#64;igalia.com" title="Philippe Normand &lt;pnormand&#64;igalia.com&gt;"> <span class="fn">Philippe Normand</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=90734#c24">comment #24</a>)
<span class="quote">&gt; Comment on <span class=""><a href="attachment.cgi?id=238352&amp;action=diff" name="attach_238352" title="Patch">attachment 238352</a> <a href="attachment.cgi?id=238352&amp;action=edit" title="Patch">[details]</a></span>
&gt; Patch
&gt; 
&gt; View in context:
&gt; <a href="https://bugs.webkit.org/attachment.cgi?id=238352&amp;action=review">https://bugs.webkit.org/attachment.cgi?id=238352&amp;action=review</a>
&gt; 
&gt; The code changes in this patch look fine, but I don't understand the test.
&gt; 
&gt; &gt; LayoutTests/media/video-frame-accurate-seek.html:8
&gt; &gt; +        &lt;p&gt;Test that setting currentTime is frame-accurate. Four videos below should be showing up, The first three at frames 12, 13, and 14. The last video should display at 6.0272 seconds.&lt;/p&gt;
&gt; 
&gt; The -&gt; the
&gt; 
&gt; &gt; LayoutTests/platform/gtk/media/video-frame-accurate-seek-expected.txt:9
&gt; &gt; +          text run at (0,17) width 356: &quot;and 14. The last video should display at 6.0272 seconds.&quot;
&gt; 
&gt; It seems like something is wrong with this test. The expected result for the
&gt; first three videos shows 12, 13, and 15, but the test says it's expected to
&gt; show 12, 13, and 14. </span >

This now works as expected \o/

<span class="quote">&gt; The last video displays 6.01 seconds, not 6.0272 or
&gt; 6.03.
&gt; </span >

That's because 6.01 means (6 * 25) + 1 frames = 151. And not 0.01 seconds.

So we do a seek to 6.0272 seconds which means 6 seconds and one half-frame (0.02 = 1/50) but the video is at 25fps so the demuxer choses instead to jump to the next full frame, if I understand correctly.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>