<?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>35967</bug_id>
          
          <creation_ts>2010-03-10 04:17:24 -0800</creation_ts>
          <short_desc>Remove extra &lt;new&gt; include and add guards to operator new/delete definitions</short_desc>
          <delta_ts>2010-03-16 09:20:17 -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>Web Template Framework</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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="Zoltan Horvath">zoltan</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>197977</commentid>
    <comment_count>0</comment_count>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2010-03-10 04:17:24 -0800</bug_when>
    <thetext>Remove extra &lt;new&gt; header include from FastAlloc.cpp since it is included in FastAlloc.h. 

Add ENABLE_GLOBAL_FASTMALLOC_NEW macro guard to operator new/delete/new []/delete [] definitions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>197979</commentid>
    <comment_count>1</comment_count>
      <attachid>50392</attachid>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2010-03-10 04:19:30 -0800</bug_when>
    <thetext>Created attachment 50392
proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>199023</commentid>
    <comment_count>2</comment_count>
      <attachid>50392</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-03-12 06:57:45 -0800</bug_when>
    <thetext>Comment on attachment 50392
proposed patch

&gt; +#if defined(ENABLE_GLOBAL_FASTMALLOC_NEW)

Should this be the following instead ? 

+#if ENABLE(GLOBAL_FASTMALLOC_NEW)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>199862</commentid>
    <comment_count>3</comment_count>
      <attachid>50392</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-15 15:24:14 -0700</bug_when>
    <thetext>Comment on attachment 50392
proposed patch

Should use the ENABLE macros.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200056</commentid>
    <comment_count>4</comment_count>
      <attachid>50773</attachid>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2010-03-16 00:51:22 -0700</bug_when>
    <thetext>Created attachment 50773
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200242</commentid>
    <comment_count>5</comment_count>
      <attachid>50773</attachid>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2010-03-16 09:20:09 -0700</bug_when>
    <thetext>Comment on attachment 50773
Patch

Clearing flags on attachment: 50773

Committed r56066: &lt;http://trac.webkit.org/changeset/56066&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>200243</commentid>
    <comment_count>6</comment_count>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2010-03-16 09:20:17 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>50392</attachid>
            <date>2010-03-10 04:19:30 -0800</date>
            <delta_ts>2010-03-16 00:51:16 -0700</delta_ts>
            <desc>proposed patch</desc>
            <filename>fastmalloc.patch</filename>
            <type>text/plain</type>
            <size>1710</size>
            <attacher name="Zoltan Horvath">zoltan</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZyBiL0phdmFTY3JpcHRDb3JlL0No
YW5nZUxvZwppbmRleCBkYjZkM2Q5Li4zYTc3Zjc4IDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg
QEAKKzIwMTAtMDMtMTAgIFpvbHRhbiBIb3J2YXRoICA8em9sdGFuQHdlYmtpdC5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVtb3ZlIGV4dHJh
IDxuZXc+IGluY2x1ZGUgYW5kIGFkZCBndWFyZHMgdG8gb3BlcmF0b3IgbmV3L2RlbGV0ZSBkZWZp
bml0aW9ucworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
MzU5NjcKKworICAgICAgICBSZW1vdmUgZXh0cmEgPG5ldz4gaGVhZGVyIGluY2x1ZGUgZnJvbSBG
YXN0QWxsb2MuY3BwIHNpbmNlIGl0IGlzIGluY2x1ZGVkIGluIAorICAgICAgICBGYXN0QWxsb2Mu
aC4gQWRkIEVOQUJMRV9HTE9CQUxfRkFTVE1BTExPQ19ORVcgbWFjcm8gZ3VhcmQgdG8gb3BlcmF0
b3IKKyAgICAgICAgbmV3L2RlbGV0ZS9uZXcgW10vZGVsZXRlIFtdIGRlZmluaXRpb25zLgorCisg
ICAgICAgICogd3RmL0Zhc3RNYWxsb2MuY3BwOgorCiAyMDEwLTAzLTEwICBLZXZpbiBPbGxpdmll
ciAgPGtldmlub0B0aGVvbGxpdmllcnMuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMg
U2VpZGVsLgpkaWZmIC0tZ2l0IGEvSmF2YVNjcmlwdENvcmUvd3RmL0Zhc3RNYWxsb2MuY3BwIGIv
SmF2YVNjcmlwdENvcmUvd3RmL0Zhc3RNYWxsb2MuY3BwCmluZGV4IGQxYTRkMjguLjc0MWVmZjkg
MTAwNjQ0Ci0tLSBhL0phdmFTY3JpcHRDb3JlL3d0Zi9GYXN0TWFsbG9jLmNwcAorKysgYi9KYXZh
U2NyaXB0Q29yZS93dGYvRmFzdE1hbGxvYy5jcHAKQEAgLTQxNyw3ICs0MTcsNiBAQCBleHRlcm4g
IkMiIGNvbnN0IGludCBqc2NvcmVfZmFzdG1hbGxvY19pbnRyb3NwZWN0aW9uID0gMDsKICNpbmNs
dWRlIDxhbGdvcml0aG0+CiAjaW5jbHVkZSA8ZXJybm8uaD4KICNpbmNsdWRlIDxsaW1pdHM+Ci0j
aW5jbHVkZSA8bmV3PgogI2luY2x1ZGUgPHB0aHJlYWQuaD4KICNpbmNsdWRlIDxzdGRhcmcuaD4K
ICNpbmNsdWRlIDxzdGRkZWYuaD4KQEAgLTM5NDMsNiArMzk0Miw4IEBAIHN0YXRpYyBpbmxpbmUg
dm9pZCogY3BwX2FsbG9jKHNpemVfdCBzaXplLCBib29sIG5vdGhyb3cpIHsKICAgfQogfQogCisj
aWYgZGVmaW5lZChFTkFCTEVfR0xPQkFMX0ZBU1RNQUxMT0NfTkVXKQorCiB2b2lkKiBvcGVyYXRv
ciBuZXcoc2l6ZV90IHNpemUpIHsKICAgdm9pZCogcCA9IGNwcF9hbGxvYyhzaXplLCBmYWxzZSk7
CiAgIC8vIFdlIGtlZXAgdGhpcyBuZXh0IGluc3RydWN0aW9uIG91dCBvZiBjcHBfYWxsb2MgZm9y
IGEgcmVhc29uOiB3aGVuCkBAIC0zOTk3LDYgKzM5OTgsOCBAQCB2b2lkIG9wZXJhdG9yIGRlbGV0
ZVtdKHZvaWQqIHAsIGNvbnN0IHN0ZDo6bm90aHJvd190JikgX19USFJPVyB7CiAgIGRvX2ZyZWUo
cCk7CiB9CiAKKyNlbmRpZgorCiBleHRlcm4gIkMiIHZvaWQqIG1lbWFsaWduKHNpemVfdCBhbGln
biwgc2l6ZV90IHNpemUpIF9fVEhST1cgewogICB2b2lkKiByZXN1bHQgPSBkb19tZW1hbGlnbihh
bGlnbiwgc2l6ZSk7CiAgIE1hbGxvY0hvb2s6Okludm9rZU5ld0hvb2socmVzdWx0LCBzaXplKTsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>50773</attachid>
            <date>2010-03-16 00:51:22 -0700</date>
            <delta_ts>2010-03-16 09:20:09 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-35967-20100316085119.patch</filename>
            <type>text/plain</type>
            <size>1697</size>
            <attacher name="Zoltan Horvath">zoltan</attacher>
            
              <data encoding="base64">SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDU2MDQ2KQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDMtMTYgIFpvbHRhbiBI
b3J2YXRoICA8em9sdGFuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgUmVtb3ZlIGV4dHJhIDxuZXc+IGluY2x1ZGUgYW5kIGFkZCBn
dWFyZHMgdG8gb3BlcmF0b3IgbmV3L2RlbGV0ZSBkZWZpbml0aW9ucworICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzU5NjcKKworICAgICAgICBSZW1vdmUg
ZXh0cmEgPG5ldz4gaGVhZGVyIGluY2x1ZGUgZnJvbSBGYXN0QWxsb2MuY3BwIHNpbmNlIGl0IGlz
IGluY2x1ZGVkIGluIAorICAgICAgICBGYXN0QWxsb2MuaC4gQWRkIEVOQUJMRShHTE9CQUxfRkFT
VE1BTExPQ19ORVcpIG1hY3JvIGd1YXJkIHRvIG9wZXJhdG9yCisgICAgICAgIG5ldy9kZWxldGUv
bmV3IFtdL2RlbGV0ZSBbXSBkZWZpbml0aW9ucy4KKworICAgICAgICAqIHd0Zi9GYXN0TWFsbG9j
LmNwcDoKKwogMjAxMC0wMy0xNSAgS3dhbmcgWXVsIFNlbyAgPHNreXVsQGNvbXBhbnkxMDAubmV0
PgogCiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVsLgpJbmRleDogSmF2YVNjcmlwdENv
cmUvd3RmL0Zhc3RNYWxsb2MuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIEphdmFTY3JpcHRDb3JlL3d0Zi9G
YXN0TWFsbG9jLmNwcAkocmV2aXNpb24gNTYwNDYpCisrKyBKYXZhU2NyaXB0Q29yZS93dGYvRmFz
dE1hbGxvYy5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTQxNyw3ICs0MTcsNiBAQCBleHRlcm4gIkMi
IGNvbnN0IGludCBqc2NvcmVfZmFzdG1hbGxvY19pCiAjaW5jbHVkZSA8YWxnb3JpdGhtPgogI2lu
Y2x1ZGUgPGVycm5vLmg+CiAjaW5jbHVkZSA8bGltaXRzPgotI2luY2x1ZGUgPG5ldz4KICNpbmNs
dWRlIDxwdGhyZWFkLmg+CiAjaW5jbHVkZSA8c3RkYXJnLmg+CiAjaW5jbHVkZSA8c3RkZGVmLmg+
CkBAIC0zOTQyLDYgKzM5NDEsOCBAQCBzdGF0aWMgaW5saW5lIHZvaWQqIGNwcF9hbGxvYyhzaXpl
X3Qgc2l6CiAgIH0KIH0KIAorI2lmIEVOQUJMRShHTE9CQUxfRkFTVE1BTExPQ19ORVcpCisKIHZv
aWQqIG9wZXJhdG9yIG5ldyhzaXplX3Qgc2l6ZSkgewogICB2b2lkKiBwID0gY3BwX2FsbG9jKHNp
emUsIGZhbHNlKTsKICAgLy8gV2Uga2VlcCB0aGlzIG5leHQgaW5zdHJ1Y3Rpb24gb3V0IG9mIGNw
cF9hbGxvYyBmb3IgYSByZWFzb246IHdoZW4KQEAgLTM5OTYsNiArMzk5Nyw4IEBAIHZvaWQgb3Bl
cmF0b3IgZGVsZXRlW10odm9pZCogcCwgY29uc3Qgc3QKICAgZG9fZnJlZShwKTsKIH0KIAorI2Vu
ZGlmCisKIGV4dGVybiAiQyIgdm9pZCogbWVtYWxpZ24oc2l6ZV90IGFsaWduLCBzaXplX3Qgc2l6
ZSkgX19USFJPVyB7CiAgIHZvaWQqIHJlc3VsdCA9IGRvX21lbWFsaWduKGFsaWduLCBzaXplKTsK
ICAgTWFsbG9jSG9vazo6SW52b2tlTmV3SG9vayhyZXN1bHQsIHNpemUpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>