Version 0.6.17.1

editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/dart/DartReconcilingStrategy.java

 /*
  * Copyright (c) 2012, 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 50 matching lines @@
   // TODO (danrubel): Replace with interface so that html editor can use this strategy
   private final DartReconcilingEditor editor;
 
   /**
    * The document with source to be reconciled. This is set by the {@link IReconciler} and should
    * not be {@code null}.
    */
   private IDocument document;
 
   /**
+   * The display in which the editor is visible.
+   */
+  private final Display display;
+
+  /**
    * The time when the context was last notified of a source change.
    */
   private volatile long notifyContextTime;
 
   /**
    * The time when the source was last modified.
    */
   private volatile long sourceChangedTime;
 
   /**
@@ skipping 15 matching lines @@
         if (event.getSource().equals(editor.getInputSource())) {
           applyAnalysisResult(event.getUnit());
         }
       }
     }
   };
 
   /**
    * Listen for changes to the source to clear the cached AST and record the last modification time.
    */
-  final IDocumentListener documentListener = new IDocumentListener() {
+  private final IDocumentListener documentListener = new IDocumentListener() {
     @Override
     public void documentAboutToBeChanged(DocumentEvent event) {
     }
 
     @Override
     public void documentChanged(DocumentEvent event) {
       sourceChangedTime = System.currentTimeMillis();
       editor.applyCompilationUnitElement(null);
 
       // Start analysis immediately if "." pressed to improve code completion response
-      // TODO (danrubel): do the same for ctrl-space ?
 
       // This may need to be modified or removed 
       // if we enable/set content assist immediate-activation character(s)
 
       if (".".equals(event.getText())) {
         updateSourceInBackground(true);
       }
     }
   };
 
   /**
    * Construct a new instance for the specified editor.
    * 
    * @param editor the editor (not {@code null})
    */
   public DartReconcilingStrategy(DartReconcilingEditor editor) {
     this.editor = editor;
+    this.display = Display.getDefault();
 
     // Prioritize analysis when editor becomes active
     editor.addViewerFocusListener(new FocusListener() {
       @Override
       public void focusGained(FocusEvent e) {
         updateAnalysisPriorityOrder(true);
       }
 
       @Override
       public void focusLost(FocusEvent e) {
@@ skipping 110 matching lines @@
           applyAnalysisResult(unit);
           return true;
         }
       }
     }
     return false;
   }
 
   /**
    * Answer the list of sources associated with the specified context that are open and visible.
+   * This method blocks until the information can be retrieved on the UI thread.
    * 
    * @param context the context (not {@code null})
    * @return a list of sources (not {@code null}, contains no {@code null}s)
    */
-  private List<Source> getVisibleSourcesForContext(AnalysisContext context) {
-    ArrayList<Source> sources = new ArrayList<Source>();
-    IWorkbench workbench = PlatformUI.getWorkbench();
-    for (IWorkbenchWindow window : workbench.getWorkbenchWindows()) {
-      for (IWorkbenchPage page : window.getPages()) {
-        IEditorReference[] allEditors = page.getEditorReferences();
-        for (IEditorReference editorRef : allEditors) {
-          IEditorPart part = editorRef.getEditor(false);
-          if (part instanceof DartEditor) {
-            DartEditor otherEditor = (DartEditor) part;
-            if (otherEditor.getInputAnalysisContext() == context && otherEditor.isVisible()) {
-              Source otherSource = otherEditor.getInputSource();
-              if (otherSource != null) {
-                sources.add(otherSource);
+  private List<Source> getVisibleSourcesForContext(final AnalysisContext context) {
+    final ArrayList<Source> sources = new ArrayList<Source>();
+    display.syncExec(new Runnable() {
+      @Override
+      public void run() {
+        IWorkbench workbench = PlatformUI.getWorkbench();
+        for (IWorkbenchWindow window : workbench.getWorkbenchWindows()) {
+          for (IWorkbenchPage page : window.getPages()) {
+            IEditorReference[] allEditors = page.getEditorReferences();
+            for (IEditorReference editorRef : allEditors) {
+              IEditorPart part = editorRef.getEditor(false);
+              if (part instanceof DartEditor) {
+                DartEditor otherEditor = (DartEditor) part;
+                if (otherEditor.getInputAnalysisContext() == context && otherEditor.isVisible()) {
+                  Source otherSource = otherEditor.getInputSource();
+                  if (otherSource != null) {
+                    sources.add(otherSource);
+                  }
+                }
               }
             }
           }
         }
       }
-    }
+    });
     return sources;
   }
 
   /**
    * Start background analysis of the context containing the source being edited.
    */
   private void performAnalysisInBackground() {
     AnalysisContext context = editor.getInputAnalysisContext();
     if (context != null) {
       ContextManager manager = editor.getInputProject();
       if (manager == null) {
         manager = DartCore.getProjectManager();
       }
       AnalysisWorker.performAnalysisInBackground(manager, context);
     }
   }
 
   /**
    * Clear the cached compilation unit and notify the context that the source has changed.
    * 
-   * @param code the new source code (not {@code null})
+   * @param code the new source code or {@code null} if the source should be pulled from disk
    */
   private void sourceChanged(String code) {
     notifyContextTime = System.currentTimeMillis();
     AnalysisContext context = editor.getInputAnalysisContext();
     Source source = editor.getInputSource();
     if (context != null && source != null) {
       context.setContents(source, code);
     }
   }
 
   /**
    * Update the order in which sources are analyzed in the context associated with the editor. This
    * is called once per instantiated editor on startup and then once for each editor as it becomes
    * active. For example, if there are 2 of 7 editors visible on startup, then this will be called
    * for the 2 visible editors.
    * 
    * @param isOpen {@code true} if the editor is open and the source should be the first source
    *          analyzed or {@code false} if the editor is closed and the source should be removed
    *          from the priority list.
    */
   private void updateAnalysisPriorityOrder(final boolean isOpen) {
-    Display.getDefault().asyncExec(new Runnable() {
-      @Override
-      public void run() {
-        // TODO (danrubel): Revisit when reviewing performance of startup, open, and activate
-        AnalysisContext context = editor.getInputAnalysisContext();
-        Source source = editor.getInputSource();
-        if (context != null && source != null) {
-          List<Source> sources = getVisibleSourcesForContext(context);
-          sources.remove(source);
-          if (isOpen) {
-            sources.add(0, source);
-          }
-          updateAnalysisPriorityOrderInBackground(sources);
-        }
+    // TODO (danrubel): Revisit when reviewing performance of startup, open, and activate
+    AnalysisContext context = editor.getInputAnalysisContext();
+    Source source = editor.getInputSource();
+    if (context != null && source != null) {
+      final List<Source> sources = getVisibleSourcesForContext(context);
+      sources.remove(source);
+      if (isOpen) {
+        sources.add(0, source);
       }
-    });
-  }
-
-  /**
-   * Update the order in which sources are analyzed in the context associated with the editor.
-   * 
-   * @param sources the list of sources to be analyzed first (not <code>null</code>, contains no
-   *          <code>null</code>s)
-   */
-  private void updateAnalysisPriorityOrderInBackground(final List<Source> sources) {
-    // TODO (danrubel): Put this back on UI thread once analysis lock contention has been resolved
-    new Thread("updateAnalysisPriorityOrderInBackground") {
-      @Override
-      public void run() {
-        AnalysisContext context = editor.getInputAnalysisContext();
-        if (context != null) {
-          context.setAnalysisPriorityOrder(sources);
-        }
-      };
-    }.start();
+      // TODO (danrubel): Keep this off the UI thread until lock contention has been resolved
+      if (Display.getCurrent() == null) {
+        context.setAnalysisPriorityOrder(sources);
+      } else {
+        new Thread("updateAnalysisPriorityOrder") {
+          @Override
+          public void run() {
+            AnalysisContext context = editor.getInputAnalysisContext();
+            if (context != null) {
+              context.setAnalysisPriorityOrder(sources);
+            }
+          };
+        }.start();
+      }
+    }
   }
 
   /**
    * Update the source being analyzed in the context associated with the editor.
    * 
    * @param isOpen {@code true} if the editor is open and the source should be cached or
    *          {@code false} if the editor is closed and the source should read from disk.
    */
   private void updateSourceInBackground(final boolean isOpen) {
-    // TODO (danrubel): Put this back on UI thread once analysis lock contention has been resolved 
+    // TODO (danrubel): Keep this off the UI thread until lock contention has been resolved
     new Thread("updateSourceInBackground") {
       @Override
       public void run() {
         sourceChanged(isOpen ? document.get() : null);
         performAnalysisInBackground();
       }
     }.start();
   }
 }