[gnome-boxes] wizard: make review cancellable

[Marc-Andre Lureau <malureau src gnome org>]

commit c61e30528eb3db8fe42d0f13fb71ae6c22ab29eb
Author: Marc-Andrà Lureau <marcandre lureau gmail com>
Date:   Wed Sep 5 14:44:30 2012 +0200

    wizard: make review cancellable
    
    Do not disable all wizard navigation buttons, but allow to cancel
    review step (back and cancel button should be always sensitive).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=683417

 src/wizard.vala |   31 ++++++++++++++++++++++++-------
 1 files changed, 24 insertions(+), 7 deletions(-)
---
diff --git a/src/wizard.vala b/src/wizard.vala
index 0a06132..cd2fdaf 100644
--- a/src/wizard.vala
+++ b/src/wizard.vala
@@ -71,14 +71,10 @@ private class Boxes.Wizard: Boxes.UI {
                     break;
 
                 case WizardPage.REVIEW:
-                    back_button.sensitive = false;
                     next_button.sensitive = false;
-                    cancel_button.sensitive = false;
-                    review.begin ((source, result) => {
-                        back_button.sensitive = true;
-                        next_button.sensitive = true;
-                        cancel_button.sensitive = true;
 
+                    review.begin ((obj, result) => {
+                        next_button.sensitive = true;
                         if (!review.end (result))
                             page = page - 1;
                     });
@@ -311,13 +307,28 @@ private class Boxes.Wizard: Boxes.UI {
         return true;
     }
 
+    private Cancellable? review_cancellable;
+
     private async bool review () {
+        // only one outstanding review () permitted
+        return_if_fail (review_cancellable == null);
+
+        review_cancellable = new Cancellable ();
+        var result = yield do_review_cancellable ();
+        review_cancellable = null;
+
+        return result;
+    }
+
+    private async bool do_review_cancellable () {
+        return_if_fail (review_cancellable != null);
+
         nokvm_label.hide ();
         summary.clear ();
 
         if (vm_creator != null && machine == null) {
             try {
-                machine = yield vm_creator.create_vm (null);
+                machine = yield vm_creator.create_vm (review_cancellable);
             } catch (IOError.CANCELLED cancel_error) { // We did this, so ignore!
                 return false;
             } catch (GLib.Error error) {
@@ -328,6 +339,9 @@ private class Boxes.Wizard: Boxes.UI {
             }
         }
 
+        if (review_cancellable.is_cancelled ())
+            return false;
+
         review_label.set_text (_("Will create a new box with the following properties:"));
 
         if (source != null) {
@@ -623,6 +637,9 @@ private class Boxes.Wizard: Boxes.UI {
     }
 
     private void destroy_machine () {
+        if (review_cancellable != null)
+            review_cancellable.cancel ();
+
         if (machine != null) {
             App.app.delete_machine (machine);
             machine = null;