Summary: | Web Inspector: AX: Console log of the Inspector does not announce output for screen readers | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Victor Tsaran <vtsaran> | ||||||||||
Component: | Web Inspector | Assignee: | James Craig <jcraig> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | commit-queue, graouts, jcraig, joepeck, timothy, vtsaran, webkit-bug-importer | ||||||||||
Priority: | P3 | Keywords: | InRadar | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Mac (Intel) | ||||||||||||
OS: | OS X 10.8 | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 118106 | ||||||||||||
Attachments: |
|
Description
Victor Tsaran
2013-05-11 19:33:05 PDT
Created attachment 205535 [details]
patch
Comment on attachment 205535 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=205535&action=review > Source/WebInspectorUI/UserInterface/LogContentView.js:38 > + this.messagesElement.setAttribute('role', 'log'); Should always use double quotes. > Source/WebInspectorUI/UserInterface/ConsoleCommandResult.js:49 > + element.setAttribute('data-labelprefix', WebInspector.UIString("Output: ")); Should always use double quotes. Created attachment 205563 [details]
patch with review feedback
Comment on attachment 205563 [details] patch with review feedback Rejecting attachment 205563 [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-01', 'apply-attachment', '--no-update', '--non-interactive', 205563, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: /ConsoleCommand.js |=================================================================== |--- UserInterface/ConsoleCommand.js (revision 152003) |+++ UserInterface/ConsoleCommand.js (working copy) -------------------------- No file to patch. Skipping patch. 1 out of 1 hunk ignored patching file ChangeLog Hunk #1 succeeded at 1 with fuzz 3. Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Timothy Hatcher']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/977513 Created attachment 205630 [details]
patch with corrected paths
previous patch was generated from a sub-directory, so the paths were wrong
Comment on attachment 205630 [details] patch with corrected paths Rejecting attachment 205630 [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', 205630, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: his was: -------------------------- |Index: UserInterface/LogContentView.css |=================================================================== |--- UserInterface/LogContentView.css (revision 152121) |+++ UserInterface/LogContentView.css (working copy) -------------------------- No file to patch. Skipping patch. 1 out of 1 hunk ignored Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Timothy Hatcher']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/899331 Created attachment 205632 [details]
patch with corrected paths (for real this time)
Heh. This must be why there is a dedicated webkit-patch script. ;-) Comment on attachment 205632 [details] patch with corrected paths (for real this time) Clearing flags on attachment: 205632 Committed r152136: <http://trac.webkit.org/changeset/152136> All reviewed patches have been landed. Closing bug. |