fixed Bug 1024299
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / phpeditor / PHPUnitEditor.java
index 34ceb6f..6755004 100644 (file)
@@ -7,15 +7,20 @@ import java.util.List;
 
 import net.sourceforge.phpdt.core.ICompilationUnit;
 import net.sourceforge.phpdt.core.IJavaElement;
+import net.sourceforge.phpdt.core.IMember;
 import net.sourceforge.phpdt.core.ISourceRange;
 import net.sourceforge.phpdt.core.ISourceReference;
 import net.sourceforge.phpdt.core.JavaCore;
 import net.sourceforge.phpdt.core.JavaModelException;
+import net.sourceforge.phpdt.core.dom.CompilationUnit;
 import net.sourceforge.phpdt.internal.compiler.parser.Scanner;
 import net.sourceforge.phpdt.internal.ui.actions.CompositeActionGroup;
 import net.sourceforge.phpdt.internal.ui.text.ContentAssistPreference;
 import net.sourceforge.phpdt.internal.ui.text.IPHPPartitions;
 import net.sourceforge.phpdt.internal.ui.text.PHPPairMatcher;
+import net.sourceforge.phpdt.internal.ui.text.SmartBackspaceManager;
+import net.sourceforge.phpdt.internal.ui.text.SmartSemicolonAutoEditStrategy;
+import net.sourceforge.phpdt.internal.ui.text.java.IJavaReconcilingListener;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionManager;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionUI;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionUI.ExitFlags;
@@ -24,7 +29,9 @@ import net.sourceforge.phpdt.ui.PreferenceConstants;
 import net.sourceforge.phpdt.ui.actions.GenerateActionGroup;
 import net.sourceforge.phpdt.ui.text.JavaTextTools;
 import net.sourceforge.phpeclipse.PHPeclipsePlugin;
+import net.sourceforge.phpeclipse.ui.editor.ShowExternalPreviewAction;
 
+import org.eclipse.core.internal.runtime.ListenerList;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IWorkspaceRoot;
 import org.eclipse.core.resources.ResourcesPlugin;
@@ -43,11 +50,11 @@ import org.eclipse.jface.preference.IPreferenceStore;
 import org.eclipse.jface.preference.PreferenceConverter;
 import org.eclipse.jface.text.BadLocationException;
 import org.eclipse.jface.text.DocumentCommand;
+import org.eclipse.jface.text.IAutoEditStrategy;
 import org.eclipse.jface.text.IDocument;
 import org.eclipse.jface.text.ILineTracker;
 import org.eclipse.jface.text.IRegion;
 import org.eclipse.jface.text.ITextOperationTarget;
-import org.eclipse.jface.text.ITextSelection;
 import org.eclipse.jface.text.ITextViewerExtension;
 import org.eclipse.jface.text.ITypedRegion;
 import org.eclipse.jface.text.IWidgetTokenKeeper;
@@ -58,7 +65,6 @@ import org.eclipse.jface.text.source.ISourceViewer;
 import org.eclipse.jface.text.source.IVerticalRuler;
 import org.eclipse.jface.text.source.SourceViewerConfiguration;
 import org.eclipse.jface.util.PropertyChangeEvent;
-import org.eclipse.jface.viewers.ISelectionProvider;
 import org.eclipse.jface.window.Window;
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.custom.VerifyKeyListener;
@@ -70,13 +76,17 @@ import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Shell;
 import org.eclipse.ui.IEditorInput;
+import org.eclipse.ui.IEditorPart;
 import org.eclipse.ui.IFileEditorInput;
+import org.eclipse.ui.IWorkbenchPage;
+import org.eclipse.ui.IWorkbenchWindow;
 import org.eclipse.ui.actions.ActionContext;
 import org.eclipse.ui.actions.ActionGroup;
 import org.eclipse.ui.dialogs.SaveAsDialog;
 import org.eclipse.ui.editors.text.IStorageDocumentProvider;
 import org.eclipse.ui.help.WorkbenchHelp;
 import org.eclipse.ui.part.FileEditorInput;
+import org.eclipse.ui.texteditor.AbstractDecoratedTextEditorPreferenceConstants;
 import org.eclipse.ui.texteditor.ContentAssistAction;
 import org.eclipse.ui.texteditor.IDocumentProvider;
 import org.eclipse.ui.texteditor.ITextEditorActionConstants;
@@ -100,277 +110,6 @@ public class PHPUnitEditor extends PHPEditor { //implements
     void customizeDocumentCommand(IDocument document, DocumentCommand command);
   };
 
-  //  class AdaptedRulerLayout extends Layout {
-  //
-  //    protected int fGap;
-  //    protected AdaptedSourceViewer fAdaptedSourceViewer;
-  //
-  //    protected AdaptedRulerLayout(int gap, AdaptedSourceViewer asv) {
-  //      fGap = gap;
-  //      fAdaptedSourceViewer = asv;
-  //    }
-  //
-  //    protected Point computeSize(Composite composite, int wHint, int hHint,
-  // boolean flushCache) {
-  //      Control[] children = composite.getChildren();
-  //      Point s = children[children.length - 1].computeSize(SWT.DEFAULT,
-  // SWT.DEFAULT, flushCache);
-  //      if (fAdaptedSourceViewer.isVerticalRulerVisible())
-  //        s.x += fAdaptedSourceViewer.getVerticalRuler().getWidth() + fGap;
-  //      return s;
-  //    }
-  //
-  //    protected void layout(Composite composite, boolean flushCache) {
-  //      Rectangle clArea = composite.getClientArea();
-  //      if (fAdaptedSourceViewer.isVerticalRulerVisible()) {
-  //
-  //        StyledText textWidget = fAdaptedSourceViewer.getTextWidget();
-  //        Rectangle trim = textWidget.computeTrim(0, 0, 0, 0);
-  //        int scrollbarHeight = trim.height;
-  //
-  //        IVerticalRuler vr = fAdaptedSourceViewer.getVerticalRuler();
-  //        int vrWidth = vr.getWidth();
-  //
-  //        int orWidth = 0;
-  //        if (fAdaptedSourceViewer.isOverviewRulerVisible()) {
-  //          OverviewRuler or = fAdaptedSourceViewer.getOverviewRuler();
-  //          orWidth = or.getWidth();
-  //          or.getControl().setBounds(clArea.width - orWidth, scrollbarHeight,
-  // orWidth, clArea.height - 3 * scrollbarHeight);
-  //        }
-  //
-  //        textWidget.setBounds(vrWidth + fGap, 0, clArea.width - vrWidth - orWidth
-  // -
-  // 2 * fGap, clArea.height);
-  //        vr.getControl().setBounds(0, 0, vrWidth, clArea.height -
-  // scrollbarHeight);
-  //
-  //      } else {
-  //        StyledText textWidget = fAdaptedSourceViewer.getTextWidget();
-  //        textWidget.setBounds(0, 0, clArea.width, clArea.height);
-  //      }
-  //    }
-  //  };
-  //
-  //  class AdaptedSourceViewer extends SourceViewer { // extends
-  // JavaCorrectionSourceViewer {
-  //
-  //    private List fTextConverters;
-  //
-  //    private OverviewRuler fOverviewRuler;
-  //    private boolean fIsOverviewRulerVisible;
-  //    /** The viewer's overview ruler hovering controller */
-  //    private AbstractHoverInformationControlManager
-  // fOverviewRulerHoveringController;
-  //
-  //    private boolean fIgnoreTextConverters = false;
-  //
-  //    private IVerticalRuler fCachedVerticalRuler;
-  //    private boolean fCachedIsVerticalRulerVisible;
-  //
-  //    public AdaptedSourceViewer(Composite parent, IVerticalRuler ruler, int
-  // styles) {
-  //      super(parent, ruler, styles); //, CompilationUnitEditor.this);
-  //
-  //      fCachedVerticalRuler = ruler;
-  //      fCachedIsVerticalRulerVisible = (ruler != null);
-  //      fOverviewRuler = new OverviewRuler(VERTICAL_RULER_WIDTH);
-  //
-  //      delayedCreateControl(parent, styles);
-  //    }
-  //
-  //    /*
-  //     * @see ISourceViewer#showAnnotations(boolean)
-  //     */
-  //    public void showAnnotations(boolean show) {
-  //      fCachedIsVerticalRulerVisible = (show && fCachedVerticalRuler != null);
-  //      // super.showAnnotations(show);
-  //    }
-  //
-  //    public IContentAssistant getContentAssistant() {
-  //      return fContentAssistant;
-  //    }
-  //
-  //    /*
-  //     * @see ITextOperationTarget#doOperation(int)
-  //     */
-  //    public void doOperation(int operation) {
-  //
-  //      if (getTextWidget() == null)
-  //        return;
-  //
-  //      switch (operation) {
-  //        case CONTENTASSIST_PROPOSALS :
-  //          String msg = fContentAssistant.showPossibleCompletions();
-  //          setStatusLineErrorMessage(msg);
-  //          return;
-  //        case UNDO :
-  //          fIgnoreTextConverters = true;
-  //          break;
-  //        case REDO :
-  //          fIgnoreTextConverters = true;
-  //          break;
-  //      }
-  //
-  //      super.doOperation(operation);
-  //    }
-  //
-  //    public void insertTextConverter(ITextConverter textConverter, int index)
-  // {
-  //      throw new UnsupportedOperationException();
-  //    }
-  //
-  //    public void addTextConverter(ITextConverter textConverter) {
-  //      if (fTextConverters == null) {
-  //        fTextConverters = new ArrayList(1);
-  //        fTextConverters.add(textConverter);
-  //      } else if (!fTextConverters.contains(textConverter))
-  //        fTextConverters.add(textConverter);
-  //    }
-  //
-  //    public void removeTextConverter(ITextConverter textConverter) {
-  //      if (fTextConverters != null) {
-  //        fTextConverters.remove(textConverter);
-  //        if (fTextConverters.size() == 0)
-  //          fTextConverters = null;
-  //      }
-  //    }
-  //
-  //    /*
-  //     * @see TextViewer#customizeDocumentCommand(DocumentCommand)
-  //     */
-  //    protected void customizeDocumentCommand(DocumentCommand command) {
-  //      super.customizeDocumentCommand(command);
-  //      if (!fIgnoreTextConverters && fTextConverters != null) {
-  //        for (Iterator e = fTextConverters.iterator(); e.hasNext();)
-  //           ((ITextConverter) e.next()).customizeDocumentCommand(getDocument(),
-  // command);
-  //      }
-  //      fIgnoreTextConverters = false;
-  //    }
-  //
-  //    public IVerticalRuler getVerticalRuler() {
-  //      return fCachedVerticalRuler;
-  //    }
-  //
-  //    public boolean isVerticalRulerVisible() {
-  //      return fCachedIsVerticalRulerVisible;
-  //    }
-  //
-  //    public OverviewRuler getOverviewRuler() {
-  //      return fOverviewRuler;
-  //    }
-  //
-  //    /*
-  //     * @see TextViewer#createControl(Composite, int)
-  //     */
-  //    protected void createControl(Composite parent, int styles) {
-  //      // do nothing here
-  //    }
-  //
-  //    protected void delayedCreateControl(Composite parent, int styles) {
-  //      //create the viewer
-  //      super.createControl(parent, styles);
-  //
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.setLayout(new AdaptedRulerLayout(GAP_SIZE, this));
-  //        fOverviewRuler.createControl(composite, this);
-  //      }
-  //    }
-  //    protected void ensureOverviewHoverManagerInstalled() {
-  //      if (fOverviewRulerHoveringController == null && fAnnotationHover != null
-  // && fHoverControlCreator != null) {
-  //        fOverviewRulerHoveringController =
-  //          new OverviewRulerHoverManager(fOverviewRuler, this, fAnnotationHover,
-  // fHoverControlCreator);
-  //        fOverviewRulerHoveringController.install(fOverviewRuler.getControl());
-  //      }
-  //    }
-  //
-  //    public void hideOverviewRuler() {
-  //      fIsOverviewRulerVisible = false;
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.layout();
-  //      }
-  //      if (fOverviewRulerHoveringController != null) {
-  //        fOverviewRulerHoveringController.dispose();
-  //        fOverviewRulerHoveringController = null;
-  //      }
-  //    }
-  //
-  //    public void showOverviewRuler() {
-  //      fIsOverviewRulerVisible = true;
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.layout();
-  //      }
-  //      ensureOverviewHoverManagerInstalled();
-  //    }
-  //
-  //    public boolean isOverviewRulerVisible() {
-  //      return fIsOverviewRulerVisible;
-  //    }
-  //
-  //    /*
-  //     * @see ISourceViewer#setDocument(IDocument, IAnnotationModel, int, int)
-  //     */
-  //    public void setDocument(
-  //      IDocument document,
-  //      IAnnotationModel annotationModel,
-  //      int visibleRegionOffset,
-  //      int visibleRegionLength) {
-  //      super.setDocument(document, annotationModel, visibleRegionOffset,
-  // visibleRegionLength);
-  //      fOverviewRuler.setModel(annotationModel);
-  //    }
-  //
-  //    // http://dev.eclipse.org/bugs/show_bug.cgi?id=19270
-  //    public void updateIndentationPrefixes() {
-  //      SourceViewerConfiguration configuration = getSourceViewerConfiguration();
-  //      String[] types = configuration.getConfiguredContentTypes(this);
-  //      for (int i = 0; i < types.length; i++) {
-  //        String[] prefixes = configuration.getIndentPrefixes(this, types[i]);
-  //        if (prefixes != null && prefixes.length > 0)
-  //          setIndentPrefixes(prefixes, types[i]);
-  //      }
-  //    }
-  //
-  //    /*
-  //     * @see IWidgetTokenOwner#requestWidgetToken(IWidgetTokenKeeper)
-  //     */
-  //    public boolean requestWidgetToken(IWidgetTokenKeeper requester) {
-  //      if (WorkbenchHelp.isContextHelpDisplayed())
-  //        return false;
-  //      return super.requestWidgetToken(requester);
-  //    }
-  //
-  //    /*
-  //     * @see
-  // org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
-  //     */
-  //    public void configure(SourceViewerConfiguration configuration) {
-  //      super.configure(configuration);
-  //      // prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
-  // IDocument.DEFAULT_CONTENT_TYPE);
-  //    }
-  //
-  //    protected void handleDispose() {
-  //      fOverviewRuler = null;
-  //
-  //      if (fOverviewRulerHoveringController != null) {
-  //        fOverviewRulerHoveringController.dispose();
-  //        fOverviewRulerHoveringController = null;
-  //      }
-  //
-  //      super.handleDispose();
-  //    }
-  //
-  //  };
   class AdaptedSourceViewer extends JavaSourceViewer {
     private List fTextConverters;
 
@@ -491,20 +230,381 @@ public class PHPUnitEditor extends PHPEditor { //implements
       return super.requestWidgetToken(requester);
     }
 
-    /*
-     * @see org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
-     */
+//    /*
+//     * @see org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
+//     */
+//    public void configure(SourceViewerConfiguration configuration) {
+//      super.configure(configuration);
+//      //                             fCorrectionAssistant= new
+//      // JavaCorrectionAssistant(CompilationUnitEditor.this);
+//      //                             fCorrectionAssistant.install(this);
+//      //TODO install SmartBracesAutoEditStrategy
+//      //                             prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
+//      // IDocument.DEFAULT_CONTENT_TYPE);
+//    }
     public void configure(SourceViewerConfiguration configuration) {
-      super.configure(configuration);
-      //                               fCorrectionAssistant= new
-      // JavaCorrectionAssistant(CompilationUnitEditor.this);
-      //                               fCorrectionAssistant.install(this);
-      //TODO install SmartBracesAutoEditStrategy
-      //                               prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
-      // IDocument.DEFAULT_CONTENT_TYPE);
-    }
+               super.configure(configuration);
+//             fCorrectionAssistant= new JavaCorrectionAssistant(CompilationUnitEditor.this);
+//             fCorrectionAssistant.install(this);
+               IAutoEditStrategy smartSemi= new SmartSemicolonAutoEditStrategy(IPHPPartitions.PHP_PARTITIONING);
+               prependAutoEditStrategy(smartSemi, IDocument.DEFAULT_CONTENT_TYPE);
+               prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_DQ);
+               prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_SQ);
+//             prependAutoEditStrategy(smartSemi, IPHPPartitions.JAVA_CHARACTER);
+       }
   };
+  /**
+        * Remembers data related to the current selection to be able to
+        * restore it later.
+        * 
+        * @since 3.0
+        */
+       private class RememberedSelection {
+               /** The remembered selection start. */
+               private RememberedOffset fStartOffset= new RememberedOffset();
+               /** The remembered selection end. */
+               private RememberedOffset fEndOffset= new RememberedOffset();
+
+               /**
+                * Remember current selection.
+                */
+               public void remember() {
+                       /* https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257
+                        * This method may be called inside an async call posted
+                        * to the UI thread, so protect against intermediate disposal
+                        * of the editor. 
+                        */
+                       ISourceViewer viewer= getSourceViewer();
+                       if (viewer != null) {
+                               IRegion selection= getSignedSelection(viewer);
+                               int startOffset= selection.getOffset();
+                               int endOffset= startOffset + selection.getLength();
+                               
+                               fStartOffset.setOffset(startOffset);
+                               fEndOffset.setOffset(endOffset);
+                       }
+               }
+
+               /**
+                * Restore remembered selection.
+                */
+               public void restore() {
+                       /* https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257
+                        * This method may be called inside an async call posted
+                        * to the UI thread, so protect against intermediate disposal
+                        * of the editor.
+                        */
+                       if (getSourceViewer() == null)
+                               return;
+                       
+                       try {
+                               
+                               int startOffset, endOffset;
+                               int revealStartOffset, revealEndOffset; 
+                               if (showsHighlightRangeOnly()) {
+                                       IJavaElement newStartElement= fStartOffset.getElement();
+                                       startOffset= fStartOffset.getRememberedOffset(newStartElement);
+                                       revealStartOffset= fStartOffset.getRevealOffset(newStartElement, startOffset);
+                                       if (revealStartOffset == -1)
+                                               startOffset= -1;
+                                       
+                                       IJavaElement newEndElement= fEndOffset.getElement();
+                                       endOffset= fEndOffset.getRememberedOffset(newEndElement);
+                                       revealEndOffset= fEndOffset.getRevealOffset(newEndElement, endOffset);
+                                       if (revealEndOffset == -1)
+                                               endOffset= -1;
+                               } else {
+                                       startOffset= fStartOffset.getOffset();
+                                       revealStartOffset= startOffset;
+                                       endOffset= fEndOffset.getOffset();
+                                       revealEndOffset= endOffset;
+                               }
+                               
+                               if (startOffset == -1) {
+                                       startOffset= endOffset; // fallback to caret offset
+                                       revealStartOffset= revealEndOffset;
+                               }
+                               
+                               if (endOffset == -1) {
+                                       endOffset= startOffset; // fallback to other offset
+                                       revealEndOffset= revealStartOffset;
+                               }
+                               
+                               IJavaElement element;
+                               if (endOffset == -1) {
+                                        // fallback to element selection
+                                       element= fEndOffset.getElement();
+                                       if (element == null)
+                                               element= fStartOffset.getElement();
+                                       if (element != null)
+                                               setSelection(element);
+                                       return;
+                               }
+                                                       
+                               if (isValidSelection(revealStartOffset, revealEndOffset - revealStartOffset) && isValidSelection(startOffset, endOffset - startOffset))
+                                       selectAndReveal(startOffset, endOffset - startOffset, revealStartOffset, revealEndOffset - revealStartOffset);
+                       } finally {
+                               fStartOffset.clear();
+                               fEndOffset.clear();
+                       }
+               }
+
+               private boolean isValidSelection(int offset, int length) {
+                       IDocumentProvider provider= getDocumentProvider();
+                       if (provider != null) {
+                               IDocument document= provider.getDocument(getEditorInput());
+                               if (document != null) {
+                                       int end= offset + length;
+                                       int documentLength= document.getLength();
+                                       return 0 <= offset  && offset <= documentLength && 0 <= end && end <= documentLength;
+                               }
+                       }
+                       return false;
+               }
+               
+       }
 
+       /**
+        * Remembers additional data for a given
+        * offset to be able restore it later.
+        * 
+        * @since 3.0
+        */
+       private class RememberedOffset {
+               /** Remembered line for the given offset */
+               private int fLine;
+               /** Remembered column for the given offset*/
+               private int fColumn;
+               /** Remembered Java element for the given offset*/
+               private IJavaElement fElement;
+               /** Remembered Java element line for the given offset*/
+               private int fElementLine;
+               
+               /**
+                * Store visual properties of the given offset.  
+                * 
+                * @param offset Offset in the document
+                */
+               public void setOffset(int offset) {
+                       try {
+                               IDocument document= getSourceViewer().getDocument();
+                               fLine= document.getLineOfOffset(offset);
+                               fColumn= offset - document.getLineOffset(fLine);
+                               fElement= getElementAt(offset, true);
+
+                               fElementLine= -1;
+                               if (fElement instanceof IMember) {
+                                       ISourceRange range= ((IMember) fElement).getNameRange();
+                                       if (range != null)
+                                               fElementLine= document.getLineOfOffset(range.getOffset());
+                               }
+                               if (fElementLine == -1)
+                                       fElementLine= document.getLineOfOffset(getOffset(fElement));
+                       } catch (BadLocationException e) {
+                               // should not happen
+                               PHPeclipsePlugin.log(e);
+                               clear();
+                       } catch (JavaModelException e) {
+                               // should not happen
+                         PHPeclipsePlugin.log(e.getStatus());
+                               clear();
+                       }
+               }
+
+               /**
+                * Return offset recomputed from stored visual properties.  
+                * 
+                * @return Offset in the document
+                */
+               public int getOffset() {
+                       IJavaElement newElement= getElement();
+                       
+                       int offset= getRememberedOffset(newElement);
+                       
+                       if (offset != -1 && !containsOffset(newElement, offset) && (offset == 0 || !containsOffset(newElement, offset - 1)))
+                               return -1;
+                       
+                       return offset;
+               }
+               
+               /**
+                * Return offset recomputed from stored visual properties.  
+                * 
+                * @param newElement Enclosing element
+                * @return Offset in the document
+                */
+               public int getRememberedOffset(IJavaElement newElement) {
+                       try {
+                               if (newElement == null)
+                                       return -1;
+                               
+                               IDocument document= getSourceViewer().getDocument();
+                               int newElementLine= -1;
+                               if (newElement instanceof IMember) {
+                                       ISourceRange range= ((IMember) newElement).getNameRange();
+                                       if (range != null)
+                                               newElementLine= document.getLineOfOffset(range.getOffset());
+                               }
+                               if (newElementLine == -1)
+                                       newElementLine= document.getLineOfOffset(getOffset(newElement));
+                               if (newElementLine == -1)
+                                       return -1;
+
+                               int newLine= fLine + newElementLine - fElementLine;
+                               if (newLine < 0 || newLine >= document.getNumberOfLines())
+                                       return -1;
+                               int maxColumn= document.getLineLength(newLine);
+                               String lineDelimiter= document.getLineDelimiter(newLine);
+                               if (lineDelimiter != null)
+                                       maxColumn= maxColumn - lineDelimiter.length();
+                               int offset;
+                               if (fColumn > maxColumn)
+                                       offset= document.getLineOffset(newLine) + maxColumn;
+                               else
+                                       offset= document.getLineOffset(newLine) + fColumn;
+
+                               return offset;
+                       } catch (BadLocationException e) {
+                               // should not happen
+                         PHPeclipsePlugin.log(e);
+                               return -1;
+                       } catch (JavaModelException e) {
+                               // should not happen
+                         PHPeclipsePlugin.log(e.getStatus());
+                               return -1;
+                       }
+               }
+               
+               /**
+                * Returns the offset used to reveal the given element based on the given selection offset.
+                * @param element the element
+                * @param offset the selection offset
+                * @return the offset to reveal the given element based on the given selection offset
+                */
+               public int getRevealOffset(IJavaElement element, int offset) {
+                       if (element == null || offset == -1)
+                               return -1;
+
+                       if (containsOffset(element, offset)) {
+                               if (offset > 0) {
+                                       IJavaElement alternateElement= getElementAt(offset, false);
+                                       if (element.getHandleIdentifier().equals(alternateElement.getParent().getHandleIdentifier()))
+                                               return offset - 1; // Solves test case 2 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
+                               }
+                               return offset;
+                       } else if (offset > 0 && containsOffset(element, offset - 1))
+                               return offset - 1; // Solves test case 1 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
+                       
+                       return -1;
+               }
+
+               /**
+                * Return Java element recomputed from stored visual properties.  
+                * 
+                * @return Java element
+                */
+               public IJavaElement getElement() {
+                       if (fElement == null)
+                               return null;
+                       
+                       return findElement(fElement);
+               }
+
+               /**
+                * Clears the stored position 
+                */
+               public void clear() {
+                       fLine= -1;
+                       fColumn= -1;
+                       fElement= null;
+                       fElementLine= -1;
+               }
+               
+               /**
+                * Does the given Java element contain the given offset?
+                * @param element Java element
+                * @param offset Offset
+                * @return <code>true</code> iff the Java element contains the offset
+                */
+               private boolean containsOffset(IJavaElement element, int offset) {
+                       int elementOffset= getOffset(element);
+                       int elementLength= getLength(element);
+                       return (elementOffset > -1 && elementLength > -1) ? (offset >= elementOffset && offset < elementOffset + elementLength) : false;
+               }
+               /**
+                * Returns the offset of the given Java element.
+                * 
+                * @param element       Java element
+                * @return Offset of the given Java element
+                */
+               private int getOffset(IJavaElement element) {
+                       if (element instanceof ISourceReference) {
+                               ISourceReference sr= (ISourceReference) element;
+                               try {
+                                       ISourceRange srcRange= sr.getSourceRange();
+                                       if (srcRange != null)
+                                               return srcRange.getOffset();
+                               } catch (JavaModelException e) {
+                               }
+                       }
+                       return -1;      
+               }
+               
+               /**
+                * Returns the length of the given Java element.
+                * 
+                * @param element       Java element
+                * @return Length of the given Java element
+                */
+               private int getLength(IJavaElement element) {
+                       if (element instanceof ISourceReference) {
+                               ISourceReference sr= (ISourceReference) element;
+                               try {
+                                       ISourceRange srcRange= sr.getSourceRange();
+                                       if (srcRange != null)
+                                               return srcRange.getLength();
+                               } catch (JavaModelException e) {
+                               }
+                       }
+                       return -1;      
+               }
+               
+               /**
+                * Returns the updated java element for the old java element.
+                * 
+                * @param element Old Java element
+                * @return Updated Java element
+                */
+               private IJavaElement findElement(IJavaElement element) {
+                       
+                       if (element == null)
+                               return null;
+                       
+                       IWorkingCopyManager manager= PHPeclipsePlugin.getDefault().getWorkingCopyManager();
+                       ICompilationUnit unit= manager.getWorkingCopy(getEditorInput());
+                       
+                       if (unit != null) {
+                               try {
+                                       
+                                       synchronized (unit) {
+//                                             unit.reconcile(ICompilationUnit.NO_AST, false, null, null);
+                                               unit.reconcile();
+                                       }
+                                       IJavaElement[] findings= unit.findElements(element);
+                                       if (findings != null && findings.length > 0)
+                                               return findings[0];
+                               
+                               } catch (JavaModelException x) {
+                                 PHPeclipsePlugin.log(x.getStatus());
+                                       // nothing found, be tolerant and go on
+                               }
+                       }
+                       
+                       return null;
+               }
+               
+       }
+       
   static class TabConverter implements ITextConverter {
     private int fTabRatio;
 
@@ -619,8 +719,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
       LinkedPositionUI.ExitListener {
     private boolean fCloseBracketsPHP = true;
 
-    private boolean fCloseStringsPHP = true;
-
+    private boolean fCloseStringsPHPDQ = true;
+    private boolean fCloseStringsPHPSQ = true;
     private boolean fCloseBracketsHTML = true;
 
     private boolean fCloseStringsHTML = true;
@@ -633,10 +733,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
       fCloseBracketsPHP = enabled;
     }
 
-    public void setCloseStringsPHPEnabled(boolean enabled) {
-      fCloseStringsPHP = enabled;
+    public void setCloseStringsPHPDQEnabled(boolean enabled) {
+      fCloseStringsPHPDQ = enabled;
+    }
+    public void setCloseStringsPHPSQEnabled(boolean enabled) {
+      fCloseStringsPHPSQ = enabled;
     }
-
     public void setCloseBracketsHTMLEnabled(boolean enabled) {
       fCloseBracketsHTML = enabled;
     }
@@ -704,7 +806,9 @@ public class PHPUnitEditor extends PHPEditor { //implements
       try {
         ITypedRegion partition = document.getPartition(offset);
         String type = partition.getType();
-        if (type.equals(IPHPPartitions.PHP_PARTITIONING)) {
+        if (type.equals(IPHPPartitions.PHP_PARTITIONING)
+            || type.equals(IDocument.DEFAULT_CONTENT_TYPE)) {
+          // you will get IDocument.DEFAULT_CONTENT_TYPE for both PHP and HTML area
           switch (event.character) {
           case '(':
             if (hasCharacterToTheRight(document, offset + length, '('))
@@ -718,7 +822,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
           // fall through
           case '"':
             if (event.character == '"') {
-              if (!fCloseStringsPHP)
+              if (!fCloseStringsPHPDQ)
                 return;
               // changed for statements like echo "" print ""
               //    if (hasIdentifierToTheLeft(document, offset)
@@ -735,46 +839,38 @@ public class PHPUnitEditor extends PHPEditor { //implements
             // &&
             // (partition.getOffset() != offset))
             //         return;
-            final char character = event.character;
-            final char closingCharacter = getPeerCharacter(character);
-            final StringBuffer buffer = new StringBuffer();
-            buffer.append(character);
-            buffer.append(closingCharacter);
-            document.replace(offset, length, buffer.toString());
-            LinkedPositionManager manager = new LinkedPositionManager(document);
-            manager.addPosition(offset + 1, 0);
+            final char characterDQ = event.character;
+            final char closingCharacterDQ = getPeerCharacter(characterDQ);
+            final StringBuffer bufferDQ = new StringBuffer();
+            bufferDQ.append(characterDQ);
+            bufferDQ.append(closingCharacterDQ);
+            document.replace(offset, length, bufferDQ.toString());
+            LinkedPositionManager managerDQ = new LinkedPositionManager(document);
+            managerDQ.addPosition(offset + 1, 0);
             fOffset = offset;
             fLength = 2;
-            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
-                manager);
-            editor.setCancelListener(this);
-            editor.setExitPolicy(new ExitPolicy(closingCharacter));
-            editor.setFinalCaretOffset(offset + 2);
-            editor.enter();
-            IRegion newSelection = editor.getSelectedRegion();
-            sourceViewer.setSelectedRange(newSelection.getOffset(),
-                newSelection.getLength());
+            LinkedPositionUI editorDQ = new LinkedPositionUI(sourceViewer,
+                managerDQ);
+            editorDQ.setCancelListener(this);
+            editorDQ.setExitPolicy(new ExitPolicy(closingCharacterDQ));
+            editorDQ.setFinalCaretOffset(offset + 2);
+            editorDQ.enter();
+            IRegion newSelectionDQ = editorDQ.getSelectedRegion();
+            sourceViewer.setSelectedRange(newSelectionDQ.getOffset(),
+                newSelectionDQ.getLength());
             event.doit = false;
-          }
-        } else if (type.equals(IPHPPartitions.HTML)
-            || type.equals(IDocument.DEFAULT_CONTENT_TYPE)) {
-          switch (event.character) {
-          case '(':
-            if (hasCharacterToTheRight(document, offset + length, '('))
-              return;
-          // fall through
-          case '[':
-            if (!fCloseBracketsHTML)
-              return;
-            if (hasIdentifierToTheRight(document, offset + length))
-              return;
-          // fall through
-          case '"':
-            if (event.character == '"') {
-              if (!fCloseStringsHTML)
+            break;
+//          fall through
+          case '\'':
+            if (event.character == '\'') {
+              if (!fCloseStringsPHPSQ)
                 return;
-              if (hasIdentifierToTheLeft(document, offset)
-                  || hasIdentifierToTheRight(document, offset + length))
+              // changed for statements like echo "" print ""
+              //    if (hasIdentifierToTheLeft(document, offset)
+              // ||
+              // hasIdentifierToTheRight(document, offset +
+              // length))
+              if (hasIdentifierToTheRight(document, offset + length))
                 return;
             }
             //     ITypedRegion partition=
@@ -784,28 +880,77 @@ public class PHPUnitEditor extends PHPEditor { //implements
             // &&
             // (partition.getOffset() != offset))
             //         return;
-            final char character = event.character;
-            final char closingCharacter = getPeerCharacter(character);
-            final StringBuffer buffer = new StringBuffer();
-            buffer.append(character);
-            buffer.append(closingCharacter);
-            document.replace(offset, length, buffer.toString());
-            LinkedPositionManager manager = new LinkedPositionManager(document);
-            manager.addPosition(offset + 1, 0);
+            final char characterSQ = event.character;
+            final char closingCharacterSQ = getPeerCharacter(characterSQ);
+            final StringBuffer bufferSQ = new StringBuffer();
+            bufferSQ.append(characterSQ);
+            bufferSQ.append(closingCharacterSQ);
+            document.replace(offset, length, bufferSQ.toString());
+            LinkedPositionManager managerSQ = new LinkedPositionManager(document);
+            managerSQ.addPosition(offset + 1, 0);
             fOffset = offset;
             fLength = 2;
-            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
-                manager);
-            editor.setCancelListener(this);
-            editor.setExitPolicy(new ExitPolicy(closingCharacter));
-            editor.setFinalCaretOffset(offset + 2);
-            editor.enter();
-            IRegion newSelection = editor.getSelectedRegion();
-            sourceViewer.setSelectedRange(newSelection.getOffset(),
-                newSelection.getLength());
+            LinkedPositionUI editorSQ = new LinkedPositionUI(sourceViewer,
+                managerSQ);
+            editorSQ.setCancelListener(this);
+            editorSQ.setExitPolicy(new ExitPolicy(closingCharacterSQ));
+            editorSQ.setFinalCaretOffset(offset + 2);
+            editorSQ.enter();
+            IRegion newSelectionSQ = editorSQ.getSelectedRegion();
+            sourceViewer.setSelectedRange(newSelectionSQ.getOffset(),
+                newSelectionSQ.getLength());
             event.doit = false;
           }
         }
+//        } else if (type.equals(IPHPPartitions.HTML)) {
+//          switch (event.character) {
+//          case '(':
+//            if (hasCharacterToTheRight(document, offset + length, '('))
+//              return;
+//          // fall through
+//          case '[':
+//            if (!fCloseBracketsHTML)
+//              return;
+//            if (hasIdentifierToTheRight(document, offset + length))
+//              return;
+//          // fall through
+//          case '"':
+//            if (event.character == '"') {
+//              if (!fCloseStringsHTML)
+//                return;
+//              if (hasIdentifierToTheLeft(document, offset)
+//                  || hasIdentifierToTheRight(document, offset + length))
+//                return;
+//            }
+//            //     ITypedRegion partition=
+//            // document.getPartition(offset);
+//            //       if (!
+//            // IDocument.DEFAULT_CONTENT_TYPE.equals(partition.getType())
+//            // &&
+//            // (partition.getOffset() != offset))
+//            //         return;
+//            final char character = event.character;
+//            final char closingCharacter = getPeerCharacter(character);
+//            final StringBuffer buffer = new StringBuffer();
+//            buffer.append(character);
+//            buffer.append(closingCharacter);
+//            document.replace(offset, length, buffer.toString());
+//            LinkedPositionManager manager = new LinkedPositionManager(document);
+//            manager.addPosition(offset + 1, 0);
+//            fOffset = offset;
+//            fLength = 2;
+//            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
+//                manager);
+//            editor.setCancelListener(this);
+//            editor.setExitPolicy(new ExitPolicy(closingCharacter));
+//            editor.setFinalCaretOffset(offset + 2);
+//            editor.enter();
+//            IRegion newSelection = editor.getSelectedRegion();
+//            sourceViewer.setSelectedRange(newSelection.getOffset(),
+//                newSelection.getLength());
+//            event.doit = false;
+//          }
+//        }
       } catch (BadLocationException e) {
       }
     }
@@ -862,8 +1007,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
   /** The remembered java element */
   private IJavaElement fRememberedElement;
 
-  /** The remembered selection */
-  private ITextSelection fRememberedSelection;
+  /**
+        * The remembered selection.
+        * @since 3.0
+        */
+ private RememberedSelection fRememberedSelection= new RememberedSelection();
+       
 
   /** The remembered php element offset */
   private int fRememberedElementOffset;
@@ -973,8 +1122,10 @@ public class PHPUnitEditor extends PHPEditor { //implements
   /** Preference key for unknown annotation indication in overview ruler */
   private final static String UNKNOWN_INDICATION_IN_OVERVIEW_RULER = PreferenceConstants.EDITOR_UNKNOWN_INDICATION_IN_OVERVIEW_RULER;
 
-  /** Preference key for automatically closing strings */
-  private final static String CLOSE_STRINGS_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_PHP;
+  /** Preference key for automatically closing double quoted strings */
+  private final static String CLOSE_STRINGS_DQ_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_DQ_PHP;
+  /** Preference key for automatically closing single quoted strings */
+  private final static String CLOSE_STRINGS_SQ_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_SQ_PHP;
 
   /** Preference key for automatically wrapping Java strings */
   private final static String WRAP_STRINGS = PreferenceConstants.EDITOR_WRAP_STRINGS;
@@ -1060,6 +1211,22 @@ public class PHPUnitEditor extends PHPEditor { //implements
   /**
    * Creates a new php unit editor.
    */
+  
+  /**
+        * Reconciling listeners.
+        * @since 3.0
+        */
+       private ListenerList fReconcilingListeners= new ListenerList();
+
+       /**
+        * Mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898
+        * for a description of the problem.
+        * <p>
+        * TODO remove once the underlying problem is solved.
+        * </p>
+        */
+       private final Object fReconcilerLock= new Object();
+  
   public PHPUnitEditor() {
     super();
     setDocumentProvider(PHPeclipsePlugin.getDefault()
@@ -1282,11 +1449,13 @@ public class PHPUnitEditor extends PHPEditor { //implements
     //    preferences.addPropertyChangeListener(fPropertyChangeListener);
     IPreferenceStore preferenceStore = getPreferenceStore();
     boolean closeBracketsPHP = preferenceStore.getBoolean(CLOSE_BRACKETS_PHP);
-    boolean closeStringsPHP = preferenceStore.getBoolean(CLOSE_STRINGS_PHP);
+    boolean closeStringsPHPDQ = preferenceStore.getBoolean(CLOSE_STRINGS_DQ_PHP);
+    boolean closeStringsPHPSQ = preferenceStore.getBoolean(CLOSE_STRINGS_SQ_PHP);
     boolean closeBracketsHTML = preferenceStore.getBoolean(CLOSE_BRACKETS_HTML);
     boolean closeStringsHTML = preferenceStore.getBoolean(CLOSE_STRINGS_HTML);
     fBracketInserter.setCloseBracketsPHPEnabled(closeBracketsPHP);
-    fBracketInserter.setCloseStringsPHPEnabled(closeStringsPHP);
+    fBracketInserter.setCloseStringsPHPDQEnabled(closeStringsPHPDQ);
+    fBracketInserter.setCloseStringsPHPSQEnabled(closeStringsPHPSQ);
     fBracketInserter.setCloseBracketsHTMLEnabled(closeBracketsHTML);
     fBracketInserter.setCloseStringsHTMLEnabled(closeStringsHTML);
     ISourceViewer sourceViewer = getSourceViewer();
@@ -1307,6 +1476,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
       return '[';
     case '"':
       return character;
+    case '\'':
+      return character;
     default:
       throw new IllegalArgumentException();
     }
@@ -1562,8 +1733,13 @@ public class PHPUnitEditor extends PHPEditor { //implements
               .getBoolean(p));
           return;
         }
-        if (CLOSE_STRINGS_PHP.equals(p)) {
-          fBracketInserter.setCloseStringsPHPEnabled(getPreferenceStore()
+        if (CLOSE_STRINGS_DQ_PHP.equals(p)) {
+          fBracketInserter.setCloseStringsPHPDQEnabled(getPreferenceStore()
+              .getBoolean(p));
+          return;
+        }
+        if (CLOSE_STRINGS_SQ_PHP.equals(p)) {
+          fBracketInserter.setCloseStringsPHPSQEnabled(getPreferenceStore()
               .getBoolean(p));
           return;
         }
@@ -1746,28 +1922,6 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /*
-   * @see AbstractTextEditor#canHandleMove(IEditorInput, IEditorInput)
-   */
-  protected boolean canHandleMove(IEditorInput originalElement,
-      IEditorInput movedElement) {
-    String oldExtension = ""; //$NON-NLS-1$
-    if (originalElement instanceof IFileEditorInput) {
-      IFile file = ((IFileEditorInput) originalElement).getFile();
-      if (file != null) {
-        String ext = file.getFileExtension();
-        if (ext != null)
-          oldExtension = ext;
-      }
-    }
-    String newExtension = ""; //$NON-NLS-1$
-    if (movedElement instanceof IFileEditorInput) {
-      IFile file = ((IFileEditorInput) movedElement).getFile();
-      if (file != null)
-        newExtension = file.getFileExtension();
-    }
-    return oldExtension.equals(newExtension);
-  }
-  /*
         * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#getInputElement()
         */
        protected IJavaElement getInputJavaElement() {
@@ -1795,6 +1949,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
       page.setInput(manager.getWorkingCopy(input));
     }
   }
+  
 
   /*
    * @see AbstractTextEditor#performSaveOperation(WorkspaceModifyOperation,
@@ -1872,10 +2027,6 @@ public class PHPUnitEditor extends PHPEditor { //implements
       } else
         performSave(false, progressMonitor);
     }
-    ShowExternalPreviewAction a = ShowExternalPreviewAction.getInstance();
-    if (a != null) {
-      a.refresh();
-    }
   }
 
   public boolean isSaveAsAllowed() {
@@ -2115,26 +2266,47 @@ public class PHPUnitEditor extends PHPEditor { //implements
     return true;
   }
 
+       /*
+        * @see org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#aboutToBeReconciled()
+        * @since 3.0
+        */
+       public void aboutToBeReconciled() {
+
+               // Notify AST provider
+//             PHPeclipsePlugin.getDefault().getASTProvider().aboutToBeReconciled(getInputJavaElement());
+               
+               // Notify listeners
+               Object[] listeners = fReconcilingListeners.getListeners();
+               for (int i = 0, length= listeners.length; i < length; ++i)
+                       ((IJavaReconcilingListener)listeners[i]).aboutToBeReconciled();
+       }
+       
   /*
-   * @see IReconcilingParticipant#reconciled()
-   */
-  public void reconciled() {
-    if (synchronizeOutlineOnCursorMove()) {
-      Shell shell = getSite().getShell();
-      if (shell != null && !shell.isDisposed()) {
-        shell.getDisplay().asyncExec(new Runnable() {
-          public void run() {
-            synchronizeOutlinePageSelection();
-          }
-        });
-      }
-    }
-  }
-
-  protected void updateStateDependentActions() {
-    super.updateStateDependentActions();
-    fGenerateActionGroup.editorStateChanged();
-  }
+        * @see org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#reconciled(CompilationUnit, boolean, IProgressMonitor)
+        * @since 3.0
+        */
+       public void reconciled(CompilationUnit ast, boolean forced, IProgressMonitor progressMonitor) {
+
+               // Always notify AST provider
+//             PHPeclipsePlugin.getDefault().getASTProvider().reconciled(ast, getInputJavaElement());
+               
+               // Notify listeners
+//             Object[] listeners = fReconcilingListeners.getListeners();
+//             for (int i = 0, length= listeners.length; i < length; ++i)
+//                     ((IJavaReconcilingListener)listeners[i]).reconciled(ast, forced, progressMonitor);
+
+               // Update Java Outline page selection
+               if (!forced && !progressMonitor.isCanceled()) {
+                       Shell shell= getSite().getShell();
+                       if (shell != null && !shell.isDisposed()) {
+                               shell.getDisplay().asyncExec(new Runnable() {
+                                       public void run() {
+                                               selectionChanged();
+                                       }
+                               });
+                       }
+               }
+       }
 
   private boolean synchronizeOutlineOnCursorMove() {
     return PreferenceConstants.getPreferenceStore().getBoolean(
@@ -2182,86 +2354,157 @@ public class PHPUnitEditor extends PHPEditor { //implements
     return -1;
   }
 
-  /*
-   * @see AbstractTextEditor#rememberSelection()
-   */
-  protected void rememberSelection() {
-    ISelectionProvider sp = getSelectionProvider();
-    fRememberedSelection = (sp == null ? null : (ITextSelection) sp
-        .getSelection());
-    if (fRememberedSelection != null) {
-      fRememberedElement = getElementAt(fRememberedSelection.getOffset(), true);
-      fRememberedElementOffset = getOffset(fRememberedElement);
-    }
-  }
+  
 
   /*
    * @see AbstractTextEditor#restoreSelection()
    */
-  protected void restoreSelection() {
-    try {
-      if (getSourceViewer() == null || fRememberedSelection == null)
-        return;
-      IJavaElement newElement = findElement(fRememberedElement);
-      int newOffset = getOffset(newElement);
-      int delta = (newOffset > -1 && fRememberedElementOffset > -1) ? newOffset
-          - fRememberedElementOffset : 0;
-      if (isValidSelection(delta + fRememberedSelection.getOffset(),
-          fRememberedSelection.getLength()))
-        selectAndReveal(delta + fRememberedSelection.getOffset(),
-            fRememberedSelection.getLength());
-    } finally {
-      fRememberedSelection = null;
-      fRememberedElement = null;
-      fRememberedElementOffset = -1;
+//  protected void restoreSelection() {
+//    try {
+//      if (getSourceViewer() == null || fRememberedSelection == null)
+//        return;
+//      IJavaElement newElement = findElement(fRememberedElement);
+//      int newOffset = getOffset(newElement);
+//      int delta = (newOffset > -1 && fRememberedElementOffset > -1) ? newOffset
+//          - fRememberedElementOffset : 0;
+//      if (isValidSelection(delta + fRememberedSelection.getOffset(),
+//          fRememberedSelection.getLength()))
+//        selectAndReveal(delta + fRememberedSelection.getOffset(),
+//            fRememberedSelection.getLength());
+//    } finally {
+//      fRememberedSelection = null;
+//      fRememberedElement = null;
+//      fRememberedElementOffset = -1;
+//    }
+//  }
+
+  /**
+        * Tells whether this is the active editor in the active page.
+        *
+        * @return <code>true</code> if this is the active editor in the active page
+        * @see IWorkbenchPage#getActiveEditor();
+        */
+       protected final boolean isActiveEditor() {
+               IWorkbenchWindow window= getSite().getWorkbenchWindow();
+               IWorkbenchPage page= window.getActivePage();
+               if (page == null)
+                       return false;
+               IEditorPart activeEditor= page.getActiveEditor();
+               return activeEditor != null && activeEditor.equals(this);
+       }
+       
+       /**
+        * Adds the given listener.
+        * Has no effect if an identical listener was not already registered.
+        * 
+        * @param listener      The reconcile listener to be added
+        * @since 3.0
+        */
+       final void addReconcileListener(IJavaReconcilingListener listener) {
+               synchronized (fReconcilingListeners) {
+                       fReconcilingListeners.add(listener);
+               }
+       }
+       
+       /**
+        * Removes the given listener.
+        * Has no effect if an identical listener was not already registered.
+        * 
+        * @param listener      the reconcile listener to be removed
+        * @since 3.0
+        */
+       final void removeReconcileListener(IJavaReconcilingListener listener) {
+               synchronized (fReconcilingListeners) {
+                       fReconcilingListeners.remove(listener);
+               }
+       }
+               
+       protected void updateStateDependentActions() {
+               super.updateStateDependentActions();
+               fGenerateActionGroup.editorStateChanged();
+       }
+       
+       /*
+        * @see AbstractTextEditor#rememberSelection()
+        */
+       protected void rememberSelection() {
+               fRememberedSelection.remember();
+       }
+       
+       /*
+        * @see AbstractTextEditor#restoreSelection()
+        */
+       protected void restoreSelection() {
+               fRememberedSelection.restore();
+       }
+       
+       /*
+        * @see AbstractTextEditor#canHandleMove(IEditorInput, IEditorInput)
+        */
+       protected boolean canHandleMove(IEditorInput originalElement, IEditorInput movedElement) {
+               
+               String oldExtension= ""; //$NON-NLS-1$
+               if (originalElement instanceof IFileEditorInput) {
+                       IFile file= ((IFileEditorInput) originalElement).getFile();
+                       if (file != null) {
+                               String ext= file.getFileExtension();
+                               if (ext != null)
+                                       oldExtension= ext;
+                       }
+               }
+               
+               String newExtension= ""; //$NON-NLS-1$
+               if (movedElement instanceof IFileEditorInput) {
+                       IFile file= ((IFileEditorInput) movedElement).getFile();
+                       if (file != null)
+                               newExtension= file.getFileExtension();
+               }
+               
+               return oldExtension.equals(newExtension);
+       }
+
+       /*
+        * @see org.eclipse.ui.texteditor.AbstractDecoratedTextEditor#isPrefQuickDiffAlwaysOn()
+        */
+       protected boolean isPrefQuickDiffAlwaysOn() {
+               // reestablishes the behaviour from AbstractDecoratedTextEditor which was hacked by JavaEditor
+               // to disable the change bar for the class file (attached source) java editor.
+               IPreferenceStore store= getPreferenceStore();
+               return store.getBoolean(AbstractDecoratedTextEditorPreferenceConstants.QUICK_DIFF_ALWAYS_ON);
+       }
+       
+       /*
+        * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#getAdapter(java.lang.Class)
+        */
+       public Object getAdapter(Class required) {
+               if (SmartBackspaceManager.class.equals(required)) {
+                       if (getSourceViewer() instanceof JavaSourceViewer) {
+                               return ((JavaSourceViewer) getSourceViewer()).getBackspaceManager();
+                       }
+               }
+
+               return super.getAdapter(required);
+       }
+  /**
+        * Returns the mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898
+        * for a description of the problem.
+        * <p>
+        * TODO remove once the underlying problem is solved.
+        * </p>
+        * @return the lock reconcilers may use to synchronize on
+        */
+       public Object getReconcilerLock() {
+               return fReconcilerLock;
+       }
+       
+  /* (non-Javadoc)
+   * @see org.eclipse.ui.texteditor.AbstractTextEditor#editorSaved()
+   */
+  protected void editorSaved() {
+    super.editorSaved();
+    ShowExternalPreviewAction a = ShowExternalPreviewAction.getInstance();
+    if (a != null) {
+      a.refresh(ShowExternalPreviewAction.PHP_TYPE);
     }
   }
-
-  //   /*
-  //    * @see
-  // org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#aboutToBeReconciled()
-  //    * @since 3.0
-  //    */
-  //   public void aboutToBeReconciled() {
-  //
-  //           // Notify AST provider
-  //           PHPeclipsePlugin.getDefault().getASTProvider().aboutToBeReconciled(getInputJavaElement());
-  //           
-  //           // Notify listeners
-  //           Object[] listeners = fReconcilingListeners.getListeners();
-  //           for (int i = 0, length= listeners.length; i < length; ++i)
-  //                   ((IJavaReconcilingListener)listeners[i]).aboutToBeReconciled();
-  //   }
-  //   
-  //   /*
-  //    * @see
-  // org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#reconciled(CompilationUnit,
-  // boolean, IProgressMonitor)
-  //    * @since 3.0
-  //    */
-  //   public void reconciled(net.sourceforge.phpdt.core.dom.CompilationUnit
-  // ast, boolean forced, IProgressMonitor progressMonitor) {
-  //
-  //           // Always notify AST provider
-  //           PHPeclipsePlugin.getDefault().getASTProvider().reconciled(ast,
-  // getInputJavaElement());
-  //           
-  //           // Notify listeners
-  //           Object[] listeners = fReconcilingListeners.getListeners();
-  //           for (int i = 0, length= listeners.length; i < length; ++i)
-  //                   ((IJavaReconcilingListener)listeners[i]).reconciled(ast, forced,
-  // progressMonitor);
-  //
-  //           // Update Java Outline page selection
-  //           if (!forced && !progressMonitor.isCanceled()) {
-  //                   Shell shell= getSite().getShell();
-  //                   if (shell != null && !shell.isDisposed()) {
-  //                           shell.getDisplay().asyncExec(new Runnable() {
-  //                                   public void run() {
-  //                                           selectionChanged();
-  //                                   }
-  //                           });
-  //                   }
-  //           }
-  //   }
 }
\ No newline at end of file