Change in ovirt-engine[ovirt-engine-4.0]: webadmin, userportal: Improve dialog cleanup logic

From Tal Nisan <tnisan@redhat.com>:
Tal Nisan has submitted this change and it was merged. Change subject: webadmin,userportal: Improve dialog cleanup logic ...................................................................... webadmin,userportal: Improve dialog cleanup logic This patch ensures that dialog related resources (Views and Models) are cleaned up when both of the following conditions are satisfied: 1, a dialog has just been closed 2, there are no more active (visible) dialogs left This fixes issues where one dialog triggers another dialog, and we cannot simply clean up former dialog's Model as it might be needed later on. Change-Id: Iedd16549f19ea3630bc59f56e6a1a2f880239b27 Bug-Url: https://bugzilla.redhat.com/1368101 Bug-Url: https://bugzilla.redhat.com/1391466 Signed-off-by: Vojtech Szocs <vszocs@redhat.com> (cherry picked from commit 13c24b64269503050fbb4a42441adc959f1b4d73) --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractPopupPresenterWidget.java 2 files changed, 44 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Greg Sheremeta: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/66134 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iedd16549f19ea3630bc59f56e6a1a2f880239b27 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review