93586 2012-08-09 00:36:03 -0700 Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.html is failing 2012-08-09 23:29:02 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 toyoshim webkit-unassigned beidson tkent yutak oldest_to_newest 691298 0 toyoshim 2012-08-09 00:36:03 -0700 The following layout test is timedout on all chromium ports fast/forms/autocomplete-off-with-default-value-does-not-clear.html This test is added by https://bugs.webkit.org/show_bug.cgi?id=93544 And patch and test looks no problem. Is there missing DRT feature to run this test? I tried this test with long timeout argument in local machine, but it didn't finish. 691306 1 toyoshim 2012-08-09 00:50:05 -0700 Update extension for now. http://trac.webkit.org/changeset/125158 Brady, Do you have any hint on this timeout? Kent, If you know the right person to handle this issue, please ping the person. Thanks, 691533 2 beidson 2012-08-09 10:09:25 -0700 (In reply to comment #1) > Update extension for now. > http://trac.webkit.org/changeset/125158 > > Brady, > Do you have any hint on this timeout? Yup, simple - The test will only work with the page cache enabled and functioning, and I think that can't happen with chromium. This was an oversight on my part, I apologize. We should skip it for all ports that don't fit in to that description. 691544 3 beidson 2012-08-09 10:16:27 -0700 There was already a section of the chromium TestExpectations dedicated to "doesn't support page cache", so I added it there in http://trac.webkit.org/changeset/125182 692126 4 yutak 2012-08-09 19:50:38 -0700 There were two duplicate lines in TestExpectations (one with SKIP, the other without). I removed the latter to fix a lint error: <http://trac.webkit.org/changeset/125241> 692216 5 beidson 2012-08-09 22:28:35 -0700 (In reply to comment #4) > There were two duplicate lines in TestExpectations (one with SKIP, the other without). I removed the latter to fix a lint error: > > <http://trac.webkit.org/changeset/125241> Ah, didn't know that's what "update extension for now." meant. Sorry about that... Better that it's in the "doesn't support page cache" block for now, anyways! 692245 6 toyoshim 2012-08-09 23:29:02 -0700 Brady, and Yuta, Thank you for handling this. > Ah, didn't know that's what "update extension for now." meant. Sorry about that... Oops... sorry, I made you confused. Yes, it was typo. I meant update TestExpectations.