Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Differentiate progress triggered by async operations ...................................................................... webadmin: Differentiate progress triggered by async operations Since there's a lot of model code checking whether getProgress() != null, in which case methods return without doing anything (this was an old workaround to avoid duplicate OK clicks), the new async events would cause some bugs. This patch changes the mechanism to be more conservative: * It now doesn't touch the model side of things. * "Manual" override is made possible even if originally the progress was triggered by an async operation, i.e. if in some flow a developer ever calls startProgress() on the model, an async operation will not stop the progress. Change-Id: I94b8a2eac59ac712f0e9744fa4bd84c66dd84c5a Signed-off-by: Lior Vernia <lvernia(a)redhat.com&gt; --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/36172 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94b8a2eac59ac712f0e9744fa4bd84c66dd84c5a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com&gt; Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server