<?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>108374</bug_id>
          
          <creation_ts>2013-01-30 13:23:03 -0800</creation_ts>
          <short_desc>Use OS() and PLATFORM() macros in WebCorePrefix.h for readability</short_desc>
          <delta_ts>2013-01-31 12:54:06 -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>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></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="David Kilzer (:ddkilzer)">ddkilzer</reporter>
          <assigned_to name="David Kilzer (:ddkilzer)">ddkilzer</assigned_to>
          <cc>benjamin</cc>
    
    <cc>bfulgham</cc>
    
    <cc>darin</cc>
    
    <cc>ddkilzer</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>mjs</cc>
    
    <cc>mrowe</cc>
    
    <cc>sam</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>820473</commentid>
    <comment_count>0</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2013-01-30 13:23:03 -0800</bug_when>
    <thetext>Use OS() and PLATFORM() macros in WebCorePrefix.h for readability</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>820478</commentid>
    <comment_count>1</comment_count>
      <attachid>185548</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2013-01-30 13:24:03 -0800</bug_when>
    <thetext>Created attachment 185548
Patch v1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>820482</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-30 13:26:16 -0800</bug_when>
    <thetext>Attachment 185548 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/WebCore/WebCorePrefix.h&apos;]&quot; exit_code: 1
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>820485</commentid>
    <comment_count>3</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2013-01-30 13:27:59 -0800</bug_when>
    <thetext>I didn&apos;t replace __APPLE__ since I wasn&apos;t sure if we wanted OS(DARWIN) here or PLATFORM(APPLE) per discussion in Bug 108007.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>820486</commentid>
    <comment_count>4</comment_count>
      <attachid>185549</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2013-01-30 13:30:07 -0800</bug_when>
    <thetext>Created attachment 185549
Patch v1 with bug URL</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>821088</commentid>
    <comment_count>5</comment_count>
      <attachid>185549</attachid>
    <who name="Build Bot">buildbot</who>
    <bug_when>2013-01-30 22:04:50 -0800</bug_when>
    <thetext>Comment on attachment 185549
Patch v1 with bug URL

Attachment 185549 did not pass win-ews (win):
Output: http://queues.webkit.org/results/16217980</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>821823</commentid>
    <comment_count>6</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2013-01-31 12:25:29 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 185549 [details])
&gt; Attachment 185549 [details] did not pass win-ews (win):
&gt; Output: http://queues.webkit.org/results/16217980

This seems bogus:

Running build-dumprendertree
Compiling DumpRenderTree failed!
Failed to execute Tools/Scripts/old-run-webkit-tests at Tools/Scripts/run-webkit-tests line 106.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>821835</commentid>
    <comment_count>7</comment_count>
      <attachid>185549</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2013-01-31 12:30:48 -0800</bug_when>
    <thetext>Comment on attachment 185549
Patch v1 with bug URL

lgtm.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>821863</commentid>
    <comment_count>8</comment_count>
      <attachid>185549</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-31 12:54:02 -0800</bug_when>
    <thetext>Comment on attachment 185549
Patch v1 with bug URL

Clearing flags on attachment: 185549

Committed r141468: &lt;http://trac.webkit.org/changeset/141468&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>821864</commentid>
    <comment_count>9</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-01-31 12:54:06 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>185548</attachid>
            <date>2013-01-30 13:24:03 -0800</date>
            <delta_ts>2013-01-30 13:30:05 -0800</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-108374-20130130132048.patch</filename>
            <type>text/plain</type>
            <size>2609</size>
            <attacher name="David Kilzer (:ddkilzer)">ddkilzer</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQxMjgwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNWQ2MzBiYmIyYTg2ZjA4
Mzc1MWE4NDcxMzM0ZTc2YTIwYzk4NzI1Yy4uYmEyOTExMTlhNzdjMzI5ZTg1NjU0MWU1N2M4NmJl
YjYwZGFiOTJkNiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTAxLTMwICBEYXZp
ZCBLaWx6ZXIgIDxkZGtpbHplckBhcHBsZS5jb20+CisKKyAgICAgICAgVXNlIE9TKCkgYW5kIFBM
QVRGT1JNKCkgbWFjcm9zIGluIFdlYkNvcmVQcmVmaXguaCBmb3IgcmVhZGFiaWxpdHkKKyAgICAg
ICAgTmVlZCB0aGUgYnVnIFVSTCAoT09QUyEpLgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgICogV2ViQ29yZVByZWZpeC5oOiBSZXBsYWNlZCBhbGwgcG9z
c2libGUgY2hlY2tzIHdpdGggT1MoKSBvcgorICAgICAgICBQTEFURk9STSgpIG1hY3JvcyBvdGhl
ciB0aGFuIF9fQVBQTEVfXy4KKwogMjAxMy0wMS0zMCAgRmxvcmluIE1hbGl0YSAgPGZtYWxpdGFA
Y2hyb21pdW0ub3JnPgogCiAgICAgICAgIERvIG5vdCByZXN0YXJ0IHRoZSBtYXRjaGVkIHByb3Bl
cnRpZXMgY2FjaGUgdGltZXIgaWYgYWN0aXZlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9X
ZWJDb3JlUHJlZml4LmggYi9Tb3VyY2UvV2ViQ29yZS9XZWJDb3JlUHJlZml4LmgKaW5kZXggYzY2
ODgwYjAxMDk0OTUwOGZkMzA5OTk3ODU2ODA3NTgxODIxNGZkMS4uOWE3YjBjYzc3YWVjNzdlZGU2
OTU5Yjc1NzQyYTY5Y2VjYWMzZTIxOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvV2ViQ29y
ZVByZWZpeC5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL1dlYkNvcmVQcmVmaXguaApAQCAtMzcsNyAr
MzcsNyBAQAogI2VuZGlmCiAjZW5kaWYKIAotI2lmIGRlZmluZWQoV0lOMzIpIHx8IGRlZmluZWQo
X1dJTjMyKQorI2lmIE9TKFdJTkRPV1MpCiAKICNpZm5kZWYgX1dJTjMyX1dJTk5UCiAjZGVmaW5l
IF9XSU4zMl9XSU5OVCAweDA1MDIKQEAgLTQ3LDcgKzQ3LDcgQEAKICNkZWZpbmUgV0lOVkVSIDB4
MDUwMgogI2VuZGlmCiAKLSNpZm5kZWYgV1RGX1VTRV9DVVJMCisjaWYgIVVTRShDVVJMKQogI2lm
bmRlZiBfV0lOU09DS0FQSV8KICNkZWZpbmUgX1dJTlNPQ0tBUElfIC8vIFByZXZlbnQgaW5jbHVz
aW9uIG9mIHdpbnNvY2suaCBpbiB3aW5kb3dzLmgKICNlbmRpZgpAQCAtNjQsNyArNjQsNyBAQAog
CiAjaW5jbHVkZSA8cHRocmVhZC5oPgogCi0jZW5kaWYgLy8gZGVmaW5lZChXSU4zMikgfHwgZGVm
aW5lZChfV0lOMzIpCisjZW5kaWYgLy8gT1MoV0lORE9XUykKIAogI2luY2x1ZGUgPHN5cy90eXBl
cy5oPgogI2luY2x1ZGUgPGZjbnRsLmg+CkBAIC03NCw3ICs3NCw3IEBACiAKIC8vIE9uIExpbnV4
IHRoaXMgY2F1c2VzIGNvbmZsaWN0cyB3aXRoIGxpYnBuZyBiZWNhdXNlIHRoZXJlIGFyZSB0d28g
aW1wbHMuIG9mCiAvLyBsb25nam1wIC0gc2VlIGhlcmU6IGh0dHBzOi8vYnVncy5sYXVuY2hwYWQu
bmV0L3VidW50dS8rc291cmNlL2xpYnBuZy8rYnVnLzIxODQwOQotI2lmbmRlZiBCVUlMRElOR19X
WF9fCisjaWYgIVBMQVRGT1JNKFdYKQogI2luY2x1ZGUgPHNldGptcC5oPgogI2VuZGlmCiAKQEAg
LTEzMiwxNCArMTMyLDE0IEBAIF9MSUJDUFBfRU5EX05BTUVTUEFDRV9TVEQKICNpbmNsdWRlIDxz
eXMvcmVzb3VyY2UuaD4KICNlbmRpZgogCi0jaWYgIWRlZmluZWQoQlVJTERJTkdfV1hfXykKKyNp
ZiAhUExBVEZPUk0oV1gpCiAjaW5jbHVkZSA8Q29yZUZvdW5kYXRpb24vQ29yZUZvdW5kYXRpb24u
aD4KLSNpZmRlZiBXVEZfUExBVEZPUk1fV0lOX0NBSVJPCisjaWYgUExBVEZPUk0oV0lOX0NBSVJP
KQogI2luY2x1ZGUgPENvbmRpdGlvbmFsTWFjcm9zLmg+CiAjaW5jbHVkZSA8d2luZG93cy5oPgog
I2Vsc2UKIAotI2lmIGRlZmluZWQoV0lOMzIpIHx8IGRlZmluZWQoX1dJTjMyKQorI2lmIE9TKFdJ
TkRPV1MpCiAvLyBGSVhNRSA8cmRhcjovL3Byb2JsZW0vODIwODg2OD4gUmVtb3ZlIHN1cHBvcnQg
Zm9yIG9ic29sZXRlIENvbG9yU3luYyBBUEksIENvcmVTZXJ2aWNlcyBoZWFkZXIgaW4gQ29yZUdy
YXBoaWNzCiAvLyBXZSBjYW4gcmVtb3ZlIHRoaXMgb25jZSB0aGUgbmV3IENvbG9yU3luYyBBUElz
IGFyZSBhdmFpbGFibGUgaW4gYW4gaW50ZXJuYWwgU2FmYXJpIFNESy4KICNpbmNsdWRlIDxDb2xv
clN5bmMvQ29sb3JTeW5jLmg+CkBAIC0xNTUsMTAgKzE1NSwxMCBAQCBfTElCQ1BQX0VORF9OQU1F
U1BBQ0VfU1RECiAjaWYgIVBMQVRGT1JNKElPUykKICNpbmNsdWRlIDxDb3JlU2VydmljZXMvQ29y
ZVNlcnZpY2VzLmg+CiAjZW5kaWYgLy8gIVBMQVRGT1JNKElPUykKLSNlbmRpZiAvLyBkZWZpbmVk
KFdJTjMyKSB8fCBkZWZpbmVkKF9XSU4zMikKKyNlbmRpZiAvLyBPUyhXSU5ET1dTKQogCiAjZW5k
aWYKLSNlbmRpZiAvLyAhZGVmaW5lZChCVUlMRElOR19XWF9fKQorI2VuZGlmIC8vICFQTEFURk9S
TShXWCkKIAogI2lmZGVmIF9fT0JKQ19fCiAjaWYgUExBVEZPUk0oSU9TKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>185549</attachid>
            <date>2013-01-30 13:30:07 -0800</date>
            <delta_ts>2013-01-31 12:54:02 -0800</delta_ts>
            <desc>Patch v1 with bug URL</desc>
            <filename>bug-108374-20130130132652.patch</filename>
            <type>text/plain</type>
            <size>2612</size>
            <attacher name="David Kilzer (:ddkilzer)">ddkilzer</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQxMjgwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNWQ2MzBiYmIyYTg2ZjA4
Mzc1MWE4NDcxMzM0ZTc2YTIwYzk4NzI1Yy4uZmNlOTg2M2RlYTFmMTdhOTlhNjZiOWVlZmY4ZmNj
ZDZhYWNkMjU5MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTAxLTMwICBEYXZp
ZCBLaWx6ZXIgIDxkZGtpbHplckBhcHBsZS5jb20+CisKKyAgICAgICAgVXNlIE9TKCkgYW5kIFBM
QVRGT1JNKCkgbWFjcm9zIGluIFdlYkNvcmVQcmVmaXguaCBmb3IgcmVhZGFiaWxpdHkKKyAgICAg
ICAgPGh0dHA6Ly93ZWJraXQub3JnL2IvMTA4Mzc0PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgICogV2ViQ29yZVByZWZpeC5oOiBSZXBsYWNlZCBhbGwg
cG9zc2libGUgY2hlY2tzIHdpdGggT1MoKSBvcgorICAgICAgICBQTEFURk9STSgpIG1hY3JvcyBv
dGhlciB0aGFuIF9fQVBQTEVfXy4KKwogMjAxMy0wMS0zMCAgRmxvcmluIE1hbGl0YSAgPGZtYWxp
dGFAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIERvIG5vdCByZXN0YXJ0IHRoZSBtYXRjaGVkIHBy
b3BlcnRpZXMgY2FjaGUgdGltZXIgaWYgYWN0aXZlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29y
ZS9XZWJDb3JlUHJlZml4LmggYi9Tb3VyY2UvV2ViQ29yZS9XZWJDb3JlUHJlZml4LmgKaW5kZXgg
YzY2ODgwYjAxMDk0OTUwOGZkMzA5OTk3ODU2ODA3NTgxODIxNGZkMS4uOWE3YjBjYzc3YWVjNzdl
ZGU2OTU5Yjc1NzQyYTY5Y2VjYWMzZTIxOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvV2Vi
Q29yZVByZWZpeC5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL1dlYkNvcmVQcmVmaXguaApAQCAtMzcs
NyArMzcsNyBAQAogI2VuZGlmCiAjZW5kaWYKIAotI2lmIGRlZmluZWQoV0lOMzIpIHx8IGRlZmlu
ZWQoX1dJTjMyKQorI2lmIE9TKFdJTkRPV1MpCiAKICNpZm5kZWYgX1dJTjMyX1dJTk5UCiAjZGVm
aW5lIF9XSU4zMl9XSU5OVCAweDA1MDIKQEAgLTQ3LDcgKzQ3LDcgQEAKICNkZWZpbmUgV0lOVkVS
IDB4MDUwMgogI2VuZGlmCiAKLSNpZm5kZWYgV1RGX1VTRV9DVVJMCisjaWYgIVVTRShDVVJMKQog
I2lmbmRlZiBfV0lOU09DS0FQSV8KICNkZWZpbmUgX1dJTlNPQ0tBUElfIC8vIFByZXZlbnQgaW5j
bHVzaW9uIG9mIHdpbnNvY2suaCBpbiB3aW5kb3dzLmgKICNlbmRpZgpAQCAtNjQsNyArNjQsNyBA
QAogCiAjaW5jbHVkZSA8cHRocmVhZC5oPgogCi0jZW5kaWYgLy8gZGVmaW5lZChXSU4zMikgfHwg
ZGVmaW5lZChfV0lOMzIpCisjZW5kaWYgLy8gT1MoV0lORE9XUykKIAogI2luY2x1ZGUgPHN5cy90
eXBlcy5oPgogI2luY2x1ZGUgPGZjbnRsLmg+CkBAIC03NCw3ICs3NCw3IEBACiAKIC8vIE9uIExp
bnV4IHRoaXMgY2F1c2VzIGNvbmZsaWN0cyB3aXRoIGxpYnBuZyBiZWNhdXNlIHRoZXJlIGFyZSB0
d28gaW1wbHMuIG9mCiAvLyBsb25nam1wIC0gc2VlIGhlcmU6IGh0dHBzOi8vYnVncy5sYXVuY2hw
YWQubmV0L3VidW50dS8rc291cmNlL2xpYnBuZy8rYnVnLzIxODQwOQotI2lmbmRlZiBCVUlMRElO
R19XWF9fCisjaWYgIVBMQVRGT1JNKFdYKQogI2luY2x1ZGUgPHNldGptcC5oPgogI2VuZGlmCiAK
QEAgLTEzMiwxNCArMTMyLDE0IEBAIF9MSUJDUFBfRU5EX05BTUVTUEFDRV9TVEQKICNpbmNsdWRl
IDxzeXMvcmVzb3VyY2UuaD4KICNlbmRpZgogCi0jaWYgIWRlZmluZWQoQlVJTERJTkdfV1hfXykK
KyNpZiAhUExBVEZPUk0oV1gpCiAjaW5jbHVkZSA8Q29yZUZvdW5kYXRpb24vQ29yZUZvdW5kYXRp
b24uaD4KLSNpZmRlZiBXVEZfUExBVEZPUk1fV0lOX0NBSVJPCisjaWYgUExBVEZPUk0oV0lOX0NB
SVJPKQogI2luY2x1ZGUgPENvbmRpdGlvbmFsTWFjcm9zLmg+CiAjaW5jbHVkZSA8d2luZG93cy5o
PgogI2Vsc2UKIAotI2lmIGRlZmluZWQoV0lOMzIpIHx8IGRlZmluZWQoX1dJTjMyKQorI2lmIE9T
KFdJTkRPV1MpCiAvLyBGSVhNRSA8cmRhcjovL3Byb2JsZW0vODIwODg2OD4gUmVtb3ZlIHN1cHBv
cnQgZm9yIG9ic29sZXRlIENvbG9yU3luYyBBUEksIENvcmVTZXJ2aWNlcyBoZWFkZXIgaW4gQ29y
ZUdyYXBoaWNzCiAvLyBXZSBjYW4gcmVtb3ZlIHRoaXMgb25jZSB0aGUgbmV3IENvbG9yU3luYyBB
UElzIGFyZSBhdmFpbGFibGUgaW4gYW4gaW50ZXJuYWwgU2FmYXJpIFNESy4KICNpbmNsdWRlIDxD
b2xvclN5bmMvQ29sb3JTeW5jLmg+CkBAIC0xNTUsMTAgKzE1NSwxMCBAQCBfTElCQ1BQX0VORF9O
QU1FU1BBQ0VfU1RECiAjaWYgIVBMQVRGT1JNKElPUykKICNpbmNsdWRlIDxDb3JlU2VydmljZXMv
Q29yZVNlcnZpY2VzLmg+CiAjZW5kaWYgLy8gIVBMQVRGT1JNKElPUykKLSNlbmRpZiAvLyBkZWZp
bmVkKFdJTjMyKSB8fCBkZWZpbmVkKF9XSU4zMikKKyNlbmRpZiAvLyBPUyhXSU5ET1dTKQogCiAj
ZW5kaWYKLSNlbmRpZiAvLyAhZGVmaW5lZChCVUlMRElOR19XWF9fKQorI2VuZGlmIC8vICFQTEFU
Rk9STShXWCkKIAogI2lmZGVmIF9fT0JKQ19fCiAjaWYgUExBVEZPUk0oSU9TKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>