<?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>95131</bug_id>
          
          <creation_ts>2012-08-27 14:59:48 -0700</creation_ts>
          <short_desc>Remove use of ClassInfo from compileGetByValOnArguments and compileGetArgumentsLength</short_desc>
          <delta_ts>2012-08-29 20:23: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>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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Hahnenberg">mhahnenberg</reporter>
          <assigned_to name="Mark Hahnenberg">mhahnenberg</assigned_to>
          <cc>dglazkov</cc>
    
    <cc>fpizlo</cc>
    
    <cc>peter+ews</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>705714</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-08-27 14:59:48 -0700</bug_when>
    <thetext>We don&apos;t need it!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>705718</commentid>
    <comment_count>1</comment_count>
      <attachid>160816</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-08-27 15:01:12 -0700</bug_when>
    <thetext>Created attachment 160816
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>705827</commentid>
    <comment_count>2</comment_count>
      <attachid>160816</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-27 16:05:17 -0700</bug_when>
    <thetext>Comment on attachment 160816
Patch

Clearing flags on attachment: 160816

Committed r126815: &lt;http://trac.webkit.org/changeset/126815&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>705828</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-27 16:05:20 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>706628</commentid>
    <comment_count>4</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-08-28 12:38:10 -0700</bug_when>
    <thetext>Reopening because we have to remove the other use too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>707877</commentid>
    <comment_count>5</comment_count>
      <attachid>161318</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-08-29 14:30:10 -0700</bug_when>
    <thetext>Created attachment 161318
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>707878</commentid>
    <comment_count>6</comment_count>
      <attachid>161318</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2012-08-29 14:30:53 -0700</bug_when>
    <thetext>Comment on attachment 161318
Patch

I&apos;ve seen this patch again!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708047</commentid>
    <comment_count>7</comment_count>
      <attachid>161318</attachid>
    <who name="Peter Beverloo (cr-android ews)">peter+ews</who>
    <bug_when>2012-08-29 16:43:16 -0700</bug_when>
    <thetext>Comment on attachment 161318
Patch

Attachment 161318 did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/13683487</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708128</commentid>
    <comment_count>8</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-08-29 18:04:48 -0700</bug_when>
    <thetext>Looks like Android bot is failing b/c it&apos;s unable to sync. Gonna CQ+ this one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708248</commentid>
    <comment_count>9</comment_count>
      <attachid>161318</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-29 20:23:14 -0700</bug_when>
    <thetext>Comment on attachment 161318
Patch

Clearing flags on attachment: 161318

Committed r127090: &lt;http://trac.webkit.org/changeset/127090&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>708249</commentid>
    <comment_count>10</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-08-29 20:23:17 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>160816</attachid>
            <date>2012-08-27 15:01:12 -0700</date>
            <delta_ts>2012-08-29 14:30:08 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-95131-20120827150104.patch</filename>
            <type>text/plain</type>
            <size>1744</size>
            <attacher name="Mark Hahnenberg">mhahnenberg</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTI2ODAyKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBA
CisyMDEyLTA4LTI3ICBNYXJrIEhhaG5lbmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CisK
KyAgICAgICAgUmVtb3ZlIHVzZSBvZiBDbGFzc0luZm8gZnJvbSBTcGVjdWxhdGl2ZUpJVDo6Y29t
cGlsZUdldEJ5VmFsT25Bcmd1bWVudHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTk1MTMxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgKiBkZmcvREZHU3BlY3VsYXRpdmVKSVQuY3BwOgorICAgICAgICAoSlND
OjpERkc6OlNwZWN1bGF0aXZlSklUOjpjb21waWxlR2V0QnlWYWxPbkFyZ3VtZW50cyk6IFdlIGRv
bid0IG5lZWQgdGhpcyBzcGVjdWxhdGlvbiBjaGVjay4gV2UgY2FuIHJlcGxhY2UgaXQgCisgICAg
ICAgIHdpdGggYW4gYXNzZXJ0IHRvIGd1YXJhbnRlZSB0aGlzLgorCiAyMDEyLTA4LTI3ICBCZW5q
YW1pbiBQb3VsYWluICA8YmVuamFtaW5Ad2Via2l0Lm9yZz4KIAogICAgICAgICBBZGQgRUNNQVNj
cmlwdCBOdW1iZXIgdG8gU3RyaW5nIGNvbnZlcnNpb24gdG8gV1RGOjpTdHJpbmcKSW5kZXg6IFNv
dXJjZS9KYXZhU2NyaXB0Q29yZS9kZmcvREZHU3BlY3VsYXRpdmVKSVQuY3BwCj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9kZmcvREZHU3BlY3VsYXRpdmVKSVQuY3BwCShyZXZp
c2lvbiAxMjY3NzEpCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvZGZnL0RGR1NwZWN1bGF0aXZl
SklULmNwcAkod29ya2luZyBjb3B5KQpAQCAtMzE2MiwxNCArMzE2Miw3IEBAIHZvaWQgU3BlY3Vs
YXRpdmVKSVQ6OmNvbXBpbGVHZXRCeVZhbE9uQXIKICAgICBpZiAoIW1fY29tcGlsZU9rYXkpCiAg
ICAgICAgIHJldHVybjsKICAgICAKLSAgICBpZiAoIWlzQXJndW1lbnRzU3BlY3VsYXRpb24obV9z
dGF0ZS5mb3JOb2RlKG5vZGUuY2hpbGQxKCkpLm1fdHlwZSkpIHsKLSAgICAgICAgc3BlY3VsYXRp
b25DaGVjaygKLSAgICAgICAgICAgIEJhZFR5cGUsIEpTVmFsdWVTb3VyY2U6OnVuYm94ZWRDZWxs
KGJhc2VSZWcpLCBub2RlLmNoaWxkMSgpLAotICAgICAgICAgICAgbV9qaXQuYnJhbmNoUHRyKAot
ICAgICAgICAgICAgICAgIE1hY3JvQXNzZW1ibGVyOjpOb3RFcXVhbCwKLSAgICAgICAgICAgICAg
ICBNYWNyb0Fzc2VtYmxlcjo6QWRkcmVzcyhiYXNlUmVnLCBKU0NlbGw6OmNsYXNzSW5mb09mZnNl
dCgpKSwKLSAgICAgICAgICAgICAgICBNYWNyb0Fzc2VtYmxlcjo6VHJ1c3RlZEltbVB0cigmQXJn
dW1lbnRzOjpzX2luZm8pKSk7Ci0gICAgfQorICAgIEFTU0VSVChtb2RlQWxyZWFkeUNoZWNrZWQo
bV9zdGF0ZS5mb3JOb2RlKG5vZGUuY2hpbGQxKCkpLCBBcnJheTo6QXJndW1lbnRzKSk7CiAgICAg
CiAgICAgbV9qaXQubG9hZFB0cigKICAgICAgICAgTWFjcm9Bc3NlbWJsZXI6OkFkZHJlc3MoYmFz
ZVJlZywgQXJndW1lbnRzOjpvZmZzZXRPZkRhdGEoKSksCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>161318</attachid>
            <date>2012-08-29 14:30:10 -0700</date>
            <delta_ts>2012-08-29 20:23:14 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-95131-20120829142959.patch</filename>
            <type>text/plain</type>
            <size>1746</size>
            <attacher name="Mark Hahnenberg">mhahnenberg</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTI3MDQ5KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBA
CisyMDEyLTA4LTI5ICBNYXJrIEhhaG5lbmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CisK
KyAgICAgICAgUmVtb3ZlIHVzZSBvZiBDbGFzc0luZm8gZnJvbSBjb21waWxlR2V0QnlWYWxPbkFy
Z3VtZW50cyBhbmQgY29tcGlsZUdldEFyZ3VtZW50c0xlbmd0aAorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTUxMzEKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGRmZy9ERkdTcGVjdWxhdGl2ZUpJVC5jcHA6
CisgICAgICAgIChKU0M6OkRGRzo6U3BlY3VsYXRpdmVKSVQ6OmNvbXBpbGVHZXRCeVZhbE9uQXJn
dW1lbnRzKTogV2UgZG9uJ3QgbmVlZCB0aGlzIHNwZWN1bGF0aW9uIGNoZWNrLiBXZSBjYW4gcmVw
bGFjZSBpdCAKKyAgICAgICAgd2l0aCBhbiBhc3NlcnQgdG8gZ3VhcmFudGVlIHRoaXMuCisKIDIw
MTItMDgtMjggIFlvbmcgTGkgIDx5b2xpQHJpbS5jb20+CiAKICAgICAgICAgRXhlY3V0YWJsZUFs
bG9jYXRvciBzaG91bGQgYmUgZGVzdHJ1Y3RlZCBhZnRlciBIZWFwCkluZGV4OiBTb3VyY2UvSmF2
YVNjcmlwdENvcmUvZGZnL0RGR1NwZWN1bGF0aXZlSklULmNwcAo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3Vy
Y2UvSmF2YVNjcmlwdENvcmUvZGZnL0RGR1NwZWN1bGF0aXZlSklULmNwcAkocmV2aXNpb24gMTI3
MDQ4KQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2RmZy9ERkdTcGVjdWxhdGl2ZUpJVC5jcHAJ
KHdvcmtpbmcgY29weSkKQEAgLTMwNjIsMTUgKzMwNjIsOCBAQCB2b2lkIFNwZWN1bGF0aXZlSklU
Ojpjb21waWxlR2V0QnlWYWxPbkFyCiAgICAgCiAgICAgaWYgKCFtX2NvbXBpbGVPa2F5KQogICAg
ICAgICByZXR1cm47Ci0gICAgCi0gICAgaWYgKCFpc0FyZ3VtZW50c1NwZWN1bGF0aW9uKG1fc3Rh
dGUuZm9yTm9kZShub2RlLmNoaWxkMSgpKS5tX3R5cGUpKSB7Ci0gICAgICAgIHNwZWN1bGF0aW9u
Q2hlY2soCi0gICAgICAgICAgICBCYWRUeXBlLCBKU1ZhbHVlU291cmNlOjp1bmJveGVkQ2VsbChi
YXNlUmVnKSwgbm9kZS5jaGlsZDEoKSwKLSAgICAgICAgICAgIG1faml0LmJyYW5jaFB0cigKLSAg
ICAgICAgICAgICAgICBNYWNyb0Fzc2VtYmxlcjo6Tm90RXF1YWwsCi0gICAgICAgICAgICAgICAg
TWFjcm9Bc3NlbWJsZXI6OkFkZHJlc3MoYmFzZVJlZywgSlNDZWxsOjpjbGFzc0luZm9PZmZzZXQo
KSksCi0gICAgICAgICAgICAgICAgTWFjcm9Bc3NlbWJsZXI6OlRydXN0ZWRJbW1QdHIoJkFyZ3Vt
ZW50czo6c19pbmZvKSkpOwotICAgIH0KKyAgCisgICAgQVNTRVJUKG1vZGVBbHJlYWR5Q2hlY2tl
ZChtX3N0YXRlLmZvck5vZGUobm9kZS5jaGlsZDEoKSksIEFycmF5OjpBcmd1bWVudHMpKTsKICAg
ICAKICAgICBtX2ppdC5sb2FkUHRyKAogICAgICAgICBNYWNyb0Fzc2VtYmxlcjo6QWRkcmVzcyhi
YXNlUmVnLCBBcmd1bWVudHM6Om9mZnNldE9mRGF0YSgpKSwK
</data>

          </attachment>
      

    </bug>

</bugzilla>