refactor: Hide waitcursor behind the IMainFormView interface

Merge pull request activescott#154 from learn-more/view_abstraction

src/LessMsi.Gui/IMainFormView.cs

@@ -22,6 +22,7 @@
 // Authors:
 //	Scott Willeke ([email protected])
 //
+using System;
 using System.Collections.Generic;
 using LessMsi.Gui.Model;
 
@@ -94,5 +95,11 @@ internal interface IMainFormView
 		/// </summary>
 	    void SetStreamSelectorSource(IEnumerable<StreamInfoView> streamNames);
 	    void SetCabContainedFileListSource(IEnumerable<CabContainedFileView> streamFiles);
+
+		/// <summary>
+		/// Start a wait cursor to indicate.
+		/// </summary>
+		/// <returns>An object that should be disposed when the operation is done.</returns>
+		IDisposable StartWaitCursor();
     }
 }

src/LessMsi.Gui/MainForm.cs

@@ -1067,6 +1067,11 @@ private ExtractionProgressDialog BeginShowingProgressDialog()
 			return progressDialog;
 		}
 
+		public IDisposable StartWaitCursor()
+		{
+			return new DisposableCursor(this);
+		}
+
 		protected bool IsFileTabSelected
 		{
 			get

src/LessMsi.Gui/MainFormPresenter.cs

@@ -113,7 +113,7 @@ private void ViewFiles(Database msidb)
 			if (msidb == null)
 				return;
 
-			using (new DisposableCursor(ViewLeakedAbstraction))
+			using (View.StartWaitCursor())
 			{
 				try
 				{
@@ -316,7 +316,7 @@ public void LoadTables()
 
 			using (var msidb = new Database(this.SelectedMsiFile.FullName, OpenDatabase.ReadOnly))
 			{
-				using (new DisposableCursor(ViewLeakedAbstraction))
+				using (View.StartWaitCursor())
 				{
 					try
 					{
@@ -425,7 +425,7 @@ private void UpdateMSiTableGrid(Database msidb, string tableName)
 
 			Status(string.Concat("Processing Table \'", tableName, "\'."));
 
-			using (new DisposableCursor(ViewLeakedAbstraction))
+			using (View.StartWaitCursor())
 			{   // clear the columns no matter what happens (in the event the table doesn't exist we don't want to show anything).
 				View.ClearTableViewGridColumns();
 				try