<?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>131932</bug_id>
          
          <creation_ts>2014-04-21 02:30:28 -0700</creation_ts>
          <short_desc>Micro-optimize the way we hand NodeLists to JSC.</short_desc>
          <delta_ts>2014-04-21 11:21:47 -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>Bindings</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="Andreas Kling">kling</reporter>
          <assigned_to name="Andreas Kling">kling</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>ggaren</cc>
    
    <cc>kangil.han</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1002517</commentid>
    <comment_count>0</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2014-04-21 02:30:28 -0700</bug_when>
    <thetext>Micro-optimize the way we hand NodeLists to JSC.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1002518</commentid>
    <comment_count>1</comment_count>
      <attachid>229795</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2014-04-21 02:30:57 -0700</bug_when>
    <thetext>Created attachment 229795
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1002521</commentid>
    <comment_count>2</comment_count>
      <attachid>229795</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-21 03:16:08 -0700</bug_when>
    <thetext>Comment on attachment 229795
Patch

Clearing flags on attachment: 229795

Committed r167589: &lt;http://trac.webkit.org/changeset/167589&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1002522</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-21 03:16:13 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1002576</commentid>
    <comment_count>4</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2014-04-21 11:21:47 -0700</bug_when>
    <thetext>Can we call this &quot;inlineAdd&quot;. &quot;Fast&quot; should never appear in any function names.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>229795</attachid>
            <date>2014-04-21 02:30:57 -0700</date>
            <delta_ts>2014-04-21 03:16:08 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-131932.diff</filename>
            <type>text/plain</type>
            <size>3623</size>
            <attacher name="Andreas Kling">kling</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCA0YjBkYWE5Li5hZTcwOWNjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkg
QEAKKzIwMTQtMDQtMjEgIEFuZHJlYXMgS2xpbmcgIDxha2xpbmdAYXBwbGUuY29tPgorCisgICAg
ICAgIE1pY3JvLW9wdGltaXplIHRoZSB3YXkgd2UgaGFuZCBOb2RlTGlzdHMgdG8gSlNDLgorICAg
ICAgICA8aHR0cHM6Ly93ZWJraXQub3JnL2IvMTMxOTMyPgorCisgICAgICAgIFVzZSBIYXNoTWFw
OjpmYXN0QWRkKCkgd2hlbiByZXR1cm5pbmcgY2FjaGVkIG5vZGUgbGlzdHMgYW5kIGNvbGxlY3Rp
b25zLgorICAgICAgICAxMC45JSBwcm9ncmVzc2lvbiBvbiBCaW5kaW5ncy9nZXQtZWxlbWVudHMt
YnktdGFnLW5hbWUuaHRtbAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgICogZG9tL05vZGVSYXJlRGF0YS5oOgorICAgICAgICAoV2ViQ29yZTo6Tm9kZUxp
c3RzTm9kZURhdGE6OmFkZENhY2hlV2l0aEF0b21pY05hbWUpOgorICAgICAgICAoV2ViQ29yZTo6
Tm9kZUxpc3RzTm9kZURhdGE6OmFkZENhY2hlV2l0aE5hbWUpOgorICAgICAgICAoV2ViQ29yZTo6
Tm9kZUxpc3RzTm9kZURhdGE6OmFkZENhY2hlV2l0aFF1YWxpZmllZE5hbWUpOgorICAgICAgICAo
V2ViQ29yZTo6Tm9kZUxpc3RzTm9kZURhdGE6OmFkZENhY2hlZENvbGxlY3Rpb24pOgorCiAyMDE0
LTA0LTIxICBDb21taXQgUXVldWUgIDxjb21taXQtcXVldWVAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBVbnJldmlld2VkLCByb2xsaW5nIG91dCByMTY3NTg0LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dl
YkNvcmUvZG9tL05vZGVSYXJlRGF0YS5oIGIvU291cmNlL1dlYkNvcmUvZG9tL05vZGVSYXJlRGF0
YS5oCmluZGV4IDk5MjQ1ZDYuLjZkMDIyMzEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2Rv
bS9Ob2RlUmFyZURhdGEuaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9kb20vTm9kZVJhcmVEYXRhLmgK
QEAgLTEyMCw3ICsxMjAsNyBAQCBwdWJsaWM6CiAgICAgdGVtcGxhdGU8dHlwZW5hbWUgVCwgdHlw
ZW5hbWUgQ29udGFpbmVyVHlwZT4KICAgICBQYXNzUmVmPFQ+IGFkZENhY2hlV2l0aEF0b21pY05h
bWUoQ29udGFpbmVyVHlwZSYgY29udGFpbmVyLCBjb25zdCBBdG9taWNTdHJpbmcmIG5hbWUpCiAg
ICAgewotICAgICAgICBOb2RlTGlzdEF0b21pY05hbWVDYWNoZU1hcDo6QWRkUmVzdWx0IHJlc3Vs
dCA9IG1fYXRvbWljTmFtZUNhY2hlcy5hZGQobmFtZWROb2RlTGlzdEtleTxUPihuYW1lKSwgbnVs
bHB0cik7CisgICAgICAgIE5vZGVMaXN0QXRvbWljTmFtZUNhY2hlTWFwOjpBZGRSZXN1bHQgcmVz
dWx0ID0gbV9hdG9taWNOYW1lQ2FjaGVzLmZhc3RBZGQobmFtZWROb2RlTGlzdEtleTxUPihuYW1l
KSwgbnVsbHB0cik7CiAgICAgICAgIGlmICghcmVzdWx0LmlzTmV3RW50cnkpCiAgICAgICAgICAg
ICByZXR1cm4gc3RhdGljX2Nhc3Q8VCY+KCpyZXN1bHQuaXRlcmF0b3ItPnZhbHVlKTsKIApAQCAt
MTMyLDcgKzEzMiw3IEBAIHB1YmxpYzoKICAgICB0ZW1wbGF0ZTx0eXBlbmFtZSBUPgogICAgIFBh
c3NSZWY8VD4gYWRkQ2FjaGVXaXRoTmFtZShDb250YWluZXJOb2RlJiBub2RlLCBjb25zdCBTdHJp
bmcmIG5hbWUpCiAgICAgewotICAgICAgICBOb2RlTGlzdE5hbWVDYWNoZU1hcDo6QWRkUmVzdWx0
IHJlc3VsdCA9IG1fbmFtZUNhY2hlcy5hZGQobmFtZWROb2RlTGlzdEtleTxUPihuYW1lKSwgbnVs
bHB0cik7CisgICAgICAgIE5vZGVMaXN0TmFtZUNhY2hlTWFwOjpBZGRSZXN1bHQgcmVzdWx0ID0g
bV9uYW1lQ2FjaGVzLmZhc3RBZGQobmFtZWROb2RlTGlzdEtleTxUPihuYW1lKSwgbnVsbHB0cik7
CiAgICAgICAgIGlmICghcmVzdWx0LmlzTmV3RW50cnkpCiAgICAgICAgICAgICByZXR1cm4gc3Rh
dGljX2Nhc3Q8VCY+KCpyZXN1bHQuaXRlcmF0b3ItPnZhbHVlKTsKIApAQCAtMTQ0LDcgKzE0NCw3
IEBAIHB1YmxpYzoKICAgICBQYXNzUmVmPFRhZ05vZGVMaXN0PiBhZGRDYWNoZVdpdGhRdWFsaWZp
ZWROYW1lKENvbnRhaW5lck5vZGUmIG5vZGUsIGNvbnN0IEF0b21pY1N0cmluZyYgbmFtZXNwYWNl
VVJJLCBjb25zdCBBdG9taWNTdHJpbmcmIGxvY2FsTmFtZSkKICAgICB7CiAgICAgICAgIFF1YWxp
ZmllZE5hbWUgbmFtZShudWxsQXRvbSwgbG9jYWxOYW1lLCBuYW1lc3BhY2VVUkkpOwotICAgICAg
ICBUYWdOb2RlTGlzdENhY2hlTlM6OkFkZFJlc3VsdCByZXN1bHQgPSBtX3RhZ05vZGVMaXN0Q2Fj
aGVOUy5hZGQobmFtZSwgbnVsbHB0cik7CisgICAgICAgIFRhZ05vZGVMaXN0Q2FjaGVOUzo6QWRk
UmVzdWx0IHJlc3VsdCA9IG1fdGFnTm9kZUxpc3RDYWNoZU5TLmZhc3RBZGQobmFtZSwgbnVsbHB0
cik7CiAgICAgICAgIGlmICghcmVzdWx0LmlzTmV3RW50cnkpCiAgICAgICAgICAgICByZXR1cm4g
KnJlc3VsdC5pdGVyYXRvci0+dmFsdWU7CiAKQEAgLTE1Niw3ICsxNTYsNyBAQCBwdWJsaWM6CiAg
ICAgdGVtcGxhdGU8dHlwZW5hbWUgVCwgdHlwZW5hbWUgQ29udGFpbmVyVHlwZT4KICAgICBQYXNz
UmVmPFQ+IGFkZENhY2hlZENvbGxlY3Rpb24oQ29udGFpbmVyVHlwZSYgY29udGFpbmVyLCBDb2xs
ZWN0aW9uVHlwZSBjb2xsZWN0aW9uVHlwZSwgY29uc3QgQXRvbWljU3RyaW5nJiBuYW1lKQogICAg
IHsKLSAgICAgICAgQ29sbGVjdGlvbkNhY2hlTWFwOjpBZGRSZXN1bHQgcmVzdWx0ID0gbV9jYWNo
ZWRDb2xsZWN0aW9ucy5hZGQobmFtZWRDb2xsZWN0aW9uS2V5KGNvbGxlY3Rpb25UeXBlLCBuYW1l
KSwgbnVsbHB0cik7CisgICAgICAgIENvbGxlY3Rpb25DYWNoZU1hcDo6QWRkUmVzdWx0IHJlc3Vs
dCA9IG1fY2FjaGVkQ29sbGVjdGlvbnMuZmFzdEFkZChuYW1lZENvbGxlY3Rpb25LZXkoY29sbGVj
dGlvblR5cGUsIG5hbWUpLCBudWxscHRyKTsKICAgICAgICAgaWYgKCFyZXN1bHQuaXNOZXdFbnRy
eSkKICAgICAgICAgICAgIHJldHVybiBzdGF0aWNfY2FzdDxUJj4oKnJlc3VsdC5pdGVyYXRvci0+
dmFsdWUpOwogCkBAIC0xNjgsNyArMTY4LDcgQEAgcHVibGljOgogICAgIHRlbXBsYXRlPHR5cGVu
YW1lIFQsIHR5cGVuYW1lIENvbnRhaW5lclR5cGU+CiAgICAgUGFzc1JlZjxUPiBhZGRDYWNoZWRD
b2xsZWN0aW9uKENvbnRhaW5lclR5cGUmIGNvbnRhaW5lciwgQ29sbGVjdGlvblR5cGUgY29sbGVj
dGlvblR5cGUpCiAgICAgewotICAgICAgICBDb2xsZWN0aW9uQ2FjaGVNYXA6OkFkZFJlc3VsdCBy
ZXN1bHQgPSBtX2NhY2hlZENvbGxlY3Rpb25zLmFkZChuYW1lZENvbGxlY3Rpb25LZXkoY29sbGVj
dGlvblR5cGUsIHN0YXJBdG9tKSwgbnVsbHB0cik7CisgICAgICAgIENvbGxlY3Rpb25DYWNoZU1h
cDo6QWRkUmVzdWx0IHJlc3VsdCA9IG1fY2FjaGVkQ29sbGVjdGlvbnMuZmFzdEFkZChuYW1lZENv
bGxlY3Rpb25LZXkoY29sbGVjdGlvblR5cGUsIHN0YXJBdG9tKSwgbnVsbHB0cik7CiAgICAgICAg
IGlmICghcmVzdWx0LmlzTmV3RW50cnkpCiAgICAgICAgICAgICByZXR1cm4gc3RhdGljX2Nhc3Q8
VCY+KCpyZXN1bHQuaXRlcmF0b3ItPnZhbHVlKTsKIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>