<?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>55220</bug_id>
          
          <creation_ts>2011-02-25 08:21:18 -0800</creation_ts>
          <short_desc>Web Inspector: &quot;length&quot; getter is invoked upon console object formatting.</short_desc>
          <delta_ts>2011-03-09 01:54:28 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>0</everconfirmed>
          <reporter name="Pavel Feldman">pfeldman</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>357928</commentid>
    <comment_count>0</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2011-02-25 08:21:18 -0800</bug_when>
    <thetext>Enter &apos;demo = {get length () {return this.a.length;}}&apos; and submit.
Read error &quot;TypeError: Cannot read property &apos;length&apos; of undefined&quot;.

Reason is that array detection in the formatting code checks the length value. We should probably switch to testing whether length is enumerated instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>357950</commentid>
    <comment_count>1</comment_count>
      <attachid>83819</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2011-02-25 08:57:42 -0800</bug_when>
    <thetext>Created attachment 83819
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>364374</commentid>
    <comment_count>2</comment_count>
      <attachid>83819</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2011-03-09 01:54:20 -0800</bug_when>
    <thetext>Comment on attachment 83819
Patch

Clearing flags on attachment: 83819

Committed r80628: &lt;http://trac.webkit.org/changeset/80628&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>364375</commentid>
    <comment_count>3</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2011-03-09 01:54:28 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>83819</attachid>
            <date>2011-02-25 08:57:42 -0800</date>
            <delta_ts>2011-03-09 01:54:20 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-55220-20110225195736.patch</filename>
            <type>text/plain</type>
            <size>5505</size>
            <attacher name="Pavel Feldman">pfeldman</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogNzk1ODMKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5n
ZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwppbmRleCA2NDY1Y2FjMjZhYjZiM2M0YjVmYWZk
YzM4NzljNTk3ZDA1ZGI5MWI3Li5mNjk5YmQ1OTE0MzcxZTc0MmYyMmQwMDhhNjE2YTJmNWI0ODJk
MzQ3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMv
Q2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMDItMjUgIFBhdmVsIEZlbGRtYW4gIDxw
ZmVsZG1hbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogImxlbmd0aCIgZ2V0dGVyIGlzIGludm9rZWQg
dXBvbiBjb25zb2xlIG9iamVjdCBmb3JtYXR0aW5nLgorICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTUyMjAKKworICAgICAgICAqIGluc3BlY3Rvci9jb25z
b2xlL2NvbnNvbGUtZm9ybWF0LWV4cGVjdGVkLnR4dDoKKyAgICAgICAgKiBpbnNwZWN0b3IvY29u
c29sZS9jb25zb2xlLWZvcm1hdC5odG1sOgorICAgICAgICAqIHBsYXRmb3JtL2Nocm9taXVtL2lu
c3BlY3Rvci9jb25zb2xlL2NvbnNvbGUtZm9ybWF0LWV4cGVjdGVkLnR4dDoKKwogMjAxMS0wMi0y
NCAgUGF2ZWwgRmVsZG1hbiAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZp
ZXdlZCBieSBZdXJ5IFNlbWlraGF0c2t5LgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvaW5zcGVj
dG9yL2NvbnNvbGUvY29uc29sZS1mb3JtYXQtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvaW5z
cGVjdG9yL2NvbnNvbGUvY29uc29sZS1mb3JtYXQtZXhwZWN0ZWQudHh0CmluZGV4IDk5NGZiMDIy
ZDAyZDQ1MzhkNmZiOTE3OGZmMmEwMjU3NWQyZmE2Y2UuLmMyMTA0ODYxOGRiZTliZDljNDE3Nzlj
YTNlYWNiZGZmZWVhM2ZmZmQgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL2luc3BlY3Rvci9jb25z
b2xlL2NvbnNvbGUtZm9ybWF0LWV4cGVjdGVkLnR4dAorKysgYi9MYXlvdXRUZXN0cy9pbnNwZWN0
b3IvY29uc29sZS9jb25zb2xlLWZvcm1hdC1leHBlY3RlZC50eHQKQEAgLTM4LDYgKzM4LDggQEAg
Q09OU09MRSBNRVNTQUdFOiBsaW5lIDUzOiBbb2JqZWN0IEF0dHJdCiBDT05TT0xFIE1FU1NBR0U6
IGxpbmUgNTQ6IFtvYmplY3QgQXR0cl0KIENPTlNPTEUgTUVTU0FHRTogbGluZSA1MzogW29iamVj
dCBBdHRyXQogQ09OU09MRSBNRVNTQUdFOiBsaW5lIDU0OiBbb2JqZWN0IEF0dHJdCitDT05TT0xF
IE1FU1NBR0U6IGxpbmUgNTM6IFtvYmplY3QgT2JqZWN0XQorQ09OU09MRSBNRVNTQUdFOiBsaW5l
IDU0OiBbb2JqZWN0IE9iamVjdF0KIFRlc3RzIHRoYXQgY29uc29sZSBsb2dnaW5nIGR1bXBzIHBy
b3BlciBtZXNzYWdlcy4KIAogY29uc29sZS1mb3JtYXQuaHRtbDoxMk1lc3NhZ2UgZm9ybWF0IG51
bWJlciAxLCAyIGFuZCAzLjUKQEAgLTExMiw0ICsxMTQsOCBAQCBjb25zb2xlLWZvcm1hdC5odG1s
OjUzaWQ9IngiCiBjb25zb2xlLWZvcm1hdC5odG1sOjU0W2lkPSJ4Il0KIGdsb2JhbHNbMTRdCiBp
ZD0ieCIKK2NvbnNvbGUtZm9ybWF0Lmh0bWw6NTNPYmplY3QKK2NvbnNvbGUtZm9ybWF0Lmh0bWw6
NTRbT2JqZWN0XQorZ2xvYmFsc1sxNV0KK09iamVjdAogCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0
cy9pbnNwZWN0b3IvY29uc29sZS9jb25zb2xlLWZvcm1hdC5odG1sIGIvTGF5b3V0VGVzdHMvaW5z
cGVjdG9yL2NvbnNvbGUvY29uc29sZS1mb3JtYXQuaHRtbAppbmRleCBmMzlmNjgxYTQ0OGRlMzdh
NjIyZjUyMTdlYzA1NjI1ZTdjZGE3YTI0Li40ODQzMGEzZWFiNmJmYTYzZTBiMTk2NzY1NzI1OTAx
YTZjNTIyNjFjIDEwMDc1NQotLS0gYS9MYXlvdXRUZXN0cy9pbnNwZWN0b3IvY29uc29sZS9jb25z
b2xlLWZvcm1hdC5odG1sCisrKyBiL0xheW91dFRlc3RzL2luc3BlY3Rvci9jb25zb2xlL2NvbnNv
bGUtZm9ybWF0Lmh0bWwKQEAgLTM0LDE1ICszNCwxNSBAQCBmdW5jdGlvbiBvbmxvYWQoKQogICAg
IH07CiAgICAgdmFyIG51bSA9IDEuMmUtMTsKICAgICB2YXIgbGlua2lmeSA9ICJodHRwOi8vd2Vi
a2l0Lm9yZy8iOwotCiAgICAgdmFyIHZhbHVlbGVzc0F0dHJpYnV0ZSA9IGRvY3VtZW50LmNyZWF0
ZUF0dHJpYnV0ZSgiYXR0ciIpOwogICAgIHZhciB2YWx1ZWRBdHRyaWJ1dGUgPSBkb2N1bWVudC5j
cmVhdGVBdHRyaWJ1dGUoImF0dHIiKTsKICAgICB2YWx1ZWRBdHRyaWJ1dGUudmFsdWUgPSAidmFs
dWUiOwogICAgIHZhciBleGlzdGluZ0F0dHJpYnV0ZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlk
KCJ4IikuYXR0cmlidXRlc1swXTsKKyAgICB2YXIgdGhyb3dpbmdMZW5ndGhHZXR0ZXIgPSB7Z2V0
IGxlbmd0aCgpIHsgdGhyb3cgIkxlbmd0aCBjYWxsZWQiOyB9fTsKIAogICAgIGdsb2JhbHMgPSBb
CiAgICAgICAgIHJlZ2V4MSwgcmVnZXgyLCBzdHIsIHN0cjIsIGVycm9yLCBub2RlLCBmdW5jLCBt
dWx0aWxpbmVmdW5jLCBudW0sIGxpbmtpZnksCi0gICAgICAgIG51bGwsIHVuZGVmaW5lZCwgdmFs
dWVsZXNzQXR0cmlidXRlLCB2YWx1ZWRBdHRyaWJ1dGUsIGV4aXN0aW5nQXR0cmlidXRlCisgICAg
ICAgIG51bGwsIHVuZGVmaW5lZCwgdmFsdWVsZXNzQXR0cmlidXRlLCB2YWx1ZWRBdHRyaWJ1dGUs
IGV4aXN0aW5nQXR0cmlidXRlLCB0aHJvd2luZ0xlbmd0aEdldHRlcgogICAgIF07CiAKICAgICBy
dW5UZXN0KCk7CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS9pbnNw
ZWN0b3IvY29uc29sZS9jb25zb2xlLWZvcm1hdC1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9w
bGF0Zm9ybS9jaHJvbWl1bS9pbnNwZWN0b3IvY29uc29sZS9jb25zb2xlLWZvcm1hdC1leHBlY3Rl
ZC50eHQKaW5kZXggYjdhMTk3ZmFkYTExYmRjMTg5MTQzMDNiMzBlMGRmYjcxYmE5MjA0OC4uZWY4
NjY4MWU1MTBjMDZiN2Y0ZWQ3NmJlM2FiYjU0ZjIwYWNhMDYyYyAxMDA2NDQKLS0tIGEvTGF5b3V0
VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vaW5zcGVjdG9yL2NvbnNvbGUvY29uc29sZS1mb3JtYXQt
ZXhwZWN0ZWQudHh0CisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL2luc3BlY3Rv
ci9jb25zb2xlL2NvbnNvbGUtZm9ybWF0LWV4cGVjdGVkLnR4dApAQCAtMzgsNiArMzgsOCBAQCBD
T05TT0xFIE1FU1NBR0U6IGxpbmUgNTM6IFtvYmplY3QgQXR0cl0KIENPTlNPTEUgTUVTU0FHRTog
bGluZSA1NDogW29iamVjdCBBdHRyXQogQ09OU09MRSBNRVNTQUdFOiBsaW5lIDUzOiBbb2JqZWN0
IEF0dHJdCiBDT05TT0xFIE1FU1NBR0U6IGxpbmUgNTQ6IFtvYmplY3QgQXR0cl0KK0NPTlNPTEUg
TUVTU0FHRTogbGluZSA1MzogW29iamVjdCBPYmplY3RdCitDT05TT0xFIE1FU1NBR0U6IGxpbmUg
NTQ6IFtvYmplY3QgT2JqZWN0XQogVGVzdHMgdGhhdCBjb25zb2xlIGxvZ2dpbmcgZHVtcHMgcHJv
cGVyIG1lc3NhZ2VzLgogCiBjb25zb2xlLWZvcm1hdC5odG1sOjEyTWVzc2FnZSBmb3JtYXQgbnVt
YmVyIDEsIDIgYW5kIDMuNQpAQCAtMTEyLDQgKzExNCw4IEBAIGNvbnNvbGUtZm9ybWF0Lmh0bWw6
NTNpZD0ieCIKIGNvbnNvbGUtZm9ybWF0Lmh0bWw6NTRbaWQ9IngiXQogZ2xvYmFsc1sxNF0KIGlk
PSJ4IgorY29uc29sZS1mb3JtYXQuaHRtbDo1M09iamVjdAorY29uc29sZS1mb3JtYXQuaHRtbDo1
NFtPYmplY3RdCitnbG9iYWxzWzE1XQorT2JqZWN0CiAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJD
b3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCAxZDY2MjYxZjAz
ZDRhZTQ4MWMxNTkzZDljYjE0NDliYTY0MjNkNDg1Li40OTE4ZWIzOTQ4NmVlNmI0NGYxYmIyNzAz
M2VjMzQ2YWYyZTE4N2Q2IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysr
IGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTEtMDItMjUg
IFBhdmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogImxlbmd0aCIg
Z2V0dGVyIGlzIGludm9rZWQgdXBvbiBjb25zb2xlIG9iamVjdCBmb3JtYXR0aW5nLgorICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTUyMjAKKworICAgICAg
ICAqIGluc3BlY3Rvci9JbmplY3RlZFNjcmlwdFNvdXJjZS5qczoKKwogMjAxMS0wMi0yNCAgUGF2
ZWwgRmVsZG1hbiAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBi
eSBZdXJ5IFNlbWlraGF0c2t5LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9y
L0luamVjdGVkU2NyaXB0U291cmNlLmpzIGIvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL0luamVj
dGVkU2NyaXB0U291cmNlLmpzCmluZGV4IGZmNmUwNzY1MGViZTMzZTVjNzVmNjBmZDZiZTQ2YjNk
NjVhY2NjMDcuLjNhM2ZjNGRiNmYwN2FmZWE4YWRlYTY3NDMzYTk5OTRlM2U4ZGIxOTIgMTAwNjQ0
Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9JbmplY3RlZFNjcmlwdFNvdXJjZS5qcwor
KysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5qZWN0ZWRTY3JpcHRTb3VyY2UuanMKQEAg
LTQ1OCwxMCArNDU4LDE0IEBAIEluamVjdGVkU2NyaXB0LnByb3RvdHlwZSA9IHsKICAgICAgICAg
aWYgKG9iaiBpbnN0YW5jZW9mIGluc3BlY3RlZFdpbmRvdy5SZWdFeHApCiAgICAgICAgICAgICBy
ZXR1cm4gInJlZ2V4cCI7CiAgICAgICAgIC8vIEZpcmVCdWcncyBhcnJheSBkZXRlY3Rpb24uCi0g
ICAgICAgIGlmIChpc0Zpbml0ZShvYmoubGVuZ3RoKSAmJiB0eXBlb2Ygb2JqLnNwbGljZSA9PT0g
ImZ1bmN0aW9uIikKLSAgICAgICAgICAgIHJldHVybiAiYXJyYXkiOwotICAgICAgICBpZiAoaXNG
aW5pdGUob2JqLmxlbmd0aCkgJiYgdHlwZW9mIG9iai5jYWxsZWUgPT09ICJmdW5jdGlvbiIpIC8v
IGFyZ3VtZW50cy4KLSAgICAgICAgICAgIHJldHVybiAiYXJyYXkiOworICAgICAgICB0cnkgewor
ICAgICAgICAgICAgaWYgKGlzRmluaXRlKG9iai5sZW5ndGgpICYmIHR5cGVvZiBvYmouc3BsaWNl
ID09PSAiZnVuY3Rpb24iKQorICAgICAgICAgICAgICAgIHJldHVybiAiYXJyYXkiOworICAgICAg
ICAgICAgaWYgKGlzRmluaXRlKG9iai5sZW5ndGgpICYmIHR5cGVvZiBvYmouY2FsbGVlID09PSAi
ZnVuY3Rpb24iKSAvLyBhcmd1bWVudHMuCisgICAgICAgICAgICAgICAgcmV0dXJuICJhcnJheSI7
CisgICAgICAgIH0gY2F0Y2ggKGUpIHsKKyAgICAgICAgICAgIHJldHVybiB0eXBlOworICAgICAg
ICB9CiAgICAgICAgIGlmIChvYmogaW5zdGFuY2VvZiBpbnNwZWN0ZWRXaW5kb3cuTm9kZUxpc3Qp
CiAgICAgICAgICAgICByZXR1cm4gImFycmF5IjsKICAgICAgICAgaWYgKG9iaiBpbnN0YW5jZW9m
IGluc3BlY3RlZFdpbmRvdy5IVE1MQ29sbGVjdGlvbikK
</data>

          </attachment>
      

    </bug>

</bugzilla>