Issue 17356. Apply selection returned from formatter to the SourceViewer.

editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/formatter/DartFormatter.java

 /*
  * Copyright (c) 2013, the Dart project authors.
  * 
  * Licensed under the Eclipse Public License v1.0 (the "License"); you may not use this file except
  * in compliance with the License. You may obtain a copy of the License at
  * 
  * http://www.eclipse.org/legal/epl-v10.html
  * 
  * Unless required by applicable law or agreed to in writing, software distributed under the License
  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
@@ skipping 163 matching lines @@
         return result;
       } catch (JSONException e) {
         DartToolsPlugin.log(e);
         throw new IOException(e);
       }
 
     }
   }
 
   public static class DartStyleRunner {
+    public static Point formatFile(IFile file, final Point selection, final IProgressMonitor monitor)
+        throws IOException, CoreException {
+      final String fileName = file.getName();
+      final String sourcePath = file.getRawLocation().makeAbsolute().toOSString();
 
-    public static void formatFile(final IFile file, final Point selection,
-        final IProgressMonitor monitor) throws IOException, CoreException {
+      final CompilationUnitChange change = new CompilationUnitChange(fileName, file);
+      change.setEdit(new MultiTextEdit());
+      change.initializeValidationData(monitor);
+      change.setSaveMode(TextFileChange.LEAVE_DIRTY);
 
-      final String sourcePath = file.getRawLocation().makeAbsolute().toOSString();
-      final CompilationUnitChange[] change = new CompilationUnitChange[1];
+      final Point newSelection = new Point(-1, 0);
 
       ExecutionUtils.runLog(new RunnableEx() {
         @Override
         public void run() throws Exception {
-
-          int selectionStart = selection != null ? selection.x : -1;
-          int selectionLength = selection != null ? selection.y : -1;
+          int initialSelectionOffset = selection != null ? selection.x : -1;
+          int initialSelectionLength = selection != null ? selection.y : -1;
 
           final CountDownLatch latch = new CountDownLatch(1);
 
           DartCore.getAnalysisServer().edit_format(
               sourcePath,
-              selectionStart,
-              selectionLength,
+              initialSelectionOffset,
+              initialSelectionLength,
               new FormatConsumer() {
-
                 @Override
-                public void computedFormat(List<SourceEdit> edits, int selectionOffset,
-                    int selectionLength) {
+                public void computedFormat(List<SourceEdit> edits, int newSelectionOffset,
+                    int newSelectionLength) {
+                  newSelection.x = newSelectionOffset;
+                  newSelection.y = newSelectionLength;
                   TextEdit[] textEdits = ServiceUtils_NEW.toLTK(edits);
-                  change[0] = new CompilationUnitChange(file.getName(), file);
-                  change[0].setEdit(new MultiTextEdit());
-                  change[0].initializeValidationData(monitor);
-                  change[0].setSaveMode(TextFileChange.LEAVE_DIRTY);
                   try {
                     for (TextEdit ltkEdit : textEdits) {
-                      change[0].addEdit(ltkEdit);
+                      change.addEdit(ltkEdit);
                     }
                   } catch (MalformedTreeException e) {
-                    throw new Error(file.getName() + " " + StringUtils.join(textEdits, " "), e);
+                    throw new Error(fileName + " " + StringUtils.join(textEdits, " "), e);
                   } finally {
                     latch.countDown();
                   }
                 }
 
                 @Override
                 public void onError(RequestError requestError) {
                   latch.countDown();
                 }
               });
-          Uninterruptibles.awaitUninterruptibly(latch, 5000, TimeUnit.MILLISECONDS);
-
-          if (change[0] != null) {
-            new PerformChangeOperation(change[0]).run(monitor);
+          boolean success = Uninterruptibles.awaitUninterruptibly(
+              latch,
+              5000,
+              TimeUnit.MILLISECONDS);
+          if (success) {
+            new PerformChangeOperation(change).run(monitor);
           }
         }
       });
-
+      return newSelection;
     }
   }
 
   public static class FormatFileAction extends WorkspaceAction {
 
     private List<IFile> files = Arrays.asList(new IFile[0]);
 
     public FormatFileAction(IShellProvider provider) {
       super(provider, "Format");
       setId(DartEditorActionDefinitionIds.QUICK_FORMAT);
@@ skipping 235 matching lines @@
   }
 
 //  private static List<String> buildArguments(IPath path) {
 //    ArrayList<String> args = new ArrayList<String>();
 //    args.add("-w");
 //    args.add(path.toOSString());
 //    return args;
 //  }
 
 }