dom-ui options: Fix incorrect handling of undefined values in pref UI....

dom-ui options: Fix incorrect handling of undefined values in pref UI.

BUG=none
TEST=Verify preference UI works properly in dom-ui options window.

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

[csilv, 2010-08-25 00:06:54]

+dhg for review.

Dunno *what* i was thinking w/ my previous CL.  This one fixes it properly.

[dhg, 2010-08-25 00:08:50]

LGTM

On Tue, Aug 24, 2010 at 5:06 PM,  <csilv@chromium.org> wrote:
> Reviewers: dhg,
>
> Message:
> +dhg for review.
>
> Dunno *what* i was thinking w/ my previous CL.  This one fixes it properly.
>
> Description:
> dom-ui options: Fix incorrect handling of undefined values in pref UI.
>
> BUG=none
> TEST=Verify preference UI works properly in dom-ui options window.
>
> Please review this at http://codereview.chromium.org/3170042/show
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     chrome/browser/resources/options/pref_ui.js
>
>
> Index: chrome/browser/resources/options/pref_ui.js
> ===================================================================
> --- chrome/browser/resources/options/pref_ui.js (revision 57240)
> +++ chrome/browser/resources/options/pref_ui.js (working copy)
> @@ -25,10 +25,10 @@
>       // Listen to pref changes.
>       Preferences.getInstance().addEventListener(this.pref,
>           function(event) {
> -            self.checked = event.value['value'] ? event.value['value']
> -                                                : event.value;
> -            self.managed = event.value['managed'] ? event.value['managed']
> -                                                  : false;
> +            self.checked = event.value['value'] != undefined ?
> +                event.value['value'] : event.value;
> +            self.managed = event.value['managed'] != undefined ?
> +                event.value['managed'] : false;
>             self.disabled = self.managed;
>           });
>
> @@ -72,10 +72,10 @@
>       // Listen to pref changes.
>       Preferences.getInstance().addEventListener(this.pref,
>           function(event) {
> -            var value = event.value['value'] ? event.value['value']
> -                                             : event.value;
> -            self.managed = event.value['managed'] ? event.value['managed']
> -                                                  : false;
> +            var value = event.value['value'] != undefined ?
> +                event.value['value'] : event.value;
> +            self.managed = event.value['managed'] != undefined ?
> +                event.value['managed'] : false;
>             self.checked = String(value) == self.value;
>             self.disabled = self.managed;
>           });
> @@ -132,10 +132,10 @@
>       // Listen to pref changes.
>       Preferences.getInstance().addEventListener(this.pref,
>           function(event) {
> -            self.value = event.value['value'] ? event.value['value']
> -                                              : event.value;
> -            self.managed = event.value['managed'] ? event.value['managed']
> -                                                  : false;
> +            self.value = event.value['value'] != undefined ?
> +                event.value['value'] : event.value;
> +            self.managed = event.value['managed'] != undefined ?
> +                event.value['managed'] : false;
>             self.disabled = self.managed;
>           });
>
> @@ -215,10 +215,10 @@
>       // Listen to pref changes.
>       Preferences.getInstance().addEventListener(this.pref,
>           function(event) {
> -            var value = event.value['value'] ? event.value['value']
> -                                             : event.value;
> -            self.managed = event.value['managed'] ? event.value['managed']
> -                                                  : false;
> +            var value = event.value['value'] != undefined ?
> +                event.value['value'] : event.value;
> +            self.managed = event.value['managed'] != undefined ?
> +                event.value['managed'] : false;
>             self.disabled = self.managed;
>             for (var i = 0; i < self.options.length; i++) {
>               if (self.options[i].value == value) {
> @@ -307,10 +307,10 @@
>       // Listen to pref changes.
>       Preferences.getInstance().addEventListener(this.pref,
>           function(event) {
> -            self.value = event.value['value'] ? event.value['value']
> -                                              : event.value;
> -            self.managed = event.value['managed'] ? event.value['managed']
> -                                                  : false;
> +            self.value = event.value['value'] != undefined ?
> +                event.value['value'] : event.value;
> +            self.managed = event.value['managed'] != undefined ?
> +                event.value['managed'] : false;
>             self.disabled = self.managed;
>           });
>
>
>
>