Merge pull request #18563 from amcasey/GH18546

Stop requiring that the full range of a declaration fall within the selection

src/harness/unittests/extractMethods.ts

@@ -768,6 +768,11 @@ function parsePrimaryExpression(): any {
             }
         }|]
     }
+}`);
+        // Selection excludes leading trivia of declaration
+        testExtractMethod("extractMethod33",
+            `function F() {
+    [#|function G() { }|]
 }`);
     });
 

src/services/refactors/extractMethod.ts

@@ -1205,7 +1205,7 @@ namespace ts.refactor.extractMethod {
             if (!declInFile) {
                 return undefined;
             }
-            if (rangeContainsRange(enclosingTextRange, declInFile)) {
+            if (rangeContainsStartEnd(enclosingTextRange, declInFile.getStart(), declInFile.end)) {
                 // declaration is located in range to be extracted - do nothing
                 return undefined;
             }