594863 - Encapsulate code in the GroupItems.removeHiddenGroups inside groupItem

Status: RESOLVED FIXED

(Firefox Graveyard :: Panorama, defect, P4)

Description

[Raymond Lee [:raymondlee]]

>+    this.groupItems.forEach(function(groupItem) {
>+      if (groupItem.canUndo) {
>+        let toClose = groupItem._children.concat();
>+        toClose.forEach(function(child) {
>+          child.close();
>+        });

This should be encapsulated inside the groupItem.

Comment 1

[Raymond Lee [:raymondlee]]

This would also be addressed when the patch in bug 599626

Comment 2

[Ian Gilman [:iangilman]]

Bug 599626 has landed, so this is fixed now.