Launcher Search box, Back and Mic buttons now focusable in ChromeVox.

Launcher Search box, Back and Mic buttons now focusable in ChromeVox.

Back and Mic buttons now have accessible names. Also added a tooltip to
Back button. Added some special logic to avoid focusing the Search box
in the search results page (because of the dual focus; it is more
important to read out the result than the search box).

BUG=458885, 459859
Committed: https://crrev.com/6213d0961fc7dfe97873706342ff67107d74494c
Cr-Commit-Position: refs/heads/master@{#317287}

[Matt Giuca, 2015-02-19 06:51:01]

Patchset #2 (id:20001) has been deleted

[Matt Giuca, 2015-02-19 06:55:56]

mgiuca@chromium.org changed reviewers:
+ calamity@chromium.org

[calamity, 2015-02-20 03:45:25]

https://codereview.chromium.org/935293002/diff/40001/ui/app_list/views/search...
File ui/app_list/views/search_box_view.h (right):

https://codereview.chromium.org/935293002/diff/40001/ui/app_list/views/search...
ui/app_list/views/search_box_view.h:74: void set_contents_view(ContentsView*
contents_view) {
This will break tests.

[Matt Giuca, 2015-02-20 05:15:17]

https://codereview.chromium.org/935293002/diff/40001/ui/app_list/views/search...
File ui/app_list/views/search_box_view.h (right):

https://codereview.chromium.org/935293002/diff/40001/ui/app_list/views/search...
ui/app_list/views/search_box_view.h:74: void set_contents_view(ContentsView*
contents_view) {
Fixed. (As discussed, we just have to use text.empty() rather than more properly
looking at the ContentsView state.)

[calamity, 2015-02-20 05:51:53]

lgtm

[Matt Giuca, 2015-02-20 06:06:39]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2015-02-20 06:07:05]

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

[Matt Giuca, 2015-02-20 08:15:40]

The CQ bit was unchecked by mgiuca@chromium.org

[Matt Giuca, 2015-02-20 08:16:25]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2015-02-20 08:17:06]

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

[Matt Giuca, 2015-02-20 10:43:08]

The CQ bit was unchecked by mgiuca@chromium.org

[Matt Giuca, 2015-02-20 10:56:34]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2015-02-20 10:57:12]

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

[commit-bot: I haz the power, 2015-02-20 11:12:47]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-02-20 11:13:13]

Patchset 3 (id:??) landed as
https://crrev.com/6213d0961fc7dfe97873706342ff67107d74494c
Cr-Commit-Position: refs/heads/master@{#317287}