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

Unified Diff: pkg/analysis_services/lib/src/refactoring/rename_constructor.dart

Issue 484733003: Import analysis_services.dart into analysis_server.dart. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 4 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: pkg/analysis_services/lib/src/refactoring/rename_constructor.dart
diff --git a/pkg/analysis_services/lib/src/refactoring/rename_constructor.dart b/pkg/analysis_services/lib/src/refactoring/rename_constructor.dart
deleted file mode 100644
index b5cfd8a39c5bb9eb96fcafbc6b7e5eb1b839f213..0000000000000000000000000000000000000000
--- a/pkg/analysis_services/lib/src/refactoring/rename_constructor.dart
+++ /dev/null
@@ -1,83 +0,0 @@
-// Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
-// for details. All rights reserved. Use of this source code is governed by a
-// BSD-style license that can be found in the LICENSE file.
-
-library services.src.refactoring.rename_constructor;
-
-import 'dart:async';
-
-import 'package:analysis_services/correction/change.dart';
-import 'package:analysis_services/correction/status.dart';
-import 'package:analysis_services/refactoring/refactoring.dart';
-import 'package:analysis_services/search/hierarchy.dart';
-import 'package:analysis_services/search/search_engine.dart';
-import 'package:analysis_services/src/correction/util.dart';
-import 'package:analysis_services/src/refactoring/naming_conventions.dart';
-import 'package:analysis_services/src/refactoring/rename.dart';
-import 'package:analyzer/src/generated/element.dart';
-import 'package:analyzer/src/generated/java_core.dart';
-
-
-/**
- * A [Refactoring] for renaming [ConstructorElement]s.
- */
-class RenameConstructorRefactoringImpl extends RenameRefactoringImpl {
- RenameConstructorRefactoringImpl(SearchEngine searchEngine,
- ConstructorElement element)
- : super(searchEngine, element);
-
- @override
- ConstructorElement get element => super.element as ConstructorElement;
-
- @override
- String get refactoringName {
- return "Rename Constructor";
- }
-
- @override
- Future<RefactoringStatus> checkFinalConditions() {
- RefactoringStatus result = new RefactoringStatus();
- _analyzePossibleConflicts(result);
- return new Future.value(result);
- }
-
- @override
- RefactoringStatus checkNewName() {
- RefactoringStatus result = super.checkNewName();
- result.addStatus(validateConstructorName(newName));
- return result;
- }
-
- @override
- Future<Change> createChange() {
- Change change = new Change(refactoringName);
- String replacement = newName.isEmpty ? '' : '.${newName}';
- // update references
- return searchEngine.searchReferences(element).then((refMatches) {
- List<SourceReference> references = getSourceReferences(refMatches);
- if (!element.isSynthetic) {
- for (SourceReference reference in references) {
- Edit edit = createReferenceEdit(reference, replacement);
- change.addEdit(reference.file, edit);
- }
- }
- return change;
- });
- }
-
- void _analyzePossibleConflicts(RefactoringStatus result) {
- // check if there are members with "newName" in the same ClassElement
- ClassElement parentClass = element.enclosingElement;
- for (Element newNameMember in getChildren(parentClass, newName)) {
- String message =
- format(
- "Class '{0}' already declares {1} with name '{2}'.",
- parentClass.displayName,
- getElementKindName(newNameMember),
- newName);
- result.addError(
- message,
- new RefactoringStatusContext.forElement(newNameMember));
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698