<?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>86965</bug_id>
          
          <creation_ts>2012-05-20 08:19:08 -0700</creation_ts>
          <short_desc>[Chromium] Print layout test result on buildbot</short_desc>
          <delta_ts>2012-05-20 12:09:50 -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>Tools / Tests</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="Gyuyoung Kim">gyuyoung.kim</reporter>
          <assigned_to name="Gyuyoung Kim">gyuyoung.kim</assigned_to>
          <cc>abarth</cc>
    
    <cc>dpranke</cc>
    
    <cc>eric</cc>
    
    <cc>jochen</cc>
    
    <cc>rniwa</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>628583</commentid>
    <comment_count>0</comment_count>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-05-20 08:19:08 -0700</bug_when>
    <thetext>Chrome build bot doesn&apos;t print layout test result unlike other ports. If chrome buildbot prints result of layout test, it will help to check layout test status for chromium port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628584</commentid>
    <comment_count>1</comment_count>
      <attachid>142904</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-05-20 08:22:45 -0700</bug_when>
    <thetext>Created attachment 142904
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628591</commentid>
    <comment_count>2</comment_count>
    <who name="">jochen</who>
    <bug_when>2012-05-20 11:00:08 -0700</bug_when>
    <thetext>The user chrome-bot is correct

Adding Adam and Eric as they run our bots</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628592</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-20 11:07:31 -0700</bug_when>
    <thetext>I thought we had moved all the buildbots to modern NRWT-aware configurations?  Lame that we&apos;re depending on this wrapper hack.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628593</commentid>
    <comment_count>4</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-20 11:10:09 -0700</bug_when>
    <thetext>It looks like we still have a separate NRWT rule:
http://trac.webkit.org/browser/trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg#L405

but the RWT rule is generic:
http://trac.webkit.org/browser/trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg#L279</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628594</commentid>
    <comment_count>5</comment_count>
    <who name="">jochen</who>
    <bug_when>2012-05-20 11:12:29 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; It looks like we still have a separate NRWT rule:
&gt; http://trac.webkit.org/browser/trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg#L405
&gt; 
&gt; but the RWT rule is generic:
&gt; http://trac.webkit.org/browser/trunk/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg#L279

The bots definitely run RWT: http://build.webkit.org/builders/Chromium%20Linux%20Release%20%28Tests%29/builds/32799/steps/layout-test/logs/stdio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628597</commentid>
    <comment_count>6</comment_count>
      <attachid>142904</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-20 11:44:09 -0700</bug_when>
    <thetext>Comment on attachment 142904
Patch

Some of this confusion is all tied up in the fact that we have only one port which still use ORWT (AppleWin).

This is not the &quot;Right&quot; fix.  But this is totally fine.  THanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628607</commentid>
    <comment_count>7</comment_count>
      <attachid>142904</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-05-20 12:09:45 -0700</bug_when>
    <thetext>Comment on attachment 142904
Patch

Clearing flags on attachment: 142904

Committed r117708: &lt;http://trac.webkit.org/changeset/117708&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>628608</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-05-20 12:09:50 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>142904</attachid>
            <date>2012-05-20 08:22:45 -0700</date>
            <delta_ts>2012-05-20 12:09:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-86965-20120521002236.patch</filename>
            <type>text/plain</type>
            <size>1517</size>
            <attacher name="Gyuyoung Kim">gyuyoung.kim</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTE3NzA3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZjk3YTZiYjI1MGM5YTA3MGQ2MGI1NDk1ZjgxYWRiMTA4
N2U1MzY1OS4uZGYzN2FmM2JmODRhMjIxNWViYmNhODE3ZmFiMGVlMzUzMmNiZjM0MSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEz
IEBACisyMDEyLTA1LTIwICBHeXV5b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ac2Ftc3VuZy5jb20+
CisKKyAgICAgICAgW0Nocm9taXVtXSBQcmludCBsYXlvdXQgdGVzdCByZXN1bHQgb24gYnVpbGRi
b3QKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTg2OTY1
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBTY3Jp
cHRzL3J1bi13ZWJraXQtdGVzdHM6IEFkZCBjaHJvbWUtYm90IHRvIGJ1aWxkYm90IHVzZXIgbGlz
dC4KKyAgICAgICAgKHJ1bm5pbmdPbkJ1aWxkQm90KToKKwogMjAxMi0wNS0xOSAgS2V2aW4gT2xs
aXZpZXIgIDxrZXZpbm9AdGhlb2xsaXZpZXJzLmNvbT4KIAogICAgICAgICBbd3hdIFVucmV2aWV3
ZWQgYnVpbGQgZml4LiBEb24ndCBzdG9wIHRoZSBidWlsZCBpZiBhIGJ1aWxkIHN5c3RlbSB1cGRh
dGUgY2hlY2sgZmFpbHMuCmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3J1bi13ZWJraXQtdGVz
dHMgYi9Ub29scy9TY3JpcHRzL3J1bi13ZWJraXQtdGVzdHMKaW5kZXggYjYyNzMyYTg3ZDU5YTA0
YzIzNTFlYmU4NjExMTkyNDJlNzRiZjM1OC4uOGQ5NmYzZjk0MTZjZjNmN2IwNWY5YWMxODQyNDRm
MzU5ZjMyYWZkMiAxMDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy9ydW4td2Via2l0LXRlc3RzCisr
KyBiL1Rvb2xzL1NjcmlwdHMvcnVuLXdlYmtpdC10ZXN0cwpAQCAtNTEsNyArNTEsNyBAQCBzdWIg
cnVubmluZ09uQnVpbGRCb3QoKQogICAgICMgVGhpcyBpcyBhIGhhY2sgdG8gZGV0ZWN0IGlmIHdl
J3JlIHJ1bm5pbmcgb24gdGhlIGJ1aWxkYm90IHNvIHdlIGNhbgogICAgICMgcGFzcyAtLXZlcmJv
c2UgdG8gbmV3LXJ1bi13ZWJraXQtdGVzdHMuICBUaGlzIHdpbGwgYmUgcmVtb3ZlZCB3aGVuIHdl
CiAgICAgIyB1cGRhdGUgdGhlIGJ1aWxkYm90IGNvbmZpZyB0byBjYWxsIG5ldy1ydW4td2Via2l0
LXRlc3RzIGV4cGxpY2l0bHkuCi0gICAgbXkgJWlzQnVpbGRCb3RVc2VyID0gKCJhcHBsZSIgPT4g
MSwgImJ1aWxkYm90IiA9PiAxLCAid2Via2l0YnVpbGRib3QiID0+IDEsICJzbGF2ZSIgPT4gMSwg
ImJ1aWxkc2xhdmUtMSIgPT4gMSk7CisgICAgbXkgJWlzQnVpbGRCb3RVc2VyID0gKCJhcHBsZSIg
PT4gMSwgImJ1aWxkYm90IiA9PiAxLCAid2Via2l0YnVpbGRib3QiID0+IDEsICJzbGF2ZSIgPT4g
MSwgImJ1aWxkc2xhdmUtMSIgPT4gMSwgImNocm9tZS1ib3QiID0+IDEpOwogICAgIHJldHVybiAk
aXNCdWlsZEJvdFVzZXJ7JEVOVnsiVVNFUiJ9fTsKIH0KIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>