<?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>30823</bug_id>
          
          <creation_ts>2009-10-27 08:29:54 -0700</creation_ts>
          <short_desc>Fix alpha handling when using wxWidgets 2.9</short_desc>
          <delta_ts>2009-11-09 12:37:58 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit wx</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="Vadim Zeitlin">vz-webkit</reporter>
          <assigned_to name="Kevin Ollivier">kevino</assigned_to>
          <cc>kevino</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>158197</commentid>
    <comment_count>0</comment_count>
      <attachid>41955</attachid>
    <who name="Vadim Zeitlin">vz-webkit</who>
    <bug_when>2009-10-27 08:29:54 -0700</bug_when>
    <thetext>Created attachment 41955
Patch fixing alpha transparency handling for wx port

Transparency in PNG images didn&apos;t work in wxWebKit because the wxBitmap &quot;has alpha&quot; internal flag was not set before wxGraphicsBitmap was created from it. This flag is set in wxPixelData dtor and, generally speaking, bitmap data can&apos;t be accessed at all while it is locked for the raw access via wxPixelData so we must ensure that the local &quot;data&quot; variable is destroyed before using bitmap in any way.

This patch does just this: in spite of many apparent changes it simply adds a block around &quot;data&quot; variable to ensure that it is destroyed (this should be more clear if the patch is applied and &quot;diff -b&quot; is done).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158200</commentid>
    <comment_count>1</comment_count>
    <who name="Vadim Zeitlin">vz-webkit</who>
    <bug_when>2009-10-27 08:32:17 -0700</bug_when>
    <thetext>Sorry, I forgot to mention that this patch was done on top of bug 30822. However even if 30822 is not applied, this one should still be as it fixes a real user-visible bug. Please let me know if I should redo the patch against the base line (OTOH it basically just adds two lines with &quot;{&quot; and &quot;}&quot; so maybe there is no need for this).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158294</commentid>
    <comment_count>2</comment_count>
      <attachid>41955</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-10-27 11:48:45 -0700</bug_when>
    <thetext>Comment on attachment 41955
Patch fixing alpha transparency handling for wx port

Your ChangeLog does not make it clear what this patch does.

I believe webkit style is to use spaces around operators:
+            p.Red() = bytes[i+2];
+            p.Green() = bytes[i+1];
+            p.Blue() = bytes[i+0];
+            p.Alpha() = bytes[i+3];</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158433</commentid>
    <comment_count>3</comment_count>
    <who name="Vadim Zeitlin">vz-webkit</who>
    <bug_when>2009-10-27 17:28:17 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 41955 [details])
&gt; Your ChangeLog does not make it clear what this patch does.

I&apos;ve tried to make it more precise while still remaining brief. I hope it&apos;s more clear now.
 
&gt; I believe webkit style is to use spaces around operators:
&gt; +            p.Red() = bytes[i+2];
&gt; +            p.Green() = bytes[i+1];
&gt; +            p.Blue() = bytes[i+0];
&gt; +            p.Alpha() = bytes[i+3];

Sorry, I just indented this code, I didn&apos;t modify it at all. I&apos;ve reformatted it now and will attach the new patch in a moment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>158434</commentid>
    <comment_count>4</comment_count>
      <attachid>41997</attachid>
    <who name="Vadim Zeitlin">vz-webkit</who>
    <bug_when>2009-10-27 17:29:13 -0700</bug_when>
    <thetext>Created attachment 41997
Patch fixing alpha transparency handling for wx port (v2)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>161061</commentid>
    <comment_count>5</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2009-11-05 20:55:17 -0800</bug_when>
    <thetext>Hi Vadim,

The patch itself does resolve the problem, but this patch won&apos;t apply cleanly because you made the patch on top of a branch that has your changes to NativeImagePtr in it. If you submit a fixed patch against stock trunk sources, I&apos;ll be happy to r+ and land it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>161290</commentid>
    <comment_count>6</comment_count>
      <attachid>42682</attachid>
    <who name="Vadim Zeitlin">vz-webkit</who>
    <bug_when>2009-11-06 18:35:08 -0800</bug_when>
    <thetext>Created attachment 42682
&quot;Standalone&quot; version of the patch 

Here is the version of the patch which should apply cleanly. As I wrote, it&apos;s completely trivial: it just adds a scope (and also spaces around &quot;+&quot; operator as requested although IMO this shouldn&apos;t be applied together with this patch at all as it has nothing to do with it...).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>161708</commentid>
    <comment_count>7</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2009-11-09 12:37:58 -0800</bug_when>
    <thetext>Landed in r50677, thanks! Note that the patch didn&apos;t conform to WebKit style since the code inside the braces was not indented, but as I wanted to get this landed, I fixed the issue before commit this time. However, most reviewers will r- a patch that doesn&apos;t conform, so things will go quicker if you make sure style issues are addressed before submission.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>41955</attachid>
            <date>2009-10-27 08:29:54 -0700</date>
            <delta_ts>2009-10-27 17:29:13 -0700</delta_ts>
            <desc>Patch fixing alpha transparency handling for wx port</desc>
            <filename>alpha-fix.diff</filename>
            <type>text/plain</type>
            <size>3765</size>
            <attacher name="Vadim Zeitlin">vz-webkit</attacher>
            
              <data encoding="base64">Y29tbWl0IGViZTBiNDM5NTgxMmVhMDc5YmNhNmQ3Y2NhZTQ5YmI3MTUwZmNmOWMKQXV0aG9yOiBW
YWRpbSBaZWl0bGluIDx2YWRpbUB3eHdpZGdldHMub3JnPgpEYXRlOiAgIFR1ZSBPY3QgMjcgMTY6
MjU6MzkgMjAwOSArMDEwMAoKICAgIEZpeCBoYW5kbGluZyBvZiBhbHBoYSBjaGFuZ2VzIGluIHd4
IHBvcnQgd2hlbiB1c2luZyAyLjkrLgogICAgCiAgICB3eFBpeGVsRGF0YSBvYmplY3QgdXNlZCBm
b3IgcmF3IGFjY2VzcyB0byB0aGUgYml0bWFwIGRhdGEgbXVzdCBiZSBkZXN0cm95ZWQKICAgIGJl
Zm9yZSB0aGUgYml0bWFwIGlzIHVzZWQgaW4gYW55IG90aGVyIHdheSwgZmFpbGluZyB0byBkbyB0
aGlzIHJlc3VsdGVkIGluCiAgICBsb3NzIG9mIGFscGhhIHRyYW5zcGFyZW5jeSB3aGVuIHNob3dp
bmcgUE5HIGltYWdlcy4KCmRpZmYgLS1naXQgYS9XZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUv
Q2hhbmdlTG9nCmluZGV4IDYzMWM0MDEuLjdhZGYzNGEgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hh
bmdlTG9nCisrKyBiL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTIgQEAKIDIwMDktMTAt
MjcgIFZhZGltIFplaXRsaW4gIDx2YWRpbUB3eHdpZGdldHMub3JnPgogCisgICAgICAgIFt3eF0g
Rml4IGhhbmRsaW5nIG9mIGFscGhhIGNoYW5uZWwgd2hlbiB1c2luZyB3eFdpZGdldHMgMi45Lgor
CisgICAgICAgICogcGxhdGZvcm0vaW1hZ2UtZGVjb2RlcnMvd3gvSW1hZ2VEZWNvZGVyV3guY3Bw
OgorICAgICAgICAoV2ViQ29yZTo6UkdCQTMyQnVmZmVyOjphc05ld05hdGl2ZUltYWdlKToKKwor
MjAwOS0xMC0yNyAgVmFkaW0gWmVpdGxpbiAgPHZhZGltQHd4d2lkZ2V0cy5vcmc+CisKICAgICAg
ICAgRGVmaW5lIE5hdGl2ZUltYWdlUHRyIGFzIHd4W0dyYXBoaWNzXUJpdG1hcCBpbiB3eCBwb3J0
IGluc3RlYWQgb2YKICAgICAgICAgZGVmaW5pbmcgaXQgYXMgYSBwb2ludGVyIHRvIHRoZW0gdG8g
YXZvaWQgdW5uZWNlc3NhcnkgaGVhcCBhbGxvY2F0aW9ucy4KIApkaWZmIC0tZ2l0IGEvV2ViQ29y
ZS9wbGF0Zm9ybS9pbWFnZS1kZWNvZGVycy93eC9JbWFnZURlY29kZXJXeC5jcHAgYi9XZWJDb3Jl
L3BsYXRmb3JtL2ltYWdlLWRlY29kZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcAppbmRleCAzZGRj
NjM4Li4xM2MyNTY0IDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRlY29kZXJz
L3d4L0ltYWdlRGVjb2Rlcld4LmNwcAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRlY29k
ZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcApAQCAtNDAsMzUgKzQwLDQxIEBAIG5hbWVzcGFjZSBX
ZWJDb3JlIHsKIE5hdGl2ZUltYWdlUHRyIFJHQkEzMkJ1ZmZlcjo6YXNOZXdOYXRpdmVJbWFnZSgp
IGNvbnN0CiB7CiAgICAgd3hCaXRtYXAgYm1wKHdpZHRoKCksIGhlaWdodCgpLCAzMik7Ci0gICAg
dHlwZWRlZiB3eFBpeGVsRGF0YTx3eEJpdG1hcCwgd3hBbHBoYVBpeGVsRm9ybWF0PiBXeFBpeGVs
RGF0YTsKLSAgICBXeFBpeGVsRGF0YSBkYXRhKGJtcCk7Ci0gICAgCi0gICAgLy8gTkI6IEl0IGFw
cGVhcnMgdGhhdCB0aGUgZGF0YSBpcyBpbiBCR1JBIGZvcm1hdCBpbnN0ZWFkIG9mIFJHQkEgZm9y
bWF0LgotICAgIC8vIFRoaXMgY29kZSB3b3JrcyBwcm9wZXJseSBvbiBib3RoIHBwYyBhbmQgaW50
ZWwsIG1lYW5pbmcgdGhlIGlzc3VlIGlzCi0gICAgLy8gbGlrZWx5IG5vdCBhbiBpc3N1ZSBvZiBi
eXRlIG9yZGVyIGdldHRpbmcgbWl4ZWQgdXAgb24gZGlmZmVyZW50IGFyY2hzLiAKLSAgICBjb25z
dCB1bnNpZ25lZCBjaGFyKiBieXRlcyA9IChjb25zdCB1bnNpZ25lZCBjaGFyKiltX2J5dGVzLmRh
dGEoKTsKLSAgICBpbnQgcm93Q291bnRlciA9IDA7Ci0gICAgbG9uZyBwaXhlbENvdW50ZXIgPSAw
OwotICAgIFd4UGl4ZWxEYXRhOjpJdGVyYXRvciBwKGRhdGEpOwotICAgIFd4UGl4ZWxEYXRhOjpJ
dGVyYXRvciByb3dTdGFydCA9IHA7IAotICAgIGZvciAoc2l6ZV90IGkgPSAwOyBpIDwgbV9ieXRl
cy5zaXplKCkgKiBzaXplb2YoUGl4ZWxEYXRhKTsgaSArPSBzaXplb2YoUGl4ZWxEYXRhKSkgewot
ICAgICAgICBwLlJlZCgpID0gYnl0ZXNbaSsyXTsKLSAgICAgICAgcC5HcmVlbigpID0gYnl0ZXNb
aSsxXTsKLSAgICAgICAgcC5CbHVlKCkgPSBieXRlc1tpKzBdOwotICAgICAgICBwLkFscGhhKCkg
PSBieXRlc1tpKzNdOworCisgICAgeworICAgICAgICB0eXBlZGVmIHd4UGl4ZWxEYXRhPHd4Qml0
bWFwLCB3eEFscGhhUGl4ZWxGb3JtYXQ+IFd4UGl4ZWxEYXRhOworICAgICAgICBXeFBpeGVsRGF0
YSBkYXRhKGJtcCk7CiAgICAgICAgIAotICAgICAgICBwKys7CisgICAgICAgIC8vIE5COiBJdCBh
cHBlYXJzIHRoYXQgdGhlIGRhdGEgaXMgaW4gQkdSQSBmb3JtYXQgaW5zdGVhZCBvZiBSR0JBIGZv
cm1hdC4KKyAgICAgICAgLy8gVGhpcyBjb2RlIHdvcmtzIHByb3Blcmx5IG9uIGJvdGggcHBjIGFu
ZCBpbnRlbCwgbWVhbmluZyB0aGUgaXNzdWUgaXMKKyAgICAgICAgLy8gbGlrZWx5IG5vdCBhbiBp
c3N1ZSBvZiBieXRlIG9yZGVyIGdldHRpbmcgbWl4ZWQgdXAgb24gZGlmZmVyZW50IGFyY2hzLiAK
KyAgICAgICAgY29uc3QgdW5zaWduZWQgY2hhciogYnl0ZXMgPSAoY29uc3QgdW5zaWduZWQgY2hh
ciopbV9ieXRlcy5kYXRhKCk7CisgICAgICAgIGludCByb3dDb3VudGVyID0gMDsKKyAgICAgICAg
bG9uZyBwaXhlbENvdW50ZXIgPSAwOworICAgICAgICBXeFBpeGVsRGF0YTo6SXRlcmF0b3IgcChk
YXRhKTsKKyAgICAgICAgV3hQaXhlbERhdGE6Okl0ZXJhdG9yIHJvd1N0YXJ0ID0gcDsgCisgICAg
ICAgIGZvciAoc2l6ZV90IGkgPSAwOyBpIDwgbV9ieXRlcy5zaXplKCkgKiBzaXplb2YoUGl4ZWxE
YXRhKTsgaSArPSBzaXplb2YoUGl4ZWxEYXRhKSkgeworICAgICAgICAgICAgcC5SZWQoKSA9IGJ5
dGVzW2krMl07CisgICAgICAgICAgICBwLkdyZWVuKCkgPSBieXRlc1tpKzFdOworICAgICAgICAg
ICAgcC5CbHVlKCkgPSBieXRlc1tpKzBdOworICAgICAgICAgICAgcC5BbHBoYSgpID0gYnl0ZXNb
aSszXTsKKyAgICAgICAgICAgIAorICAgICAgICAgICAgcCsrOwogCi0gICAgICAgIHBpeGVsQ291
bnRlcisrOwotICAgICAgICBpZiAoKHBpeGVsQ291bnRlciAlIHdpZHRoKCkpID09IDApIHsKLSAg
ICAgICAgICAgIHJvd0NvdW50ZXIrKzsKLSAgICAgICAgICAgIHAgPSByb3dTdGFydDsKLSAgICAg
ICAgICAgIHAuTW92ZVRvKGRhdGEsIDAsIHJvd0NvdW50ZXIpOworICAgICAgICAgICAgcGl4ZWxD
b3VudGVyKys7CisgICAgICAgICAgICBpZiAoKHBpeGVsQ291bnRlciAlIHdpZHRoKCkpID09IDAp
IHsKKyAgICAgICAgICAgICAgICByb3dDb3VudGVyKys7CisgICAgICAgICAgICAgICAgcCA9IHJv
d1N0YXJ0OworICAgICAgICAgICAgICAgIHAuTW92ZVRvKGRhdGEsIDAsIHJvd0NvdW50ZXIpOwor
ICAgICAgICAgICAgfQogICAgICAgICB9Ci0gICAgfQogI2lmICF3eENIRUNLX1ZFUlNJT04oMiw5
LDApCi0gICAgYm1wLlVzZUFscGhhKCk7CisgICAgICAgIGJtcC5Vc2VBbHBoYSgpOwogI2VuZGlm
CisgICAgfSAvLyBlbnN1cmUgdGhhdCBXeFBpeGVsRGF0YSBpcyBkZXN0cm95ZWQgYXMgaXQgdW5s
b2NrcyB0aGUgYml0bWFwIGRhdGEgaW4KKyAgICAgIC8vIGl0cyBkdG9yIGFuZCB3ZSBjYW4ndCBh
Y2Nlc3MgaXQgKG5vdGFibHkgaW4gQ3JlYXRlQml0bWFwKCkgYmVsb3cpCisgICAgICAvLyBiZWZv
cmUgdGhpcyBpcyBkb25lCisKICAgICBBU1NFUlQoYm1wLklzT2soKSk7CiAKICNpZiBVU0UoV1hH
QykK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>41997</attachid>
            <date>2009-10-27 17:29:13 -0700</date>
            <delta_ts>2009-11-06 18:35:08 -0800</delta_ts>
            <desc>Patch fixing alpha transparency handling for wx port (v2)</desc>
            <filename>alpha-fix-2.diff</filename>
            <type>text/plain</type>
            <size>3890</size>
            <attacher name="Vadim Zeitlin">vz-webkit</attacher>
            
              <data encoding="base64">Y29tbWl0IDcwZmU2OThhZGFiNDA1OTMwZjQ2ODVlY2RiYzI1N2M4YmNjZTM3MTkKQXV0aG9yOiBW
YWRpbSBaZWl0bGluIDx2YWRpbUB3eHdpZGdldHMub3JnPgpEYXRlOiAgIFR1ZSBPY3QgMjcgMTY6
MjU6MzkgMjAwOSArMDEwMAoKICAgIEZpeCBoYW5kbGluZyBvZiBhbHBoYSBjaGFuZ2VzIGluIHd4
IHBvcnQgd2hlbiB1c2luZyAyLjkrLgogICAgCiAgICB3eFBpeGVsRGF0YSBvYmplY3QgdXNlZCBm
b3IgcmF3IGFjY2VzcyB0byB0aGUgYml0bWFwIGRhdGEgbXVzdCBiZSBkZXN0cm95ZWQKICAgIGJl
Zm9yZSB0aGUgYml0bWFwIGlzIHVzZWQgaW4gYW55IG90aGVyIHdheSwgZmFpbGluZyB0byBkbyB0
aGlzIHJlc3VsdGVkIGluCiAgICBsb3NzIG9mIGFscGhhIHRyYW5zcGFyZW5jeSB3aGVuIHNob3dp
bmcgUE5HIGltYWdlcy4KCmRpZmYgLS1naXQgYS9XZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUv
Q2hhbmdlTG9nCmluZGV4IDYzMWM0MDEuLjQ5YzM5MmEgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hh
bmdlTG9nCisrKyBiL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTQgQEAKIDIwMDktMTAt
MjcgIFZhZGltIFplaXRsaW4gIDx2YWRpbUB3eHdpZGdldHMub3JnPgogCisgICAgICAgIFt3eF0g
Rml4IGhhbmRsaW5nIG9mIGFscGhhIGNoYW5uZWwgd2hlbiB1c2luZyB3eFdpZGdldHMgMi45OiBp
dCB3YXMKKyAgICAgICAgc2ltcGx5IGlnbm9yZWQgYmVmb3JlIHJlc3VsdGluZyBpbiB0cmFuc3Bh
cmVudCBhcmVhcyBiZWluZyBibGFjayBpbiBQTkcKKyAgICAgICAgaW1hZ2VzIGZvciBleGFtcGxl
LgorCisgICAgICAgICogcGxhdGZvcm0vaW1hZ2UtZGVjb2RlcnMvd3gvSW1hZ2VEZWNvZGVyV3gu
Y3BwOgorICAgICAgICAoV2ViQ29yZTo6UkdCQTMyQnVmZmVyOjphc05ld05hdGl2ZUltYWdlKToK
KworMjAwOS0xMC0yNyAgVmFkaW0gWmVpdGxpbiAgPHZhZGltQHd4d2lkZ2V0cy5vcmc+CisKICAg
ICAgICAgRGVmaW5lIE5hdGl2ZUltYWdlUHRyIGFzIHd4W0dyYXBoaWNzXUJpdG1hcCBpbiB3eCBw
b3J0IGluc3RlYWQgb2YKICAgICAgICAgZGVmaW5pbmcgaXQgYXMgYSBwb2ludGVyIHRvIHRoZW0g
dG8gYXZvaWQgdW5uZWNlc3NhcnkgaGVhcCBhbGxvY2F0aW9ucy4KIApkaWZmIC0tZ2l0IGEvV2Vi
Q29yZS9wbGF0Zm9ybS9pbWFnZS1kZWNvZGVycy93eC9JbWFnZURlY29kZXJXeC5jcHAgYi9XZWJD
b3JlL3BsYXRmb3JtL2ltYWdlLWRlY29kZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcAppbmRleCAz
ZGRjNjM4Li4wMzRhZjE2IDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRlY29k
ZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRl
Y29kZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcApAQCAtNDAsMzUgKzQwLDQxIEBAIG5hbWVzcGFj
ZSBXZWJDb3JlIHsKIE5hdGl2ZUltYWdlUHRyIFJHQkEzMkJ1ZmZlcjo6YXNOZXdOYXRpdmVJbWFn
ZSgpIGNvbnN0CiB7CiAgICAgd3hCaXRtYXAgYm1wKHdpZHRoKCksIGhlaWdodCgpLCAzMik7Ci0g
ICAgdHlwZWRlZiB3eFBpeGVsRGF0YTx3eEJpdG1hcCwgd3hBbHBoYVBpeGVsRm9ybWF0PiBXeFBp
eGVsRGF0YTsKLSAgICBXeFBpeGVsRGF0YSBkYXRhKGJtcCk7Ci0gICAgCi0gICAgLy8gTkI6IEl0
IGFwcGVhcnMgdGhhdCB0aGUgZGF0YSBpcyBpbiBCR1JBIGZvcm1hdCBpbnN0ZWFkIG9mIFJHQkEg
Zm9ybWF0LgotICAgIC8vIFRoaXMgY29kZSB3b3JrcyBwcm9wZXJseSBvbiBib3RoIHBwYyBhbmQg
aW50ZWwsIG1lYW5pbmcgdGhlIGlzc3VlIGlzCi0gICAgLy8gbGlrZWx5IG5vdCBhbiBpc3N1ZSBv
ZiBieXRlIG9yZGVyIGdldHRpbmcgbWl4ZWQgdXAgb24gZGlmZmVyZW50IGFyY2hzLiAKLSAgICBj
b25zdCB1bnNpZ25lZCBjaGFyKiBieXRlcyA9IChjb25zdCB1bnNpZ25lZCBjaGFyKiltX2J5dGVz
LmRhdGEoKTsKLSAgICBpbnQgcm93Q291bnRlciA9IDA7Ci0gICAgbG9uZyBwaXhlbENvdW50ZXIg
PSAwOwotICAgIFd4UGl4ZWxEYXRhOjpJdGVyYXRvciBwKGRhdGEpOwotICAgIFd4UGl4ZWxEYXRh
OjpJdGVyYXRvciByb3dTdGFydCA9IHA7IAotICAgIGZvciAoc2l6ZV90IGkgPSAwOyBpIDwgbV9i
eXRlcy5zaXplKCkgKiBzaXplb2YoUGl4ZWxEYXRhKTsgaSArPSBzaXplb2YoUGl4ZWxEYXRhKSkg
ewotICAgICAgICBwLlJlZCgpID0gYnl0ZXNbaSsyXTsKLSAgICAgICAgcC5HcmVlbigpID0gYnl0
ZXNbaSsxXTsKLSAgICAgICAgcC5CbHVlKCkgPSBieXRlc1tpKzBdOwotICAgICAgICBwLkFscGhh
KCkgPSBieXRlc1tpKzNdOworCisgICAgeworICAgICAgICB0eXBlZGVmIHd4UGl4ZWxEYXRhPHd4
Qml0bWFwLCB3eEFscGhhUGl4ZWxGb3JtYXQ+IFd4UGl4ZWxEYXRhOworICAgICAgICBXeFBpeGVs
RGF0YSBkYXRhKGJtcCk7CiAgICAgICAgIAotICAgICAgICBwKys7CisgICAgICAgIC8vIE5COiBJ
dCBhcHBlYXJzIHRoYXQgdGhlIGRhdGEgaXMgaW4gQkdSQSBmb3JtYXQgaW5zdGVhZCBvZiBSR0JB
IGZvcm1hdC4KKyAgICAgICAgLy8gVGhpcyBjb2RlIHdvcmtzIHByb3Blcmx5IG9uIGJvdGggcHBj
IGFuZCBpbnRlbCwgbWVhbmluZyB0aGUgaXNzdWUgaXMKKyAgICAgICAgLy8gbGlrZWx5IG5vdCBh
biBpc3N1ZSBvZiBieXRlIG9yZGVyIGdldHRpbmcgbWl4ZWQgdXAgb24gZGlmZmVyZW50IGFyY2hz
LiAKKyAgICAgICAgY29uc3QgdW5zaWduZWQgY2hhciogYnl0ZXMgPSAoY29uc3QgdW5zaWduZWQg
Y2hhciopbV9ieXRlcy5kYXRhKCk7CisgICAgICAgIGludCByb3dDb3VudGVyID0gMDsKKyAgICAg
ICAgbG9uZyBwaXhlbENvdW50ZXIgPSAwOworICAgICAgICBXeFBpeGVsRGF0YTo6SXRlcmF0b3Ig
cChkYXRhKTsKKyAgICAgICAgV3hQaXhlbERhdGE6Okl0ZXJhdG9yIHJvd1N0YXJ0ID0gcDsgCisg
ICAgICAgIGZvciAoc2l6ZV90IGkgPSAwOyBpIDwgbV9ieXRlcy5zaXplKCkgKiBzaXplb2YoUGl4
ZWxEYXRhKTsgaSArPSBzaXplb2YoUGl4ZWxEYXRhKSkgeworICAgICAgICAgICAgcC5SZWQoKSA9
IGJ5dGVzW2kgKyAyXTsKKyAgICAgICAgICAgIHAuR3JlZW4oKSA9IGJ5dGVzW2kgKyAxXTsKKyAg
ICAgICAgICAgIHAuQmx1ZSgpID0gYnl0ZXNbaSArIDBdOworICAgICAgICAgICAgcC5BbHBoYSgp
ID0gYnl0ZXNbaSArIDNdOworICAgICAgICAgICAgCisgICAgICAgICAgICBwKys7CiAKLSAgICAg
ICAgcGl4ZWxDb3VudGVyKys7Ci0gICAgICAgIGlmICgocGl4ZWxDb3VudGVyICUgd2lkdGgoKSkg
PT0gMCkgewotICAgICAgICAgICAgcm93Q291bnRlcisrOwotICAgICAgICAgICAgcCA9IHJvd1N0
YXJ0OwotICAgICAgICAgICAgcC5Nb3ZlVG8oZGF0YSwgMCwgcm93Q291bnRlcik7CisgICAgICAg
ICAgICBwaXhlbENvdW50ZXIrKzsKKyAgICAgICAgICAgIGlmICgocGl4ZWxDb3VudGVyICUgd2lk
dGgoKSkgPT0gMCkgeworICAgICAgICAgICAgICAgIHJvd0NvdW50ZXIrKzsKKyAgICAgICAgICAg
ICAgICBwID0gcm93U3RhcnQ7CisgICAgICAgICAgICAgICAgcC5Nb3ZlVG8oZGF0YSwgMCwgcm93
Q291bnRlcik7CisgICAgICAgICAgICB9CiAgICAgICAgIH0KLSAgICB9CiAjaWYgIXd4Q0hFQ0tf
VkVSU0lPTigyLDksMCkKLSAgICBibXAuVXNlQWxwaGEoKTsKKyAgICAgICAgYm1wLlVzZUFscGhh
KCk7CiAjZW5kaWYKKyAgICB9IC8vIGVuc3VyZSB0aGF0IFd4UGl4ZWxEYXRhIGlzIGRlc3Ryb3ll
ZCBhcyBpdCB1bmxvY2tzIHRoZSBiaXRtYXAgZGF0YSBpbgorICAgICAgLy8gaXRzIGR0b3IgYW5k
IHdlIGNhbid0IGFjY2VzcyBpdCAobm90YWJseSBpbiBDcmVhdGVCaXRtYXAoKSBiZWxvdykKKyAg
ICAgIC8vIGJlZm9yZSB0aGlzIGlzIGRvbmUKKwogICAgIEFTU0VSVChibXAuSXNPaygpKTsKIAog
I2lmIFVTRShXWEdDKQo=
</data>
<flag name="review"
          id="23483"
          type_id="1"
          status="-"
          setter="kevino"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>42682</attachid>
            <date>2009-11-06 18:35:08 -0800</date>
            <delta_ts>2009-11-09 12:35:45 -0800</delta_ts>
            <desc>&quot;Standalone&quot; version of the patch </desc>
            <filename>alpha-fix-3.diff</filename>
            <type>text/plain</type>
            <size>2057</size>
            <attacher name="Vadim Zeitlin">vz-webkit</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
MDI3ZTlhYS4uOGYzMDI3YiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAorMjAwOS0xMS0wNiAgVmFkaW0gWmVpdGxp
biAgPHZhZGltQHd4d2lkZ2V0cy5vcmc+CisKKyAgICAgICAgW3d4XSBGaXggaGFuZGxpbmcgb2Yg
YWxwaGEgY2hhbm5lbCB3aGVuIHVzaW5nIHd4V2lkZ2V0cyAyLjk6IGl0IHdhcworICAgICAgICBz
aW1wbHkgaWdub3JlZCBiZWZvcmUgcmVzdWx0aW5nIGluIHRyYW5zcGFyZW50IGFyZWFzIGJlaW5n
IGJsYWNrIGluIFBORworICAgICAgICBpbWFnZXMgZm9yIGV4YW1wbGUuCisKKyAgICAgICAgKiBw
bGF0Zm9ybS9pbWFnZS1kZWNvZGVycy93eC9JbWFnZURlY29kZXJXeC5jcHA6CisgICAgICAgIChX
ZWJDb3JlOjpSR0JBMzJCdWZmZXI6OmFzTmV3TmF0aXZlSW1hZ2UpOgorCiAyMDA5LTEwLTE4ICBL
ZXZpbiBPbGxpdmllciAgPGtldmlub0B0aGVvbGxpdmllcnMuY29tPgogCiAgICAgICAgIENVUkwg
YnVpbGQgZml4LCB1c2UgcHJvcGVyIGhlYWRlciBuYW1lLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9w
bGF0Zm9ybS9pbWFnZS1kZWNvZGVycy93eC9JbWFnZURlY29kZXJXeC5jcHAgYi9XZWJDb3JlL3Bs
YXRmb3JtL2ltYWdlLWRlY29kZXJzL3d4L0ltYWdlRGVjb2Rlcld4LmNwcAppbmRleCAzY2FkZjFj
Li42ZTI1NzI4IDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRlY29kZXJzL3d4
L0ltYWdlRGVjb2Rlcld4LmNwcAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2ltYWdlLWRlY29kZXJz
L3d4L0ltYWdlRGVjb2Rlcld4LmNwcApAQCAtNDAsNiArNDAsOCBAQCBuYW1lc3BhY2UgV2ViQ29y
ZSB7CiBOYXRpdmVJbWFnZVB0ciBSR0JBMzJCdWZmZXI6OmFzTmV3TmF0aXZlSW1hZ2UoKSBjb25z
dAogewogICAgIHd4Qml0bWFwKiBibXAgPSBuZXcgd3hCaXRtYXAod2lkdGgoKSwgaGVpZ2h0KCks
IDMyKTsKKworICAgIHsKICAgICB0eXBlZGVmIHd4UGl4ZWxEYXRhPHd4Qml0bWFwLCB3eEFscGhh
UGl4ZWxGb3JtYXQ+IFd4UGl4ZWxEYXRhOwogICAgIFd4UGl4ZWxEYXRhIGRhdGEoKmJtcCk7CiAg
ICAgCkBAIC01MiwxMCArNTQsMTAgQEAgTmF0aXZlSW1hZ2VQdHIgUkdCQTMyQnVmZmVyOjphc05l
d05hdGl2ZUltYWdlKCkgY29uc3QKICAgICBXeFBpeGVsRGF0YTo6SXRlcmF0b3IgcChkYXRhKTsK
ICAgICBXeFBpeGVsRGF0YTo6SXRlcmF0b3Igcm93U3RhcnQgPSBwOyAKICAgICBmb3IgKHNpemVf
dCBpID0gMDsgaSA8IG1fYnl0ZXMuc2l6ZSgpICogc2l6ZW9mKFBpeGVsRGF0YSk7IGkgKz0gc2l6
ZW9mKFBpeGVsRGF0YSkpIHsKLSAgICAgICAgcC5SZWQoKSA9IGJ5dGVzW2krMl07Ci0gICAgICAg
IHAuR3JlZW4oKSA9IGJ5dGVzW2krMV07Ci0gICAgICAgIHAuQmx1ZSgpID0gYnl0ZXNbaSswXTsK
LSAgICAgICAgcC5BbHBoYSgpID0gYnl0ZXNbaSszXTsKKyAgICAgICAgICAgIHAuUmVkKCkgPSBi
eXRlc1tpICsgMl07CisgICAgICAgICAgICBwLkdyZWVuKCkgPSBieXRlc1tpICsgMV07CisgICAg
ICAgICAgICBwLkJsdWUoKSA9IGJ5dGVzW2kgKyAwXTsKKyAgICAgICAgICAgIHAuQWxwaGEoKSA9
IGJ5dGVzW2kgKyAzXTsKICAgICAgICAgCiAgICAgICAgIHArKzsKIApAQCAtNjksNiArNzEsMTAg
QEAgTmF0aXZlSW1hZ2VQdHIgUkdCQTMyQnVmZmVyOjphc05ld05hdGl2ZUltYWdlKCkgY29uc3QK
ICNpZiAhd3hDSEVDS19WRVJTSU9OKDIsOSwwKQogICAgIGJtcC0+VXNlQWxwaGEoKTsKICNlbmRp
ZgorICAgIH0gLy8gZW5zdXJlIHRoYXQgV3hQaXhlbERhdGEgaXMgZGVzdHJveWVkIGFzIGl0IHVu
bG9ja3MgdGhlIGJpdG1hcCBkYXRhIGluCisgICAgICAvLyBpdHMgZHRvciBhbmQgd2UgY2FuJ3Qg
YWNjZXNzIGl0IChub3RhYmx5IGluIENyZWF0ZUJpdG1hcCgpIGJlbG93KQorICAgICAgLy8gYmVm
b3JlIHRoaXMgaXMgZG9uZQorCiAgICAgQVNTRVJUKGJtcC0+SXNPaygpKTsKIAogI2lmIFVTRShX
WEdDKQo=
</data>
<flag name="review"
          id="24245"
          type_id="1"
          status="+"
          setter="kevino"
    />
    <flag name="commit-queue"
          id="24246"
          type_id="3"
          status="-"
          setter="kevino"
    />
          </attachment>
      

    </bug>

</bugzilla>