Bug 89224

Summary: [WK2][GTK] Complete implementation of AccessibilityController in WKTR for GTK
Product: WebKit Reporter: Mario Sanchez Prada <mario>
Component: AccessibilityAssignee: Mario Sanchez Prada <mario>
Status: RESOLVED FIXED    
Severity: Normal CC: cfleizach, cgarcia
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 89223    
Bug Blocks: 72588    
Attachments:
Description Flags
Patch proposal
none
Patch proposal cfleizach: review+

Mario Sanchez Prada
Reported 2012-06-15 09:30:11 PDT
In order to be able to unskip accessibility tests in WK2 like the following ones: platform/gtk/accessibility/aria-slider-required-attributes.html platform/gtk/accessibility/combo-box-collapsed-selection-changed.html ...we need to implement the following functions in WKTR's AccessibilityController for GTK: void AccessibilityController::logAccessibilityEvents() void AccessibilityController::resetToConsistentState() This bug is for that.
Attachments
Patch proposal (16.62 KB, patch)
2012-06-15 10:05 PDT, Mario Sanchez Prada
no flags
Patch proposal (14.68 KB, patch)
2012-07-18 03:29 PDT, Mario Sanchez Prada
cfleizach: review+
Mario Sanchez Prada
Comment 1 2012-06-15 10:05:51 PDT
Created attachment 147844 [details] Patch proposal
chris fleizach
Comment 2 2012-07-06 09:49:16 PDT
Comment on attachment 147844 [details] Patch proposal try to get patch to build again, i'll review
chris fleizach
Comment 3 2012-07-14 01:11:43 PDT
Comment on attachment 147844 [details] Patch proposal need to get building on the bots
Mario Sanchez Prada
Comment 4 2012-07-18 03:29:26 PDT
Created attachment 152977 [details] Patch proposal New patch now that patch for bug 89223 has been pushed
chris fleizach
Comment 5 2012-07-18 09:28:51 PDT
Comment on attachment 152977 [details] Patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=152977&action=review looks ok > Tools/WebKitTestRunner/InjectedBundle/gtk/AccessibilityControllerGtk.cpp:14 > + * * Neither the name of Google Inc. nor the names of its you probably don't need to include the Google copyright info
Mario Sanchez Prada
Comment 6 2012-07-19 02:04:23 PDT
Note You need to log in before you can comment on or make changes to this bug.