<?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>62655</bug_id>
          
          <creation_ts>2011-06-14 11:20:32 -0700</creation_ts>
          <short_desc>REGRESSION: Drawing video into canvas gets about one frame every 5 seconds (AVFoundation)</short_desc>
          <delta_ts>2011-06-14 13:35:25 -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>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://www.craftymind.com/factory/html5video/CanvasVideo.html</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="Jer Noble">jer.noble</reporter>
          <assigned_to name="Jer Noble">jer.noble</assigned_to>
          <cc>eric.carlson</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>420593</commentid>
    <comment_count>0</comment_count>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2011-06-14 11:20:32 -0700</bug_when>
    <thetext>REGRESSION: Drawing video into canvas gets about one frame every 5 seconds (AVFoundation)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420599</commentid>
    <comment_count>1</comment_count>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2011-06-14 11:25:12 -0700</bug_when>
    <thetext>http://www.craftymind.com/factory/html5video/CanvasVideo.html has terrible performance on Lion. On SL, it&apos;s pretty smooth, but on Lion the animation is really choppy.

Sampling shows lots of time in -[AVAssetImageGenerator copyCGImageAtTime:actualTime:error:] (attached.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420600</commentid>
    <comment_count>2</comment_count>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2011-06-14 11:25:26 -0700</bug_when>
    <thetext>&lt;rdar://problem/9569220&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420601</commentid>
    <comment_count>3</comment_count>
      <attachid>97143</attachid>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2011-06-14 11:26:38 -0700</bug_when>
    <thetext>Created attachment 97143
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420602</commentid>
    <comment_count>4</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-06-14 11:27:59 -0700</bug_when>
    <thetext>Note that frequency of new frames is directly related to the key frame frequency in the video file because we are not resetting AVFoundation&apos;s default of snapping the time value to the nearest key frame time.

This makes the title is slightly misleading, as we could get one frame every five seconds, one frame every minute, ten frames per second, etc.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420604</commentid>
    <comment_count>5</comment_count>
      <attachid>97143</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2011-06-14 11:28:47 -0700</bug_when>
    <thetext>Comment on attachment 97143
Patch

Nice simple fix!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420697</commentid>
    <comment_count>6</comment_count>
    <who name="Jer Noble">jer.noble</who>
    <bug_when>2011-06-14 13:35:25 -0700</bug_when>
    <thetext>Committed r88847: &lt;http://trac.webkit.org/changeset/88847&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>97143</attachid>
            <date>2011-06-14 11:26:38 -0700</date>
            <delta_ts>2011-06-14 11:28:47 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-62655-20110614112637.patch</filename>
            <type>text/plain</type>
            <size>2096</size>
            <attacher name="Jer Noble">jer.noble</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODg2NTQKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCBkODBhMTlmOGEzNzI1ZGUy
MjNjMjdhMjJlNzBkYWYwMzk0NjUzODA5Li4zZTg3Yzg1YTA2YzBlYzMzMjNlOWFmZDMzMTc5ZWVi
OGVkOGVlMzQzIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkgQEAKKzIwMTEtMDYtMTQgIEplciBO
b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgUkVHUkVTU0lPTjogRHJhd2luZyB2aWRlbyBpbnRvIGNhbnZh
cyBnZXRzIGFib3V0IG9uZSBmcmFtZSBldmVyeSA1IHNlY29uZHMgKEFWRm91bmRhdGlvbikKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTYyNjU1CisKKyAg
ICAgICAgTm8gbmV3IHRlc3RzOyB3ZSBkb24ndCBoYXZlIGFueSBtZWRpYS1lbmdpbmUgc3BlY2lm
aWMgdGVzdHMuCisKKyAgICAgICAgQVZBc3NldEltYWdlR2VuZXJhdG9yIHdpbGwsIGJ5IGRlZmF1
bHQsIHJldHVybiB0aGUgbmVhcmVzdCBrZXlmcmFtZS4gIFRvIGdldCB0aGUgZ2VuZXJhdG9yIHRv
IHJldHVybgorICAgICAgICB0aGUgaW1hZ2UgZnJvbSB0aGUgY3VycmVudCB0aW1lIGluc3RlYWQg
b2YgdGhlIG5lYXJlc3Qga2V5ZnJhbWUsIHNldCB0aGUgcmVxdWVzdGVkIHRpbWUgdG9sZXJhbmNl
IHRvIAorICAgICAgICB6ZXJvICh3aGljaCBkZWZhdWx0cyB0byBpbmZpbml0eSkuCisKKyAgICAg
ICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9hdmZvdW5kYXRpb24vTWVkaWFQbGF5ZXJQcml2YXRlQVZG
b3VuZGF0aW9uT2JqQy5tbToKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZUFW
Rm91bmRhdGlvbk9iakM6OmNyZWF0ZUNvbnRleHRWaWRlb1JlbmRlcmVyKToKKwogMjAxMS0wNi0x
MSAgSmVyIE5vYmxlICA8amVyLm5vYmxlQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBi
eSBBbmRlcnMgQ2FybHNzb24uCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9n
cmFwaGljcy9hdmZvdW5kYXRpb24vTWVkaWFQbGF5ZXJQcml2YXRlQVZGb3VuZGF0aW9uT2JqQy5t
bSBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2F2Zm91bmRhdGlvbi9NZWRpYVBs
YXllclByaXZhdGVBVkZvdW5kYXRpb25PYmpDLm1tCmluZGV4IDBlZWM5MzNmMjUwOGZiYmI4YmQy
ODc1ZjBjZTVlM2YzNTM5Yjc2MzMuLjVhMDNjNzY5YmI3MjNmN2ExZTI4OTkzNjhlMTFiYmRhMzZm
OWJjODMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2F2Zm91
bmRhdGlvbi9NZWRpYVBsYXllclByaXZhdGVBVkZvdW5kYXRpb25PYmpDLm1tCisrKyBiL1NvdXJj
ZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2F2Zm91bmRhdGlvbi9NZWRpYVBsYXllclByaXZh
dGVBVkZvdW5kYXRpb25PYmpDLm1tCkBAIC0yMDEsNiArMjAxLDggQEAgdm9pZCBNZWRpYVBsYXll
clByaXZhdGVBVkZvdW5kYXRpb25PYmpDOjpjcmVhdGVDb250ZXh0VmlkZW9SZW5kZXJlcigpCiAK
ICAgICBbbV9pbWFnZUdlbmVyYXRvci5nZXQoKSBzZXRBcGVydHVyZU1vZGU6QVZBc3NldEltYWdl
R2VuZXJhdG9yQXBlcnR1cmVNb2RlQ2xlYW5BcGVydHVyZV07CiAgICAgW21faW1hZ2VHZW5lcmF0
b3IuZ2V0KCkgc2V0QXBwbGllc1ByZWZlcnJlZFRyYWNrVHJhbnNmb3JtOllFU107CisgICAgW21f
aW1hZ2VHZW5lcmF0b3IuZ2V0KCkgc2V0UmVxdWVzdGVkVGltZVRvbGVyYW5jZUJlZm9yZTprQ01U
aW1lWmVyb107CisgICAgW21faW1hZ2VHZW5lcmF0b3IuZ2V0KCkgc2V0UmVxdWVzdGVkVGltZVRv
bGVyYW5jZUFmdGVyOmtDTVRpbWVaZXJvXTsKIAogICAgIExPRyhNZWRpYSwgIk1lZGlhUGxheWVy
UHJpdmF0ZUFWRm91bmRhdGlvbk9iakM6OmNyZWF0ZUltYWdlR2VuZXJhdG9yKCVwKSAtIHJldHVy
bmluZyAlcCIsIHRoaXMsIG1faW1hZ2VHZW5lcmF0b3IuZ2V0KCkpOwogfQo=
</data>
<flag name="review"
          id="91009"
          type_id="1"
          status="+"
          setter="eric.carlson"
    />
          </attachment>
      

    </bug>

</bugzilla>