Remove support for isAMD since php is now also commonjs

src/vs/platform/plugins/common/abstractPluginService.ts

@@ -233,11 +233,3 @@ export abstract class AbstractPluginService<T extends ActivatedPlugin> implement
 
 	protected abstract _actualActivatePlugin(pluginDescription: IPluginDescription): TPromise<T>;
 }
-
-export function loadAMDModule<T>(moduleId: string): TPromise<T> {
-	return new TPromise<T>((c, e, p) => {
-		require([moduleId], (r: T) => {
-			c(r);
-		}, e);
-	});
-}

src/vs/platform/plugins/common/nativePluginService.ts

@@ -10,7 +10,7 @@ import * as paths from 'vs/base/common/paths';
 import Severity from 'vs/base/common/severity';
 import {TPromise} from 'vs/base/common/winjs.base';
 import {IMessageService} from 'vs/platform/message/common/message';
-import {AbstractPluginService, ActivatedPlugin, IPluginContext, IPluginMemento, loadAMDModule} from 'vs/platform/plugins/common/abstractPluginService';
+import {AbstractPluginService, ActivatedPlugin, IPluginContext, IPluginMemento} from 'vs/platform/plugins/common/abstractPluginService';
 import {IMessage, IPluginDescription, IPluginStatus} from 'vs/platform/plugins/common/plugins';
 import {PluginsRegistry} from 'vs/platform/plugins/common/pluginsRegistry';
 import {PluginHostStorage} from 'vs/platform/storage/common/remotable.storage';
@@ -339,10 +339,6 @@ export class PluginHostPluginService extends AbstractPluginService<ExtHostPlugin
 	}
 
 	protected _loadPluginModule(pluginDescription: IPluginDescription): TPromise<IPluginModule> {
-		if (pluginDescription.isAMD) {
-			return loadAMDModule(uriFromPath(pluginDescription.main));
-		}
-
 		return loadCommonJSModule(pluginDescription.main);
 	}
 
@@ -429,18 +425,3 @@ function loadCommonJSModule<T>(modulePath: string): TPromise<T> {
 	}
 	return TPromise.as(r);
 }
-
-
-// TODO@Alex: Duplicated in:
-// * src\bootstrap.js
-// * src\vs\workbench\electron-main\bootstrap.js
-// * src\vs\platform\plugins\common\nativePluginService.ts
-function uriFromPath(_path) {
-	let pathName = _path.replace(/\\/g, '/');
-
-	if (pathName.length > 0 && pathName.charAt(0) !== '/') {
-		pathName = '/' + pathName;
-	}
-
-	return encodeURI('file://' + pathName);
-}
\ No newline at end of file

src/vs/platform/plugins/common/plugins.ts

@@ -22,7 +22,6 @@ export interface IPluginDescription {
 	};
 	main?: string;
 	contributes?: { [point: string]: any; };
-	isAMD: boolean;
 }
 
 export interface IActivationEventListener {

src/vs/platform/plugins/common/pluginsRegistry.ts

@@ -360,10 +360,6 @@ const schema: IJSONSchema = {
 				// extensions will fill in
 			},
 			default: {}
-		},
-		isAMD: {
-			description: nls.localize('vscode.extension.isAMD', 'Indicated whether VS Code should load your code as AMD or CommonJS. Default: false.'),
-			type: 'boolean'
 		}
 	}
 };