Remove InlineEditableItemList from tab order.

Remove InlineEditableItemList from tab order.

Remove InlineEditableItemList (but not the InlineEditableItem items
inside of it) from the tab order. Giving the list focus is useless, so
skip it and go straight to focusing the first item.

Improves "Manage search engines" dialog by making focus go directly to
the first item in each list.

Tested with all other InlineEditableItemList subclasses:
autofill_options_list.js
content_settings_exception_area.js
browser_options_startup_page_list.js
language_dictionary_overlay_world_list.js

This change does not negatively impact any of these classes.

Depends on https://codereview.chromium.org/666823003 to function properly.
BUG=276874
Committed: https://crrev.com/4fc8dad0d6d9bf818e96ac19fbcad54373dd898e
Cr-Commit-Position: refs/heads/master@{#300678}

[bondd, 2014-10-21 01:08:58]

bondd@chromium.org changed reviewers:
+ dbeam@chromium.org

[bondd, 2014-10-21 01:08:59]

bondd@chromium.org changed required reviewers:
+ dbeam@chromium.org

[Dan Beam, 2014-10-21 17:57:55]

dbeam@chromium.org changed reviewers:
+ hcarmona@chromium.org

[Dan Beam, 2014-10-21 17:58:18]

hcarmona@: are you OK with this approach? (you added the tabindex to lists
themselves originally)

[hcarmona, 2014-10-21 18:40:11]

On 2014/10/21 17:58:18, Dan Beam wrote:
> hcarmona@: are you OK with this approach? (you added the tabindex to lists
> themselves originally)

A tabindex of -1 was added to prevent the close button from being tab focus-able
in lists where there are no other buttons (like chrome://settings/languages).
LGTM if the LanguageOptionsWebUITest.TestListTabAccessibility test still passes.

[Dan Beam, 2014-10-21 19:04:26]

lgtm

[bondd, 2014-10-21 19:05:10]

The CQ bit was checked by bondd@chromium.org

[bondd, 2014-10-21 19:05:18]

The CQ bit was unchecked by bondd@chromium.org

[bondd, 2014-10-21 19:05:29]

The CQ bit was checked by bondd@chromium.org

[bondd, 2014-10-21 19:05:45]

The CQ bit was unchecked by bondd@chromium.org

[bondd, 2014-10-21 19:13:01]

The CQ bit was checked by bondd@chromium.org

[bondd, 2014-10-21 19:13:06]

The CQ bit was unchecked by bondd@chromium.org

[bondd, 2014-10-21 19:14:30]

The CQ bit was checked by bondd@chromium.org

[commit-bot: I haz the power, 2014-10-21 19:16:29]

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

[commit-bot: I haz the power, 2014-10-21 20:19:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-21 20:19:44]

Try jobs failed on following builders:
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bondd, 2014-10-22 14:48:42]

The CQ bit was checked by bondd@chromium.org

[commit-bot: I haz the power, 2014-10-22 14:49:26]

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

[commit-bot: I haz the power, 2014-10-22 14:51:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-22 14:51:48]

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