<?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>131197</bug_id>
          
          <creation_ts>2014-04-03 17:46:18 -0700</creation_ts>
          <short_desc>All Heap::writeBarriers should be inline</short_desc>
          <delta_ts>2014-04-29 15:30:18 -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>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>
          
          <blocked>121074</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Hahnenberg">mhahnenberg</reporter>
          <assigned_to name="Mark Hahnenberg">mhahnenberg</assigned_to>
          <cc>commit-queue</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>997397</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-03 17:46:18 -0700</bug_when>
    <thetext>One is in a JSCellInlines.h, another is in Heap.cpp. These are all critical enough and small enough to belong in HeapInlines.h</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997398</commentid>
    <comment_count>1</comment_count>
      <attachid>228565</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-03 17:48:20 -0700</bug_when>
    <thetext>Created attachment 228565
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997404</commentid>
    <comment_count>2</comment_count>
      <attachid>228565</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-03 17:59:29 -0700</bug_when>
    <thetext>Comment on attachment 228565
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997441</commentid>
    <comment_count>3</comment_count>
      <attachid>228565</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-03 19:24:55 -0700</bug_when>
    <thetext>Comment on attachment 228565
Patch

Clearing flags on attachment: 228565

Committed r166759: &lt;http://trac.webkit.org/changeset/166759&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997442</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-03 19:24:57 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>228565</attachid>
            <date>2014-04-03 17:48:20 -0700</date>
            <delta_ts>2014-04-03 19:24:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-131197-20140403174631.patch</filename>
            <type>text/plain</type>
            <size>4670</size>
            <attacher name="Mark Hahnenberg">mhahnenberg</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTY2NzU1KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIyIEBA
CisyMDE0LTA0LTAzICBNYXJrIEhhaG5lbmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CisK
KyAgICAgICAgQWxsIEhlYXA6OndyaXRlQmFycmllcnMgc2hvdWxkIGJlIGlubGluZQorICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTMxMTk3CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgT25lIGlzIGluIGEgSlND
ZWxsSW5saW5lcy5oLCBhbm90aGVyIGlzIGluIEhlYXAuY3BwLiBUaGVzZSBhcmUgYWxsIGNyaXRp
Y2FsIAorICAgICAgICBlbm91Z2ggYW5kIHNtYWxsIGVub3VnaCB0byBiZWxvbmcgaW4gSGVhcElu
bGluZXMuaC4gQWxzbyBhZGRlZCB0aGUgcHJvcGVyIAorICAgICAgICBFTkFCTEUoR0dDKSBpZmRl
ZnMgdG8gbWluaW1pemUgdGhlIGNvc3Qgb2YgQysrIGJhcnJpZXJzIGZvciAhRU5BQkxFKEdHQykg
YnVpbGRzLgorCisgICAgICAgICogaGVhcC9IZWFwLmNwcDoKKyAgICAgICAgKEpTQzo6SGVhcDo6
d3JpdGVCYXJyaWVyKTogRGVsZXRlZC4KKyAgICAgICAgKiBoZWFwL0hlYXAuaDoKKyAgICAgICAg
KiBoZWFwL0hlYXBJbmxpbmVzLmg6CisgICAgICAgIChKU0M6OkhlYXA6OndyaXRlQmFycmllcik6
CisgICAgICAgICogcnVudGltZS9KU0NlbGxJbmxpbmVzLmg6CisgICAgICAgIChKU0M6OkhlYXA6
OndyaXRlQmFycmllcik6IERlbGV0ZWQuCisKIDIwMTQtMDQtMDMgIE1hcmsgTGFtICA8bWFyay5s
YW1AYXBwbGUuY29tPgogCiAgICAgICAgIEZpeCBiaXQgcm90IGluIEFSTXY3IEpJVCBwcm9iZSBt
ZWNoYW5pc20uCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9IZWFwLmNwcAo9PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9IZWFwLmNwcAkocmV2aXNpb24g
MTY2NzQzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvSGVhcC5jcHAJKHdvcmtpbmcg
Y29weSkKQEAgLTEyOTYsMjEgKzEyOTYsNiBAQCB2b2lkIEhlYXA6OnpvbWJpZnlEZWFkT2JqZWN0
cygpCiAgICAgbV9vYmplY3RTcGFjZS5mb3JFYWNoRGVhZENlbGw8Wm9tYmlmeT4oaXRlcmF0aW9u
U2NvcGUpOwogfQogCi12b2lkIEhlYXA6OndyaXRlQmFycmllcihjb25zdCBKU0NlbGwqIGZyb20p
Ci17Ci0jaWYgRU5BQkxFKEdHQykKLSAgICBBU1NFUlRfR0NfT0JKRUNUX0xPT0tTX1ZBTElEKGNv
bnN0X2Nhc3Q8SlNDZWxsKj4oZnJvbSkpOwotICAgIGlmICghZnJvbSB8fCAhZnJvbS0+aXNNYXJr
ZWQoKSkgewotICAgICAgICBBU1NFUlQoIWZyb20gfHwgIWlzTWFya2VkKGZyb20pKTsKLSAgICAg
ICAgcmV0dXJuOwotICAgIH0KLSAgICBBU1NFUlQoaXNNYXJrZWQoZnJvbSkpOwotICAgIGFkZFRv
UmVtZW1iZXJlZFNldChmcm9tKTsKLSNlbHNlCi0gICAgVU5VU0VEX1BBUkFNKGZyb20pOwotI2Vu
ZGlmCi19Ci0KIHZvaWQgSGVhcDo6Zmx1c2hXcml0ZUJhcnJpZXJCdWZmZXIoSlNDZWxsKiBjZWxs
KQogewogI2lmIEVOQUJMRShHR0MpCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9I
ZWFwLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvSGVhcC5oCShy
ZXZpc2lvbiAxNjY3NDMpCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9IZWFwLmgJKHdv
cmtpbmcgY29weSkKQEAgLTEwMCw3ICsxMDAsNyBAQCBwdWJsaWM6CiAKICAgICBKU19FWFBPUlRf
UFJJVkFURSB2b2lkIGFkZFRvUmVtZW1iZXJlZFNldChjb25zdCBKU0NlbGwqKTsKICAgICBzdGF0
aWMgYm9vbCBpc1dyaXRlQmFycmllckVuYWJsZWQoKTsKLSAgICBKU19FWFBPUlRfUFJJVkFURSB2
b2lkIHdyaXRlQmFycmllcihjb25zdCBKU0NlbGwqKTsKKyAgICB2b2lkIHdyaXRlQmFycmllcihj
b25zdCBKU0NlbGwqKTsKICAgICB2b2lkIHdyaXRlQmFycmllcihjb25zdCBKU0NlbGwqLCBKU1Zh
bHVlKTsKICAgICB2b2lkIHdyaXRlQmFycmllcihjb25zdCBKU0NlbGwqLCBKU0NlbGwqKTsKIApJ
bmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvSGVhcElubGluZXMuaAo9PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
Ci0tLSBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9IZWFwSW5saW5lcy5oCShyZXZpc2lvbiAx
NjY3NDMpCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaGVhcC9IZWFwSW5saW5lcy5oCSh3b3Jr
aW5nIGNvcHkpCkBAIC0yOCw2ICsyOCw3IEBACiAKICNpbmNsdWRlICJIZWFwLmgiCiAjaW5jbHVk
ZSAiSlNDZWxsLmgiCisjaW5jbHVkZSAiU3RydWN0dXJlLmgiCiAKIG5hbWVzcGFjZSBKU0Mgewog
CkBAIC0xMDUsOSArMTA2LDUwIEBAIGlubGluZSB2b2lkIEhlYXA6OndyaXRlQmFycmllcihjb25z
dCBKU0MKICNpZiBFTkFCTEUoV1JJVEVfQkFSUklFUl9QUk9GSUxJTkcpCiAgICAgV3JpdGVCYXJy
aWVyQ291bnRlcnM6OmNvdW50V3JpdGVCYXJyaWVyKCk7CiAjZW5kaWYKKyNpZiBFTkFCTEUoR0dD
KQogICAgIGlmICghdG8uaXNDZWxsKCkpCiAgICAgICAgIHJldHVybjsKICAgICB3cml0ZUJhcnJp
ZXIoZnJvbSwgdG8uYXNDZWxsKCkpOworI2Vsc2UKKyAgICBVTlVTRURfUEFSQU0oZnJvbSk7Cisg
ICAgVU5VU0VEX1BBUkFNKHRvKTsKKyNlbmRpZgorfQorCitpbmxpbmUgdm9pZCBIZWFwOjp3cml0
ZUJhcnJpZXIoY29uc3QgSlNDZWxsKiBmcm9tLCBKU0NlbGwqIHRvKQoreworI2lmIEVOQUJMRShX
UklURV9CQVJSSUVSX1BST0ZJTElORykKKyAgICBXcml0ZUJhcnJpZXJDb3VudGVyczo6Y291bnRX
cml0ZUJhcnJpZXIoKTsKKyNlbmRpZgorI2lmIEVOQUJMRShHR0MpCisgICAgaWYgKCFmcm9tIHx8
ICFmcm9tLT5pc01hcmtlZCgpKSB7CisgICAgICAgIEFTU0VSVCghZnJvbSB8fCAhaXNNYXJrZWQo
ZnJvbSkpOworICAgICAgICByZXR1cm47CisgICAgfQorICAgIGlmICghdG8gfHwgdG8tPmlzTWFy
a2VkKCkpIHsKKyAgICAgICAgQVNTRVJUKCF0byB8fCBpc01hcmtlZCh0bykpOworICAgICAgICBy
ZXR1cm47CisgICAgfQorICAgIGFkZFRvUmVtZW1iZXJlZFNldChmcm9tKTsKKyNlbHNlCisgICAg
VU5VU0VEX1BBUkFNKGZyb20pOworICAgIFVOVVNFRF9QQVJBTSh0byk7CisjZW5kaWYKK30KKwor
aW5saW5lIHZvaWQgSGVhcDo6d3JpdGVCYXJyaWVyKGNvbnN0IEpTQ2VsbCogZnJvbSkKK3sKKyNp
ZiBFTkFCTEUoR0dDKQorICAgIEFTU0VSVF9HQ19PQkpFQ1RfTE9PS1NfVkFMSUQoY29uc3RfY2Fz
dDxKU0NlbGwqPihmcm9tKSk7CisgICAgaWYgKCFmcm9tIHx8ICFmcm9tLT5pc01hcmtlZCgpKSB7
CisgICAgICAgIEFTU0VSVCghZnJvbSB8fCAhaXNNYXJrZWQoZnJvbSkpOworICAgICAgICByZXR1
cm47CisgICAgfQorICAgIEFTU0VSVChpc01hcmtlZChmcm9tKSk7CisgICAgYWRkVG9SZW1lbWJl
cmVkU2V0KGZyb20pOworI2Vsc2UKKyAgICBVTlVTRURfUEFSQU0oZnJvbSk7CisjZW5kaWYKIH0K
IAogaW5saW5lIHZvaWQgSGVhcDo6cmVwb3J0RXh0cmFNZW1vcnlDb3N0KHNpemVfdCBjb3N0KQpJ
bmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvSlNDZWxsSW5saW5lcy5oCj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL0pTQ2VsbElubGluZXMuaAko
cmV2aXNpb24gMTY2NzQzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvSlNDZWxs
SW5saW5lcy5oCSh3b3JraW5nIGNvcHkpCkBAIC0yNDIsMjIgKzI0Miw2IEBAIGlubGluZSBUcmlT
dGF0ZSBKU0NlbGw6OnB1cmVUb0Jvb2xlYW4oKSAKICAgICByZXR1cm4gTWl4ZWRUcmlTdGF0ZTsK
IH0KIAotaW5saW5lIHZvaWQgSGVhcDo6d3JpdGVCYXJyaWVyKGNvbnN0IEpTQ2VsbCogZnJvbSwg
SlNDZWxsKiB0bykKLXsKLSNpZiBFTkFCTEUoV1JJVEVfQkFSUklFUl9QUk9GSUxJTkcpCi0gICAg
V3JpdGVCYXJyaWVyQ291bnRlcnM6OmNvdW50V3JpdGVCYXJyaWVyKCk7Ci0jZW5kaWYKLSAgICBp
ZiAoIWZyb20gfHwgIWZyb20tPmlzTWFya2VkKCkpIHsKLSAgICAgICAgQVNTRVJUKCFmcm9tIHx8
ICFpc01hcmtlZChmcm9tKSk7Ci0gICAgICAgIHJldHVybjsKLSAgICB9Ci0gICAgaWYgKCF0byB8
fCB0by0+aXNNYXJrZWQoKSkgewotICAgICAgICBBU1NFUlQoIXRvIHx8IGlzTWFya2VkKHRvKSk7
Ci0gICAgICAgIHJldHVybjsKLSAgICB9Ci0gICAgYWRkVG9SZW1lbWJlcmVkU2V0KGZyb20pOwot
fQotCiB9IC8vIG5hbWVzcGFjZSBKU0MKIAogI2VuZGlmIC8vIEpTQ2VsbElubGluZXNfaAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>