Currently the blockquote element is exposed via ATK_ROLE_PANEL, which is completely wrong (PANEL is appropriate for a generic container that holds widgets; not a block of text.) ATK_ROLE_BLOCK_QUOTE was added to ATK about one year ago. And for environments with older versions of ATK, the fallback mapping should be the same as that of a div element.
<rdar://problem/18995323>
Created attachment 241671 [details] Patch
Comment on attachment 241671 [details] Patch I'm rather surprised that the EFL EWS claims that this patch doesn't build. There's an ATK version check for the added role. And looking at the associated output from from the failure, I'm not seeing anything that suggests the build failure is due to this patch. But having said that, in the spirit of due diligence, I set up a build environment for WebKitEfl. WebKitEfl builds and runs without my patch. WebKitEfl also builds and runs with my patch applied. <insert shrug here> So I'm going to assume the EWS failure was some fluke or other issue not related to this patch. If the failure really was caused by my patch, feel free to roll it out and tell me how I broke your port so that I can fix it.
Comment on attachment 241671 [details] Patch Rejecting attachment 241671 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 241671, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: FAILED -- saving rejects to file Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp.rej patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/platform/efl/accessibility/roles-exposed-expected.txt patching file LayoutTests/platform/gtk/accessibility/roles-exposed-expected.txt Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Chris Fleizach']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/5847257488293888
Created attachment 241679 [details] Patch
Bah. Was uploading a master-compatible version of the same, reviewed patch. (Because it no longer applies cleanly after subsequent commits I made.) I forgot to use --no-review with webkit-patch. On the bright side, we can see if the EFL EWS still spits up. ;) Chris, since it's probably not kosher for me to r+ my own patch, assuming the EFL bot doesn't spit up again, mind re-r+ing it?
Comment on attachment 241679 [details] Patch Was chatting with Martin Robinson: <mr obinson> Oh, definitely if it's just a rebase to merge with the current code. <mr obinson> I'd just commit in that case. Doing that now.
Comment on attachment 241679 [details] Patch Clearing flags on attachment: 241679 Committed r176162: <http://trac.webkit.org/changeset/176162>
All reviewed patches have been landed. Closing bug.