remo unused code

#38414

Author: isidorn

src/vs/base/common/actions.ts

@@ -41,33 +41,6 @@ export interface IActionItem {
 	dispose(): void;
 }
 
-/**
- * Checks if the provided object is compatible
- * with the IAction interface.
- * @param thing an object
- */
-export function isAction(thing: any): thing is IAction {
-	if (!thing) {
-		return false;
-	} else if (thing instanceof Action) {
-		return true;
-	} else if (typeof thing.id !== 'string') {
-		return false;
-	} else if (typeof thing.label !== 'string') {
-		return false;
-	} else if (typeof thing.class !== 'string') {
-		return false;
-	} else if (typeof thing.enabled !== 'boolean') {
-		return false;
-	} else if (typeof thing.checked !== 'boolean') {
-		return false;
-	} else if (typeof thing.run !== 'function') {
-		return false;
-	} else {
-		return true;
-	}
-}
-
 export interface IActionChangeEvent {
 	label?: string;
 	tooltip?: string;

src/vs/base/test/common/actions.test.ts

@@ -25,7 +25,7 @@ export function registerCommands(): void {
 	KeybindingsRegistry.registerCommandAndKeybindingRule({
 		id: 'debug.logToDebugConsole',
 		weight: KeybindingsRegistry.WEIGHT.workbenchContrib(),
-		handler(accessor: ServicesAccessor, value: string) {
+		handler: (accessor: ServicesAccessor, value: string) => {
 			if (typeof value === 'string') {
 				const debugService = accessor.get(IDebugService);
 				// Use warning as severity to get the orange color for messages coming from the debug extension

src/vs/workbench/parts/debug/electron-browser/debugCommands.ts

@@ -225,33 +225,6 @@ export class FileStat implements IFileStat {
 		this.hasChildren = this.children.length > 0;
 	}
 
-	/**
-	 * Returns true if this stat is a directory that contains a child with the given name.
-	 *
-	 * @param ignoreCase if true, will check for the name ignoring case.
-	 * @param type the type of stat to check for.
-	 */
-	public hasChild(name: string, ignoreCase?: boolean, type: StatType = StatType.ANY): boolean {
-		for (let i = 0; i < this.children.length; i++) {
-			const child = this.children[i];
-			if ((type === StatType.FILE && child.isDirectory) || (type === StatType.FOLDER && !child.isDirectory)) {
-				continue;
-			}
-
-			// Check for Identity
-			if (child.name === name) {
-				return true;
-			}
-
-			// Also consider comparing without case
-			if (ignoreCase && child.name.toLowerCase() === name.toLowerCase()) {
-				return true;
-			}
-		}
-
-		return false;
-	}
-
 	/**
 	 * Removes a child element from this folder.
 	 */
@@ -383,10 +356,6 @@ export class NewStatPlaceholder extends FileStat {
 		throw new Error('Can\'t perform operations in NewStatPlaceholder.');
 	}
 
-	public hasChild(name: string, ignoreCase?: boolean): boolean {
-		return false;
-	}
-
 	public removeChild(child: NewStatPlaceholder): void {
 		throw new Error('Can\'t perform operations in NewStatPlaceholder.');
 	}

src/vs/workbench/parts/files/common/explorerModel.ts

@@ -44,16 +44,9 @@ suite('Files - View Model', () => {
 		const s = createStat('/path/to/stat', 'sName', true, false, 8096, d);
 
 		const child1 = createStat('/path/to/stat/foo', 'foo', true, false, 8096, d);
-		const child2 = createStat('/path/to/stat/bar.html', 'bar', false, false, 8096, d);
 		const child4 = createStat('/otherpath/to/other/otherbar.html', 'otherbar.html', false, false, 8096, d);
 
-		assert(!s.hasChild(child1.name));
-		assert(!s.hasChild(child2.name));
-
 		s.addChild(child1);
-		assert(s.hasChild(child1.name));
-		assert(!s.hasChild(child1.name.toUpperCase()));
-		assert(s.hasChild(child1.name.toUpperCase(), true));
 
 		assert(s.children.length === 1);
 		assert(s.hasChildren);