Bug 90948

Summary:

[EFL] Move tests that require ENABLE_INPUT_TYPE_* to TestExpectations

Product:

WebKit

Reporter:

Chris Dumez <cdumez>

Component:

WebKit EFL

Assignee:

Chris Dumez <cdumez>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

gyuyoung.kim, lucas.de.marchi, rakuco, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	cdumez: commit-queue-
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none

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	Details Formatted Diff Diff
Patch (2.40 KB, patch) 2012-07-11 00:52 PDT, Chris Dumez	cdumez: commit-queue-	Details Formatted Diff Diff
Patch (2.75 KB, patch) 2012-07-11 00:58 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (2.75 KB, patch) 2012-07-11 01:00 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (2.27 KB, patch) 2012-07-11 01:01 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (2.33 KB, patch) 2012-07-11 01:05 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (2.33 KB, patch) 2012-07-11 01:06 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Show Obsolete (6) View All Add attachment proposed patch, testcase, etc.

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

Created attachment 151633 [details] Patch

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.