Skip over the URL input when the NTP is used for the Home Button.

Skip over the URL input when the NTP is used for the Home Button.

BUG=633915
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0
Cr-Commit-Position: refs/heads/master@{#424745}

[hcarmona, 2016-10-08 00:40:42]

Description was changed from

==========
Skip over the URL input when the NTP is used for the Home Button.

BUG=633915
==========

to

==========
Skip over the URL input when the NTP is used for the Home Button.

BUG=633915
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[hcarmona, 2016-10-08 00:46:49]

The CQ bit was checked by hcarmona@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-08 00:47:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hcarmona, 2016-10-08 00:47:33]

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

[hcarmona, 2016-10-08 00:47:33]

PTAL

[commit-bot: I haz the power, 2016-10-08 01:51:25]

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

[commit-bot: I haz the power, 2016-10-08 01:51:26]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-10-12 06:39:40]

lgtm

https://codereview.chromium.org/2403623002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/controls/settings_input.html (right):

https://codereview.chromium.org/2403623002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/controls/settings_input.html:22:
tabindex$=[[getTabindex_(canTab)]]>
put quotes around this I.e. tabindex="[[...]]"

[hcarmona, 2016-10-12 14:39:04]

The CQ bit was checked by hcarmona@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 14:39:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hcarmona, 2016-10-12 14:40:19]

The CQ bit was unchecked by hcarmona@chromium.org

[hcarmona, 2016-10-12 14:40:19]

The CQ bit was checked by hcarmona@chromium.org

[hcarmona, 2016-10-12 14:40:19]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2403623002/#ps20001
(title: "put quotes and rebase")

[commit-bot: I haz the power, 2016-10-12 14:40:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-12 15:39:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-12 15:41:49]

Description was changed from

==========
Skip over the URL input when the NTP is used for the Home Button.

BUG=633915
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Skip over the URL input when the NTP is used for the Home Button.

BUG=633915
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0
Cr-Commit-Position: refs/heads/master@{#424745}
==========

[commit-bot: I haz the power, 2016-10-12 15:41:50]

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