<?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>46452</bug_id>
          
          <creation_ts>2010-09-23 23:53:35 -0700</creation_ts>
          <short_desc>[chromium] Updated test expectations to match the bots using new auto-update script.</short_desc>
          <delta_ts>2011-01-18 22:02:37 -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>0</everconfirmed>
          <reporter name="James Kozianski">koz</reporter>
          <assigned_to name="James Kozianski">koz</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>noel.gordon</cc>
    
    <cc>ojan</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>284261</commentid>
    <comment_count>0</comment_count>
    <who name="James Kozianski">koz</who>
    <bug_when>2010-09-23 23:53:35 -0700</bug_when>
    <thetext>[chromium] Updated test expectations to match the bots using new auto-update script.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284263</commentid>
    <comment_count>1</comment_count>
      <attachid>68656</attachid>
    <who name="James Kozianski">koz</who>
    <bug_when>2010-09-23 23:56:49 -0700</bug_when>
    <thetext>Created attachment 68656
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284299</commentid>
    <comment_count>2</comment_count>
      <attachid>68656</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-24 02:13:33 -0700</bug_when>
    <thetext>Comment on attachment 68656
Patch

Clearing flags on attachment: 68656

Committed r68244: &lt;http://trac.webkit.org/changeset/68244&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284300</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-24 02:13:38 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284544</commentid>
    <comment_count>4</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-24 11:00:20 -0700</bug_when>
    <thetext>Where can I learn more about this script?  What will the work flow be for converting BUG_AUTO into bugs?

As a feature request, maybe it can guess when to use SLOW if the test is only failing in debug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284650</commentid>
    <comment_count>5</comment_count>
    <who name="James Kozianski">koz</who>
    <bug_when>2010-09-24 13:24:26 -0700</bug_when>
    <thetext>Ojan and I have been working on improving the webkit/tools/layout_tests/webkitpy/layout_tests/update_expectations_from_dashboard.py script. Its usage is undocumented, but in short it takes JSON data as input generated from http://test-results.appspot.com/dashboards/flakiness_dashboard.html#expectationsUpdate=true and uses it to modify test_expectations.txt.

The modifications to the script are unreviewed, but once committed I&apos;ll add a section to the wiki explaining how to use it at http://dev.chromium.org/developers/testing/flakiness-dashboard.

&gt; What will the work flow be for converting BUG_AUTO into bugs?

I&apos;m not sure - Ojan, can you chime in?


&gt; As a feature request, maybe it can guess when to use SLOW if the test is only failing in debug.

Yep, that sounds like a good idea. Could you provide a more specific heuristic?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>284688</commentid>
    <comment_count>6</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-24 14:16:27 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; &gt; As a feature request, maybe it can guess when to use SLOW if the test is only failing in debug.
&gt; 
&gt; Yep, that sounds like a good idea. Could you provide a more specific heuristic?

I&apos;m not sure.  I just bring it up because it looks like the last 5 entries to test_expectations.txt are just slow tests.  But maybe not.  One of the tests is for Release builds.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>285049</commentid>
    <comment_count>7</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-09-25 14:02:00 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Where can I learn more about this script?  

The working version is still not checked in. We&apos;ll document it once it&apos;s usable. This was our first pass at actually using it.

&gt; What will the work flow be for converting BUG_AUTO into bugs?

That&apos;s a good question. I don&apos;t have a good answer. I was planning on bringing this up on chromium-dev soon. I&apos;m open to suggestions.

&gt; As a feature request, maybe it can guess when to use SLOW if the test is only failing in debug.

I&apos;ve been meaning to get rid of SLOW. I think it&apos;s too complicated. Instead, we should just have a long timeout but give a short timeout to tests that we expect to timeout. How does that sound?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>285497</commentid>
    <comment_count>8</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-27 09:29:46 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (In reply to comment #4)
&gt; &gt; What will the work flow be for converting BUG_AUTO into bugs?
&gt; 
&gt; That&apos;s a good question. I don&apos;t have a good answer. I was planning on bringing this up on chromium-dev soon. I&apos;m open to suggestions.

I think whoever runs the tool should fill in bug numbers, right?  We shouldn&apos;t check in expectations without bugs filed.

&gt; &gt; As a feature request, maybe it can guess when to use SLOW if the test is only failing in debug.
&gt; 
&gt; I&apos;ve been meaning to get rid of SLOW. I think it&apos;s too complicated. Instead, we should just have a long timeout but give a short timeout to tests that we expect to timeout. How does that sound?

I suspect that over time, more tests will time out (the long timeout) and the full test run will gradually get slower.  I prefer fast by default with exceptions to make things slower.

Maybe you&apos;re hoping that the auto-update script will detect and mark tests as slow?  I&apos;m not sure how easy that will be to do.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>68656</attachid>
            <date>2010-09-23 23:56:49 -0700</date>
            <delta_ts>2010-09-24 02:13:33 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-46452-20100924165646.patch</filename>
            <type>text/plain</type>
            <size>4168</size>
            <attacher name="James Kozianski">koz</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA5YzdhZWEzOTE4OTVkYjc0NzljNzhlMDM4MTM3YjIyNGM1YWJmNDJjLi45ZmVkMzg2
ZDk0OGM1YzZjOGZkN2Y4MWE4YzM3MDI2ZWEyMDNjYWE5IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAK
KzIwMTAtMDktMjMgIEphbWVzIEtvemlhbnNraSAgPGtvekBjaHJvbWl1bS5vcmc+CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW2Nocm9taXVtXSBVcGRh
dGVkIHRlc3QgZXhwZWN0YXRpb25zIHRvIG1hdGNoIHRoZSBib3RzIHVzaW5nIG5ldyBhdXRvLXVw
ZGF0ZSBzY3JpcHQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD00NjQ1MgorCisgICAgICAgIFVwZGF0ZWQgZXhwZWN0YXRpb25zIHRvIG1hdGNoIHRoZSBi
b3RzIHVzaW5nIHRoZSBuZXcgYXV0by11cGRhdGUgc2NyaXB0LgorCisgICAgICAgICogcGxhdGZv
cm0vY2hyb21pdW0vdGVzdF9leHBlY3RhdGlvbnMudHh0OgorCiAyMDEwLTA5LTIzICBTaW1vbiBG
cmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh
biBCZXJuc3RlaW4uCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS90
ZXN0X2V4cGVjdGF0aW9ucy50eHQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS90ZXN0
X2V4cGVjdGF0aW9ucy50eHQKaW5kZXggOTMxY2MxNmMxYjY5MTRhNTI3YzUxZTJiNTgxYjQ3MzA5
YWVlNGIxNi4uZWYyYjM1MGFiY2RhODllNzliNDFkZjdlNDhlZTY5NjJmN2E4ODIyMyAxMDA2NDQK
LS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vdGVzdF9leHBlY3RhdGlvbnMudHh0
CisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4
dApAQCAtMjQ5Myw3ICsyNDkzLDkgQEAgQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJk
LmZpbGxTdHlsZS5wYXJzZS5jdXJyZW50LmJhc2ljLmh0bWwgPSBURVhUCiBCVUc0NTk5MSA6IGNh
bnZhcy9waGlsaXAvdGVzdHMvMmQuZmlsbFN0eWxlLnBhcnNlLmN1cnJlbnQuY2hhbmdlZC5odG1s
ID0gVEVYVAogQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmZpbGxTdHlsZS5wYXJz
ZS5jdXJyZW50LnJlbW92ZWQuaHRtbCA9IFRFWFQKIEJVRzQ1OTkxIDogY2FudmFzL3BoaWxpcC90
ZXN0cy8yZC5maWxsU3R5bGUucGFyc2Uuc3lzdGVtLmh0bWwgPSBURVhUCi1CVUc0NTk5MSA6IGNh
bnZhcy9waGlsaXAvdGVzdHMvMmQuZ3JhZGllbnQucmFkaWFsLmNvbmUudG9wLmh0bWwgPSBURVhU
IFRJTUVPVVQKK0JVRzQ1OTkxIE1BQyBXSU4gREVCVUcgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJk
LmdyYWRpZW50LnJhZGlhbC5jb25lLnRvcC5odG1sID0gVEVYVAorQlVHNDU5OTEgUkVMRUFTRSA6
IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuZ3JhZGllbnQucmFkaWFsLmNvbmUudG9wLmh0bWwgPSBU
RVhUCitCVUc0NTk5MSBMSU5VWCBERUJVRyA6IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuZ3JhZGll
bnQucmFkaWFsLmNvbmUudG9wLmh0bWwgPSBURVhUIFRJTUVPVVQKIEJVRzQ1OTkxIDogY2FudmFz
L3BoaWxpcC90ZXN0cy8yZC5ncmFkaWVudC5yYWRpYWwubmVnYXRpdmUuaHRtbCA9IFRFWFQKIEJV
RzQ1OTkxIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5pbWFnZURhdGEuY3JlYXRlMS50eXBlLmh0
bWwgPSBURVhUCiBCVUc0NTk5MSA6IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuaW1hZ2VEYXRhLmNy
ZWF0ZTIudHlwZS5odG1sID0gVEVYVApAQCAtMjUwMiw5ICsyNTA0LDkgQEAgQlVHNDU5OTEgOiBj
YW52YXMvcGhpbGlwL3Rlc3RzLzJkLmltYWdlRGF0YS5nZXQudHlwZS5odG1sID0gVEVYVAogQlVH
NDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmltYWdlRGF0YS5vYmplY3Qucm91bmQuaHRt
bCA9IFRFWFQKIEJVRzQ1OTkxIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5pbWFnZURhdGEub2Jq
ZWN0LndyYXAuaHRtbCA9IFRFWFQKIEJVRzQ1OTkxIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5t
aXNzaW5nYXJncy5odG1sID0gVEVYVAotQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJk
LnBhdGguYXJjVG8uY29pbmNpZGUuMS5odG1sID0gVEVYVAorQlVHNDU5OTEgTElOVVggV0lOIERF
QlVHIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5wYXRoLmFyY1RvLmNvaW5jaWRlLjEuaHRtbCA9
IFRFWFQKIEJVRzQ1OTkxIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5wYXRoLnJlY3Qud2luZGlu
Zy5odG1sID0gVEVYVAotQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLnBhdGguc3Ry
b2tlLnBydW5lLmFyYy5odG1sID0gVEVYVAorQlVHNDU5OTEgV0lOIERFQlVHIDogY2FudmFzL3Bo
aWxpcC90ZXN0cy8yZC5wYXRoLnN0cm9rZS5wcnVuZS5hcmMuaHRtbCA9IFRFWFQKIEJVRzQ1OTkx
IDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5wYXR0ZXJuLmltYWdlLmJyb2tlbi5odG1sID0gVEVY
VAogQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLnBhdHRlcm4uaW1hZ2UuaW5jb21w
bGV0ZS5odG1sID0gVEVYVAogQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLnBhdHRl
cm4uaW1hZ2UudW5kZWZpbmVkLmh0bWwgPSBURVhUCkBAIC0yNTc2LDcgKzI1NzgsOCBAQCBCVUc0
NTk5MSBNQUMgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmdyYWRpZW50LnJhZGlhbC5jb25lLmZy
b250Lmh0bWwgPSBURVhUCiBCVUc0NTk5MSBNQUMgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmdy
YWRpZW50LnJhZGlhbC5pbnNpZGUyLmh0bWwgPSBURVhUCiBCVUc0NTk5MSBNQUMgOiBjYW52YXMv
cGhpbGlwL3Rlc3RzLzJkLmdyYWRpZW50LnJhZGlhbC5pbnNpZGUzLmh0bWwgPSBURVhUCiBCVUc0
NTk5MSBNQUMgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmdyYWRpZW50LnJhZGlhbC5vdXRzaWRl
MS5odG1sID0gVEVYVAotQlVHNDU5OTEgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmltYWdlRGF0
YS5nZXQuc291cmNlLm91dHNpZGUuaHRtbCA9IFRFWFQKK0JVRzQ1OTkxIERFQlVHIDogY2FudmFz
L3BoaWxpcC90ZXN0cy8yZC5pbWFnZURhdGEuZ2V0LnNvdXJjZS5vdXRzaWRlLmh0bWwgPSBURVhU
CitCVUc0NTk5MSBNQUMgUkVMRUFTRSA6IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuaW1hZ2VEYXRh
LmdldC5zb3VyY2Uub3V0c2lkZS5odG1sID0gVEVYVAogQlVHNDU5OTEgTUFDIDogY2FudmFzL3Bo
aWxpcC90ZXN0cy8yZC5saW5lLmNhcC5jbG9zZWQuaHRtbCA9IFRFWFQKIEJVRzQ1OTkxIE1BQyA6
IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQubGluZS5qb2luLnBhcmFsbGVsLmh0bWwgPSBURVhUCiBC
VUc0NTk5MSBNQUMgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJkLmxpbmUubWl0ZXIubGluZWVkZ2Uu
aHRtbCA9IFRFWFQKQEAgLTMxMDAsMyArMzEwMyw5IEBAIEJVR0pBUEhFVCA6IGh0dHAvdGVzdHMv
bmF2aWdhdGlvbi9waW5nLWNyb3NzLW9yaWdpbi5odG1sID0gVEVYVCBUSU1FT1VUCiAKIC8vIFN0
YXJ0ZWQgY3Jhc2hpbmcgYXMgcGFydCBvZiB0aGUgV0sgcm9sbCA2ODEwMDo2ODE3OC4KIEJVRzU2
NzMyIDogc3RvcmFnZS9pbmRleGVkZGIvdHJhbnNhY3Rpb24tYmFzaWNzLmh0bWwgPSBDUkFTSAor
CitCVUdfQVVUTyBMSU5VWCBXSU4gREVCVUcgOiBhbmltYXRpb25zL3N1c3BlbmQtcmVzdW1lLWFu
aW1hdGlvbi5odG1sID0gUEFTUyBURVhUCitCVUdfQVVUTyBMSU5VWCBSRUxFQVNFIDogYW5pbWF0
aW9ucy8zZC90cmFuc2Zvcm0tb3JpZ2luLXZzLWZ1bmN0aW9ucy5odG1sID0gUEFTUyBURVhUCitC
VUdfQVVUTyBXSU4gREVCVUcgOiBhbmltYXRpb25zLzNkL3RyYW5zZm9ybS1vcmlnaW4tdnMtZnVu
Y3Rpb25zLmh0bWwgPSBQQVNTIFRFWFQKK0JVR19BVVRPIFdJTiBERUJVRyA6IGFuaW1hdGlvbnMv
bXVsdGlwbGUta2V5ZnJhbWVzLmh0bWwgPSBQQVNTIFRFWFQKK0JVR19BVVRPIFdJTiBERUJVRyA6
IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQudGV4dC5kcmF3LmFsaWduLmNlbnRlci5odG1sID0gUEFT
UyBURVhUCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>