debug: get rid of SimpleActionProvider

src/vs/workbench/parts/debug/browser/debugViewer.ts

@@ -119,33 +119,6 @@ function renderRenameBox(debugService: debug.IDebugService, contextViewService:
 	}));
 }
 
-export class SimpleActionProvider implements renderer.IActionProvider {
-
-	constructor() {
-		// noop
-	}
-
-	public hasActions(tree: tree.ITree, element: any): boolean {
-		return false;
-	}
-
-	public getActions(tree: tree.ITree, element: any): TPromise<actions.IAction[]> {
-		return Promise.as([]);
-	}
-
-	public hasSecondaryActions(tree: tree.ITree, element: any): boolean {
-		return false;
-	}
-
-	public getSecondaryActions(tree: tree.ITree, element: any): TPromise<actions.IAction[]> {
-		return Promise.as([]);
-	}
-
-	public getActionItem(tree: tree.ITree, element: any, action: actions.IAction): actionbar.IActionItem {
-		return null;
-	}
-}
-
 export class BaseDebugController extends treedefaults.DefaultController {
 
 	constructor(protected debugService: debug.IDebugService, private contextMenuService: IContextMenuService, private actionProvider: renderer.IActionProvider, private focusOnContextMenu = true) {
@@ -317,12 +290,11 @@ export class CallStackRenderer implements tree.IRenderer {
 
 // variables
 
-export class VariablesActionProvider extends SimpleActionProvider {
+export class VariablesActionProvider implements renderer.IActionProvider {
 
 	private instantiationService: IInstantiationService;
 
 	constructor(instantiationService: IInstantiationService) {
-		super();
 		this.instantiationService = instantiationService;
 	}
 
@@ -330,6 +302,10 @@ export class VariablesActionProvider extends SimpleActionProvider {
 		return false;
 	}
 
+	public getActions(tree: tree.ITree, element: any): TPromise<actions.IAction[]> {
+		return Promise.as([]);
+	}
+
 	public hasSecondaryActions(tree: tree.ITree, element: any): boolean {
 		return element instanceof model.Variable;
 	}
@@ -344,6 +320,10 @@ export class VariablesActionProvider extends SimpleActionProvider {
 
 		return Promise.as(actions);
 	}
+
+	public getActionItem(tree: tree.ITree, element: any, action: actions.IAction): actionbar.IActionItem {
+		return null;
+	}
 }
 
 export class VariablesDataSource implements tree.IDataSource {
@@ -449,12 +429,11 @@ export class VariablesRenderer implements tree.IRenderer {
 
 // watch expressions
 
-export class WatchExpressionsActionProvider extends SimpleActionProvider {
+export class WatchExpressionsActionProvider implements renderer.IActionProvider {
 
 	private instantiationService: IInstantiationService;
 
 	constructor(instantiationService: IInstantiationService) {
-		super();
 		this.instantiationService = instantiationService;
 	}
 
@@ -501,6 +480,10 @@ export class WatchExpressionsActionProvider extends SimpleActionProvider {
 
 		return Promise.as(actions);
 	}
+
+	public getActionItem(tree: tree.ITree, element: any, action: actions.IAction): actionbar.IActionItem {
+		return null;
+	}
 }
 
 export class WatchExpressionsDataSource implements tree.IDataSource {
@@ -669,10 +652,10 @@ export class WatchExpressionsController extends BaseDebugController {
 
 // breakpoints
 
-export class BreakpointsActionProvider extends SimpleActionProvider {
+export class BreakpointsActionProvider implements renderer.IActionProvider {
 
 	constructor(private instantiationService: IInstantiationService) {
-		super();
+		// noop
 	}
 
 	public hasActions(tree: tree.ITree, element: any): boolean {
@@ -717,6 +700,10 @@ export class BreakpointsActionProvider extends SimpleActionProvider {
 
 		return Promise.as(actions);
 	}
+
+	public getActionItem(tree: tree.ITree, element: any, action: actions.IAction): actionbar.IActionItem {
+		return null;
+	}
 }
 
 export class BreakpointsDataSource implements tree.IDataSource {