Summary: | [WK2][GTK] Implement AccessibilityUIElement in WKTR for GTK | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mario Sanchez Prada <mario> | ||||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cfleizach, cgarcia | ||||||||
Priority: | P2 | Keywords: | Gtk | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 89226 | ||||||||||
Bug Blocks: | 72588, 89224 | ||||||||||
Attachments: |
|
Description
Mario Sanchez Prada
2012-06-15 09:25:52 PDT
Created attachment 147842 [details]
Patch proposal
The idea with this patch is to provide basic support for accessibility in GTK to WKTR, and it certainly does that, since almost 100 a11y tests are now passing...
However, some others are not passing yet and some are even crashing. Those will be addressed later in separate bugs.
Created attachment 147853 [details]
Patch proposal
Ooops! Sorry, this is the right one (some tests to be skipped missing in the previous patch)
Created attachment 148083 [details] Patch proposal New patch. Implementing AccessibilityUIElements using AtkValue interface in a better way (see Martin's comments for bug 89226) and adding two more tests that are randomly crashing to the Skipped file. Comment on attachment 148083 [details] Patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=148083&action=review looks good. a few minor nits > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:181 > +{ seems like you could probably implement this method pretty easily > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:256 > + AtkObject* parent = atk_object_get_parent(ATK_OBJECT(m_element)); extra white space after = sign > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:863 > + // FIXME: implement two FIXME's here > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:869 > + // FIXME: implement two FIXMEs here Thanks Chris for reviewing this, just got back from holidays today, so please understand my late reply. (In reply to comment #4) > (From update of attachment 148083 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=148083&action=review > > looks good. a few minor nits > > > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:181 > > +{ > > seems like you could probably implement this method pretty easily Indeed. I just haven't done it so since I was more worried at this point about porting DRT stuff to WKTR rather than adding new stuff. Still, I agree implementation for this one is very simple, so I just added it. > > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:256 > > + AtkObject* parent = atk_object_get_parent(ATK_OBJECT(m_element)); > > extra white space after = sign For some reason check-webkit-style didn't get this one. Thanks for pointing it out. > > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:863 > > + // FIXME: implement > > two FIXME's here > > > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityUIElementGtk.cpp:869 > > + // FIXME: implement > > two FIXMEs here Both fixed Committed r122940: <http://trac.webkit.org/changeset/122940> |