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

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>28234</bug_id>
          
          <creation_ts>2015-03-19 00:17:22 +0000</creation_ts>
          <short_desc>[MSE] &quot;highest presentation timestamp&quot; is ambiguous/misleading</short_desc>
          <delta_ts>2015-04-09 18:53:11 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>Media Source Extensions</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>CR</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Cunningham">chcunningham</reporter>
          <assigned_to name="Matt Wolenetz">wolenetz</assigned_to>
          <cc>acolwell</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-media</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>118669</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Cunningham">chcunningham</who>
    <bug_when>2015-03-19 00:17:22 +0000</bug_when>
    <thetext>&quot;set highest presentation timestamp for track buffer to frame end timestamp.&quot;

To me, the name &quot;highest presentation timestamp&quot; implies a definition as the presentation timestamp set on the frame which is furthest into the timeline for the coded frame group. You see in the quote above that it actually the presentation timestamp + the duration for that frame... 

I think this would be more easily understood if we called it &quot;highest frame end timestamp&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>118693</commentid>
    <comment_count>1</comment_count>
    <who name="Aaron Colwell">acolwell</who>
    <bug_when>2015-03-19 16:28:20 +0000</bug_when>
    <thetext>How about the following changes?

1. Change &quot;highest presentation timestamp&quot; to &quot;highest end timestamp&quot;
2. Change the definition of the variable to say &quot;stores the highest coded frame end timestamp across all coded frames in the current coded frame group that were appended to this track buffer.&quot;

This makes the definition more consistent with the &quot;group end timestamp&quot; name and definition.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>118694</commentid>
    <comment_count>2</comment_count>
    <who name="Chris Cunningham">chcunningham</who>
    <bug_when>2015-03-19 17:30:13 +0000</bug_when>
    <thetext>Aaron, that sounds great to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119310</commentid>
    <comment_count>3</comment_count>
    <who name="Matt Wolenetz">wolenetz</who>
    <bug_when>2015-04-07 19:54:17 +0000</bug_when>
    <thetext>Pull request for the fix recommended in Comment 1 is pending at https://github.com/w3c/media-source/pull/4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119404</commentid>
    <comment_count>4</comment_count>
    <who name="Matt Wolenetz">wolenetz</who>
    <bug_when>2015-04-09 18:53:11 +0000</bug_when>
    <thetext>Change committed...
https://github.com/w3c/media-source/commit/3ea5e00384f132dc02df04a2c0d806cdb15cfde1</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>