<?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>63080</bug_id>
          
          <creation_ts>2011-06-21 11:11:38 -0700</creation_ts>
          <short_desc>Unit-tests step on test-only bot is broken</short_desc>
          <delta_ts>2011-06-21 12:58:54 -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>0</everconfirmed>
          <reporter name="Dmitry Lomov">dslomov</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>aroben</cc>
    
    <cc>dslomov</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>424576</commentid>
    <comment_count>0</comment_count>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-21 11:11:38 -0700</bug_when>
    <thetext>Removing the step until run-api-tests is fixed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424578</commentid>
    <comment_count>1</comment_count>
      <attachid>98013</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-21 11:13:25 -0700</bug_when>
    <thetext>Created attachment 98013
Fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424621</commentid>
    <comment_count>2</comment_count>
      <attachid>98013</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-06-21 11:46:25 -0700</bug_when>
    <thetext>Comment on attachment 98013
Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=98013&amp;action=review

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:-564
&gt; -        # FIXME: Only running in debug mode due to https://bugs.webkit.org/show_bug.cgi?id=61812
&gt; -        if configuration == &quot;debug&quot;:
&gt; -            self.addStep(RunUnitTests)

Presumably BuildAndTestFactory doesn&apos;t need this fix?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424622</commentid>
    <comment_count>3</comment_count>
      <attachid>98013</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-06-21 11:47:40 -0700</bug_when>
    <thetext>Comment on attachment 98013
Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=98013&amp;action=review

&gt;&gt; Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:-564
&gt;&gt; -            self.addStep(RunUnitTests)
&gt; 
&gt; Presumably BuildAndTestFactory doesn&apos;t need this fix?

Indeed, SnowLeopard Intel Leaks (which uses BuildAndTestFactory) doesn&apos;t show the issue you&apos;re trying to fix: http://build.webkit.org/builders/SnowLeopard%20Intel%20Leaks/builds/17616/steps/run-api-tests/logs/stdio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424626</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-21 11:48:39 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 98013 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=98013&amp;action=review
&gt; 
&gt; &gt;&gt; Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:-564
&gt; &gt;&gt; -            self.addStep(RunUnitTests)
&gt; &gt; 
&gt; &gt; Presumably BuildAndTestFactory doesn&apos;t need this fix?
&gt; 
&gt; Indeed, SnowLeopard Intel Leaks (which uses BuildAndTestFactory) doesn&apos;t show the issue you&apos;re trying to fix: http://build.webkit.org/builders/SnowLeopard%20Intel%20Leaks/builds/17616/steps/run-api-tests/logs/stdio

Cool - I&apos;ll be back :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424633</commentid>
    <comment_count>5</comment_count>
      <attachid>98030</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-21 11:53:57 -0700</bug_when>
    <thetext>Created attachment 98030
Not disabling on build-and-test bots; disabling on non-(win and mac)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424649</commentid>
    <comment_count>6</comment_count>
      <attachid>98030</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-06-21 12:08:53 -0700</bug_when>
    <thetext>Comment on attachment 98030
Not disabling on build-and-test bots; disabling on non-(win and mac)

View in context: https://bugs.webkit.org/attachment.cgi?id=98030&amp;action=review

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg:559
&gt; +        if configuration == &quot;debug&quot; and (platform == &apos;mac&apos; or platform == &apos;win&apos;):

An even better way to write this is:

if configuration == &apos;debug&apos; and platform in (&apos;mac&apos;, &apos;win&apos;):</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424688</commentid>
    <comment_count>7</comment_count>
      <attachid>98030</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-06-21 12:58:50 -0700</bug_when>
    <thetext>Comment on attachment 98030
Not disabling on build-and-test bots; disabling on non-(win and mac)

Clearing flags on attachment: 98030

Committed r89373: &lt;http://trac.webkit.org/changeset/89373&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>424689</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-06-21 12:58:54 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>98013</attachid>
            <date>2011-06-21 11:13:25 -0700</date>
            <delta_ts>2011-06-21 11:53:57 -0700</delta_ts>
            <desc>Fix</desc>
            <filename>63080.patch</filename>
            <type>text/plain</type>
            <size>1878</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtpdC5vcmctY29u
ZmlnL21hc3Rlci5jZmcgYi9Ub29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3Jn
LWNvbmZpZy9tYXN0ZXIuY2ZnCmluZGV4IDVjMmY0YzUuLjdmODJkYWQgMTAwNjQ0Ci0tLSBhL1Rv
b2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtpdC5vcmctY29uZmlnL21hc3Rlci5jZmcK
KysrIGIvVG9vbHMvQnVpbGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvbWFz
dGVyLmNmZwpAQCAtNTM1LDEwICs1MzUsNiBAQCBjbGFzcyBUZXN0RmFjdG9yeShGYWN0b3J5KToK
ICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1bkphdmFTY3JpcHRDb3JlVGVzdHMsIGJ1aWxkSlNDVG9v
bD1GYWxzZSkKICAgICAgICAgc2VsZi5hZGRTdGVwKHNlbGYuVGVzdENsYXNzLCBidWlsZEpTQ1Rv
b2w9KHBsYXRmb3JtICE9ICd3aW4nKSkKIAotICAgICAgICAjIEZJWE1FOiBPbmx5IHJ1bm5pbmcg
aW4gZGVidWcgbW9kZSBkdWUgdG8gaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTYxODEyCi0gICAgICAgIGlmIGNvbmZpZ3VyYXRpb24gPT0gImRlYnVnIjoKLSAgICAgICAg
ICAgIHNlbGYuYWRkU3RlcChSdW5Vbml0VGVzdHMpCi0KICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1
blB5dGhvblRlc3RzKQogICAgICAgICBzZWxmLmFkZFN0ZXAoUnVuUGVybFRlc3RzKQogICAgICAg
ICBzZWxmLmFkZFN0ZXAoUnVuQmluZGluZ3NUZXN0cykKQEAgLTU1OSw5ICs1NTUsNiBAQCBjbGFz
cyBCdWlsZEFuZFRlc3RGYWN0b3J5KEZhY3RvcnkpOgogICAgICAgICBpZiBwbGF0Zm9ybS5zdGFy
dHN3aXRoKCJjaHJvbWl1bSIpOgogICAgICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1bkNocm9taXVt
V2ViS2l0VW5pdFRlc3RzKQogICAgICAgICBzZWxmLmFkZFN0ZXAoc2VsZi5UZXN0Q2xhc3MpCi0g
ICAgICAgICMgRklYTUU6IE9ubHkgcnVubmluZyBpbiBkZWJ1ZyBtb2RlIGR1ZSB0byBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjE4MTIKLSAgICAgICAgaWYgY29uZmln
dXJhdGlvbiA9PSAiZGVidWciOgotICAgICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1blVuaXRUZXN0
cykKICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1blB5dGhvblRlc3RzKQogICAgICAgICAjIENocm9t
aXVtIFdpbiBydW5zIGluIG5vbi1DeWd3aW4gZW52aXJvbm1lbnQsIHdoaWNoIGlzIG5vdCB5ZXQg
Zml0CiAgICAgICAgICMgZm9yIHJ1bm5pbmcgdGVzdHMuIFRoaXMgY2FuIGJlIHJlbW92ZWQgb25j
ZSBidWcgNDgxNjYgaXMgZml4ZWQuCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cgYi9Ub29s
cy9DaGFuZ2VMb2cKaW5kZXggOTExNzkwMi4uMWFhN2I0MyAxMDA2NDQKLS0tIGEvVG9vbHMvQ2hh
bmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDExLTA2LTIx
ICBEbWl0cnkgTG9tb3YgIDxkc2xvbW92QGdvb2dsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTYzMDgwCisgICAgICAgIFVuaXQtdGVzdHMgc3RlcCBvbiB0ZXN0LW9ubHkg
Ym90IGlzIGJyb2tlbi4KKyAgICAgICAgUmVtb3ZpbmcgdGhlIHN0ZXAgdW50aWwgcnVuLWFwaS10
ZXN0cyBpcyBmaXhlZC4KKworICAgICAgICAqIEJ1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtp
dC5vcmctY29uZmlnL21hc3Rlci5jZmc6CisKIDIwMTEtMDYtMjAgIERtaXRyeSBMb21vdiAgPGRz
bG9tb3ZAZ29vZ2xlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXZpZCBMZXZpbi4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>98030</attachid>
            <date>2011-06-21 11:53:57 -0700</date>
            <delta_ts>2011-06-21 12:58:50 -0700</delta_ts>
            <desc>Not disabling on build-and-test bots; disabling on non-(win and mac)</desc>
            <filename>63080.patch</filename>
            <type>text/plain</type>
            <size>1841</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtpdC5vcmctY29u
ZmlnL21hc3Rlci5jZmcgYi9Ub29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3Jn
LWNvbmZpZy9tYXN0ZXIuY2ZnCmluZGV4IDVjMmY0YzUuLjQzNjdjNmMgMTAwNjQ0Ci0tLSBhL1Rv
b2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtpdC5vcmctY29uZmlnL21hc3Rlci5jZmcK
KysrIGIvVG9vbHMvQnVpbGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvbWFz
dGVyLmNmZwpAQCAtNTM1LDEwICs1MzUsNiBAQCBjbGFzcyBUZXN0RmFjdG9yeShGYWN0b3J5KToK
ICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1bkphdmFTY3JpcHRDb3JlVGVzdHMsIGJ1aWxkSlNDVG9v
bD1GYWxzZSkKICAgICAgICAgc2VsZi5hZGRTdGVwKHNlbGYuVGVzdENsYXNzLCBidWlsZEpTQ1Rv
b2w9KHBsYXRmb3JtICE9ICd3aW4nKSkKIAotICAgICAgICAjIEZJWE1FOiBPbmx5IHJ1bm5pbmcg
aW4gZGVidWcgbW9kZSBkdWUgdG8gaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTYxODEyCi0gICAgICAgIGlmIGNvbmZpZ3VyYXRpb24gPT0gImRlYnVnIjoKLSAgICAgICAg
ICAgIHNlbGYuYWRkU3RlcChSdW5Vbml0VGVzdHMpCi0KICAgICAgICAgc2VsZi5hZGRTdGVwKFJ1
blB5dGhvblRlc3RzKQogICAgICAgICBzZWxmLmFkZFN0ZXAoUnVuUGVybFRlc3RzKQogICAgICAg
ICBzZWxmLmFkZFN0ZXAoUnVuQmluZGluZ3NUZXN0cykKQEAgLTU2MCw3ICs1NTYsNyBAQCBjbGFz
cyBCdWlsZEFuZFRlc3RGYWN0b3J5KEZhY3RvcnkpOgogICAgICAgICAgICAgc2VsZi5hZGRTdGVw
KFJ1bkNocm9taXVtV2ViS2l0VW5pdFRlc3RzKQogICAgICAgICBzZWxmLmFkZFN0ZXAoc2VsZi5U
ZXN0Q2xhc3MpCiAgICAgICAgICMgRklYTUU6IE9ubHkgcnVubmluZyBpbiBkZWJ1ZyBtb2RlIGR1
ZSB0byBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjE4MTIKLSAgICAg
ICAgaWYgY29uZmlndXJhdGlvbiA9PSAiZGVidWciOgorICAgICAgICBpZiBjb25maWd1cmF0aW9u
ID09ICJkZWJ1ZyIgYW5kIChwbGF0Zm9ybSA9PSAnbWFjJyBvciBwbGF0Zm9ybSA9PSAnd2luJyk6
CiAgICAgICAgICAgICBzZWxmLmFkZFN0ZXAoUnVuVW5pdFRlc3RzKQogICAgICAgICBzZWxmLmFk
ZFN0ZXAoUnVuUHl0aG9uVGVzdHMpCiAgICAgICAgICMgQ2hyb21pdW0gV2luIHJ1bnMgaW4gbm9u
LUN5Z3dpbiBlbnZpcm9ubWVudCwgd2hpY2ggaXMgbm90IHlldCBmaXQKZGlmZiAtLWdpdCBhL1Rv
b2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA5MTE3OTAyLi4xYWE3YjQzIDEw
MDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMg
KzEsMTMgQEAKKzIwMTEtMDYtMjEgIERtaXRyeSBMb21vdiAgPGRzbG9tb3ZAZ29vZ2xlLmNvbT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjMwODAKKyAgICAgICAgVW5pdC10ZXN0
cyBzdGVwIG9uIHRlc3Qtb25seSBib3QgaXMgYnJva2VuLgorICAgICAgICBSZW1vdmluZyB0aGUg
c3RlcCB1bnRpbCBydW4tYXBpLXRlc3RzIGlzIGZpeGVkLgorCisgICAgICAgICogQnVpbGRTbGF2
ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvbWFzdGVyLmNmZzoKKwogMjAxMS0wNi0y
MCAgRG1pdHJ5IExvbW92ICA8ZHNsb21vdkBnb29nbGUuY29tPgogCiAgICAgICAgIFJldmlld2Vk
IGJ5IERhdmlkIExldmluLgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>