Fix groupby-picker.

Fix groupby-picker.

Currently, when localStorage[settingsKey] contains a grouping key that is not in
possibleGroups, then |set currentGroupKeys| gets confused and tries to access a
non-existent group element, causing a TypeError.

This was broken by https://codereview.chromium.org/2458873002

https://bugs.chromium.org/p/chromium/issues/detail?id=661711

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/b6402725257bb09d6015a386c7ef5b61811b14f3

[benjhayden, 2016-11-07 22:10:16]

Patchset #1 (id:1) has been deleted

[benjhayden, 2016-11-07 22:10:16]

Patchset #1 (id:1) has been deleted

[benjhayden, 2016-11-07 22:10:16]

Patchset #1 (id:1) has been deleted

[benjhayden, 2016-11-07 22:10:20]

Patchset #1 (id:20001) has been deleted

[benjhayden, 2016-11-07 22:11:14]

benjhayden@chromium.org changed reviewers:
+ eakuefner@chromium.org, nednguyen@google.com

[benjhayden, 2016-11-07 22:11:15]

The new test fails without the fix and passes with it.

PTAL :-)

[benjhayden, 2016-11-07 22:15:06]

Description was changed from

==========
Fix groupby-picker.

Currently, when localStorage[settingsKey] contains a grouping key that is not in
possibleGroups, then |set currentGroupKeys| gets confused and tries to access a
non-existent group element, causing a TypeError.

https://bugs.chromium.org/p/chromium/issues/detail?id=661711
==========

to

==========
Fix groupby-picker.

Currently, when localStorage[settingsKey] contains a grouping key that is not in
possibleGroups, then |set currentGroupKeys| gets confused and tries to access a
non-existent group element, causing a TypeError.

This was broken by https://codereview.chromium.org/2458873002

https://bugs.chromium.org/p/chromium/issues/detail?id=661711
==========

[eakuefner, 2016-11-07 22:48:06]

Thanks Ben, and thanks especially for adding a test!

lgtm

[benjhayden, 2016-11-08 00:27:23]

The CQ bit was checked by benjhayden@chromium.org

[commit-bot: I haz the power, 2016-11-08 00:27:34]

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-11-08 00:57:43]

Description was changed from

==========
Fix groupby-picker.

Currently, when localStorage[settingsKey] contains a grouping key that is not in
possibleGroups, then |set currentGroupKeys| gets confused and tries to access a
non-existent group element, causing a TypeError.

This was broken by https://codereview.chromium.org/2458873002

https://bugs.chromium.org/p/chromium/issues/detail?id=661711
==========

to

==========
Fix groupby-picker.

Currently, when localStorage[settingsKey] contains a grouping key that is not in
possibleGroups, then |set currentGroupKeys| gets confused and tries to access a
non-existent group element, causing a TypeError.

This was broken by https://codereview.chromium.org/2458873002

https://bugs.chromium.org/p/chromium/issues/detail?id=661711

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-11-08 00:57:44]

Committed patchset #1 (id:40001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...