Set default control on Terms of Service screen

Set default control on Terms of Service screen

This CL addresses the a11y issue that the Terms of Service screen does
not have any focused control by default. The focus is now placed on the
back button by default and shifts to the accept button when the Terms of
Service have finished loading.

BUG=175730
TEST=Manual


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182096

[bartfab (slow), 2013-02-12 15:21:24]

Hi Nikita,

Could you take a look please?

[Nikita (slow), 2013-02-12 20:51:48]

lgtm

https://chromiumcodereview.appspot.com/12221143/diff/1/chrome/browser/resourc...
File chrome/browser/resources/chromeos/login/oobe_screen_terms_of_service.js
(right):

https://chromiumcodereview.appspot.com/12221143/diff/1/chrome/browser/resourc...
chrome/browser/resources/chromeos/login/oobe_screen_terms_of_service.js:52: if
(!$('tos-back-button').blurred)
nit: You could check what's the current document.activeElement is so that you
won't need additional property (blurred).

[bartfab (slow), 2013-02-12 23:22:59]

https://chromiumcodereview.appspot.com/12221143/diff/1/chrome/browser/resourc...
File chrome/browser/resources/chromeos/login/oobe_screen_terms_of_service.js
(right):

https://chromiumcodereview.appspot.com/12221143/diff/1/chrome/browser/resourc...
chrome/browser/resources/chromeos/login/oobe_screen_terms_of_service.js:52: if
(!$('tos-back-button').blurred)
On 2013/02/12 20:51:48, Nikita Kostylev wrote:
> nit: You could check what's the current document.activeElement is so that you
> won't need additional property (blurred).

The property actually is needed to achieve the semantics I am after: I want to
avoid changing the focus if the user has modified the focus manually in any way,
even if that included moving the focus back to the button that was focused
initially. So I do not want to find out whether the default button is focused
right now, I want to find out whether it had ever lost focus in the past.

[commit-bot: I haz the power, 2013-02-12 23:27:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@chromium.org/12221143/1

[commit-bot: I haz the power, 2013-02-13 02:41:44]

Change committed as 182096