The accessibility/placeholder.html layout test is failing on GTK WK2 port. Here's the diff: --- /home/zan/Dev/webkit/webkit/WebKitBuild/Release/layout-test-results/accessibility/placeholder-expected.txt +++ /home/zan/Dev/webkit/webkit/WebKitBuild/Release/layout-test-results/accessibility/placeholder-actual.txt @@ -4,8 +4,8 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS fieldElement.stringAttributeValue('AXPlaceholderValue') is 'search' -PASS pass.stringAttributeValue('AXPlaceholderValue') is 'Password' +FAIL fieldElement.stringAttributeValue('AXPlaceholderValue') should be search. Was . +FAIL pass.stringAttributeValue('AXPlaceholderValue') should be Password. Was . PASS successfullyParsed is true TEST COMPLETE
Created attachment 202792 [details] Patch
Comment on attachment 202792 [details] Patch Looks good to me
Created attachment 202806 [details] Patch
(In reply to comment #3) > Created an attachment (id=202806) [details] > Patch Deleted an empty line. Applied Mario's informal suggestion.
Comment on attachment 202806 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=202806&action=review > LayoutTests/platform/gtk-wk2/TestExpectations:-307 > -webkit.org/b/106344 accessibility/placeholder.html [ Failure ] Can't we unskip this on efl-wk2 as well ?
(In reply to comment #5) > (From update of attachment 202806 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=202806&action=review > > > LayoutTests/platform/gtk-wk2/TestExpectations:-307 > > -webkit.org/b/106344 accessibility/placeholder.html [ Failure ] > > Can't we unskip this on efl-wk2 as well ? Yes we can, but there is a separate bug (112031) regarding this test on efl-wk2.
As soon this patch lands, I'm going to enabled it on efl-wk2.
Comment on attachment 202806 [details] Patch rs=me according to GTK committer's informal review. However, it would be good if this patch is landed by GTK committer/reviewer.
Comment on attachment 202806 [details] Patch (In reply to comment #8) > (From update of attachment 202806 [details]) > rs=me according to GTK committer's informal review. However, it would be good if this patch is landed by GTK committer/reviewer. I'm fine with this, so setting cq+ now Btw, I just realized we might like to use atk_attribute_set_free() (and probably just a raw pointer) for freeing the memory allocated by atk_object_get_attributes, instead of the method attributesClear(), but I'd rather leave that for a different commit, if others agree on that too.
Comment on attachment 202806 [details] Patch Rejecting attachment 202806 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 202806, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: h fuzz 3. patching file LayoutTests/platform/gtk-wk2/TestExpectations Hunk #1 FAILED at 304. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/gtk-wk2/TestExpectations.rej patching file Tools/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Gyuyoung Kim']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/692618
Created attachment 203571 [details] Rebased patch
Comment on attachment 203571 [details] Rebased patch Clearing flags on attachment: 203571 Committed r151103: <http://trac.webkit.org/changeset/151103>
All reviewed patches have been landed. Closing bug.