Bug 93586 - Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.html is failing
Summary: Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.htm...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-09 00:36 PDT by Takashi Toyoshima
Modified: 2012-08-09 23:29 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Takashi Toyoshima 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.
Comment 1 Takashi Toyoshima 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,
Comment 2 Brady Eidson 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.
Comment 3 Brady Eidson 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
Comment 4 Yuta Kitamura 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>
Comment 5 Brady Eidson 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!
Comment 6 Takashi Toyoshima 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.