<?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>51962</bug_id>
          
          <creation_ts>2011-01-05 15:19:07 -0800</creation_ts>
          <short_desc>minor code cleanup for code review tool</short_desc>
          <delta_ts>2011-01-05 19:20:10 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</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="Ojan Vafai">ojan</reporter>
          <assigned_to name="Ojan Vafai">ojan</assigned_to>
          <cc>abarth</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>329824</commentid>
    <comment_count>0</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2011-01-05 15:19:07 -0800</bug_when>
    <thetext>minor code cleanup for code review tool</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>329825</commentid>
    <comment_count>1</comment_count>
      <attachid>78054</attachid>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2011-01-05 15:21:18 -0800</bug_when>
    <thetext>Created attachment 78054
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>329977</commentid>
    <comment_count>2</comment_count>
      <attachid>78054</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-05 19:20:05 -0800</bug_when>
    <thetext>Comment on attachment 78054
Patch

Clearing flags on attachment: 78054

Committed r75131: &lt;http://trac.webkit.org/changeset/75131&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>329978</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-05 19:20:10 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>78054</attachid>
            <date>2011-01-05 15:21:18 -0800</date>
            <delta_ts>2011-01-05 19:20:05 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-51962-20110105152117.patch</filename>
            <type>text/plain</type>
            <size>2489</size>
            <attacher name="Ojan Vafai">ojan</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYnNpdGVzL2J1Z3Mud2Via2l0Lm9yZy9DaGFuZ2VMb2cgYi9XZWJzaXRl
cy9idWdzLndlYmtpdC5vcmcvQ2hhbmdlTG9nCmluZGV4IDQ2OTI3YzRmM2NlMGE5Y2E1NDVlNTZk
NWVjNzE5ODEzN2NjNWYxM2EuLjBhZDAxNDBjMjYyM2FiNTk2YjRhNDcxNmNjZTU4MmI4NjJkZGU2
NzYgMTAwNjQ0Ci0tLSBhL1dlYnNpdGVzL2J1Z3Mud2Via2l0Lm9yZy9DaGFuZ2VMb2cKKysrIGIv
V2Vic2l0ZXMvYnVncy53ZWJraXQub3JnL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDEx
LTAxLTA1ICBPamFuIFZhZmFpICA8b2phbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgbWlub3IgY29kZSBjbGVhbnVwIGZvciBj
b2RlIHJldmlldyB0b29sCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD01MTk2MgorCisgICAgICAgIENvbnNvbGlkYXRlcyBzb21lIHF1ZXJpZXMgdG8gdXNp
bmcgc2hhcmVkIGZ1bmN0aW9ucy4KKworICAgICAgICAqIGNvZGUtcmV2aWV3LmpzOgorCiAyMDEx
LTAxLTA0ICBPamFuIFZhZmFpICA8b2phbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgSW5jcmVt
ZW50IHZlcnNpb24gbnVtYmVyIHRvIGF2b2lkIGdldHRpbmcgdGhlIGNhY2hlZCBKUyBmaWxlLgpk
aWZmIC0tZ2l0IGEvV2Vic2l0ZXMvYnVncy53ZWJraXQub3JnL2NvZGUtcmV2aWV3LmpzIGIvV2Vi
c2l0ZXMvYnVncy53ZWJraXQub3JnL2NvZGUtcmV2aWV3LmpzCmluZGV4IGRhYTk5ZTc4NzEwYjI5
YWE4NTU0NTE3ZjNiNDMzNjE3OTA0NjVkZTYuLjY5OTQxMDllY2RhOGVlNDlmMjZlOWMyNDdjYmRm
NmE1MmFkOGQ5NGEgMTAwNjQ0Ci0tLSBhL1dlYnNpdGVzL2J1Z3Mud2Via2l0Lm9yZy9jb2RlLXJl
dmlldy5qcworKysgYi9XZWJzaXRlcy9idWdzLndlYmtpdC5vcmcvY29kZS1yZXZpZXcuanMKQEAg
LTUzMyw2ICs1MzMsMTAgQEAKICAgICByZXR1cm4gTnVtYmVyKGxpbmUucXVlcnlTZWxlY3Rvcign
LnRvJykudGV4dENvbnRlbnQpOwogICB9CiAKKyAgZnVuY3Rpb24gdGV4dENvbnRlbnRzRm9yKGxp
bmUpIHsKKyAgICByZXR1cm4gJCgnLnRleHQnLCBsaW5lKS50ZXh0KCk7CisgIH0KKwogICBmdW5j
dGlvbiBsaW5lTnVtYmVyRm9yRmlyc3ROb25Db250ZXh0TGluZShwYXRjaGVkX2ZpbGUsIGxpbmUs
IHByZXZfbGluZSwgY29udGV4dCwgaHVua19udW0pIHsKICAgICBpZiAoY29udGV4dC5sZW5ndGgp
IHsKICAgICAgIHZhciBwcmV2X2xpbmVfbnVtID0gZnJvbUxpbmVOdW1iZXIocHJldl9saW5lKSAt
IDE7CkBAIC01NjAsNyArNTY0LDcgQEAKIAogICAgICAgZm9yICh2YXIgaiA9IDAsIGxpbmVzX2xl
biA9IGxpbmVzLmxlbmd0aDsgaiA8IGxpbmVzX2xlbjsgaisrKSB7CiAgICAgICAgIHZhciBsaW5l
ID0gbGluZXNbal07Ci0gICAgICAgIHZhciBsaW5lX2NvbnRlbnRzID0gbGluZS5xdWVyeVNlbGVj
dG9yKCcudGV4dCcpLnRleHRDb250ZW50OworICAgICAgICB2YXIgbGluZV9jb250ZW50cyA9IHRl
eHRDb250ZW50c0ZvcihsaW5lKTsKICAgICAgICAgaWYgKCQobGluZSkuaGFzQ2xhc3MoJ2FkZCcp
KSB7CiAgICAgICAgICAgaWYgKGN1cnJlbnRfbGluZSA9PSAtMSkgewogICAgICAgICAgICAgY3Vy
cmVudF9saW5lID0gbGluZU51bWJlckZvckZpcnN0Tm9uQ29udGV4dExpbmUocGF0Y2hlZF9maWxl
LCBsaW5lLCBsaW5lc1tqLTFdLCBjb250ZXh0LCBodW5rX251bSk7CkBAIC03ODMsOCArNzg3LDcg
QEAKICAgICAgICAgYWN0aW9uID0gJysnOwogICAgICAgZWxzZSBpZiAoJCh0aGlzKS5oYXNDbGFz
cygncmVtb3ZlJykpCiAgICAgICAgIGFjdGlvbiA9ICctJzsKLSAgICAgIHZhciB0ZXh0ID0gJCh0
aGlzKS5jaGlsZHJlbignLnRleHQnKS50ZXh0KCk7Ci0gICAgICBzbmlwcGV0cy5wdXNoKGluZGVu
dCArIGFjdGlvbiArIHRleHQpOworICAgICAgc25pcHBldHMucHVzaChpbmRlbnQgKyBhY3Rpb24g
KyB0ZXh0Q29udGVudHNGb3IodGhpcykpOwogICAgIH0pOwogICAgIHJldHVybiBzbmlwcGV0cy5q
b2luKCdcbicpOwogICB9CkBAIC03OTksNyArODAyLDcgQEAKIAogICBmdW5jdGlvbiBzbmlwcGV0
Rm9yKGxpbmUsIGluZGVudCkgewogICAgIHZhciBmaWxlX25hbWUgPSBmaWxlTmFtZUZvcihsaW5l
KTsKLSAgICB2YXIgbGluZV9udW1iZXIgPSBsaW5lLmhhc0NsYXNzKCdyZW1vdmUnKSA/ICctJyAr
IGxpbmUuY2hpbGRyZW4oJy5mcm9tJykudGV4dCgpIDogbGluZS5jaGlsZHJlbignLnRvJykudGV4
dCgpOworICAgIHZhciBsaW5lX251bWJlciA9IGxpbmUuaGFzQ2xhc3MoJ3JlbW92ZScpID8gJy0n
ICsgZnJvbUxpbmVOdW1iZXIobGluZVswXSkgOiB0b0xpbmVOdW1iZXIobGluZVswXSk7CiAgICAg
cmV0dXJuIGluZGVudCArIGZpbGVfbmFtZSArICc6JyArIGxpbmVfbnVtYmVyICsgJ1xuJyArIGNv
bnRleHRTbmlwcGV0Rm9yKGxpbmUsIGluZGVudCk7CiAgIH0KIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>