<?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>59428</bug_id>
          
          <creation_ts>2011-04-25 23:48:45 -0700</creation_ts>
          <short_desc>Enable strict mode for OwnPtr and PassOwnPtr</short_desc>
          <delta_ts>2011-05-01 18:26:19 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</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>
          <dependson>59666</dependson>
    
    <dependson>59667</dependson>
    
    <dependson>59861</dependson>
    
    <dependson>59876</dependson>
    
    <dependson>59878</dependson>
    
    <dependson>59881</dependson>
    
    <dependson>59898</dependson>
          <blocked>59909</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Barth">abarth</reporter>
          <assigned_to name="Adam Barth">abarth</assigned_to>
          <cc>abarth</cc>
    
    <cc>buildbot</cc>
    
    <cc>darin</cc>
    
    <cc>dglazkov</cc>
    
    <cc>eric</cc>
    
    <cc>gustavo.noronha</cc>
    
    <cc>gustavo</cc>
    
    <cc>levin</cc>
    
    <cc>webkit-ews</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>392309</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-25 23:48:45 -0700</bug_when>
    <thetext>Enable strict mode for OwnPtr</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392310</commentid>
    <comment_count>1</comment_count>
      <attachid>91073</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-25 23:49:26 -0700</bug_when>
    <thetext>Created attachment 91073
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392313</commentid>
    <comment_count>2</comment_count>
      <attachid>91073</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2011-04-25 23:56:56 -0700</bug_when>
    <thetext>Comment on attachment 91073
Patch

pending ews results.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392317</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 00:12:28 -0700</bug_when>
    <thetext>Attachment 91073 did not build on chromium:
Build output: http://queues.webkit.org/results/8509542</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392321</commentid>
    <comment_count>4</comment_count>
    <who name="Early Warning System Bot">webkit-ews</who>
    <bug_when>2011-04-26 00:19:32 -0700</bug_when>
    <thetext>Attachment 91073 did not build on qt:
Build output: http://queues.webkit.org/results/8509543</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392338</commentid>
    <comment_count>5</comment_count>
    <who name="Build Bot">buildbot</who>
    <bug_when>2011-04-26 00:41:44 -0700</bug_when>
    <thetext>Attachment 91073 did not build on win:
Build output: http://queues.webkit.org/results/8506647</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392343</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 00:55:45 -0700</bug_when>
    <thetext>Attachment 91073 did not build on chromium:
Build output: http://queues.webkit.org/results/8508569</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392369</commentid>
    <comment_count>7</comment_count>
    <who name="Collabora GTK+ EWS bot">gustavo.noronha</who>
    <bug_when>2011-04-26 03:01:40 -0700</bug_when>
    <thetext>Attachment 91073 did not build on gtk:
Build output: http://queues.webkit.org/results/8507579</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393763</commentid>
    <comment_count>8</comment_count>
      <attachid>91337</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-27 13:40:17 -0700</bug_when>
    <thetext>Created attachment 91337
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393764</commentid>
    <comment_count>9</comment_count>
      <attachid>91337</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2011-04-27 13:41:34 -0700</bug_when>
    <thetext>Comment on attachment 91337
Patch

This will make it easier for other ports to break the mac platform, but I guess it will get fixed quickly and it shouldn&apos;t happen too often.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393777</commentid>
    <comment_count>10</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-27 13:55:51 -0700</bug_when>
    <thetext>&gt; This will make it easier for other ports to break the mac platform, but I guess it will get fixed quickly and it shouldn&apos;t happen too often.

Yeah, this is just an intermediate state.  We shouldn&apos;t linger here very long.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393781</commentid>
    <comment_count>11</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-27 13:59:04 -0700</bug_when>
    <thetext>Committed r85084: &lt;http://trac.webkit.org/changeset/85084&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393790</commentid>
    <comment_count>12</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-27 14:03:36 -0700</bug_when>
    <thetext>Re-opening because we need to enable this on other platforms.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393815</commentid>
    <comment_count>13</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-27 14:19:02 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/85084 might have broken Leopard Intel Debug (Build)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393844</commentid>
    <comment_count>14</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-04-27 14:41:49 -0700</bug_when>
    <thetext>We also want to enable the strict mode for PassOwnPtr.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393849</commentid>
    <comment_count>15</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-04-27 14:45:56 -0700</bug_when>
    <thetext>(In reply to comment #14)
&gt; We also want to enable the strict mode for PassOwnPtr.

Yep.  I&apos;m testing compile for that one now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>394122</commentid>
    <comment_count>16</comment_count>
      <attachid>91337</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2011-04-27 19:29:28 -0700</bug_when>
    <thetext>Comment on attachment 91337
Patch

Clearing r+ since it has been landed (to move out of the list to be committed).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396104</commentid>
    <comment_count>17</comment_count>
      <attachid>91860</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-05-01 17:33:22 -0700</bug_when>
    <thetext>Created attachment 91860
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396109</commentid>
    <comment_count>18</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-05-01 17:33:50 -0700</bug_when>
    <thetext>Committed r85445: &lt;http://trac.webkit.org/changeset/85445&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396156</commentid>
    <comment_count>19</comment_count>
      <attachid>91860</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-05-01 18:02:54 -0700</bug_when>
    <thetext>Comment on attachment 91860
Patch

Oh, well technically we didn&apos;t do strict PassOwnPtr, but that can be another bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91073</attachid>
            <date>2011-04-25 23:49:26 -0700</date>
            <delta_ts>2011-04-27 13:40:12 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59428-20110425234924.patch</filename>
            <type>text/plain</type>
            <size>1179</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gODQ4ODYpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAK
KzIwMTEtMDQtMjUgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBFbmFibGUgc3RyaWN0IG1vZGUg
Zm9yIE93blB0cgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NTk0MjgKKworICAgICAgICBUaGVyZSBhcmUgc3RpbGwgZm91ciB3aGl0ZWxpc3RlZCBmaWxl
cyBpbiBXZWJDb3JlIHRoYXQgbmVlZCB0byBiZQorICAgICAgICBmaXhlZCwgYnV0IHRoaXMgcGF0
Y2ggc2hvdWxkIHByZXZlbnQgcmVncmVzc2lvbnMuCisKKyAgICAgICAgKiB3dGYvT3duUHRyLmg6
CisKIDIwMTEtMDQtMjUgIEdlb2ZmcmV5IEdhcmVuICA8Z2dhcmVuQGFwcGxlLmNvbT4KIAogICAg
ICAgICBSZXZpZXdlZCBieSBPbGl2ZXIgSHVudC4KSW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29y
ZS93dGYvT3duUHRyLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9P
d25QdHIuaAkocmV2aXNpb24gODQ4ODApCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvd3RmL093
blB0ci5oCSh3b3JraW5nIGNvcHkpCkBAIC0yOCw5ICsyOCw2IEBACiAjaW5jbHVkZSA8YWxnb3Jp
dGhtPgogI2luY2x1ZGUgPG1lbW9yeT4KIAotLy8gUmVtb3ZlIHRoaXMgb25jZSB3ZSBtYWtlIGFs
bCBXZWJLaXQgY29kZSBjb21wYXRpYmxlIHdpdGggc3RyaWN0ZXIgcnVsZXMgYWJvdXQgT3duUHRy
LgotI2RlZmluZSBMT09TRV9PV05fUFRSCi0KIG5hbWVzcGFjZSBXVEYgewogCiAgICAgLy8gVW5s
aWtlIG1vc3Qgb2Ygb3VyIHNtYXJ0IHBvaW50ZXJzLCBPd25QdHIgY2FuIHRha2UgZWl0aGVyIHRo
ZSBwb2ludGVyIHR5cGUgb3IgdGhlIHBvaW50ZWQtdG8gdHlwZS4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91337</attachid>
            <date>2011-04-27 13:40:17 -0700</date>
            <delta_ts>2011-05-01 17:33:16 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59428-20110427134016.patch</filename>
            <type>text/plain</type>
            <size>1020</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gODUwNzgpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAK
KzIwMTEtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBFbmFibGUgc3RyaWN0IG1vZGUg
Zm9yIE93blB0cgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NTk0MjgKKworICAgICAgICBUaGlzIHBhdGNoIGVuYWJsZXMgc3RyaWN0IG1vZGUgZm9yIE93
blB0ciBvbiBQTEFURk9STShNQUMpIG9ubHkuCisKKyAgICAgICAgKiB3dGYvT3duUHRyLmg6CisK
IDIwMTEtMDQtMjcgIFN0ZXZlIEJsb2NrICA8c3RldmVibG9ja0Bnb29nbGUuY29tPgogCiAgICAg
ICAgIFJldmlld2VkIGJ5IERhdmlkIExldmluLgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3Jl
L3d0Zi9Pd25QdHIuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvd3RmL093
blB0ci5oCShyZXZpc2lvbiA4NTA3NCkKKysrIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS93dGYvT3du
UHRyLmgJKHdvcmtpbmcgY29weSkKQEAgLTI5LDcgKzI5LDkgQEAKICNpbmNsdWRlIDxtZW1vcnk+
CiAKIC8vIFJlbW92ZSB0aGlzIG9uY2Ugd2UgbWFrZSBhbGwgV2ViS2l0IGNvZGUgY29tcGF0aWJs
ZSB3aXRoIHN0cmljdGVyIHJ1bGVzIGFib3V0IE93blB0ci4KKyNpZiAhUExBVEZPUk0oTUFDKQog
I2RlZmluZSBMT09TRV9PV05fUFRSCisjZW5kaWYKIAogbmFtZXNwYWNlIFdURiB7CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91860</attachid>
            <date>2011-05-01 17:33:22 -0700</date>
            <delta_ts>2011-05-01 18:02:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59428-20110501173320.patch</filename>
            <type>text/plain</type>
            <size>1163</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gODU0NDQpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAK
KzIwMTEtMDUtMDEgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KKworICAgICAgICBS
ZXZpZXdlZCBieSBEYXZpZCBMZXZpbi4KKworICAgICAgICBFbmFibGUgc3RyaWN0IG1vZGUgZm9y
IE93blB0ciBhbmQgUGFzc093blB0cgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9NTk0MjgKKworICAgICAgICAqIHd0Zi9Pd25QdHIuaDoKKwogMjAxMS0w
NS0wMSAgUGF0cmljayBHYW5zdGVyZXIgIDxwYXJvZ2FAd2Via2l0Lm9yZz4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBBZGFtIEJhcnRoLgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9P
d25QdHIuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvd3RmL093blB0ci5o
CShyZXZpc2lvbiA4NTQ0NCkKKysrIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS93dGYvT3duUHRyLmgJ
KHdvcmtpbmcgY29weSkKQEAgLTI4LDExICsyOCw2IEBACiAjaW5jbHVkZSA8YWxnb3JpdGhtPgog
I2luY2x1ZGUgPG1lbW9yeT4KIAotLy8gUmVtb3ZlIHRoaXMgb25jZSB3ZSBtYWtlIGFsbCBXZWJL
aXQgY29kZSBjb21wYXRpYmxlIHdpdGggc3RyaWN0ZXIgcnVsZXMgYWJvdXQgT3duUHRyLgotI2lm
ICFQTEFURk9STShNQUMpICYmICFQTEFURk9STShDSFJPTUlVTSkgJiYgIVBMQVRGT1JNKFFUKSAm
JiAhUExBVEZPUk0oR1RLKSAmJiAhUExBVEZPUk0oV0lOKQotI2RlZmluZSBMT09TRV9PV05fUFRS
Ci0jZW5kaWYKLQogbmFtZXNwYWNlIFdURiB7CiAKICAgICAvLyBVbmxpa2UgbW9zdCBvZiBvdXIg
c21hcnQgcG9pbnRlcnMsIE93blB0ciBjYW4gdGFrZSBlaXRoZXIgdGhlIHBvaW50ZXIgdHlwZSBv
ciB0aGUgcG9pbnRlZC10byB0eXBlLgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>