Handle truncated XML frame info gracefully

When the XML frame has insufficient information, gracefully handle the
case by returning the input as-is, instead of the earlier behavior of
throwing an internal exception and failing.

Author: arvindshmicrosoft

Engine/ModuleInfoHelper.cs

@@ -96,15 +96,15 @@ await Task.Run(() => Parallel.ForEach(listOfCallStacks, currItem => {
                                     var pdbGuid = reader.GetAttribute("guid");
                                     var pdbAge = reader.GetAttribute("age");
                                     string uniqueModuleName;
-                                    // createe a map of the last mapped module names to handle cases when the frame is "truncated" and the above PDB details are not available
+                                    // Create a map of the last mapped module names to handle cases when the frame is "truncated" and the above PDB details are not available
                                     if (pdbGuid != null && pdbAge != null) {
                                         uniqueModuleName = $"{pdbGuid.Replace("-", string.Empty).ToUpper()}{pdbAge}";
                                         if (latestMappedModuleNames.ContainsKey(moduleName)) latestMappedModuleNames[moduleName] = uniqueModuleName;
                                         else latestMappedModuleNames.Add(moduleName, uniqueModuleName);
                                     } else {
                                         if (!latestMappedModuleNames.TryGetValue(moduleName, out uniqueModuleName)) {
-                                            anyTaskFailed = true;
-                                            return;
+                                            outCallstack.AppendLine(line);
+                                            continue;
                                         }
                                     }
                                     lock (syms) {

Tests/Tests.cs

@@ -616,7 +616,7 @@ private string PrepareLargeXEventInput() {
             // modify the input to not have any prior PDB info - this will be an "error" case
             input = "Frame = <frame id=\"02\" name=\"sqldk.dll\" address = \"0x100440609\"/>\r\n<frame id=\"03\" name=\"sqldk.dll\" address=\"0x10042249f\" />\n";
             ret = await csr.ResolveCallstacksAsync(await csr.GetListofCallStacksAsync(input, false, cts), pdbPath, false, null, false, true, false, true, false, false, null, cts);
-            Assert.IsTrue(ret.StartsWith("Unable to determine symbol information from XML frames"));
+            Assert.AreEqual("<frame id=\"02\" name=\"sqldk.dll\" address = \"0x100440609\"/>\r\n<frame id=\"03\" name=\"sqldk.dll\" address=\"0x10042249f\" />", ret.Trim());
         }
 
         /// End-to-end test with XE histogram target and XML frames.