Revert "Map the paths back to https:"

This reverts commit 0edf650.
microsoft · sheetalkamat · Dec 9, 2020 · Jul 18, 2020 · Jul 23, 2020 · Aug 31, 2020
commit 16ff1ce041fd1dc4f2419c7b9fbba65694a318ea
diff --git a/src/tsserver/webServer.ts b/src/tsserver/webServer.ts
@@ -140,17 +140,12 @@ namespace ts.server {
         return typeof cmdLineVerbosity === "undefined" ? nullLogger : new MainProcessLogger(cmdLineVerbosity);
     }
 
-    interface WebServerHost extends ServerHost {
-        getWebPath: (path: string) => string | undefined;
-    }
-
     function createWebSystem(args: string[]) {
         Debug.assert(ts.sys === undefined);
         const returnEmptyString = () => "";
         // Later we could map ^memfs:/ to do something special if we want to enable more functionality like module resolution or something like that
         const getWebPath = (path: string) => startsWith(path, directorySeparator) ? path.replace(directorySeparator, executingDirectoryPath) : undefined;
-        const sys: WebServerHost = {
-            getWebPath,
+        const sys: ServerHost = {
             args,
             newLine: "\r\n", // This can be configured by clients
             useCaseSensitiveFileNames: false, // Use false as the default on web since that is the safest option
@@ -238,8 +233,10 @@ namespace ts.server {
     function startWebSession(options: StartSessionOptions, logger: Logger, cancellationToken: ServerCancellationToken) {
         class WorkerSession extends Session<{}> {
             constructor() {
+                const host = sys as ServerHost;
+
                 super({
-                    host: sys as WebServerHost,
+                    host,
                     cancellationToken,
                     ...options,
                     typingsInstaller: nullTypingsInstaller,
@@ -251,9 +248,6 @@ namespace ts.server {
             }
 
             public send(msg: protocol.Message) {
-                // Updates to file paths
-                this.updateWebPaths(msg);
-
                 if (msg.type === "event" && !this.canUseEvents) {
                     if (this.logger.hasLevel(LogLevel.verbose)) {
                         this.logger.info(`Session does not support events: ignored event: ${JSON.stringify(msg)}`);
@@ -266,30 +260,6 @@ namespace ts.server {
                 postMessage(msg);
             }
 
-            private updateWebPaths(obj: any) {
-                if (isArray(obj)) {
-                    obj.forEach(ele => this.updateWebPaths(ele));
-                }
-                else if (typeof obj === "object") {
-                    for (const id in obj) {
-                        if (hasProperty(obj, id)) {
-                            const value = obj[id];
-                            if ((id === "file" || id === "fileName" || id === "renameFilename") && isString(value)) {
-                                const webpath = (sys as WebServerHost).getWebPath(value);
-                                if (webpath) obj[id] = webpath;
-                            }
-                            else if ((id === "files" || id === "fileNames") && isArray(value) && value.every(isString)) {
-                                obj[id] = value.map(ele => (sys as WebServerHost).getWebPath(ele) || ele);
-                            }
-                            else {
-                                this.updateWebPaths(value);
-                            }
-                        }
-                    }
-
-                }
-            }
-
             protected parseMessage(message: {}): protocol.Request {
                 return <protocol.Request>message;
             }
@@ -309,6 +279,8 @@ namespace ts.server {
                     this.onMessage(message.data);
                 });
             }
+
+            // TODO:: Update all responses to use webPath
         }
 
         const session = new WorkerSession();