Remove unused stackFrameId argument

Author: andyleejordan

src/PowerShellEditorServices/Services/DebugAdapter/DebugService.cs

@@ -320,9 +320,8 @@ public VariableDetailsBase[] GetVariables(int variableReferenceId)
         /// walk the cached variable data for the specified stack frame.
         /// </summary>
         /// <param name="variableExpression">The variable expression string to evaluate.</param>
-        /// <param name="stackFrameId">The ID of the stack frame in which the expression should be evaluated.</param>
         /// <returns>A VariableDetailsBase object containing the result.</returns>
-        public VariableDetailsBase GetVariableFromExpression(string variableExpression, int stackFrameId)
+        public VariableDetailsBase GetVariableFromExpression(string variableExpression)
         {
             // NOTE: From a watch we will get passed expressions that are not naked variables references.
             // Probably the right way to do this would be to examine the AST of the expr before calling
@@ -517,14 +516,12 @@ public async Task<string> SetVariableAsync(int variableContainerReferenceId, str
         /// PowerShellContext.
         /// </summary>
         /// <param name="expressionString">The expression string to execute.</param>
-        /// <param name="stackFrameId">The ID of the stack frame in which the expression should be executed.</param>
         /// <param name="writeResultAsOutput">
         /// If true, writes the expression result as host output rather than returning the results.
         /// In this case, the return value of this function will be null.</param>
         /// <returns>A VariableDetails object containing the result.</returns>
         public async Task<VariableDetails> EvaluateExpressionAsync(
             string expressionString,
-            int stackFrameId,
             bool writeResultAsOutput)
         {
             var command = new PSCommand().AddScript(expressionString);

src/PowerShellEditorServices/Services/DebugAdapter/Handlers/DebugEvaluateHandler.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -61,16 +61,14 @@ public async Task<EvaluateResponseBody> Handle(EvaluateRequestArguments request,
                 if (_debugContext.IsStopped)
                 {
                     // First check to see if the watch expression refers to a naked variable reference.
-                    result =
-                        _debugService.GetVariableFromExpression(request.Expression, request.FrameId);
+                    result = _debugService.GetVariableFromExpression(request.Expression);
 
                     // If the expression is not a naked variable reference, then evaluate the expression.
                     if (result == null)
                     {
                         result =
                             await _debugService.EvaluateExpressionAsync(
                                 request.Expression,
-                                request.FrameId,
                                 isFromRepl).ConfigureAwait(false);
                     }
                 }