Made the revoke buttons in the App Info dialog tab-navigable

Made the revoke buttons in the App Info dialog tab-navigable

Added alt-text and focusability to the revoke buttons in the App Info
dialog so they can be navigated to with the keyboard. This allows
permissions to be revoked for apps for users who are navigating with a
screen reader or only using the keyboard.

BUG=425311
Committed: https://crrev.com/15c2942b68a2e52c881c8e45aa8e0fbe52157d3f
Cr-Commit-Position: refs/heads/master@{#302405}

[sashab, 2014-10-30 04:45:13]

sashab@chromium.org changed reviewers:
+ benwells@chromium.org

[benwells, 2014-10-30 21:49:07]

https://codereview.chromium.org/689863002/diff/1/chrome/app/generated_resourc...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/689863002/diff/1/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:2245: +        Revoke this permission
Will this help someone using a screen reader? Should it be Revoke %1 and we fill
in the permission?

[sashab, 2014-10-31 02:10:25]

https://codereview.chromium.org/689863002/diff/1/chrome/app/generated_resourc...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/689863002/diff/1/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:2245: +        Revoke this permission
On 2014/10/30 21:49:06, benwells wrote:
> Will this help someone using a screen reader? Should it be Revoke %1 and we
fill
> in the permission?

It's a good point. Originally I got ack on just this message, but confirmed with
dmazzoni and apparently ChromeVox doesn't support reading static text in native
dialogs yet, so all alt-text should contain any actual static text they need.
Updated it to 'Revoke the permission <blah>'.

[benwells, 2014-11-02 21:08:46]

lgtm

[sashab, 2014-11-02 21:53:17]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2014-11-02 21:54:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/689863002/20001

[commit-bot: I haz the power, 2014-11-02 22:35:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-02 22:36:04]

Patchset 2 (id:??) landed as
https://crrev.com/15c2942b68a2e52c881c8e45aa8e0fbe52157d3f
Cr-Commit-Position: refs/heads/master@{#302405}