Summary: | REGRESSION(r172826) [GTK] [CSS] 4 Layout Tests at fast/forms/*placeholder* and fast/css/placeholder-shown-basics.html fail | ||
---|---|---|---|
Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> |
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | benjamin, cgarcia, dtrebbien, jfernandez, koivisto, rego |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Carlos Alberto Lopez Perez
2014-08-26 10:14:44 PDT
There is another failure related to this bug. r172826 added a new layout test: fast/css/placeholder-shown-basics.html. This test is failing on platform GTK since it was added. The diff is: http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r172826%20%282346%29/fast/css/placeholder-shown-basics-diffs.html Some of it may just need a rebaseline. After r172826, the placeholder no longer gets a RenderObject when not visible. placeholder-shown-basics.html does not look right. Can you please debug it? It looks like the <textarea> gets a placeholder but its isPlaceholderVisible() is false. (In reply to comment #2) > Some of it may just need a rebaseline. After r172826, the placeholder no longer gets a RenderObject when not visible. > > placeholder-shown-basics.html does not look right. > > Can you please debug it? It looks like the <textarea> gets a placeholder but its isPlaceholderVisible() is false. Checking this manually. The last field is color, not <textarea>. It looks like the problem is simply that WebKitGTK does not support the color type. If you want we can remove it from the test, it is not very important. Can you test the other 4 manually? They may just need a rebase. I have marked all this tests as failing on the GTK TestExpectations on http://trac.webkit.org/r172981 (In reply to comment #3) > (In reply to comment #2) > > Some of it may just need a rebaseline. After r172826, the placeholder no longer gets a RenderObject when not visible. > > > > placeholder-shown-basics.html does not look right. > > > > Can you please debug it? It looks like the <textarea> gets a placeholder but its isPlaceholderVisible() is false. > > Checking this manually. The last field is color, not <textarea>. > > It looks like the problem is simply that WebKitGTK does not support the color type. If you want we can remove it from the test, it is not very important. Yes, GTK still don't supports that (bug 98935). Given that there are already a bunch of tests for input type color at fast/forms/color maybe removing that from this test can be a good idea so ports that still that don't implement the input color type can pass the test. > > Can you test the other 4 manually? They may just need a rebase. How can I test them for correctness? any guidelines? Thanks. (In reply to comment #4) > I have marked all this tests as failing on the GTK TestExpectations on http://trac.webkit.org/r172981 Updated the :placeholder-shown part in http://trac.webkit.org/changeset/173042 (In reply to comment #4) > > Can you test the other 4 manually? They may just need a rebase. > > How can I test them for correctness? any guidelines? > > Thanks. There is a flag defining if the placeholder should be displayed when an input field is focused (shouldShowPlaceholderWhenFocused). I believe it is false for GTK, and the placeholder should be hidden if the input field is focused. With that assumption: On every one of those tests, the input field should be focused, and the placeholder should not be displayed. Typing text should not show the placeholder. Emptying the input field and removing the focus should show the placeholder. From the test results you linked I think it is all good. If you confirm manually, you can just rebaseline those 4 results. On a completely unrelated: what is the native behavior of GTK for placeholder? Does it also hide on focus? If not we could unify and simplify the code. |