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

[Code Review]

>From Tal Nisan <tnisan at 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 at 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 at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>