Fixing test failures

microsoft · Sep 28, 2017 · 70217ca · 70217ca
1 parent afa17ae
commit 70217ca
Showing 1 changed file with 17 additions and 4 deletions.
diff --git a/src/chrome/chromeDebugAdapter.ts b/src/chrome/chromeDebugAdapter.ts
@@ -503,7 +503,10 @@ export abstract class ChromeDebugAdapter implements IDebugAdapter {
                 const pausedScriptUrl = this._scriptsById.get(pausedScriptId).url;
 
                 const mappedUrl = this._pathTransformer.scriptParsed(pausedScriptUrl);
-                const normalizedMappedUrl = path.normalize(mappedUrl);
+                let normalizedMappedUrl: string;
+                if (mappedUrl !== undefined) {
+                    normalizedMappedUrl = path.normalize(mappedUrl);
+                }
 
                 // If the file has unbound breakpoints, make sure to resolve them first and then decide to continue or not
                 if (this._pendingBreakpointsByUrl.has(normalizedMappedUrl)) {
@@ -665,7 +668,10 @@ export abstract class ChromeDebugAdapter implements IDebugAdapter {
 
         const resolvePendingBPs = (source: string) => {
             source = source && this.fixPathCasing(source);
-            const normalizedSource = path.normalize(source);
+            let normalizedSource: string;
+            if (source !== undefined) {
+                normalizedSource = path.normalize(source);
+            }
 
             if (this._pendingBreakpointsByUrl.has(normalizedSource)) {
                 this.resolvePendingBreakpoint(this._pendingBreakpointsByUrl.get(normalizedSource))
@@ -674,7 +680,10 @@ export abstract class ChromeDebugAdapter implements IDebugAdapter {
         };
 
         const mappedUrl = this._pathTransformer.scriptParsed(script.url);
-        const normalizedMappedUrl = path.normalize(mappedUrl);
+        let normalizedMappedUrl: string;
+        if (mappedUrl !== undefined) {
+            normalizedMappedUrl = path.normalize(mappedUrl);
+        }
 
         const sourceMapsP = this._sourceMapTransformer.scriptParsed(mappedUrl, script.sourceMapURL).then(sources => {
             if (this._hasTerminated) {
@@ -958,7 +967,11 @@ export abstract class ChromeDebugAdapter implements IDebugAdapter {
             column: params.location.columnNumber
         };
         const scriptPath = this._pathTransformer.breakpointResolved(bp, script.url);
-        const normalizedScriptPath = path.normalize(scriptPath);
+        let normalizedScriptPath: string;
+        if (scriptPath !== undefined) {
+            normalizedScriptPath = path.normalize(scriptPath);
+        }
+
         if (this._pendingBreakpointsByUrl.has(normalizedScriptPath)) {
             // If we set these BPs before the script was loaded, remove from the pending list
             this._pendingBreakpointsByUrl.delete(normalizedScriptPath);