<?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>59828</bug_id>
          
          <creation_ts>2011-04-29 14:06:23 -0700</creation_ts>
          <short_desc>HTMLMediaElement should not seek to time 0 when readyState reaches HAVE_METADATA</short_desc>
          <delta_ts>2011-05-02 10:53:52 -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>Media</component>
          <version>528+ (Nightly build)</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>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Carlson">eric.carlson</reporter>
          <assigned_to name="Eric Carlson">eric.carlson</assigned_to>
          <cc>atwilson</cc>
    
    <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>395492</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-04-29 14:06:23 -0700</bug_when>
    <thetext>At some point in the past, the spec required a seek to time 0 once metadata is loaded. The spec text has been changed to say:

   If either the media resource or the address of the current 
   media resource indicate a particular start time, then set 
   the initial playback position to that time, seek to that time, 
   and let jumped be true. Ignore any resulting exceptions 
   (if the position is out of range, it is effectively ignored).

WebKit doesn&apos;t support starting at an offset, so we should not force a seek to time 0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>395831</commentid>
    <comment_count>1</comment_count>
      <attachid>91807</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-04-30 15:40:25 -0700</bug_when>
    <thetext>Created attachment 91807
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>395924</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-05-01 09:48:58 -0700</bug_when>
    <thetext>So this is not at all visible from javascript (that surprises me).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396080</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-05-01 15:56:44 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; So this is not at all visible from javascript (that surprises me).

It isn&apos;t visible because it is a noop - the media file has just been opened and it is no playing, so &quot;seeking&quot; to time zero does nothing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396090</commentid>
    <comment_count>4</comment_count>
      <attachid>91807</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-05-01 16:30:12 -0700</bug_when>
    <thetext>Comment on attachment 91807
Proposed patch

OK.  Would be nice to say that in the ChagneLog. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396466</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-05-02 10:53:52 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/85488 (with improved ChangeLog text ;-) )</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91807</attachid>
            <date>2011-04-30 15:40:25 -0700</date>
            <delta_ts>2011-05-01 16:30:12 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>patch_1.txt</filename>
            <type>text/plain</type>
            <size>1284</size>
            <attacher name="Eric Carlson">eric.carlson</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDg1NDAyKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTEtMDQtMzAgIEVyaWMgQ2Fy
bHNvbiAgPGVyaWMuY2FybHNvbkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C
T0RZIChPT1BTISkuCisKKyAgICAgICAgSFRNTE1lZGlhRWxlbWVudCBzaG91bGQgbm90IHNlZWsg
dG8gdGltZSAwIHdoZW4gcmVhZHlTdGF0ZSByZWFjaGVzIEhBVkVfTUVUQURBVEEKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU5ODI4CisKKyAgICAgICAg
Tm8gbmV3IHRlc3RzIHJlcXVpcmVkLCBubyBmdW5jdGlvbmFsIGNoYW5nZS4KKworICAgICAgICAq
IGh0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpIVE1MTWVkaWFF
bGVtZW50OjpzZXRSZWFkeVN0YXRlKTogRG9uJ3Qgc2VlayB0byB0aW1lIDAgd2hlbiByZWFkeVN0
YXRlIHJlYWNoZXMKKyAgICAgICAgICAgIEhBVkVfTUVUQURBVEEuCisKIDIwMTEtMDQtMzAgIFBh
dmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgTm90IHJldmll
d2VkOiBpbnNwZWN0b3IgdG9vbGJhciB0aXRsZXMgd2VyZSAycHggb2ZmLgpJbmRleDogU291cmNl
L1dlYkNvcmUvaHRtbC9IVE1MTWVkaWFFbGVtZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv
V2ViQ29yZS9odG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwCShyZXZpc2lvbiA4NTE3OSkKKysrIFNv
dXJjZS9XZWJDb3JlL2h0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAg
LTk4NCw3ICs5ODQsNiBAQCB2b2lkIEhUTUxNZWRpYUVsZW1lbnQ6OnNldFJlYWR5U3RhdGUoTWVk
CiAgICAgICAgICAgICBtZWRpYUNvbnRyb2xzKCktPmxvYWRlZE1ldGFkYXRhKCk7CiAgICAgICAg
IGlmIChyZW5kZXJlcigpKQogICAgICAgICAgICAgcmVuZGVyZXIoKS0+dXBkYXRlRnJvbUVsZW1l
bnQoKTsKLSAgICAgICAgbV9wbGF5ZXItPnNlZWsoMCk7CiAgICAgfQogCiAgICAgYm9vbCBzaG91
bGRVcGRhdGVEaXNwbGF5U3RhdGUgPSBmYWxzZTsK
</data>
<flag name="review"
          id="84687"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>