<?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>86408</bug_id>
          
          <creation_ts>2012-05-14 16:18:48 -0700</creation_ts>
          <short_desc>Increase Debug Logging in MarkStack::validate()</short_desc>
          <delta_ts>2012-05-21 15:22:14 -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="Michael Saboff">msaboff</reporter>
          <assigned_to name="Michael Saboff">msaboff</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>623410</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2012-05-14 16:18:48 -0700</bug_when>
    <thetext>Improve the diagnostic information in MarkStack::validate() beyond crashing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>623419</commentid>
    <comment_count>1</comment_count>
      <attachid>141806</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2012-05-14 16:23:49 -0700</bug_when>
    <thetext>Created attachment 141806
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>623423</commentid>
    <comment_count>2</comment_count>
      <attachid>141806</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2012-05-14 16:29:00 -0700</bug_when>
    <thetext>Comment on attachment 141806
Patch

rs=me, but change to dataLog().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>623442</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2012-05-14 16:44:58 -0700</bug_when>
    <thetext>Committed r117013: &lt;http://trac.webkit.org/changeset/117013&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>629585</commentid>
    <comment_count>4</comment_count>
      <attachid>141806</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-21 15:22:14 -0700</bug_when>
    <thetext>Comment on attachment 141806
Patch

Cleared review? from attachment 141806 so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>141806</attachid>
            <date>2012-05-14 16:23:49 -0700</date>
            <delta_ts>2012-05-21 15:22:13 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>86408.patch</filename>
            <type>text/plain</type>
            <size>2395</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTE3MDExKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBA
CisyMDEyLTA1LTE0ICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgorCisgICAg
ICAgIEluY3JlYXNlIERlYnVnIExvZ2dpbmcgaW4gTWFya1N0YWNrOjp2YWxpZGF0ZSgpCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NjQwOAorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZGVkIHNvbWUgZGVz
Y3JpcHRpdmUgZnByaW50ZidzIGZvciB0aGUgY29uZGl0aW9ucyBhbmQKKyAgICAgICAgdmFsdWVz
IHdoZW4gYSBjZWxsIHZhbGlkYXRpb24gZmFpbHMuOndxCisKKyAgICAgICAgKiBoZWFwL01hcmtT
dGFjay5jcHA6CisgICAgICAgIChKU0M6Ok1hcmtTdGFjazo6dmFsaWRhdGUpOgorCiAyMDEyLTA1
LTE0ICBDYXJsb3MgR2FyY2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KIAogICAgICAg
ICBVbnJldmlld2VkLiBGaXggbWFrZSBkaXN0Y2hlY2suCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlw
dENvcmUvaGVhcC9NYXJrU3RhY2suY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0
Q29yZS9oZWFwL01hcmtTdGFjay5jcHAJKHJldmlzaW9uIDExNjk1OCkKKysrIFNvdXJjZS9KYXZh
U2NyaXB0Q29yZS9oZWFwL01hcmtTdGFjay5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTUzNSwxNiAr
NTM1LDI5IEBAIHZvaWQgU2xvdFZpc2l0b3I6OmZpbmFsaXplVW5jb25kaXRpb25hbEYKICNpZiBF
TkFCTEUoR0NfVkFMSURBVElPTikKIHZvaWQgTWFya1N0YWNrOjp2YWxpZGF0ZShKU0NlbGwqIGNl
bGwpCiB7Ci0gICAgaWYgKCFjZWxsKQorICAgIGlmICghY2VsbCkgeworICAgICAgICBmcHJpbnRm
KHN0ZGVyciwgImNlbGwgaXMgTlVMTFxuIik7CiAgICAgICAgIENSQVNIKCk7CisgICAgfQogCi0g
ICAgaWYgKCFjZWxsLT5zdHJ1Y3R1cmUoKSkKKyAgICBpZiAoIWNlbGwtPnN0cnVjdHVyZSgpKSB7
CisgICAgICAgIGZwcmludGYoc3RkZXJyLCAiY2VsbCBhdCAlcCBoYXMgYSBudWxsIHN0cnVjdHVy
ZVxuIiAsIGNlbGwpOwogICAgICAgICBDUkFTSCgpOworICAgIH0KIAogICAgIC8vIEJvdGggdGhl
IGNlbGwncyBzdHJ1Y3R1cmUsIGFuZCB0aGUgY2VsbCdzIHN0cnVjdHVyZSdzIHN0cnVjdHVyZSBz
aG91bGQgYmUgdGhlIFN0cnVjdHVyZSBTdHJ1Y3R1cmUuCiAgICAgLy8gSSBoYXRlIHRoaXMgc2Vu
dGVuY2UuCi0gICAgaWYgKGNlbGwtPnN0cnVjdHVyZSgpLT5zdHJ1Y3R1cmUoKS0+SlNDZWxsOjpj
bGFzc0luZm8oKSAhPSBjZWxsLT5zdHJ1Y3R1cmUoKS0+SlNDZWxsOjpjbGFzc0luZm8oKSkKKyAg
ICBpZiAoY2VsbC0+c3RydWN0dXJlKCktPnN0cnVjdHVyZSgpLT5KU0NlbGw6OmNsYXNzSW5mbygp
ICE9IGNlbGwtPnN0cnVjdHVyZSgpLT5KU0NlbGw6OmNsYXNzSW5mbygpKSB7CisgICAgICAgIGNv
bnN0IGNoYXIqIHBhcmVudENsYXNzTmFtZSA9IDA7CisgICAgICAgIGNvbnN0IGNoYXIqIG91ckNs
YXNzTmFtZSA9IDA7CisgICAgICAgIGlmIChjZWxsLT5zdHJ1Y3R1cmUoKS0+c3RydWN0dXJlKCkg
JiYgY2VsbC0+c3RydWN0dXJlKCktPnN0cnVjdHVyZSgpLT5KU0NlbGw6OmNsYXNzSW5mbygpKQor
ICAgICAgICAgICAgcGFyZW50Q2xhc3NOYW1lID0gY2VsbC0+c3RydWN0dXJlKCktPnN0cnVjdHVy
ZSgpLT5KU0NlbGw6OmNsYXNzSW5mbygpLT5jbGFzc05hbWU7CisgICAgICAgIGlmIChjZWxsLT5z
dHJ1Y3R1cmUoKS0+SlNDZWxsOjpjbGFzc0luZm8oKSkKKyAgICAgICAgICAgIG91ckNsYXNzTmFt
ZSA9IGNlbGwtPnN0cnVjdHVyZSgpLT5KU0NlbGw6OmNsYXNzSW5mbygpLT5jbGFzc05hbWU7Cisg
ICAgICAgIGZwcmludGYoc3RkZXJyLCAicGFyZW50IHN0cnVjdHVyZSAoJXAgPCVzPikgb2YgY2Vs
bCBhdCAlcCBkb2Vzbid0IG1hdGNoIGNlbGwncyBzdHJ1Y3R1cmUgKCVwIDwlcz4pXG4iLAorICAg
ICAgICAgICAgICAgIGNlbGwtPnN0cnVjdHVyZSgpLT5zdHJ1Y3R1cmUoKSwgcGFyZW50Q2xhc3NO
YW1lLCBjZWxsLCBjZWxsLT5zdHJ1Y3R1cmUoKSwgb3VyQ2xhc3NOYW1lKTsKICAgICAgICAgQ1JB
U0goKTsKKyAgICB9CiB9CiAjZWxzZQogdm9pZCBNYXJrU3RhY2s6OnZhbGlkYXRlKEpTQ2VsbCop
Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>