<?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>35391</bug_id>
          
          <creation_ts>2010-02-25 09:40:37 -0800</creation_ts>
          <short_desc>diff_parser should not log to the root logger</short_desc>
          <delta_ts>2010-02-26 08:19:53 -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>Tools / Tests</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>1</everconfirmed>
          <reporter name="Chris Jerdonek">cjerdonek</reporter>
          <assigned_to name="Chris Jerdonek">cjerdonek</assigned_to>
          <cc>abarth</cc>
    
    <cc>cjerdonek</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>hamaji</cc>
    
    <cc>levin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>193771</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Jerdonek">cjerdonek</who>
    <bug_when>2010-02-25 09:40:37 -0800</bug_when>
    <thetext>See summary.  See also the similar report for the Python layout test scripts:

https://bugs.webkit.org/show_bug.cgi?id=35194

See the first comment there for more info.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>193781</commentid>
    <comment_count>1</comment_count>
      <attachid>49502</attachid>
    <who name="Chris Jerdonek">cjerdonek</who>
    <bug_when>2010-02-25 09:59:36 -0800</bug_when>
    <thetext>Created attachment 49502
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>193906</commentid>
    <comment_count>2</comment_count>
      <attachid>49502</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-02-25 16:00:04 -0800</bug_when>
    <thetext>Comment on attachment 49502
Proposed patch

LGTM.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>193969</commentid>
    <comment_count>3</comment_count>
    <who name="Chris Jerdonek">cjerdonek</who>
    <bug_when>2010-02-25 17:58:18 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 49502 [details])
&gt; LGTM.

Thanks, Eric!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>194105</commentid>
    <comment_count>4</comment_count>
      <attachid>49502</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-02-26 08:19:49 -0800</bug_when>
    <thetext>Comment on attachment 49502
Proposed patch

Clearing flags on attachment: 49502

Committed r55284: &lt;http://trac.webkit.org/changeset/55284&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>194106</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-02-26 08:19:53 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>49502</attachid>
            <date>2010-02-25 09:59:36 -0800</date>
            <delta_ts>2010-02-26 08:19:48 -0800</delta_ts>
            <desc>Proposed patch</desc>
            <filename>_patch-35391-1.diff</filename>
            <type>text/plain</type>
            <size>2004</size>
            <attacher name="Chris Jerdonek">cjerdonek</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBlMzU0YzNhLi4yM2E2NTE0IDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDIt
MjUgIENocmlzIEplcmRvbmVrICA8Y2plcmRvbmVrQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQ2hhbmdlZCB0aGUgZGlmZl9wYXJz
ZXIgbW9kdWxlIHRvIGxvZyB0byBhIG1vZHVsZS1zcGVjaWZpYworICAgICAgICBsb2dnaW5nLmxv
Z2dlciByYXRoZXIgdGhhbiB0aGUgcm9vdCBsb2dnZXIuCisKKyAgICAgICAgaHR0cHM6Ly9idWdz
LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM1MzkxCisKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2RpZmZfcGFyc2VyLnB5OgorCiAyMDEwLTAyLTI0ICBDaHJpcyBKZXJkb25layAgPGNq
ZXJkb25la0B3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgpkaWZmIC0tZ2l0IGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9kaWZmX3BhcnNlci5w
eSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvZGlmZl9wYXJzZXIucHkKaW5kZXggN2Rj
ZTdlOC4uMDllNTRkYiAxMDA2NDQKLS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9k
aWZmX3BhcnNlci5weQorKysgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2RpZmZfcGFy
c2VyLnB5CkBAIC0zMSw2ICszMSw3IEBACiBpbXBvcnQgbG9nZ2luZwogaW1wb3J0IHJlCiAKK19s
b2cgPSBsb2dnaW5nLmdldExvZ2dlcigid2Via2l0cHkuZGlmZl9wYXJzZXIiKQogCiBfcmVnZXhw
X2NvbXBpbGVfY2FjaGUgPSB7fQogCkBAIC0xMzgsNyArMTM5LDggQEAgY2xhc3MgRGlmZlBhcnNl
cjoKICAgICAgICAgICAgIGxpbmVzX2NoYW5nZWQgPSBtYXRjaChyIl5AQCAtKD9QPE9sZFN0YXJ0
TGluZT5cZCspKCxcZCspPyBcKyg/UDxOZXdTdGFydExpbmU+XGQrKSgsXGQrKT8gQEAiLCBsaW5l
KQogICAgICAgICAgICAgaWYgbGluZXNfY2hhbmdlZDoKICAgICAgICAgICAgICAgICBpZiBzdGF0
ZSAhPSBfREVDTEFSRURfRklMRV9QQVRIIGFuZCBzdGF0ZSAhPSBfUFJPQ0VTU0lOR19DSFVOSzoK
LSAgICAgICAgICAgICAgICAgICAgbG9nZ2luZy5lcnJvcignVW5leHBlY3RlZCBsaW5lIGNoYW5n
ZSB3aXRob3V0IGZpbGUgcGF0aCBkZWNsYXJhdGlvbjogJXInICUgbGluZSkKKyAgICAgICAgICAg
ICAgICAgICAgX2xvZy5lcnJvcignVW5leHBlY3RlZCBsaW5lIGNoYW5nZSB3aXRob3V0IGZpbGUg
cGF0aCAnCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ2RlY2xhcmF0aW9uOiAlcicg
JSBsaW5lKQogICAgICAgICAgICAgICAgIG9sZF9kaWZmX2xpbmUgPSBpbnQobGluZXNfY2hhbmdl
ZC5ncm91cCgnT2xkU3RhcnRMaW5lJykpCiAgICAgICAgICAgICAgICAgbmV3X2RpZmZfbGluZSA9
IGludChsaW5lc19jaGFuZ2VkLmdyb3VwKCdOZXdTdGFydExpbmUnKSkKICAgICAgICAgICAgICAg
ICBzdGF0ZSA9IF9QUk9DRVNTSU5HX0NIVU5LCkBAIC0xNTksNCArMTYxLDUgQEAgY2xhc3MgRGlm
ZlBhcnNlcjoKICAgICAgICAgICAgICAgICAgICAgIyBOb3RoaW5nIHRvIGRvLiAgV2UgbWF5IHN0
aWxsIGhhdmUgc29tZSBhZGRlZCBsaW5lcy4KICAgICAgICAgICAgICAgICAgICAgcGFzcwogICAg
ICAgICAgICAgICAgIGVsc2U6Ci0gICAgICAgICAgICAgICAgICAgIGxvZ2dpbmcuZXJyb3IoJ1Vu
ZXhwZWN0ZWQgZGlmZiBmb3JtYXQgd2hlbiBwYXJzaW5nIGEgY2h1bms6ICVyJyAlIGxpbmUpCisg
ICAgICAgICAgICAgICAgICAgIF9sb2cuZXJyb3IoJ1VuZXhwZWN0ZWQgZGlmZiBmb3JtYXQgd2hl
biBwYXJzaW5nIGEgJworICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdjaHVuazogJXIn
ICUgbGluZSkK
</data>

          </attachment>
      

    </bug>

</bugzilla>