RESOLVED FIXED 93586
Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.html is failing
https://bugs.webkit.org/show_bug.cgi?id=93586
Summary Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.htm...
Takashi Toyoshima
Reported 2012-08-09 00:36:03 PDT
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.
Attachments
Takashi Toyoshima
Comment 1 2012-08-09 00:50:05 PDT
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,
Brady Eidson
Comment 2 2012-08-09 10:09:25 PDT
(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.
Brady Eidson
Comment 3 2012-08-09 10:16:27 PDT
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
Yuta Kitamura
Comment 4 2012-08-09 19:50:38 PDT
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>
Brady Eidson
Comment 5 2012-08-09 22:28:35 PDT
(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!
Takashi Toyoshima
Comment 6 2012-08-09 23:29:02 PDT
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.
Note You need to log in before you can comment on or make changes to this bug.