<?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>43192</bug_id>
          
          <creation_ts>2010-07-29 08:46:35 -0700</creation_ts>
          <short_desc>Warning fix on platforms where XP_MACOSX is undefined.</short_desc>
          <delta_ts>2010-07-29 09:14: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>Tools / Tests</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>
          
          <blocked>43191</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>257698</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-07-29 08:46:35 -0700</bug_when>
    <thetext>Warnings on Qt buildbot:

/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/WebKitTools/DumpRenderTree/TestNetscapePlugIn/PluginObject.h:28:5: warning: &quot;XP_MACOSX&quot; is not defined
/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/WebKitTools/DumpRenderTree/TestNetscapePlugIn/PluginObject.h:69:5: warning: &quot;XP_MACOSX&quot; is not defined
/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/WebKitTools/DumpRenderTree/TestNetscapePlugIn/PluginObject.h:83:5: warning: &quot;XP_MACOSX&quot; is not defined</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>257699</commentid>
    <comment_count>1</comment_count>
      <attachid>62950</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-07-29 08:48:52 -0700</bug_when>
    <thetext>Created attachment 62950
accidentally uploaded

We can use #if defined(XP_MACOSX) instead of #if XP_MACOSX ,
because XP_MACOSX is 1 when it is defined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>257700</commentid>
    <comment_count>2</comment_count>
      <attachid>62951</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-07-29 08:51:10 -0700</bug_when>
    <thetext>Created attachment 62951
proposed patch

We can use #if defined(XP_MACOSX) instead of #if XP_MACOSX ,
because XP_MACOSX is 1 when it is defined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>257701</commentid>
    <comment_count>3</comment_count>
      <attachid>62951</attachid>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-07-29 08:55:47 -0700</bug_when>
    <thetext>Comment on attachment 62951
proposed patch

r=me , can you wait green builtbots before commit, please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>257715</commentid>
    <comment_count>4</comment_count>
      <attachid>62951</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-07-29 09:14:03 -0700</bug_when>
    <thetext>Comment on attachment 62951
proposed patch

Clearing flags on attachment: 62951

Committed r64282: &lt;http://trac.webkit.org/changeset/64282&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>257716</commentid>
    <comment_count>5</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-07-29 09:14:11 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>62950</attachid>
            <date>2010-07-29 08:48:52 -0700</date>
            <delta_ts>2010-07-29 08:49:56 -0700</delta_ts>
            <desc>accidentally uploaded</desc>
            <filename>1.patch</filename>
            <type>text/plain</type>
            <size>1389</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0L2Zhc3QvYm9yZGVycy9kaWZmZXJl
bnQtY29sb3ItYm9yZGVycy1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9m
YXN0L2JvcmRlcnMvZGlmZmVyZW50LWNvbG9yLWJvcmRlcnMtZXhwZWN0ZWQudHh0Cm5ldyBmaWxl
IG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAuLjBiNzhhMDcKLS0tIC9kZXYvbnVsbAorKysgYi9M
YXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9mYXN0L2JvcmRlcnMvZGlmZmVyZW50LWNvbG9yLWJvcmRl
cnMtZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEsMTQgQEAKK2xheWVyIGF0ICgwLDApIHNpemUgODAw
eDYwMAorICBSZW5kZXJWaWV3IGF0ICgwLDApIHNpemUgODAweDYwMAorbGF5ZXIgYXQgKDAsMCkg
c2l6ZSA4MDB4NjAwCisgIFJlbmRlckJsb2NrIHtIVE1MfSBhdCAoMCwwKSBzaXplIDgwMHg2MDAK
KyAgICBSZW5kZXJCb2R5IHtCT0RZfSBhdCAoOCw4KSBzaXplIDc4NHg1NzIKKyAgICAgIFJlbmRl
ckJsb2NrIHtQfSBhdCAoMCwwKSBzaXplIDc4NHg2MAorICAgICAgICBSZW5kZXJUZXh0IHsjdGV4
dH0gYXQgKDAsMCkgc2l6ZSA3NjF4NjAKKyAgICAgICAgICB0ZXh0IHJ1biBhdCAoMCwwKSB3aWR0
aCA3NjE6ICJXaGVuIHR3byBib3JkZXIgc2lkZXMgaGF2ZSBkaWZmZXJlbnQgY29sb3JzLCB0aGUg
ZWRnZSBqb2luaW5nIHRoZW0gc2hvdWxkIGJlIGFudGktYWxpYXNlZC4gV2hlbiB0d28gYm9yZGVy
IgorICAgICAgICAgIHRleHQgcnVuIGF0ICgwLDIwKSB3aWR0aCA3NTE6ICJzaWRlcyBoYXZlIHRo
ZSBzYW1lIGNvbG9yLCB0aGUgZWRnZSBqb2luaW5nIHRoZW0gc2hvdWxkIG5vdCBiZSBhbnRpLWFs
aWFzZWQgYW5kIG5vIGNvcm5lciBqb2ludHMgc2hvdWxkIGJlIgorICAgICAgICAgIHRleHQgcnVu
IGF0ICgwLDQwKSB3aWR0aCA1MDogInZpc2libGUuIgorICAgICAgUmVuZGVyQmxvY2sge0RJVn0g
YXQgKDIwLDgwKSBzaXplIDI0MHg4MCBbYm9yZGVyOiAoMjBweCBzb2xpZCAjRkYwMDAwKSAoMjBw
eCBzb2xpZCAjMDAwMDAwKV0KKyAgICAgIFJlbmRlckJsb2NrIHtESVZ9IGF0ICgyMCwxODApIHNp
emUgMjQweDgwIFtib3JkZXI6ICgyMHB4IHNvbGlkICNGRjAwMDApICgyMHB4IHNvbGlkICMwMDAw
MDApICgyMHB4IHNvbGlkICNGRjAwMDApXQorICAgICAgUmVuZGVyQmxvY2sge0RJVn0gYXQgKDIw
LDI4MCkgc2l6ZSAyNDB4ODAgW2JvcmRlcjogKDIwcHggc29saWQgI0ZGMDAwMCkgKDIwcHggc29s
aWQgIzAwMDAwMCldCisgICAgICBSZW5kZXJCbG9jayB7RElWfSBhdCAoMjAsMzgwKSBzaXplIDI0
MHg4MCBbYm9yZGVyOiAoMjBweCBzb2xpZCAjRkYwMDAwKSAoMjBweCBzb2xpZCAjMDAwMDAwKSAo
MjBweCBzb2xpZCAjRkYwMDAwKV0K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>62951</attachid>
            <date>2010-07-29 08:51:10 -0700</date>
            <delta_ts>2010-07-29 09:14:03 -0700</delta_ts>
            <desc>proposed patch</desc>
            <filename>1.patch</filename>
            <type>text/plain</type>
            <size>1750</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0R1bXBSZW5kZXJUcmVlL1Rlc3ROZXRzY2FwZVBsdWdJbi9QbHVn
aW5PYmplY3QuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9EdW1wUmVuZGVyVHJlZS9UZXN0
TmV0c2NhcGVQbHVnSW4vUGx1Z2luT2JqZWN0LmgJKHJldmlzaW9uIDY0MjgwKQorKysgV2ViS2l0
VG9vbHMvRHVtcFJlbmRlclRyZWUvVGVzdE5ldHNjYXBlUGx1Z0luL1BsdWdpbk9iamVjdC5oCSh3
b3JraW5nIGNvcHkpCkBAIC0yNSw3ICsyNSw3IEBACiAKICNpbmNsdWRlIDxXZWJLaXQvbnBmdW5j
dGlvbnMuaD4KIAotI2lmIFhQX01BQ09TWAorI2lmIGRlZmluZWQoWFBfTUFDT1NYKQogI2lmICFk
ZWZpbmVkKE1BQ19PU19YX1ZFUlNJT05fMTBfNSkgfHwgTUFDX09TX1hfVkVSU0lPTl9NQVhfQUxM
T1dFRCA8IE1BQ19PU19YX1ZFUlNJT05fMTBfNQogI2RlZmluZSBCVUlMRElOR19PTl9USUdFUiAx
CiAjZWxpZiAhZGVmaW5lZChNQUNfT1NfWF9WRVJTSU9OXzEwXzYpIHx8IE1BQ19PU19YX1ZFUlNJ
T05fTUFYX0FMTE9XRUQgPCBNQUNfT1NfWF9WRVJTSU9OXzEwXzYKQEAgLTY2LDcgKzY2LDcgQEAK
ICNpZmRlZiBYUF9NQUNPU1gKICAgICBOUEV2ZW50TW9kZWwgZXZlbnRNb2RlbDsKICNlbmRpZgot
I2lmIFhQX01BQ09TWCAmJiAhZGVmaW5lZChCVUlMRElOR19PTl9USUdFUikKKyNpZiBkZWZpbmVk
KFhQX01BQ09TWCkgJiYgIWRlZmluZWQoQlVJTERJTkdfT05fVElHRVIpCiAgICAgdm9pZCogY29y
ZUFuaW1hdGlvbkxheWVyOwogI2VuZGlmCiAgICAgTlBXaW5kb3cgbGFzdFdpbmRvdzsKQEAgLTgw
LDcgKzgwLDcgQEAKIGV4dGVybiBib29sIHRlc3REb2N1bWVudE9wZW4oTlBQIG5wcCk7CiBleHRl
cm4gYm9vbCB0ZXN0V2luZG93T3BlbihOUFAgbnBwKTsKIAotI2lmIFhQX01BQ09TWCAmJiAhZGVm
aW5lZChCVUlMRElOR19PTl9USUdFUikKKyNpZiBkZWZpbmVkKFhQX01BQ09TWCkgJiYgIWRlZmlu
ZWQoQlVJTERJTkdfT05fVElHRVIpCiBleHRlcm4gdm9pZCogY3JlYXRlQ29yZUFuaW1hdGlvbkxh
eWVyKCk7CiAjZW5kaWYKIApJbmRleDogV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIFdlYktpdFRvb2xzL0NoYW5nZUxvZwkocmV2aXNpb24gNjQyODApCisrKyBXZWJLaXRUb29s
cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAxMC0wNy0yOSAg
Q3NhYmEgT3N6dHJvZ29uw6FjICA8b3NzeUB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFdhcm5pbmcgZml4IG9uIHBsYXRmb3JtcyB3
aGVyZSBYUF9NQUNPU1ggaXMgdW5kZWZpbmVkLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDMxOTIKKworICAgICAgICAqIER1bXBSZW5kZXJUcmVlL1Rl
c3ROZXRzY2FwZVBsdWdJbi9QbHVnaW5PYmplY3QuaDoKKyAgICAgICAgVXNlICNpZiBkZWZpbmVk
KFhQX01BQ09TWCkgaW5zdGVhZCBvZiAjaWYgWFBfTUFDT1NYIC4KKwogMjAxMC0wNy0yOSAgQWRh
bSBSb2JlbiAgPGFyb2JlbkBhcHBsZS5jb20+CiAKICAgICAgICAgR2l2ZSBmaW5kLWluY2x1ZGVk
LWZyYW1ld29yay1oZWFkZXJzIG91ciBzdGFuZGFyZCBsaWNlbnNlCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>