WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 76549
Bug 76531
[GTK] platform/gtk/accessibility/combo-box-collapsed-selection-changed.html is failing after
r105253
https://bugs.webkit.org/show_bug.cgi?id=76531
Summary
[GTK] platform/gtk/accessibility/combo-box-collapsed-selection-changed.html i...
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
Details
Formatted Diff
Diff
Patch proposal
(6.35 KB, patch)
2012-01-19 04:39 PST
,
Mario Sanchez Prada
no flags
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug