Add a disabled property to DOMUI lists

Add a disabled property to DOMUI lists

Also restricts the extra highlighting of options page list element to just the lead element, since that code was changing anyway to handle disabling.

BUG=60838
TEST=Startup page list in DOMUI options should look and act disable when the corresponding option is not selected.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=68389

[arv (Not doing code reviews), 2010-12-04 01:03:27]

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/option...
File chrome/browser/resources/options/options_page.css (right):

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/option...
chrome/browser/resources/options/options_page.css:370: list.disabled >
[lead][selected],
Use disabled attribute

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
File chrome/browser/resources/shared/js/cr/ui/list.js (right):

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
chrome/browser/resources/shared/js/cr/ui/list.js:104: disabled_: false,
cr.defineProperty(List, 'disabled', cr.PropertyKind.BOOL_ATTR);

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
chrome/browser/resources/shared/js/cr/ui/list.js:310: if (this.disabled)
Do this before "var target"

[stuartmorgan, 2010-12-04 01:36:51]

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/option...
File chrome/browser/resources/options/options_page.css (right):

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/option...
chrome/browser/resources/options/options_page.css:370: list.disabled >
[lead][selected],
On 2010/12/04 01:03:27, arv wrote:
> Use disabled attribute

Done.

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
File chrome/browser/resources/shared/js/cr/ui/list.js (right):

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
chrome/browser/resources/shared/js/cr/ui/list.js:104: disabled_: false,
On 2010/12/04 01:03:27, arv wrote:
> cr.defineProperty(List, 'disabled', cr.PropertyKind.BOOL_ATTR);

Ah-ha, this is the magic I was missing; I originally tried to do it as a
property but since it wasn't a real property CSS ignored it, and I didn't
realize I could declare it to make it work. Thanks for the pointer!

http://codereview.chromium.org/5608003/diff/1/chrome/browser/resources/shared...
chrome/browser/resources/shared/js/cr/ui/list.js:310: if (this.disabled)
On 2010/12/04 01:03:27, arv wrote:
> Do this before "var target"

Done.

[arv (Not doing code reviews), 2010-12-06 21:31:44]

LGTM