Bug 106344 – [GTK][WK2] accessibility/placeholder.html is failing

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Zan Dobersek]

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

[Krzysztof Czech]

Created attachment 202792 [details]
Patch

[Mario Sanchez Prada]

Comment on attachment 202792 [details]
Patch

Looks good to me

[Krzysztof Czech]

Created attachment 202806 [details]
Patch

[Krzysztof Czech]

(In reply to comment #3)
> Created an attachment (id=202806) [details]
> Patch

Deleted an empty line. Applied Mario's informal suggestion.

[Gyuyoung Kim]

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 ?

[Krzysztof Czech]

(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.

[Krzysztof Czech]

As soon this patch lands, I'm going to enabled it on efl-wk2.

[Gyuyoung Kim]

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.

[Mario Sanchez Prada]

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.

[WebKit Commit Bot]

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

[Krzysztof Czech]

Created attachment 203571 [details]
Rebased patch

[WebKit Commit Bot]

Comment on attachment 203571 [details]
Rebased patch

Clearing flags on attachment: 203571

Committed r151103: <http://trac.webkit.org/changeset/151103>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.