RESOLVED FIXED Bug 90948
[EFL] Move tests that require ENABLE_INPUT_TYPE_* to TestExpectations
https://bugs.webkit.org/show_bug.cgi?id=90948
Summary [EFL] Move tests that require ENABLE_INPUT_TYPE_* to TestExpectations
Chris Dumez
Reported 2012-07-11 00:47:24 PDT
The fast/forms/input-in-table-cell-no-value.html introduced in r122290 is failing on EFL port because ENABLE_INPUT_TYPE_DATE flag is not enabled on EFL port.
Attachments
Patch (2.40 KB, patch)
2012-07-11 00:49 PDT, Chris Dumez
no flags
Patch (2.40 KB, patch)
2012-07-11 00:52 PDT, Chris Dumez
cdumez: commit-queue-
Patch (2.75 KB, patch)
2012-07-11 00:58 PDT, Chris Dumez
no flags
Patch (2.75 KB, patch)
2012-07-11 01:00 PDT, Chris Dumez
no flags
Patch (2.27 KB, patch)
2012-07-11 01:01 PDT, Chris Dumez
no flags
Patch (2.33 KB, patch)
2012-07-11 01:05 PDT, Chris Dumez
no flags
Patch (2.33 KB, patch)
2012-07-11 01:06 PDT, Chris Dumez
no flags
Chris Dumez
Comment 1 2012-07-11 00:49:38 PDT
Created attachment 151626 [details] Patch Could someone please cq+ ?
Chris Dumez
Comment 2 2012-07-11 00:52:06 PDT
Created attachment 151629 [details] Patch Fix bug number.
Chris Dumez
Comment 3 2012-07-11 00:55:21 PDT
Just received feedback from Bug 89209 that the date input might not be the issue for this failure after all. Marking cq- until we have a better understanding.
Chris Dumez
Comment 4 2012-07-11 00:58:42 PDT
Created attachment 151631 [details] Patch Move only the other tests to TestExpectations for now.
Chris Dumez
Comment 5 2012-07-11 01:00:06 PDT
Created attachment 151632 [details] Patch Fix typo.
Chris Dumez
Comment 6 2012-07-11 01:01:13 PDT
Chris Dumez
Comment 7 2012-07-11 01:05:00 PDT
Created attachment 151635 [details] Patch Ok, now it was agreed in Bug 89209 that the new test should be skipped as well.
Chris Dumez
Comment 8 2012-07-11 01:06:18 PDT
WebKit Review Bot
Comment 9 2012-07-11 02:07:14 PDT
Comment on attachment 151638 [details] Patch Clearing flags on attachment: 151638 Committed r122314: <http://trac.webkit.org/changeset/122314>
WebKit Review Bot
Comment 10 2012-07-11 02:07:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.