rename CoolParameter -> ParameterBase -> Parameter

src/Features/Core/Portable/ChangeSignature/AbstractChangeSignatureService.cs

@@ -136,7 +136,7 @@ internal ChangeSignatureResult ChangeSignature(Document document, int position,
                 return new ChangeSignatureAnalyzedContext(CannotChangeSignatureReason.IncorrectKind);
             }
 
-            var parameterConfiguration = ParameterConfiguration.Create(symbol.GetParameters().Select(p => new ExistingParameter(p)).ToList<ParameterBase>(), symbol is IMethodSymbol && (symbol as IMethodSymbol).IsExtensionMethod, selectedIndex);
+            var parameterConfiguration = ParameterConfiguration.Create(symbol.GetParameters().Select(p => new ExistingParameter(p)).ToList<Parameter>(), symbol is IMethodSymbol && (symbol as IMethodSymbol).IsExtensionMethod, selectedIndex);
             if (!parameterConfiguration.IsChangeable())
             {
                 return new ChangeSignatureAnalyzedContext(CannotChangeSignatureReason.InsufficientParameters);

src/Features/Core/Portable/ChangeSignature/ParameterBase.cs → src/Features/Core/Portable/ChangeSignature/Parameter.cs

@@ -2,14 +2,14 @@
 
 namespace Microsoft.CodeAnalysis.ChangeSignature
 {
-    internal abstract class ParameterBase
+    internal abstract class Parameter
     {
         public abstract bool HasExplicitDefaultValue { get; }
         public abstract string Name { get; }
         public abstract IParameterSymbol Symbol { get; }
     }
 
-    internal class ExistingParameter : ParameterBase
+    internal class ExistingParameter : Parameter
     {
         public override IParameterSymbol Symbol { get; }
 
@@ -22,7 +22,7 @@ public ExistingParameter(IParameterSymbol param)
         public override string Name => Symbol.Name;
     }
 
-    internal class AddedParameter : ParameterBase
+    internal class AddedParameter : Parameter
     {
         public AddedParameter(string type, string parameter, string callsite)
         {

src/Features/Core/Portable/ChangeSignature/ParameterConfiguration.cs

 // Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
 
-using System;
 using System.Collections.Generic;
 using System.Linq;
 
@@ -8,13 +7,13 @@ namespace Microsoft.CodeAnalysis.ChangeSignature
 {
     internal sealed class ParameterConfiguration
     {
-        public readonly ParameterBase ThisParameter;
-        public readonly List<ParameterBase> ParametersWithoutDefaultValues;
-        public readonly List<ParameterBase> RemainingEditableParameters;
-        public readonly ParameterBase ParamsParameter;
+        public readonly Parameter ThisParameter;
+        public readonly List<Parameter> ParametersWithoutDefaultValues;
+        public readonly List<Parameter> RemainingEditableParameters;
+        public readonly Parameter ParamsParameter;
         public readonly int SelectedIndex;
 
-        public ParameterConfiguration(ParameterBase thisParameter, List<ParameterBase> parametersWithoutDefaultValues, List<ParameterBase> remainingEditableParameters, ParameterBase paramsParameter, int selectedIndex)
+        public ParameterConfiguration(Parameter thisParameter, List<Parameter> parametersWithoutDefaultValues, List<Parameter> remainingEditableParameters, Parameter paramsParameter, int selectedIndex)
         {
             ThisParameter = thisParameter;
             ParametersWithoutDefaultValues = parametersWithoutDefaultValues;
@@ -23,12 +22,12 @@ public ParameterConfiguration(ParameterBase thisParameter, List<ParameterBase> p
             SelectedIndex = selectedIndex;
         }
 
-        public static ParameterConfiguration Create(List<ParameterBase> parameters, bool isExtensionMethod, int selectedIndex)
+        public static ParameterConfiguration Create(List<Parameter> parameters, bool isExtensionMethod, int selectedIndex)
         {
-            ParameterBase thisParameter = null;
-            var parametersWithoutDefaultValues = new List<ParameterBase>();
-            var remainingReorderableParameters = new List<ParameterBase>();
-            ParameterBase paramsParameter = null;
+            Parameter thisParameter = null;
+            var parametersWithoutDefaultValues = new List<Parameter>();
+            var remainingReorderableParameters = new List<Parameter>();
+            Parameter paramsParameter = null;
 
             if (parameters.Count > 0 && isExtensionMethod)
             {
@@ -58,12 +57,12 @@ public static ParameterConfiguration Create(List<ParameterBase> parameters, bool
 
         internal ParameterConfiguration WithoutAddedParameters()
         {
-            return Create(ToListOfParameters().OfType<ExistingParameter>().ToList<ParameterBase>(), ThisParameter != null, selectedIndex: 0);
+            return Create(ToListOfParameters().OfType<ExistingParameter>().ToList<Parameter>(), ThisParameter != null, selectedIndex: 0);
         }
 
-        public List<ParameterBase> ToListOfParameters()
+        public List<Parameter> ToListOfParameters()
         {
-            var list = new List<ParameterBase>();
+            var list = new List<Parameter>();
 
             if (ThisParameter != null)
             {
@@ -81,7 +80,7 @@ public List<ParameterBase> ToListOfParameters()
             return list;
         }
 
-        // TODO probably remove thiis check. It was created when we didn't support Add Parameters to parameterless methods.
+        // TODO probably remove this check. It was created when we didn't support Add Parameters to parameterless methods.
         public bool IsChangeable()
         {
             return true;

src/VisualStudio/Core/Def/Implementation/ChangeSignature/ChangeSignatureDialogViewModel.cs

@@ -523,7 +523,7 @@ public ParameterViewModel(ChangeSignatureDialogViewModel changeSignatureDialogVi
                 this.changeSignatureDialogViewModel = changeSignatureDialogViewModel;
             }
 
-            internal abstract ParameterBase CreateParameter();
+            internal abstract Parameter CreateParameter();
 
             public abstract string InitialIndex { get; }
         }
@@ -548,7 +548,7 @@ public AddedParameterViewModel(ChangeSignatureDialogViewModel changeSignatureDia
             public override bool IsDisabled => false;
             public override string Callsite => _addParameterViewModel.CallsiteValue;
 
-            internal override ParameterBase CreateParameter()
+            internal override Parameter CreateParameter()
                 => new AddedParameter(Type, Parameter, Callsite);
 
             public override string InitialIndex => "NEW";
@@ -558,14 +558,14 @@ public class ExistingParameterViewModel : ParameterViewModel
         {
             public IParameterSymbol ParameterSymbol { get; }
 
-            public ExistingParameterViewModel(ChangeSignatureDialogViewModel changeSignatureDialogViewModel, ParameterBase parameter, int initialIndex)
+            public ExistingParameterViewModel(ChangeSignatureDialogViewModel changeSignatureDialogViewModel, Parameter parameter, int initialIndex)
                 : base(changeSignatureDialogViewModel)
             {
                 ParameterSymbol = (parameter as ExistingParameter).Symbol;
                 InitialIndex = initialIndex.ToString();
             }
 
-            internal override ParameterBase CreateParameter()
+            internal override Parameter CreateParameter()
             {
                 return new ExistingParameter(ParameterSymbol);
             }

src/VisualStudio/Core/Test/ChangeSignature/ChangeSignatureViewModelTests.vb

@@ -436,7 +436,7 @@ class MyClass
 
                 Dim viewModel = New ChangeSignatureDialogViewModel(
                     New TestNotificationService(),
-                    ParameterConfiguration.Create(symbol.GetParameters().Select(Function(p) DirectCast(New ExistingParameter(p), ParameterBase)).ToList(), symbol.IsExtensionMethod(), selectedIndex:=0),
+                    ParameterConfiguration.Create(symbol.GetParameters().Select(Function(p) DirectCast(New ExistingParameter(p), Parameter)).ToList(), symbol.IsExtensionMethod(), selectedIndex:=0),
                     symbol,
                     workspace.ExportProvider.GetExportedValue(Of IClassificationFormatMapService)().GetClassificationFormatMap("text"),
                     workspace.ExportProvider.GetExportedValue(Of ClassificationTypeMap)())