Bug 76531

Summary: [GTK] platform/gtk/accessibility/combo-box-collapsed-selection-changed.html is failing after r105253
Product: WebKit Reporter: Mario Sanchez Prada <mario>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: pnormand, tkent, webkit.review.bot
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 76620    
Bug Blocks: 25531    
Attachments:
Description Flags
Patch proposal
none
Patch proposal none

Mario Sanchez Prada
Reported 2012-01-18 04:56:32 PST
This test has started failing after r105253, which made the 'selected' and 'focused' events to be emitted anymore for AtkObjects representing list boxes and menu lists. As the failure is localized in the GTK port only, I think it's sensible to skip the test for now, while working on a real patch that gets everything working fine in that port back again (see original bug 53146)
Attachments
Patch proposal (6.29 KB, patch)
2012-01-19 04:38 PST, Mario Sanchez Prada 		no flags
DetailsFormatted DiffDiff
Patch proposal (6.35 KB, patch)
2012-01-19 04:39 PST, Mario Sanchez Prada 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mario Sanchez Prada
Comment 1 2012-01-18 04:59:24 PST
This blocks basic a11y support
Mario Sanchez Prada
Comment 2 2012-01-19 04:38:02 PST
Created attachment 123105 [details] Patch proposal
Mario Sanchez Prada
Comment 3 2012-01-19 04:39:51 PST
Created attachment 123106 [details] Patch proposal Sorry, this one is the good one.
Martin Robinson
Comment 4 2012-01-19 09:29:01 PST
Comment on attachment 123106 [details] Patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=123106&action=review > Source/WebCore/html/HTMLSelectElement.cpp:978 > -#if !PLATFORM(WIN) || OS(WINCE) > +#if !PLATFORM(GTK) && !PLATFORM(WIN) || OS(WINCE) Couldn't you achieve the same effect by returning something different from popsMenuByArrowKeys?
Kent Tamura
Comment 5 2012-01-19 16:16:44 PST
Wasn't it fixed by http://trac.webkit.org/changeset/105286 ?
Mario Sanchez Prada
Comment 6 2012-01-20 03:40:20 PST
(In reply to comment #5) > Wasn't it fixed by http://trac.webkit.org/changeset/105286 ? You are right. I didn't realize of that commit, which actually looks like a better fix than this one. So, I'm now resolving this one as duplicate, and soon pushing an unreviewed patch to unskip this test. Thanks! *** This bug has been marked as a duplicate of bug 76549 ***
Note You need to log in before you can comment on or make changes to this bug.