<?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>16920</bug_id>
          
          <creation_ts>2012-05-03 14:55:56 +0000</creation_ts>
          <short_desc>&lt;track&gt; Should run &quot;rules for updating the text track rendering&quot; when cue properties or styles have changed</short_desc>
          <delta_ts>2013-07-02 22:22:30 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WHATWG</product>
          <component>HTML</component>
          <version>unspecified</version>
          <rep_platform>Other</rep_platform>
          <op_sys>other</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://www.whatwg.org/specs/web-apps/current-work/#media-playback</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>Unsorted</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>contributor</reporter>
          <assigned_to name="Ian &apos;Hixie&apos; Hickson">ian</assigned_to>
          <cc>ian</cc>
    
    <cc>mike</cc>
    
    <cc>silviapfeiffer1</cc>
    
    <cc>zcorpan</cc>
          
          <qa_contact>contributor</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>67290</commentid>
    <comment_count>0</comment_count>
    <who name="">contributor</who>
    <bug_when>2012-05-03 14:55:56 +0000</bug_when>
    <thetext>Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html
Multipage: http://www.whatwg.org/C#media-playback
Complete: http://www.whatwg.org/c#media-playback

Comment:
Should run &quot;rules for updating the text track rendering&quot; when cue properties
or styles have changed

Posted from: 88.131.66.80 by simonp@opera.com
User agent: Opera/9.80 (Macintosh; Intel Mac OS X 10.7.3; U; en) Presto/2.10.229 Version/11.62</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67291</commentid>
    <comment_count>1</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2012-05-03 15:00:00 +0000</bug_when>
    <thetext>When the following properties have changed for an active cue, the rules for updating the text track rendering should be run:

the properties corresponding to the &apos;font&apos; shorthand, including &apos;line-height&apos;
&apos;white-space&apos;

When an active cue gets one of the following properties changed, the rules for updating the text track rendering should be run:
vertical
snapToLines
line
position
size
align
text

Also when changing startTime and endTime we might want to check if the cue should be currently active or not.

This should be done even if the video is paused, except if video playback hasn&apos;t ever started yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67473</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2012-05-08 10:48:09 +0000</bug_when>
    <thetext>Also when the native controls become visible or not visible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>70835</commentid>
    <comment_count>3</comment_count>
    <who name="">contributor</who>
    <bug_when>2012-07-18 17:35:50 +0000</bug_when>
    <thetext>This bug was cloned to create bug 18184 as part of operation convergence.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>71537</commentid>
    <comment_count>4</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2012-07-26 23:31:20 +0000</bug_when>
    <thetext>Not sure I agree about doing it when the controls appear/disappear, but I guess we should at least allow it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75131</commentid>
    <comment_count>5</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2012-10-02 22:45:51 +0000</bug_when>
    <thetext>Turns out the spec already does it for when the controls appear/disappear.

Still need to do startTime/endTime changes. Actually, it&apos;s more than that, because it looks like we don&apos;t yet handle showing captions when they&apos;re added while the video is paused, either.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75132</commentid>
    <comment_count>6</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2012-10-02 22:47:05 +0000</bug_when>
    <thetext>(Not quite sure how to fix that.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75133</commentid>
    <comment_count>7</comment_count>
    <who name="">contributor</who>
    <bug_when>2012-10-02 22:48:08 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r7430.
Check-in comment: Updating cues&apos; styles and properties dynamically should be visible immediately
http://html5.org/tools/web-apps-tracker?from=7429&amp;to=7430</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86219</commentid>
    <comment_count>8</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2013-04-15 18:14:09 +0000</bug_when>
    <thetext>I&apos;m not 100% sure this fix (see diff in next comment) really works, so if you see a problem with it please don&apos;t hesitate to reopen this bug (or file a new one).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86220</commentid>
    <comment_count>9</comment_count>
    <who name="">contributor</who>
    <bug_when>2013-04-15 18:15:42 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r7835.
Check-in comment: Make adding a cue, or changing its timings, work when the video is paused.
http://html5.org/tools/web-apps-tracker?from=7834&amp;to=7835</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86274</commentid>
    <comment_count>10</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2013-04-16 08:06:21 +0000</bug_when>
    <thetext>I think you overlooked this part in the two commits above:

&quot;This should be done even if the video is paused, except if video playback hasn&apos;t ever started yet.&quot;

Otherwise it looks good I think.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>86466</commentid>
    <comment_count>11</comment_count>
    <who name="Silvia Pfeiffer">silviapfeiffer1</who>
    <bug_when>2013-04-22 12:59:11 +0000</bug_when>
    <thetext>I also think that this line was changed by error:

-&lt;!-- EDITOR NOTES  -*- mode: Text; fill-column: 100 -*-
+ne&lt;!-- EDITOR NOTES  -*- mode: Text; fill-column: 100 -*-</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88661</commentid>
    <comment_count>12</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2013-06-04 20:52:08 +0000</bug_when>
    <thetext>zcorpan: What&apos;s the right way to trigger this when paused-but-not-started? Just call &quot;time marches on&quot; in the paragraph that adds the stuff to &quot;list of newly introduced cues&quot; when the element is paused, unless its &quot;show poster flag&quot; set is set?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88688</commentid>
    <comment_count>13</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2013-06-04 23:06:07 +0000</bug_when>
    <thetext>Yeah... or maybe bail out of &quot;time marches on&quot; as the first step if &quot;show poster flag&quot; is set.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89203</commentid>
    <comment_count>14</comment_count>
    <who name="">contributor</who>
    <bug_when>2013-06-12 19:38:37 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r7965.
Check-in comment: Make cues that are added while a video is paused get rendered before it gets unpaused.
http://html5.org/tools/web-apps-tracker?from=7964&amp;to=7965</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89208</commentid>
    <comment_count>15</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2013-06-12 20:42:51 +0000</bug_when>
    <thetext>Don&apos;t the cases that invoke &quot;time marches on&quot; in http://html5.org/tools/web-apps-tracker?from=7834&amp;to=7835 (except the first one) also need to check the show poster flag? Does the new commit add anything that wasn&apos;t already required?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89531</commentid>
    <comment_count>16</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2013-06-19 00:34:03 +0000</bug_when>
    <thetext>In answering your questions, I concluded that this was a big mess, and I tried fixing this all over again. Take a look at the next text, let me know what you think. Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89555</commentid>
    <comment_count>17</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2013-06-19 06:37:55 +0000</bug_when>
    <thetext>http://html5.org/r/7990</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89557</commentid>
    <comment_count>18</comment_count>
    <who name="Simon Pieters">zcorpan</who>
    <bug_when>2013-06-19 07:04:21 +0000</bug_when>
    <thetext>+      false, &lt;span&gt;queue a task&lt;/span&gt; to &lt;span&gt;fire a simple event&lt;/span&gt; named &lt;code
+      title=&quot;event-media-play&quot;&gt;play&lt;/code&gt; at the element, &lt;span&gt;queue a task&lt;/span&gt; to &lt;span&gt;fire a
+      simple event&lt;/span&gt; named &lt;code title=&quot;event-media-playing&quot;&gt;playing&lt;/code&gt; at the element, and
+      then, if the element&apos;s &lt;span&gt;show poster flag&lt;/span&gt; to false, first set it to false and then
+      run the &lt;i&gt;time marches on&lt;/i&gt; steps.&lt;/p&gt;

&quot;then&quot; being after the tasks have been queued? Or after they have run? If the latter, maybe it would be clearer to queue a third task?

Also s/to false/is true/

Otherwise it looks good, I think.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90159</commentid>
    <comment_count>19</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2013-07-02 22:22:22 +0000</bug_when>
    <thetext>I meant it to happen synchronously, before the tasks run, after they&apos;re queued. Normally I&apos;d queue them last, but then it&apos;s unclear if the queuing depends on the flag or not, etc. Anyway I made this better by using a list instead of a single paragraph.

Thanks for reviewing it!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90160</commentid>
    <comment_count>20</comment_count>
    <who name="">contributor</who>
    <bug_when>2013-07-02 22:22:30 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r8012.
Check-in comment: Clarify how autoplay and subtitles interact. Mostly editorial cleanup, though I change a &apos;false&apos; to a &apos;true&apos;, so not entirely.
http://html5.org/tools/web-apps-tracker?from=8011&amp;to=8012</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>