<?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>20849</bug_id>
          
          <creation_ts>2008-09-15 00:18:54 -0700</creation_ts>
          <short_desc>Cache property names  for getEnumerablePropertyNames in the StructureID</short_desc>
          <delta_ts>2008-09-15 00:28:04 -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>Mac</rep_platform>
          <op_sys>OS X 10.5</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="Sam Weinig">sam</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>91179</commentid>
    <comment_count>0</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2008-09-15 00:18:54 -0700</bug_when>
    <thetext>Patch forthcoming.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91180</commentid>
    <comment_count>1</comment_count>
      <attachid>23430</attachid>
    <who name="Sam Weinig">sam</who>
    <bug_when>2008-09-15 00:22:49 -0700</bug_when>
    <thetext>Created attachment 23430
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91181</commentid>
    <comment_count>2</comment_count>
      <attachid>23430</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2008-09-15 00:25:06 -0700</bug_when>
    <thetext>Comment on attachment 23430
patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91182</commentid>
    <comment_count>3</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2008-09-15 00:28:04 -0700</bug_when>
    <thetext>Fixed in r36429.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>23430</attachid>
            <date>2008-09-15 00:22:49 -0700</date>
            <delta_ts>2008-09-15 00:25:06 -0700</delta_ts>
            <desc>patch</desc>
            <filename>cache.diff</filename>
            <type>text/plain</type>
            <size>6442</size>
            <attacher name="Sam Weinig">sam</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDM2NDI4
KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjMgQEAKKzIwMDgtMDkt
MTUgIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBQYXRjaCBmb3IgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTIwODQ5CisgICAgICAgIENhY2hlIHByb3BlcnR5IG5hbWVzIGZv
ciBnZXRFbnVtZXJhYmxlUHJvcGVydHlOYW1lcyBpbiB0aGUgU3RydWN0dXJlSUQuCisKKyAgICAg
ICAgfjAuNSUgc3BlZWR1cCBvbiBTdW5zcGlkZXIgb3ZlcmFsbCAoOS43JSBzcGVlZHVwIG9uIHN0
cmluZy1mYXN0YSkuICB+MSUgc3BlZWR1cAorICAgICAgICBvbiB0aGUgdjggdGVzdCBzdWl0ZS4K
KworICAgICAgICAqIGtqcy9KU09iamVjdC5jcHA6CisgICAgICAgIChKU0M6OkpTT2JqZWN0Ojpn
ZXRQcm9wZXJ0eU5hbWVzKToKKyAgICAgICAgKiBranMvUHJvcGVydHlNYXAuY3BwOgorICAgICAg
ICAoSlNDOjpQcm9wZXJ0eU1hcDo6Z2V0RW51bWVyYWJsZVByb3BlcnR5TmFtZXMpOgorICAgICAg
ICAqIGtqcy9Qcm9wZXJ0eU1hcC5oOgorICAgICAgICAqIGtqcy9TdHJ1Y3R1cmVJRC5jcHA6Cisg
ICAgICAgIChKU0M6OlN0cnVjdHVyZUlEOjpTdHJ1Y3R1cmVJRCk6CisgICAgICAgIChKU0M6OlN0
cnVjdHVyZUlEOjpnZXRFbnVtZXJhYmxlUHJvcGVydHlOYW1lcyk6CisgICAgICAgICoga2pzL1N0
cnVjdHVyZUlELmg6CisKIDIwMDgtMDktMTQgIE1hY2llaiBTdGFjaG93aWFrICA8bWpzQGFwcGxl
LmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBDYW1lcm9uIFp3YXJpY2guCkluZGV4OiBranMv
SlNPYmplY3QuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGtqcy9KU09iamVjdC5jcHAJKHJldmlzaW9uIDM2
NDE2KQorKysga2pzL0pTT2JqZWN0LmNwcAkod29ya2luZyBjb3B5KQpAQCAtNDMzLDcgKzQzMyw3
IEBAIGJvb2wgSlNPYmplY3Q6OmdldFByb3BlcnR5QXR0cmlidXRlcyhFeGUKIAogdm9pZCBKU09i
amVjdDo6Z2V0UHJvcGVydHlOYW1lcyhFeGVjU3RhdGUqIGV4ZWMsIFByb3BlcnR5TmFtZUFycmF5
JiBwcm9wZXJ0eU5hbWVzKQogewotICAgIG1fc3RydWN0dXJlSUQtPnByb3BlcnR5TWFwKCkuZ2V0
RW51bWVyYWJsZVByb3BlcnR5TmFtZXMocHJvcGVydHlOYW1lcyk7CisgICAgbV9zdHJ1Y3R1cmVJ
RC0+Z2V0RW51bWVyYWJsZVByb3BlcnR5TmFtZXMocHJvcGVydHlOYW1lcyk7CiAKICAgICAvLyBB
ZGQgcHJvcGVydGllcyBmcm9tIHRoZSBzdGF0aWMgaGFzaHRhYmxlcyBvZiBwcm9wZXJ0aWVzCiAg
ICAgZm9yIChjb25zdCBDbGFzc0luZm8qIGluZm8gPSBjbGFzc0luZm8oKTsgaW5mbzsgaW5mbyA9
IGluZm8tPnBhcmVudENsYXNzKSB7CkluZGV4OiBranMvUHJvcGVydHlNYXAuY3BwCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIGtqcy9Qcm9wZXJ0eU1hcC5jcHAJKHJldmlzaW9uIDM2NDE2KQorKysga2pzL1Byb3Bl
cnR5TWFwLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjMsNyArMjMsNiBAQAogCiAjaW5jbHVkZSAi
SlNPYmplY3QuaCIKICNpbmNsdWRlICJwcm90ZWN0LmgiCi0jaW5jbHVkZSAiUHJvcGVydHlOYW1l
QXJyYXkuaCIKICNpbmNsdWRlIDxhbGdvcml0aG0+CiAjaW5jbHVkZSA8d3RmL0Fzc2VydGlvbnMu
aD4KICNpbmNsdWRlIDx3dGYvRmFzdE1hbGxvYy5oPgpAQCAtNDczLDcgKzQ3Miw3IEBAIHN0YXRp
YyBpbnQgY29tcGFyZVByb3BlcnR5TWFwRW50cnlJbmRpY2UKICAgICByZXR1cm4gMDsKIH0KIAot
dm9pZCBQcm9wZXJ0eU1hcDo6Z2V0RW51bWVyYWJsZVByb3BlcnR5TmFtZXMoUHJvcGVydHlOYW1l
QXJyYXkmIHByb3BlcnR5TmFtZXMpIGNvbnN0Cit2b2lkIFByb3BlcnR5TWFwOjpnZXRFbnVtZXJh
YmxlUHJvcGVydHlOYW1lcyhWZWN0b3I8VVN0cmluZzo6UmVwKj4mIHByb3BlcnR5TmFtZXMpIGNv
bnN0CiB7CiAgICAgaWYgKCFtX3RhYmxlKQogICAgICAgICByZXR1cm47CkBAIC00OTIsMTMgKzQ5
MSw5IEBAIHZvaWQgUHJvcGVydHlNYXA6OmdldEVudW1lcmFibGVQcm9wZXJ0eU4KICAgICAgICAg
ICAgICAgICArK2k7CiAgICAgICAgICAgICB9CiAgICAgICAgIH0KLSAgICAgICAgaWYgKCFwcm9w
ZXJ0eU5hbWVzLnNpemUoKSkgewotICAgICAgICAgICAgZm9yIChpbnQgayA9IDA7IGsgPCBpOyAr
K2spCi0gICAgICAgICAgICAgICAgcHJvcGVydHlOYW1lcy5hZGRLbm93blVuaXF1ZShhW2tdLT5r
ZXkpOwotICAgICAgICB9IGVsc2UgewotICAgICAgICAgICAgZm9yIChpbnQgayA9IDA7IGsgPCBp
OyArK2spCi0gICAgICAgICAgICAgICAgcHJvcGVydHlOYW1lcy5hZGQoYVtrXS0+a2V5KTsKLSAg
ICAgICAgfQorICAgICAgICBwcm9wZXJ0eU5hbWVzLnJlc2VydmVDYXBhY2l0eShpKTsKKyAgICAg
ICAgZm9yIChpbnQgayA9IDA7IGsgPCBpOyArK2spCisgICAgICAgICAgICBwcm9wZXJ0eU5hbWVz
LmFwcGVuZChhW2tdLT5rZXkpOwogICAgICAgICByZXR1cm47CiAgICAgfQogCkBAIC01MTcsOCAr
NTEyLDkgQEAgdm9pZCBQcm9wZXJ0eU1hcDo6Z2V0RW51bWVyYWJsZVByb3BlcnR5TgogICAgIHFz
b3J0KHNvcnRlZEVudW1lcmFibGVzLmRhdGEoKSwgcCAtIHNvcnRlZEVudW1lcmFibGVzLmRhdGEo
KSwgc2l6ZW9mKEVudHJ5KiksIGNvbXBhcmVQcm9wZXJ0eU1hcEVudHJ5SW5kaWNlcyk7CiAKICAg
ICAvLyBQdXQgdGhlIGtleXMgb2YgdGhlIHNvcnRlZCBlbnRyaWVzIGludG8gdGhlIGxpc3QuCi0g
ICAgZm9yIChFbnRyeSoqIHEgPSBzb3J0ZWRFbnVtZXJhYmxlcy5kYXRhKCk7IHEgIT0gcDsgKytx
KQotICAgICAgICBwcm9wZXJ0eU5hbWVzLmFkZChxWzBdLT5rZXkpOworICAgIHByb3BlcnR5TmFt
ZXMucmVzZXJ2ZUNhcGFjaXR5KHNvcnRlZEVudW1lcmFibGVzLnNpemUoKSk7CisgICAgZm9yIChz
aXplX3QgaSA9IDA7IGkgPCBzb3J0ZWRFbnVtZXJhYmxlcy5zaXplKCk7ICsraSkKKyAgICAgICAg
cHJvcGVydHlOYW1lcy5hcHBlbmQoc29ydGVkRW51bWVyYWJsZXNbaV0tPmtleSk7CiB9CiAKICNp
ZiBET19QUk9QRVJUWU1BUF9DT05TVEVOQ1lfQ0hFQ0sKSW5kZXg6IGtqcy9Qcm9wZXJ0eU1hcC5o
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIGtqcy9Qcm9wZXJ0eU1hcC5oCShyZXZpc2lvbiAzNjQxNikKKysrIGtq
cy9Qcm9wZXJ0eU1hcC5oCSh3b3JraW5nIGNvcHkpCkBAIC0zMCw3ICszMCw2IEBAIG5hbWVzcGFj
ZSBKU0MgewogCiAgICAgY2xhc3MgSlNPYmplY3Q7CiAgICAgY2xhc3MgSlNWYWx1ZTsKLSAgICBj
bGFzcyBQcm9wZXJ0eU5hbWVBcnJheTsKIAogICAgIHR5cGVkZWYgSlNWYWx1ZSoqIFByb3BlcnR5
U3RvcmFnZTsKIApAQCAtOTQsNyArOTMsNyBAQCBuYW1lc3BhY2UgSlNDIHsKICAgICAgICAgc2l6
ZV90IGdldE9mZnNldChjb25zdCBJZGVudGlmaWVyJiBwcm9wZXJ0eU5hbWUpOwogICAgICAgICBz
aXplX3QgZ2V0T2Zmc2V0KGNvbnN0IElkZW50aWZpZXImIHByb3BlcnR5TmFtZSwgdW5zaWduZWQm
IGF0dHJpYnV0ZXMpOwogCi0gICAgICAgIHZvaWQgZ2V0RW51bWVyYWJsZVByb3BlcnR5TmFtZXMo
UHJvcGVydHlOYW1lQXJyYXkmKSBjb25zdDsKKyAgICAgICAgdm9pZCBnZXRFbnVtZXJhYmxlUHJv
cGVydHlOYW1lcyhWZWN0b3I8VVN0cmluZzo6UmVwKj4mKSBjb25zdDsKIAogICAgICAgICBib29s
IGhhc0dldHRlclNldHRlclByb3BlcnRpZXMoKSBjb25zdCB7IHJldHVybiBtX2dldHRlclNldHRl
ckZsYWc7IH0KICAgICAgICAgdm9pZCBzZXRIYXNHZXR0ZXJTZXR0ZXJQcm9wZXJ0aWVzKGJvb2wg
ZikgeyBtX2dldHRlclNldHRlckZsYWcgPSBmOyB9CkluZGV4OiBranMvU3RydWN0dXJlSUQuY3Bw
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIGtqcy9TdHJ1Y3R1cmVJRC5jcHAJKHJldmlzaW9uIDM2NDE2KQorKysg
a2pzL1N0cnVjdHVyZUlELmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjgsMTMgKzI4LDE0IEBACiAK
ICNpbmNsdWRlICJpZGVudGlmaWVyLmgiCiAjaW5jbHVkZSAiSlNPYmplY3QuaCIKKyNpbmNsdWRl
ICJQcm9wZXJ0eU5hbWVBcnJheS5oIjsKICNpbmNsdWRlIDx3dGYvUmVmUHRyLmg+CiAKIHVzaW5n
IG5hbWVzcGFjZSBzdGQ7CiAKIG5hbWVzcGFjZSBKU0MgewogCi0gICAgU3RydWN0dXJlSUQ6OlN0
cnVjdHVyZUlEKEpTVmFsdWUqIHByb3RvdHlwZSwgSlNUeXBlIHR5cGUpCitTdHJ1Y3R1cmVJRDo6
U3RydWN0dXJlSUQoSlNWYWx1ZSogcHJvdG90eXBlLCBKU1R5cGUgdHlwZSkKICAgICA6IG1faXNE
aWN0aW9uYXJ5KGZhbHNlKQogICAgICwgbV90eXBlKHR5cGUpCiAgICAgLCBtX3Byb3RvdHlwZShw
cm90b3R5cGUpCkBAIC00Nyw2ICs0OCwyMCBAQCBuYW1lc3BhY2UgSlNDIHsKICAgICBBU1NFUlQo
bV9wcm90b3R5cGUtPmlzT2JqZWN0KCkgfHwgbV9wcm90b3R5cGUtPmlzTnVsbCgpKTsKIH0KIAor
dm9pZCBTdHJ1Y3R1cmVJRDo6Z2V0RW51bWVyYWJsZVByb3BlcnR5TmFtZXMoUHJvcGVydHlOYW1l
QXJyYXkmIHByb3BlcnR5TmFtZXMpIGNvbnN0Cit7CisgICAgaWYgKG1fY2FjaGVkUHJvcGVydHlO
YW1lQXJyYXkuaXNFbXB0eSgpKQorICAgICAgICBtX3Byb3BlcnR5TWFwLmdldEVudW1lcmFibGVQ
cm9wZXJ0eU5hbWVzKG1fY2FjaGVkUHJvcGVydHlOYW1lQXJyYXkpOworCisgICAgaWYgKCFwcm9w
ZXJ0eU5hbWVzLnNpemUoKSkgeworICAgICAgICBmb3IgKHNpemVfdCBpID0gMDsgaSA8IG1fY2Fj
aGVkUHJvcGVydHlOYW1lQXJyYXkuc2l6ZSgpOyArK2kpCisgICAgICAgICAgICBwcm9wZXJ0eU5h
bWVzLmFkZEtub3duVW5pcXVlKG1fY2FjaGVkUHJvcGVydHlOYW1lQXJyYXlbaV0pOworICAgIH0g
ZWxzZSB7CisgICAgICAgIGZvciAoc2l6ZV90IGkgPSAwOyBpIDwgbV9jYWNoZWRQcm9wZXJ0eU5h
bWVBcnJheS5zaXplKCk7ICsraSkKKyAgICAgICAgICAgIHByb3BlcnR5TmFtZXMuYWRkKG1fY2Fj
aGVkUHJvcGVydHlOYW1lQXJyYXlbaV0pOworICAgIH0KK30KKwogdm9pZCBTdHJ1Y3R1cmVJRDo6
dHJhbnNpdGlvblRvKFN0cnVjdHVyZUlEKiBvbGRTdHJ1Y3R1cmVJRCwgU3RydWN0dXJlSUQqIG5l
d1N0cnVjdHVyZUlELCBKU09iamVjdCogc2xvdEJhc2UpCiB7CiAgICAgaWYgKCFzbG90QmFzZS0+
dXNpbmdJbmxpbmVTdG9yYWdlKCkgJiYgb2xkU3RydWN0dXJlSUQtPm1fcHJvcGVydHlNYXAuc2l6
ZSgpICE9IG5ld1N0cnVjdHVyZUlELT5tX3Byb3BlcnR5TWFwLnNpemUoKSkKSW5kZXg6IGtqcy9T
dHJ1Y3R1cmVJRC5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGtqcy9TdHJ1Y3R1cmVJRC5oCShyZXZpc2lvbiAz
NjQxNikKKysrIGtqcy9TdHJ1Y3R1cmVJRC5oCSh3b3JraW5nIGNvcHkpCkBAIC0zOSw2ICszOSw3
IEBACiBuYW1lc3BhY2UgSlNDIHsKIAogICAgIGNsYXNzIEpTVmFsdWU7CisgICAgY2xhc3MgUHJv
cGVydHlOYW1lQXJyYXk7CiAgICAgY2xhc3MgU3RydWN0dXJlSURDaGFpbjsKIAogICAgIHN0cnVj
dCBUcmFuc2l0aW9uVGFibGVIYXNoIHsKQEAgLTEwNyw2ICsxMDgsOCBAQCBuYW1lc3BhY2UgSlND
IHsKICAgICAgICAgY29uc3QgUHJvcGVydHlNYXAmIHByb3BlcnR5TWFwKCkgY29uc3QgeyByZXR1
cm4gbV9wcm9wZXJ0eU1hcDsgfQogICAgICAgICBQcm9wZXJ0eU1hcCYgcHJvcGVydHlNYXAoKSB7
IHJldHVybiBtX3Byb3BlcnR5TWFwOyB9CiAKKyAgICAgICAgdm9pZCBnZXRFbnVtZXJhYmxlUHJv
cGVydHlOYW1lcyhQcm9wZXJ0eU5hbWVBcnJheSYpIGNvbnN0OworCiAgICAgICAgIHN0YXRpYyB2
b2lkIHRyYW5zaXRpb25UbyhTdHJ1Y3R1cmVJRCogb2xkU3RydWN0dXJlSUQsIFN0cnVjdHVyZUlE
KiBuZXdTdHJ1Y3R1cmVJRCwgSlNPYmplY3QqIHNsb3RCYXNlKTsKIAogICAgIHByaXZhdGU6CkBA
IC0xMzAsNiArMTMzLDggQEAgbmFtZXNwYWNlIEpTQyB7CiAgICAgICAgIHNpemVfdCBtX3RyYW5z
aXRpb25Db3VudDsKICAgICAgICAgVHJhbnNpdGlvblRhYmxlIG1fdHJhbnNpdGlvblRhYmxlOwog
CisgICAgICAgIG11dGFibGUgVmVjdG9yPFVTdHJpbmc6OlJlcCo+IG1fY2FjaGVkUHJvcGVydHlO
YW1lQXJyYXk7CisKICAgICAgICAgUHJvcGVydHlNYXAgbV9wcm9wZXJ0eU1hcDsKICAgICB9Owog
Cg==
</data>
<flag name="review"
          id="10457"
          type_id="1"
          status="+"
          setter="mjs"
    />
          </attachment>
      

    </bug>

</bugzilla>