<?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>58095</bug_id>
          
          <creation_ts>2011-04-07 15:56:43 -0700</creation_ts>
          <short_desc>[GTK] Fix the WebKit2 build for older versions of GTK+</short_desc>
          <delta_ts>2011-04-08 12:47:20 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P3</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Martin Robinson">mrobinson</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>alex</cc>
    
    <cc>eric</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>381805</commentid>
    <comment_count>0</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2011-04-07 15:56:43 -0700</bug_when>
    <thetext>WebKit2 uses gdk_window_create_similar_surface which is only availabe in GTK+ &gt;= 2.22.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>382404</commentid>
    <comment_count>1</comment_count>
      <attachid>88845</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2011-04-08 11:35:36 -0700</bug_when>
    <thetext>Created attachment 88845
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>382405</commentid>
    <comment_count>2</comment_count>
      <attachid>88845</attachid>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2011-04-08 11:38:11 -0700</bug_when>
    <thetext>Comment on attachment 88845
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>382407</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-08 11:39:00 -0700</bug_when>
    <thetext>Attachment 88845 did not pass style-queue:

Failed to run &quot;[&apos;Tools/Scripts/check-webkit-style&apos;, &apos;--diff-files&apos;, u&apos;Source/WebCore/ChangeLog&apos;, u&apos;Source/WebCor...&quot; exit_code: 1

Source/WebCore/platform/gtk/GtkVersioning.h:50:  The parameter name &quot;window&quot; adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/gtk/GtkVersioning.h:50:  The parameter name &quot;content&quot; adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/gtk/GtkVersioning.h:50:  gdk_window_create_similar_surface is incorrectly named. Don&apos;t use underscores in your identifier names.  [readability/naming] [4]
Total errors found: 3 in 5 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>382423</commentid>
    <comment_count>4</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2011-04-08 11:48:09 -0700</bug_when>
    <thetext>Committed r83319: &lt;http://trac.webkit.org/changeset/83319&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>382462</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-08 12:47:20 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/83319 might have broken Qt Linux Release
The following tests are not passing:
http/tests/loading/preload-append-scan.php</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>88845</attachid>
            <date>2011-04-08 11:35:36 -0700</date>
            <delta_ts>2011-04-08 11:38:11 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-58095-20110408113535.patch</filename>
            <type>text/plain</type>
            <size>3800</size>
            <attacher name="Martin Robinson">mrobinson</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODMyMjYKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCBjNmZlYjNhOWZkZDFmMGM1
ZmViMWNhYjcxZWRhMDkzMWNhODQyNDgwLi5lY2NmMGZiYzMwY2RhNWY1NDkyZDUyYTZiN2EyZGE3
ZDM2NzYyZTBiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTEtMDQtMDggIE1hcnRp
biBSb2JpbnNvbiAgPG1yb2JpbnNvbkBpZ2FsaWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5
IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtHVEtdIEZpeCB0aGUgV2ViS2l0MiBidWlsZCBm
b3Igb2xkZXIgdmVyc2lvbnMgb2YgR1RLKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9NTgwOTUKKworICAgICAgICBObyBuZXcgdGVzdHMuIFRoaXMgaXMg
anVzdCBhIGJ1aWxkIGZpeC4KKworICAgICAgICAqIHBsYXRmb3JtL2d0ay9HdGtWZXJzaW9uaW5n
LmM6CisgICAgICAgIChnZGtfd2luZG93X2NyZWF0ZV9zaW1pbGFyX3N1cmZhY2UpOiBSZWltcGxl
bWVudCB0aGlzIG1ldGhvZCBmb3Igb2xkZXIgdmVyc2lvbnMgb2YgR0RLLgorICAgICAgICAqIHBs
YXRmb3JtL2d0ay9HdGtWZXJzaW9uaW5nLmg6IEFkZCB0aGUgbmV3IG1ldGhvZCB0byB0aGUgaGVh
ZGVyLgorCiAyMDExLTA0LTA3ICBEYW4gQmVybnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CiAKICAg
ICAgICAgQnVpbGQgZml4LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3Rr
L0d0a1ZlcnNpb25pbmcuYyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2d0ay9HdGtWZXJzaW9u
aW5nLmMKaW5kZXggYzM0MDdlYWExMThkZjFiZTY0NjEzMGQyYmVhMjUxOTExNGQxMDRkNi4uMTEz
OGQ1NjIwNWIwMWE1ZWYyM2MxZDEzYjQ4YzAyMTM2ZTAwNmM0OSAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYkNvcmUvcGxhdGZvcm0vZ3RrL0d0a1ZlcnNpb25pbmcuYworKysgYi9Tb3VyY2UvV2ViQ29y
ZS9wbGF0Zm9ybS9ndGsvR3RrVmVyc2lvbmluZy5jCkBAIC0yNzEsMyArMjcxLDEzIEBAIGdib29s
ZWFuIGdfc2lnbmFsX2FjY3VtdWxhdG9yX2ZpcnN0X3dpbnMoR1NpZ25hbEludm9jYXRpb25IaW50
ICppbnZvY2F0aW9uSGludCwKIH0KICNlbmRpZgogCisjaWYgIUdUS19DSEVDS19WRVJTSU9OKDIs
IDIyLCAwKQorY2Fpcm9fc3VyZmFjZV90ICpnZGtfd2luZG93X2NyZWF0ZV9zaW1pbGFyX3N1cmZh
Y2UoR2RrV2luZG93ICp3aW5kb3csIGNhaXJvX2NvbnRlbnRfdCBjb250ZW50LCBpbnQgd2lkdGgs
IGludCBoZWlnaHQpCit7CisgICAgY2Fpcm9fdCAqY2Fpcm9Db250ZXh0ID0gZ2RrX2NhaXJvX2Ny
ZWF0ZSh3aW5kb3cpOworICAgIGNhaXJvX3N1cmZhY2VfdCAqY2Fpcm9TdXJmYWNlID0gY2Fpcm9f
Z2V0X3RhcmdldChjYWlyb0NvbnRleHQpOworICAgIGNhaXJvX3N1cmZhY2VfdCAqbmV3U3VyZmFj
ZSA9IGNhaXJvX3N1cmZhY2VfY3JlYXRlX3NpbWlsYXIoY2Fpcm9TdXJmYWNlLCBjb250ZW50LCB3
aWR0aCwgaGVpZ2h0KTsKKyAgICBjYWlyb19kZXN0cm95KGNhaXJvQ29udGV4dCk7CisgICAgcmV0
dXJuIG5ld1N1cmZhY2U7Cit9CisjZW5kaWYgLy8gR1RLX0NIRUNLX1ZFUlNJT04oMiwgMjIsIDAp
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ndGsvR3RrVmVyc2lvbmluZy5o
IGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3RrL0d0a1ZlcnNpb25pbmcuaAppbmRleCA3MGUx
YmJlYjgzMjI5ZGM5NTBjZjU5M2YxMTlhODliMGRmNjcyZjA3Li5iNDRmYzM4MzkyNjI2NTZjMzAw
MzE0YWY4OTU2YmRmYzIxMzFhNTcyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9y
bS9ndGsvR3RrVmVyc2lvbmluZy5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2d0ay9H
dGtWZXJzaW9uaW5nLmgKQEAgLTQ2LDYgKzQ2LDEwIEBAIEdka1BpeGJ1ZiogZ2RrX3BpeGJ1Zl9n
ZXRfZnJvbV9zdXJmYWNlKGNhaXJvX3N1cmZhY2VfdCogc3VyZmFjZSwgaW50IHNyY1gsIGludCBz
CiAjZGVmaW5lIGdka193aW5kb3dfZ2V0X3Zpc3VhbCBnZGtfZHJhd2FibGVfZ2V0X3Zpc3VhbAog
I2VuZGlmIC8vIEdUS19DSEVDS19WRVJTSU9OKDIsIDIzLCAwKQogCisjaWYgIUdUS19DSEVDS19W
RVJTSU9OKDIsIDIyLCAwKQorY2Fpcm9fc3VyZmFjZV90KiBnZGtfd2luZG93X2NyZWF0ZV9zaW1p
bGFyX3N1cmZhY2UoR2RrV2luZG93KiB3aW5kb3csIGNhaXJvX2NvbnRlbnRfdCBjb250ZW50LCBp
bnQgd2lkdGgsIGludCBoZWlnaHQpOworI2VuZGlmIC8vIEdUS19DSEVDS19WRVJTSU9OKDIsIDIy
LCAwKQorCiAjaWYgIUdUS19DSEVDS19WRVJTSU9OKDIsIDIxLCAyKQogI2RlZmluZSBnZGtfdmlz
dWFsX2dldF9kZXB0aCh2aXN1YWwpICh2aXN1YWwpLT5kZXB0aAogI2RlZmluZSBnZGtfdmlzdWFs
X2dldF9iaXRzX3Blcl9yZ2IodmlzdWFsKSAodmlzdWFsKS0+Yml0c19wZXJfcmdiCmRpZmYgLS1n
aXQgYS9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cK
aW5kZXggODY1MzJiYTczMTNlZGVkYTg2MjVkOGVlN2M4NGFiMjg2OTIzZmQyOC4uNDZiNzdjMTBk
YmQzM2VlZjI1MDRjMDkyZGVjMTM5ZDcwMjdjYTllOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktp
dDIvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEy
IEBACisyMDExLTA0LTA4ICBNYXJ0aW4gUm9iaW5zb24gIDxtcm9iaW5zb25AaWdhbGlhLmNvbT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbR1RLXSBG
aXggdGhlIFdlYktpdDIgYnVpbGQgZm9yIG9sZGVyIHZlcnNpb25zIG9mIEdUSysKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU4MDk1CisKKyAgICAgICAg
KiBVSVByb2Nlc3MvZ3RrL0NodW5rZWRVcGRhdGVEcmF3aW5nQXJlYVByb3h5R3RrLmNwcDogSW5j
bHVkZSB0aGUgR3RrVmVyc2lvbmluZy5oIGhlYWRlci4KKwogMjAxMS0wNC0wNyAgQW5kcmV3IFNj
aGVya3VzICA8c2NoZXJrdXNAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmVydCBFTkFCTEVf
VFJBQ0sgcGF0Y2ggZHVlIHRvIGNvbXBpbGUgZmFpbHVyZXMuCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViS2l0Mi9VSVByb2Nlc3MvZ3RrL0NodW5rZWRVcGRhdGVEcmF3aW5nQXJlYVByb3h5R3RrLmNw
cCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvQ2h1bmtlZFVwZGF0ZURyYXdpbmdBcmVh
UHJveHlHdGsuY3BwCmluZGV4IGE1YzcwOGYzNGRiMmE4NDgxNzhiN2Q4NzA4Yzg5ZGU2NWNhMzE0
ZjAuLjg5YTllZjc0MjFkZTBlMjgxNzJmOGM4MWY3OTYzYzZjN2UzNDE0ZmMgMTAwNjQ0Ci0tLSBh
L1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvQ2h1bmtlZFVwZGF0ZURyYXdpbmdBcmVhUHJv
eHlHdGsuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvQ2h1bmtlZFVwZGF0
ZURyYXdpbmdBcmVhUHJveHlHdGsuY3BwCkBAIC0zMiw3ICszMiw3IEBACiAjaW5jbHVkZSAiVXBk
YXRlQ2h1bmsuaCIKICNpbmNsdWRlICJXZWJQcm9jZXNzUHJveHkuaCIKICNpbmNsdWRlICJXZWJW
aWV3LmgiCi0KKyNpbmNsdWRlIDxXZWJDb3JlL0d0a1ZlcnNpb25pbmcuaD4KICNpbmNsdWRlIDxn
ZGsvZ2RrLmg+CiAKIHVzaW5nIG5hbWVzcGFjZSBXZWJDb3JlOwo=
</data>
<flag name="review"
          id="81434"
          type_id="1"
          status="+"
          setter="xan.lopez"
    />
          </attachment>
      

    </bug>

</bugzilla>