Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java

Issue 12340077: Initial binding of RenameRefactoring service to Eclipse. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java
diff --git a/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java b/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java
index e42b14010f88dfd3dd5ecbd88667cfb3058a2c89..ebc62349e79e4592bab772c532896467790e52ec 100644
--- a/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java
+++ b/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/internal/refactoring/RenameRefactoringImpl.java
@@ -51,6 +51,11 @@ public abstract class RenameRefactoringImpl extends RenameRefactoring {
@Override
public RefactoringStatus checkInitialConditions(ProgressMonitor pm) throws Exception {
+ return new RefactoringStatus();
+ }
+
+ @Override
+ public RefactoringStatus checkNewName(String newName) {
RefactoringStatus result = new RefactoringStatus();
if (Objects.equal(newName, element.getName())) {
result.addFatalError("Choose another name.");
@@ -58,6 +63,16 @@ public abstract class RenameRefactoringImpl extends RenameRefactoring {
return result;
}
+ @Override
+ public String getCurrentName() {
+ return element.getName();
+ }
+
+ @Override
+ public String getNewName() {
+ return newName;
+ }
+
/**
* @return the {@link Set} with all direct and indirect sub {@link ClassElement}s of the given.
*/

Powered by Google App Engine
This is Rietveld 408576698