<?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>136408</bug_id>
          
          <creation_ts>2014-08-30 22:23:35 -0700</creation_ts>
          <short_desc>SVGImage::drawPatternForContainer creates a buffer without respecting the destination&apos;s acceleration setting</short_desc>
          <delta_ts>2014-08-31 00:42:11 -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>SVG</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></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="Tim Horton">thorton</reporter>
          <assigned_to name="Tim Horton">thorton</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>d-r</cc>
    
    <cc>fmalita</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>mitz</cc>
    
    <cc>pdr</cc>
    
    <cc>schenney</cc>
    
    <cc>sergio</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1032372</commentid>
    <comment_count>0</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2014-08-30 22:23:35 -0700</bug_when>
    <thetext>Mismatched acceleration settings == terrible performance.

rdar://problem/12013317</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032373</commentid>
    <comment_count>1</comment_count>
      <attachid>237419</attachid>
    <who name="Tim Horton">thorton</who>
    <bug_when>2014-08-30 22:25:17 -0700</bug_when>
    <thetext>Created attachment 237419
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032374</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-08-30 22:27:32 -0700</bug_when>
    <thetext>Attachment 237419 did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032380</commentid>
    <comment_count>3</comment_count>
      <attachid>237419</attachid>
    <who name="">mitz</who>
    <bug_when>2014-08-31 00:04:05 -0700</bug_when>
    <thetext>Comment on attachment 237419
patch

Any way to test this? Is this a performance enhancement or something else?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032383</commentid>
    <comment_count>4</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2014-08-31 00:34:01 -0700</bug_when>
    <thetext>Just performance, no way to test (unless we were to add some mechanism to log when someone paints from an unaccelerated buffer to an accelerated context or vice versa, and then wrote some tests around that).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032384</commentid>
    <comment_count>5</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2014-08-31 00:42:11 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/173143</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>237419</attachid>
            <date>2014-08-30 22:25:17 -0700</date>
            <delta_ts>2014-08-31 00:39:36 -0700</delta_ts>
            <desc>patch</desc>
            <filename>accel.diff</filename>
            <type>text/plain</type>
            <size>1923</size>
            <attacher name="Tim Horton">thorton</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAzZmE0Yjc4Li4yNTA0ZmRiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTgg
QEAKKzIwMTQtMDgtMzAgIFRpbSBIb3J0b24gIDx0aW1vdGh5X2hvcnRvbkBhcHBsZS5jb20+CisK
KyAgICAgICAgU1ZHSW1hZ2U6OmRyYXdQYXR0ZXJuRm9yQ29udGFpbmVyIGNyZWF0ZXMgYSBidWZm
ZXIgd2l0aG91dCByZXNwZWN0aW5nIHRoZSBkZXN0aW5hdGlvbidzIGFjY2VsZXJhdGlvbiBzZXR0
aW5nCisgICAgICAgIE5lZWQgdGhlIGJ1ZyBVUkwgKE9PUFMhKS4KKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIHN2Zy9ncmFwaGljcy9TVkdJbWFnZS5j
cHA6CisgICAgICAgIChXZWJDb3JlOjpTVkdJbWFnZTo6ZHJhd1BhdHRlcm5Gb3JDb250YWluZXIp
OgorICAgICAgICBVc2UgSW1hZ2VCdWZmZXI6OmNyZWF0ZUNvbXBhdGlibGVCdWZmZXIsIHdoaWNo
IHBhc3NlcyB0aGUgZGVzdGluYXRpb24ncworICAgICAgICBhY2NlbGVyYXRpb24gc2V0dGluZyB0
aHJvdWdoIHRvIHRoZSBJbWFnZUJ1ZmZlciBjb25zdHJ1Y3Rvci4KKworICAgICAgICBJIGRpZG4n
dCB1c2UgR3JhcGhpY3NDb250ZXh0OjpjcmVhdGVDb21wYXRpYmxlQnVmZmVyIGJlY2F1c2UgYWRq
dXN0aW5nCisgICAgICAgIGhvdyB0aGlzIGZ1bmN0aW9uIGFwcGxpZXMgdGhlIGRlc3RpbmF0aW9u
J3MgQ1RNIGlzIG91dHNpZGUgdGhlIHNjb3BlIG9mIHRoaXMgcGF0Y2guCisKIDIwMTQtMDgtMzAg
IEpvc2VwaCBQZWNvcmFybyAgPHBlY29yYXJvQGFwcGxlLmNvbT4KIAogICAgICAgICBDb252ZXJ0
IHN0cmluZyBsaXRlcmFscyB0byBjaGFyYWN0ZXIgbGl0ZXJhbHMgaW4gbWFrZVN0cmluZyB1c2Fn
ZQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvc3ZnL2dyYXBoaWNzL1NWR0ltYWdlLmNwcCBi
L1NvdXJjZS9XZWJDb3JlL3N2Zy9ncmFwaGljcy9TVkdJbWFnZS5jcHAKaW5kZXggMDQxMDJkYi4u
YTQ4OTE5MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvc3ZnL2dyYXBoaWNzL1NWR0ltYWdl
LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9zdmcvZ3JhcGhpY3MvU1ZHSW1hZ2UuY3BwCkBAIC0x
OTQsNyArMTk0LDcgQEAgdm9pZCBTVkdJbWFnZTo6ZHJhd1BhdHRlcm5Gb3JDb250YWluZXIoR3Jh
cGhpY3NDb250ZXh0KiBjb250ZXh0LCBjb25zdCBGbG9hdFNpemUKICAgICBGbG9hdFJlY3QgaW1h
Z2VCdWZmZXJTaXplID0gem9vbWVkQ29udGFpbmVyUmVjdDsKICAgICBpbWFnZUJ1ZmZlclNpemUu
c2NhbGUoaW1hZ2VCdWZmZXJTY2FsZS53aWR0aCgpLCBpbWFnZUJ1ZmZlclNjYWxlLmhlaWdodCgp
KTsKIAotICAgIHN0ZDo6dW5pcXVlX3B0cjxJbWFnZUJ1ZmZlcj4gYnVmZmVyID0gSW1hZ2VCdWZm
ZXI6OmNyZWF0ZShleHBhbmRlZEludFNpemUoaW1hZ2VCdWZmZXJTaXplLnNpemUoKSksIDEpOwor
ICAgIHN0ZDo6dW5pcXVlX3B0cjxJbWFnZUJ1ZmZlcj4gYnVmZmVyID0gSW1hZ2VCdWZmZXI6OmNy
ZWF0ZUNvbXBhdGlibGVCdWZmZXIoZXhwYW5kZWRJbnRTaXplKGltYWdlQnVmZmVyU2l6ZS5zaXpl
KCkpLCAxLCBDb2xvclNwYWNlRGV2aWNlUkdCLCBjb250ZXh0LCB0cnVlKTsKICAgICBpZiAoIWJ1
ZmZlcikgLy8gRmFpbGVkIHRvIGFsbG9jYXRlIGJ1ZmZlci4KICAgICAgICAgcmV0dXJuOwogICAg
IGRyYXdGb3JDb250YWluZXIoYnVmZmVyLT5jb250ZXh0KCksIGNvbnRhaW5lclNpemUsIHpvb20s
IGltYWdlQnVmZmVyU2l6ZSwgem9vbWVkQ29udGFpbmVyUmVjdCwgQ29sb3JTcGFjZURldmljZVJH
QiwgQ29tcG9zaXRlU291cmNlT3ZlciwgQmxlbmRNb2RlTm9ybWFsKTsK
</data>
<flag name="review"
          id="262146"
          type_id="1"
          status="+"
          setter="mitz"
    />
          </attachment>
      

    </bug>

</bugzilla>