Bug 2020683 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Right now, `default:` is a valid Gecko preference experiment stanza, e.g.:

```
  gecko-nimbus-validation:
    description: "Feature for validating Nimbus Gecko functionality"
    variables:
      test-preference:
        description: "A test gecko preference"
        type: String
        default: ""
        gecko-pref:
          pref: "gecko.nimbus.test"
          branch: "default"

```

We should remove `default` as a valid element for now because:
1. Gecko preferences should have their default value set Gecko side
2. Desktop does not support this stanza
3. We do not have a mechanism to capture and set default values

This could be a possible enhancement or change in the future, but for right now, it makes the most sense to prevent it to prevent confusion.
Revision 1 by
on 
Right now, `default:` is a valid Gecko preference experiment stanza in Android, e.g.:

```
  gecko-nimbus-validation:
    description: "Feature for validating Nimbus Gecko functionality"
    variables:
      test-preference:
        description: "A test gecko preference"
        type: String
        default: ""
        gecko-pref:
          pref: "gecko.nimbus.test"
          branch: "default"

```

We should remove `default` as a valid element for now because:
1. Gecko preferences should have their default value set Gecko side
2. Desktop does not support this stanza
3. We do not have a mechanism to capture and set default values

This could be a possible enhancement or change in the future, but for right now, it makes the most sense to prevent it to prevent confusion.

Back to Bug 2020683 Comment 0