<?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>148766</bug_id>
          
          <creation_ts>2015-09-03 18:13:28 -0700</creation_ts>
          <short_desc>WatchpointsOnStructureStubInfo doesn&apos;t need to be reference counted</short_desc>
          <delta_ts>2015-09-03 18:21:15 -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>JavaScriptCore</component>
          <version>Other</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>148718</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Filip Pizlo">fpizlo</reporter>
          <assigned_to name="Filip Pizlo">fpizlo</assigned_to>
          <cc>barraclough</cc>
    
    <cc>basile_clement</cc>
    
    <cc>benjamin</cc>
    
    <cc>ggaren</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mhahnenb</cc>
    
    <cc>msaboff</cc>
    
    <cc>nrotem</cc>
    
    <cc>oliver</cc>
    
    <cc>saam</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1123254</commentid>
    <comment_count>0</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2015-09-03 18:13:28 -0700</bug_when>
    <thetext>It doesn&apos;t need to be reference counted because the only RefPtr to it is in StructureStubInfo.  Therefore, it can be a unique_ptr.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1123255</commentid>
    <comment_count>1</comment_count>
      <attachid>260546</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2015-09-03 18:14:25 -0700</bug_when>
    <thetext>Created attachment 260546
the patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1123256</commentid>
    <comment_count>2</comment_count>
      <attachid>260546</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2015-09-03 18:16:35 -0700</bug_when>
    <thetext>Comment on attachment 260546
the patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1123258</commentid>
    <comment_count>3</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2015-09-03 18:21:15 -0700</bug_when>
    <thetext>Landed in http://trac.webkit.org/changeset/189328</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>260546</attachid>
            <date>2015-09-03 18:14:25 -0700</date>
            <delta_ts>2015-09-03 18:16:35 -0700</delta_ts>
            <desc>the patch</desc>
            <filename>blah.patch</filename>
            <type>text/plain</type>
            <size>4258</size>
            <attacher name="Filip Pizlo">fpizlo</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTg5MzI3KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIyIEBA
CisyMDE1LTA5LTAzICBGaWxpcCBQaXpsbyAgPGZwaXpsb0BhcHBsZS5jb20+CisKKyAgICAgICAg
V2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0dWJJbmZvIGRvZXNuJ3QgbmVlZCB0byBiZSByZWZlcmVu
Y2UgY291bnRlZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9MTQ4NzY2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgSXQgZG9lc24ndCBuZWVkIHRvIGJlIHJlZmVyZW5jZSBjb3VudGVkIGJlY2F1c2UgdGhlIG9u
bHkgUmVmUHRyIHRvIGl0IGlzIGluCisgICAgICAgIFN0cnVjdHVyZVN0dWJJbmZvLiBUaGVyZWZv
cmUsIGl0IGNhbiBiZSBhIHVuaXF1ZV9wdHIuCisKKyAgICAgICAgKiBieXRlY29kZS9TdHJ1Y3R1
cmVTdHViQ2xlYXJpbmdXYXRjaHBvaW50LmNwcDoKKyAgICAgICAgKEpTQzo6V2F0Y2hwb2ludHNP
blN0cnVjdHVyZVN0dWJJbmZvOjphZGRXYXRjaHBvaW50KToKKyAgICAgICAgKEpTQzo6V2F0Y2hw
b2ludHNPblN0cnVjdHVyZVN0dWJJbmZvOjplbnN1cmVSZWZlcmVuY2VBbmRBZGRXYXRjaHBvaW50
KToKKyAgICAgICAgKiBieXRlY29kZS9TdHJ1Y3R1cmVTdHViQ2xlYXJpbmdXYXRjaHBvaW50Lmg6
CisgICAgICAgIChKU0M6OldhdGNocG9pbnRzT25TdHJ1Y3R1cmVTdHViSW5mbzo6V2F0Y2hwb2lu
dHNPblN0cnVjdHVyZVN0dWJJbmZvKToKKyAgICAgICAgKEpTQzo6V2F0Y2hwb2ludHNPblN0cnVj
dHVyZVN0dWJJbmZvOjpjb2RlQmxvY2spOgorICAgICAgICAqIGJ5dGVjb2RlL1N0cnVjdHVyZVN0
dWJJbmZvLmg6CisgICAgICAgIChKU0M6OmdldFN0cnVjdHVyZVN0dWJJbmZvQ29kZU9yaWdpbik6
CisKIDIwMTUtMDktMDMgIEJhc2lsZSBDbGVtZW50ICA8YmFzaWxlX2NsZW1lbnRAYXBwbGUuY29t
PgogCiAgICAgICAgIEphdmFTY3JpcHQgZnVuY3Rpb25zIHNob3VsZCByZXN0b3JlIHRoZSBzdGFj
ayBwb2ludGVyIGFmdGVyIGEgY2FsbApJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVj
b2RlL1N0cnVjdHVyZVN0dWJDbGVhcmluZ1dhdGNocG9pbnQuY3BwCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNv
dXJjZS9KYXZhU2NyaXB0Q29yZS9ieXRlY29kZS9TdHJ1Y3R1cmVTdHViQ2xlYXJpbmdXYXRjaHBv
aW50LmNwcAkocmV2aXNpb24gMTg5MzIzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVj
b2RlL1N0cnVjdHVyZVN0dWJDbGVhcmluZ1dhdGNocG9pbnQuY3BwCSh3b3JraW5nIGNvcHkpCkBA
IC03NiwxMSArNzYsMTEgQEAgU3RydWN0dXJlU3R1YkNsZWFyaW5nV2F0Y2hwb2ludCogV2F0Y2hw
bwogfQogCiBTdHJ1Y3R1cmVTdHViQ2xlYXJpbmdXYXRjaHBvaW50KiBXYXRjaHBvaW50c09uU3Ry
dWN0dXJlU3R1YkluZm86OmVuc3VyZVJlZmVyZW5jZUFuZEFkZFdhdGNocG9pbnQoCi0gICAgUmVm
UHRyPFdhdGNocG9pbnRzT25TdHJ1Y3R1cmVTdHViSW5mbz4mIGhvbGRlclJlZiwgQ29kZUJsb2Nr
KiBjb2RlQmxvY2ssCisgICAgc3RkOjp1bmlxdWVfcHRyPFdhdGNocG9pbnRzT25TdHJ1Y3R1cmVT
dHViSW5mbz4mIGhvbGRlclJlZiwgQ29kZUJsb2NrKiBjb2RlQmxvY2ssCiAgICAgU3RydWN0dXJl
U3R1YkluZm8qIHN0dWJJbmZvLCBjb25zdCBPYmplY3RQcm9wZXJ0eUNvbmRpdGlvbiYga2V5KQog
ewogICAgIGlmICghaG9sZGVyUmVmKQotICAgICAgICBob2xkZXJSZWYgPSBhZG9wdFJlZihuZXcg
V2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0dWJJbmZvKGNvZGVCbG9jaywgc3R1YkluZm8pKTsKKyAg
ICAgICAgaG9sZGVyUmVmID0gc3RkOjptYWtlX3VuaXF1ZTxXYXRjaHBvaW50c09uU3RydWN0dXJl
U3R1YkluZm8+KGNvZGVCbG9jaywgc3R1YkluZm8pOwogICAgIGVsc2UgewogICAgICAgICBBU1NF
UlQoaG9sZGVyUmVmLT5tX2NvZGVCbG9jayA9PSBjb2RlQmxvY2spOwogICAgICAgICBBU1NFUlQo
aG9sZGVyUmVmLT5tX3N0dWJJbmZvID09IHN0dWJJbmZvKTsKSW5kZXg6IFNvdXJjZS9KYXZhU2Ny
aXB0Q29yZS9ieXRlY29kZS9TdHJ1Y3R1cmVTdHViQ2xlYXJpbmdXYXRjaHBvaW50LmgKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb2RlL1N0cnVjdHVyZVN0dWJDbGVh
cmluZ1dhdGNocG9pbnQuaAkocmV2aXNpb24gMTg5MzIyKQorKysgU291cmNlL0phdmFTY3JpcHRD
b3JlL2J5dGVjb2RlL1N0cnVjdHVyZVN0dWJDbGVhcmluZ1dhdGNocG9pbnQuaAkod29ya2luZyBj
b3B5KQpAQCAtMzMsOCArMzMsNiBAQAogCiAjaW5jbHVkZSA8d3RmL0Zhc3RNYWxsb2MuaD4KICNp
bmNsdWRlIDx3dGYvTm9uY29weWFibGUuaD4KLSNpbmNsdWRlIDx3dGYvUmVmQ291bnRlZC5oPgot
I2luY2x1ZGUgPHd0Zi9SZWZQdHIuaD4KIAogbmFtZXNwYWNlIEpTQyB7CiAKQEAgLTcyLDcgKzcw
LDkgQEAgcHJpdmF0ZToKICAgICBzdGQ6OnVuaXF1ZV9wdHI8U3RydWN0dXJlU3R1YkNsZWFyaW5n
V2F0Y2hwb2ludD4gbV9uZXh0OwogfTsKIAotY2xhc3MgV2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0
dWJJbmZvIDogcHVibGljIFJlZkNvdW50ZWQ8V2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0dWJJbmZv
PiB7CitjbGFzcyBXYXRjaHBvaW50c09uU3RydWN0dXJlU3R1YkluZm8geworICAgIFdURl9NQUtF
X05PTkNPUFlBQkxFKFdhdGNocG9pbnRzT25TdHJ1Y3R1cmVTdHViSW5mbyk7CisgICAgV1RGX01B
S0VfRkFTVF9BTExPQ0FURUQ7CiBwdWJsaWM6CiAgICAgV2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0
dWJJbmZvKENvZGVCbG9jayogY29kZUJsb2NrLCBTdHJ1Y3R1cmVTdHViSW5mbyogc3R1YkluZm8p
CiAgICAgICAgIDogbV9jb2RlQmxvY2soY29kZUJsb2NrKQpAQCAtODUsNyArODUsNyBAQCBwdWJs
aWM6CiAgICAgU3RydWN0dXJlU3R1YkNsZWFyaW5nV2F0Y2hwb2ludCogYWRkV2F0Y2hwb2ludChj
b25zdCBPYmplY3RQcm9wZXJ0eUNvbmRpdGlvbiYga2V5KTsKICAgICAKICAgICBzdGF0aWMgU3Ry
dWN0dXJlU3R1YkNsZWFyaW5nV2F0Y2hwb2ludCogZW5zdXJlUmVmZXJlbmNlQW5kQWRkV2F0Y2hw
b2ludCgKLSAgICAgICAgUmVmUHRyPFdhdGNocG9pbnRzT25TdHJ1Y3R1cmVTdHViSW5mbz4mIGhv
bGRlclJlZiwKKyAgICAgICAgc3RkOjp1bmlxdWVfcHRyPFdhdGNocG9pbnRzT25TdHJ1Y3R1cmVT
dHViSW5mbz4mIGhvbGRlclJlZiwKICAgICAgICAgQ29kZUJsb2NrKiwgU3RydWN0dXJlU3R1Yklu
Zm8qLCBjb25zdCBPYmplY3RQcm9wZXJ0eUNvbmRpdGlvbiYga2V5KTsKICAgICAKICAgICBDb2Rl
QmxvY2sqIGNvZGVCbG9jaygpIGNvbnN0IHsgcmV0dXJuIG1fY29kZUJsb2NrOyB9CkluZGV4OiBT
b3VyY2UvSmF2YVNjcmlwdENvcmUvYnl0ZWNvZGUvU3RydWN0dXJlU3R1YkluZm8uaAo9PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvYnl0ZWNvZGUvU3RydWN0dXJlU3R1YkluZm8u
aAkocmV2aXNpb24gMTg5MzIzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb2RlL1N0
cnVjdHVyZVN0dWJJbmZvLmgJKHdvcmtpbmcgY29weSkKQEAgLTIzMCw3ICsyMzAsNyBAQCBzdHJ1
Y3QgU3RydWN0dXJlU3R1YkluZm8gewogCiAgICAgUmVmUHRyPEpJVFN0dWJSb3V0aW5lPiBzdHVi
Um91dGluZTsKICAgICBDb2RlTG9jYXRpb25DYWxsIGNhbGxSZXR1cm5Mb2NhdGlvbjsKLSAgICBS
ZWZQdHI8V2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0dWJJbmZvPiB3YXRjaHBvaW50czsKKyAgICBz
dGQ6OnVuaXF1ZV9wdHI8V2F0Y2hwb2ludHNPblN0cnVjdHVyZVN0dWJJbmZvPiB3YXRjaHBvaW50
czsKIH07CiAKIGlubGluZSBDb2RlT3JpZ2luIGdldFN0cnVjdHVyZVN0dWJJbmZvQ29kZU9yaWdp
bihTdHJ1Y3R1cmVTdHViSW5mbyYgc3RydWN0dXJlU3R1YkluZm8pCg==
</data>
<flag name="review"
          id="285730"
          type_id="1"
          status="+"
          setter="saam"
    />
          </attachment>
      

    </bug>

</bugzilla>