<?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>3971</bug_id>
          
          <creation_ts>2005-07-12 16:32:47 -0700</creation_ts>
          <short_desc>JS test suite depends on JS 1.2 behavior</short_desc>
          <delta_ts>2005-07-25 14:01: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>412</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</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="Geoffrey Garen">ggaren</reporter>
          <assigned_to name="Geoffrey Garen">ggaren</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>14478</commentid>
    <comment_count>0</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2005-07-12 16:32:47 -0700</bug_when>
    <thetext>The JS test suite attempts to use version() to travel back to the days of JS 1.2 and test for behaviors that 
are directly contradictory to the ECMA3 spec. Since we only support JS 1.5/ECMA3, and since version() is 
only a dummy function in our code, some tests raise false positives.

Test failures:
        js1_2/Array/tostring_1.js
        js1_2/Array/tostring_2.js</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>14479</commentid>
    <comment_count>1</comment_count>
      <attachid>2935</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2005-07-12 16:34:04 -0700</bug_when>
    <thetext>Created attachment 2935
Patch

This patch makes the tests expect JS 1.5 results.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>14774</commentid>
    <comment_count>2</comment_count>
      <attachid>2935</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2005-07-18 07:54:16 -0700</bug_when>
    <thetext>Comment on attachment 2935
Patch

I&apos;d like the change to the tests to at least document what&apos;s going on rather
than simply commenting out the code.

When we get new versions of the JS test suite, we may need to merge this change
in, and it would be good to make sure it&apos;s &quot;self-documenting&quot;.

I must admit I don&apos;t entirely understand why removing this code completely is
the best fix, but I think a comment explaining why would probably convince me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>14816</commentid>
    <comment_count>3</comment_count>
      <attachid>2991</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2005-07-18 11:38:09 -0700</bug_when>
    <thetext>Created attachment 2991
Patch

Updated patch to include comment.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>2935</attachid>
            <date>2005-07-12 16:34:04 -0700</date>
            <delta_ts>2005-07-18 11:38:09 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>2373</size>
            <attacher name="Geoffrey Garen">ggaren</attacher>
            
              <data encoding="base64">PyB0ZXN0cy9tb3ppbGxhL2pzMV8yLy5EU19TdG9yZQpJbmRleDoga2pzL2FycmF5X29iamVjdC5j
cHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290L0phdmFTY3JpcHRDb3JlL2tqcy9hcnJh
eV9vYmplY3QuY3BwLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjQzCmRpZmYgLXUgLXIxLjQzIGFy
cmF5X29iamVjdC5jcHAKLS0tIGtqcy9hcnJheV9vYmplY3QuY3BwCTMgSnVsIDIwMDUgMTA6NDc6
NTIgLTAwMDAJMS40MworKysga2pzL2FycmF5X29iamVjdC5jcHAJMTIgSnVsIDIwMDUgMjM6MTQ6
MTMgLTAwMDAKQEAgLTQ3NCw3ICs0NzQsNyBAQAogICAgICAgaWYgKGVsZW1lbnQudHlwZSgpICE9
IFVuZGVmaW5lZFR5cGUgJiYgZWxlbWVudC50eXBlKCkgIT0gTnVsbFR5cGUpCiAgICAgICAgIHN0
ciArPSBlbGVtZW50LnRvU3RyaW5nKGV4ZWMpOwogICAgICAgaWYgKCBleGVjLT5oYWRFeGNlcHRp
b24oKSApCi0JYnJlYWs7CisgICAgICAgIGJyZWFrOwogICAgIH0KICAgICByZXN1bHQgPSBTdHJp
bmcoc3RyKTsKICAgICBicmVhazsKSW5kZXg6IHRlc3RzL21vemlsbGEvanMxXzIvQXJyYXkvdG9z
dHJpbmdfMS5qcwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3ZzL3Jvb3QvSmF2YVNjcmlwdENvcmUv
dGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18xLmpzLHYKcmV0cmlldmluZyByZXZp
c2lvbiAxLjEKZGlmZiAtdSAtcjEuMSB0b3N0cmluZ18xLmpzCi0tLSB0ZXN0cy9tb3ppbGxhL2pz
MV8yL0FycmF5L3Rvc3RyaW5nXzEuanMJMTMgT2N0IDIwMDMgMDI6MDU6MzMgLTAwMDAJMS4xCisr
KyB0ZXN0cy9tb3ppbGxhL2pzMV8yL0FycmF5L3Rvc3RyaW5nXzEuanMJMTIgSnVsIDIwMDUgMjM6
MTQ6MTMgLTAwMDAKQEAgLTQzLDEyICs0MywxMiBAQAogCiAgICAgdmFyIFZFUlNJT04gPSAwOwog
Ci0gICAgaWYgKCB0eXBlb2YgdmVyc2lvbiA9PSAiZnVuY3Rpb24iICkgewotICAgICAgICB2ZXJz
aW9uKDEyMCk7Ci0gICAgICAgIFZFUlNJT04gPSAiMTIwIjsKLSAgICB9IGVsc2UgewotICAgICAg
ICBmdW5jdGlvbiB2ZXJzaW9uKCkgeyByZXR1cm4gMDsgfTsKLSAgICB9CisvLyAgICBpZiAoIHR5
cGVvZiB2ZXJzaW9uID09ICJmdW5jdGlvbiIgKSB7CisvLyAgICAgICAgdmVyc2lvbigxMjApOwor
Ly8gICAgICAgIFZFUlNJT04gPSAiMTIwIjsKKy8vICAgIH0gZWxzZSB7CisvLyAgICAgICAgZnVu
Y3Rpb24gdmVyc2lvbigpIHsgcmV0dXJuIDA7IH07CisvLyAgICB9CiAKICAgICB0ZXN0Y2FzZXNb
dGMrK10gPSBuZXcgVGVzdENhc2UgKCBTRUNUSU9OLAogICAgICAgICAgICAgICAgICAgICAgICAg
InZhciBhID0gbmV3IEFycmF5KCk7IGEudG9TdHJpbmcoKSIsCkluZGV4OiB0ZXN0cy9tb3ppbGxh
L2pzMV8yL0FycmF5L3Rvc3RyaW5nXzIuanMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290
L0phdmFTY3JpcHRDb3JlL3Rlc3RzL21vemlsbGEvanMxXzIvQXJyYXkvdG9zdHJpbmdfMi5qcyx2
CnJldHJpZXZpbmcgcmV2aXNpb24gMS4xCmRpZmYgLXUgLXIxLjEgdG9zdHJpbmdfMi5qcwotLS0g
dGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18yLmpzCTEzIE9jdCAyMDAzIDAyOjA1
OjMzIC0wMDAwCTEuMQorKysgdGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18yLmpz
CTEyIEp1bCAyMDA1IDIzOjE0OjEzIC0wMDAwCkBAIC00MSwxMyArNDEsMTMgQEAKICAgICB2YXIg
YSA9IFtdOwogCiAKLSAgICBpZiAoIHR5cGVvZiB2ZXJzaW9uID09ICJmdW5jdGlvbiIgKSB7Ci0g
ICAgICAgIHdyaXRlTGluZVRvTG9nKCJ2ZXJzaW9uIDEyMCIpOwotICAgICAgICB2ZXJzaW9uKDEy
MCk7Ci0gICAgICAgIFZFUlNJT04gPSAiMTIwIjsKLSAgICB9IGVsc2UgewotICAgICAgICBmdW5j
dGlvbiB2ZXJzaW9uKCkgeyByZXR1cm4gMDsgfTsKLSAgICB9CisvLyAgICBpZiAoIHR5cGVvZiB2
ZXJzaW9uID09ICJmdW5jdGlvbiIgKSB7CisvLyAgICAgICAgd3JpdGVMaW5lVG9Mb2coInZlcnNp
b24gMTIwIik7CisvLyAgICAgICAgdmVyc2lvbigxMjApOworLy8gICAgICAgIFZFUlNJT04gPSAi
MTIwIjsKKy8vICAgIH0gZWxzZSB7CisvLyAgICAgICAgZnVuY3Rpb24gdmVyc2lvbigpIHsgcmV0
dXJuIDA7IH07CisvLyAgICB9CiAKICAgICB0ZXN0Y2FzZXNbdGMrK10gPSBuZXcgVGVzdENhc2Ug
KCBTRUNUSU9OLAogICAgICAgICAiYS50b1N0cmluZygpIiwK
</data>
<flag name="review"
          id="227"
          type_id="1"
          status="-"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>2991</attachid>
            <date>2005-07-18 11:38:09 -0700</date>
            <delta_ts>2005-07-22 23:00:43 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>2693</size>
            <attacher name="Geoffrey Garen">ggaren</attacher>
            
              <data encoding="base64">SW5kZXg6IC4vdGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18xLmpzCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KUkNTIGZpbGU6IC9jdnMvcm9vdC9KYXZhU2NyaXB0Q29yZS90ZXN0cy9tb3ppbGxhL2pzMV8y
L0FycmF5L3Rvc3RyaW5nXzEuanMsdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuMQpkaWZmIC1wIC11
IC1yMS4xIC4vdGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18xLmpzCi0tLSAuL3Rl
c3RzL21vemlsbGEvanMxXzIvQXJyYXkvdG9zdHJpbmdfMS5qcwkxMyBPY3QgMjAwMyAwMjowNToz
MyAtMDAwMAkxLjEKKysrIC4vdGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0cmluZ18xLmpz
CTE4IEp1bCAyMDA1IDE4OjE5OjQ5IC0wMDAwCkBAIC00MywxMiArNDMsMjYgQEAKIAogICAgIHZh
ciBWRVJTSU9OID0gMDsKIAorICAgIC8qIFRoaXMgdGVzdCBhc3N1bWVzIHRoYXQgaWYgdmVyc2lv
bigpIGV4aXN0cywgaXQgY2FuIHNldCB0aGUgSmF2YVNjcmlwdCAKKyAgICAgKiBpbnRlcnByZXRl
ciB0byBhbiBhcmJpdHJhcnkgdmVyc2lvbi4gVG8gcHJldmVudCB1bmhhbmRsZWQgZXhjZXB0aW9u
cyBpbgorICAgICAqIG90aGVyIHRlc3RzLCB0ZXN0a2pzIGltcGxlbWVudHMgdmVyc2lvbigpIGFz
IGEgc3R1YiBmdW5jdGlvbiwgYnV0IAorICAgICAqIEphdmFTY3JpcHRDb3JlIGRvZXNuJ3Qgc3Vw
cG9ydCBzZXR0aW5nIHRoZSBKYXZhU2NyaXB0IGVuZ2luZSdzIHZlcnNpb24uCisgICAgIAorICAg
ICAqIENvbW1lbnRpbmcgb3V0IHRoZSBmb2xsb3dpbmcgbGluZXMgZm9yY2VzIHRoZSB0ZXN0IHRv
IGV4cGVjdCBKYXZhU2NyaXB0CisgICAgICogMS41IHJlc3VsdHMuCisgICAgIAorICAgICAqIElm
IEphdmFTY3JpcHRDb3JlIGNoYW5nZXMgdG8gc3VwcG9ydCB2ZXJzaW9uaW5nLCB0aGlzIHRlc3Qg
c2hvdWxkIHNwbGl0CisgICAgICogaW50byBhIDEuMiB0ZXN0IGluIGpzMV8yLyBhbmQgYSAxLjUg
dGVzdCBpbiBqczFfNS8uCisgICAgICovCisgICAgCisgICAgLyoKICAgICBpZiAoIHR5cGVvZiB2
ZXJzaW9uID09ICJmdW5jdGlvbiIgKSB7CiAgICAgICAgIHZlcnNpb24oMTIwKTsKICAgICAgICAg
VkVSU0lPTiA9ICIxMjAiOwogICAgIH0gZWxzZSB7CiAgICAgICAgIGZ1bmN0aW9uIHZlcnNpb24o
KSB7IHJldHVybiAwOyB9OwogICAgIH0KKyAgICAqLwogCiAgICAgdGVzdGNhc2VzW3RjKytdID0g
bmV3IFRlc3RDYXNlICggU0VDVElPTiwKICAgICAgICAgICAgICAgICAgICAgICAgICJ2YXIgYSA9
IG5ldyBBcnJheSgpOyBhLnRvU3RyaW5nKCkiLApJbmRleDogLi90ZXN0cy9tb3ppbGxhL2pzMV8y
L0FycmF5L3Rvc3RyaW5nXzIuanMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290L0phdmFT
Y3JpcHRDb3JlL3Rlc3RzL21vemlsbGEvanMxXzIvQXJyYXkvdG9zdHJpbmdfMi5qcyx2CnJldHJp
ZXZpbmcgcmV2aXNpb24gMS4xCmRpZmYgLXAgLXUgLXIxLjEgLi90ZXN0cy9tb3ppbGxhL2pzMV8y
L0FycmF5L3Rvc3RyaW5nXzIuanMKLS0tIC4vdGVzdHMvbW96aWxsYS9qczFfMi9BcnJheS90b3N0
cmluZ18yLmpzCTEzIE9jdCAyMDAzIDAyOjA1OjMzIC0wMDAwCTEuMQorKysgLi90ZXN0cy9tb3pp
bGxhL2pzMV8yL0FycmF5L3Rvc3RyaW5nXzIuanMJMTggSnVsIDIwMDUgMTg6MTk6NDkgLTAwMDAK
QEAgLTQwLDcgKzQwLDIxIEBACiAKICAgICB2YXIgYSA9IFtdOwogCisgICAgdmFyIFZFUlNJT04g
PSAwOwogCisgICAgLyogVGhpcyB0ZXN0IGFzc3VtZXMgdGhhdCBpZiB2ZXJzaW9uKCkgZXhpc3Rz
LCBpdCBjYW4gc2V0IHRoZSBKYXZhU2NyaXB0IAorICAgICAqIGludGVycHJldGVyIHRvIGFuIGFy
Yml0cmFyeSB2ZXJzaW9uLiBUbyBwcmV2ZW50IHVuaGFuZGxlZCBleGNlcHRpb25zIGluCisgICAg
ICogb3RoZXIgdGVzdHMsIHRlc3RranMgaW1wbGVtZW50cyB2ZXJzaW9uKCkgYXMgYSBzdHViIGZ1
bmN0aW9uLCBidXQgCisgICAgICogSmF2YVNjcmlwdENvcmUgZG9lc24ndCBzdXBwb3J0IHNldHRp
bmcgdGhlIEphdmFTY3JpcHQgZW5naW5lJ3MgdmVyc2lvbi4KKyAgICAgCisgICAgICogQ29tbWVu
dGluZyBvdXQgdGhlIGZvbGxvd2luZyBsaW5lcyBmb3JjZXMgdGhlIHRlc3QgdG8gZXhwZWN0IEph
dmFTY3JpcHQKKyAgICAgKiAxLjUgcmVzdWx0cy4KKyAgICAgCisgICAgICogSWYgSmF2YVNjcmlw
dENvcmUgY2hhbmdlcyB0byBzdXBwb3J0IHZlcnNpb25pbmcsIHRoaXMgdGVzdCBzaG91bGQgc3Bs
aXQKKyAgICAgKiBpbnRvIGEgMS4yIHRlc3QgaW4ganMxXzIvIGFuZCBhIDEuNSB0ZXN0IGluIGpz
MV81Ly4KKyAgICAgKi8KKyAgICAKKyAgICAvKgogICAgIGlmICggdHlwZW9mIHZlcnNpb24gPT0g
ImZ1bmN0aW9uIiApIHsKICAgICAgICAgd3JpdGVMaW5lVG9Mb2coInZlcnNpb24gMTIwIik7CiAg
ICAgICAgIHZlcnNpb24oMTIwKTsKQEAgLTQ4LDYgKzYyLDcgQEAKICAgICB9IGVsc2UgewogICAg
ICAgICBmdW5jdGlvbiB2ZXJzaW9uKCkgeyByZXR1cm4gMDsgfTsKICAgICB9CisgICAgKi8KIAog
ICAgIHRlc3RjYXNlc1t0YysrXSA9IG5ldyBUZXN0Q2FzZSAoIFNFQ1RJT04sCiAgICAgICAgICJh
LnRvU3RyaW5nKCkiLAo=
</data>
<flag name="review"
          id="246"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>