Use SubMonintor in FileBufferOperationAction

done() call not necessary here and convert can also show the label and
set the number of work unit.

See https://www.eclipse.org/articles/Article-Progress-Monitors/article.html

Author: vogella

bundles/org.eclipse.ui.editors/src/org/eclipse/ui/editors/text/FileBufferOperationAction.java

@@ -198,20 +198,15 @@ protected IStatus run(IProgressMonitor monitor) {
 	}
 
 	protected final IPath[] generateLocations(IFile[] files, IProgressMonitor progressMonitor) {
-		progressMonitor.beginTask(TextEditorMessages.FileBufferOperationAction_collectionFiles_label, files.length);
-		try {
-			Set<IPath> locations= new HashSet<>();
-			for (IFile file : files) {
-				IPath fullPath = file.getFullPath();
-				if (isAcceptableLocation(fullPath))
-					locations.add(fullPath);
-				progressMonitor.worked(1);
-			}
-			return locations.toArray(new IPath[locations.size()]);
-
-		} finally {
-			progressMonitor.done();
+		SubMonitor subMonitor= SubMonitor.convert(progressMonitor, TextEditorMessages.FileBufferOperationAction_collectionFiles_label, files.length);
+		Set<IPath> locations= new HashSet<>();
+		for (IFile file : files) {
+			IPath fullPath= file.getFullPath();
+			if (isAcceptableLocation(fullPath))
+				locations.add(fullPath);
+			subMonitor.worked(1);
 		}
+		return locations.toArray(new IPath[locations.size()]);
 	}
 
 	/**