NEW - bug 210710: review uses of progress service

https://bugs.eclipse.org/bugs/show_bug.cgi?id=210710

5 files changed, 5 insertions, 2 deletions

diff --git a/org.eclipse.mylyn.bugzilla.ui/src/org/eclipse/mylyn/internal/bugzilla/ui/search/BugzillaSearchPage.java b/org.eclipse.mylyn.bugzilla.ui/src/org/eclipse/mylyn/internal/bugzilla/ui/search/BugzillaSearchPage.java
index 8c1085aeb..ebd696e4e 100644
--- a/org.eclipse.mylyn.bugzilla.ui/src/org/eclipse/mylyn/internal/bugzilla/ui/search/BugzillaSearchPage.java
+++ b/org.eclipse.mylyn.bugzilla.ui/src/org/eclipse/mylyn/internal/bugzilla/ui/search/BugzillaSearchPage.java
@@ -1237,7 +1237,7 @@ public class BugzillaSearchPage extends AbstractRepositoryQueryPage implements L
 					getContainer().run(true, false, updateRunnable);
 				} else {
 					IProgressService service = PlatformUI.getWorkbench().getProgressService();
-					service.run(true, false, updateRunnable);
+					service.busyCursorWhile(updateRunnable);
 				}
 
 			} catch (InvocationTargetException e) {
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/planner/TaskActivityEditorInput.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/planner/TaskActivityEditorInput.java
index 2fde42a04..4ad35ee23 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/planner/TaskActivityEditorInput.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/planner/TaskActivityEditorInput.java
@@ -60,6 +60,7 @@ public class TaskActivityEditorInput implements IEditorInput {
 		taskReportGenerator.addCollector(inProgressTaskCollector);
 
 		try {
+			// TODO consider using IProgressService.busyCursorWhile(): bug 210710
 			IProgressService service = PlatformUI.getWorkbench().getProgressService();
 			service.run(false, true, taskReportGenerator);
 			while (!taskReportGenerator.isFinished())
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataExportWizard.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataExportWizard.java
index 685dbc2a6..0f226f71f 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataExportWizard.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataExportWizard.java
@@ -174,6 +174,7 @@ public class TaskDataExportWizard extends Wizard implements IExportWizard {
 		IProgressService service = PlatformUI.getWorkbench().getProgressService();
 
 		try {
+			// TODO use the wizard's progress service or IProgressService.busyCursorWhile(): bug 210710 
 			service.run(true, false, job);
 		} catch (InvocationTargetException e) {
 			StatusHandler.fail(e, "Could not export files", true);
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataImportWizard.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataImportWizard.java
index 0a893595e..cc224a1d4 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataImportWizard.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/wizards/TaskDataImportWizard.java
@@ -164,6 +164,7 @@ public class TaskDataImportWizard extends Wizard implements IImportWizard {
 		IProgressService service = PlatformUI.getWorkbench().getProgressService();
 
 		try {
+			// TODO use the wizard's progress service or IProgressService.busyCursorWhile(): bug 210710
 			service.run(true, false, job);
 		} catch (InvocationTargetException e) {
 			StatusHandler.fail(e, "Could not import files", true);
diff --git a/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/wizard/TracCustomQueryPage.java b/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/wizard/TracCustomQueryPage.java
index 023cf9d72..b935db901 100644
--- a/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/wizard/TracCustomQueryPage.java
+++ b/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/wizard/TracCustomQueryPage.java
@@ -387,7 +387,7 @@ public class TracCustomQueryPage extends AbstractRepositoryQueryPage {
 					scontainer.getRunnableContext().run(true, true, runnable);
 				} else {
 					IProgressService service = PlatformUI.getWorkbench().getProgressService();
-					service.run(true, true, runnable);
+					service.busyCursorWhile(runnable);
 				}
 			} catch (InvocationTargetException e) {
 				StatusHandler.displayStatus("Error updating attributes", TracCorePlugin.toStatus(e.getCause(),