Bug 31524

Summary: AX: order AccessibilityRole alphabetically
Product: WebKit Reporter: chris fleizach <cfleizach>
Component: AccessibilityAssignee: chris fleizach <cfleizach>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
patch
bdakin: review+
patch to check building (no review necessary, want to check builders)
none
patch to check building (no review necessary)
none
patch to check building (no review) none

chris fleizach
Reported 2009-11-15 17:50:51 PST
we need to re-order AccessibilityRole list to be alphabetically (except UnknownRole which == 0)
Attachments
patch (11.93 KB, patch)
2011-04-21 15:50 PDT, chris fleizach 		bdakin: review+
DetailsFormatted DiffDiff
patch to check building (no review necessary, want to check builders) (12.48 KB, patch)
2011-04-21 17:55 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
patch to check building (no review necessary) (13.56 KB, patch)
2011-04-21 18:57 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
patch to check building (no review) (12.48 KB, patch)
2011-04-21 19:15 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
chris fleizach
Comment 1 2009-12-13 13:43:47 PST
the last ARIA role…
chris fleizach
Comment 2 2011-04-21 15:50:17 PDT
Created attachment 90620 [details] patch
WebKit Review Bot
Comment 3 2011-04-21 16:34:53 PDT
Attachment 90620 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8498138
Beth Dakin
Comment 4 2011-04-21 17:36:24 PDT
Comment on attachment 90620 [details] patch You should look into the linux-builder failure though.
chris fleizach
Comment 5 2011-04-21 17:55:59 PDT
Created attachment 90644 [details] patch to check building (no review necessary, want to check builders)
WebKit Review Bot
Comment 6 2011-04-21 17:59:21 PDT
Attachment 90620 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8486592
chris fleizach
Comment 7 2011-04-21 17:59:44 PDT
Comment on attachment 90644 [details] patch to check building (no review necessary, want to check builders) no reviewnecessary
WebKit Review Bot
Comment 8 2011-04-21 18:30:19 PDT
Attachment 90644 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8495247
chris fleizach
Comment 9 2011-04-21 18:57:41 PDT
Created attachment 90651 [details] patch to check building (no review necessary)
WebKit Review Bot
Comment 10 2011-04-21 19:08:52 PDT
Attachment 90651 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8493322
chris fleizach
Comment 11 2011-04-21 19:15:30 PDT
Created attachment 90653 [details] patch to check building (no review)
chris fleizach
Comment 12 2011-04-21 20:14:01 PDT
http://trac.webkit.org/changeset/84593
Eric Seidel (no email)
Comment 13 2011-04-25 21:37:06 PDT
Comment on attachment 90653 [details] patch to check building (no review) Cleared review? from attachment 90653 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).
Note You need to log in before you can comment on or make changes to this bug.