allow parsing frames with eval url

Author: vzaidman

front_end/panels/console/ErrorStackParser.ts

@@ -22,20 +22,27 @@ export interface ParsedErrorFrame {
 }
 
 export type SpecialHermesStackTraceFrameTypes =
-  // functions implemented in c++.
-  // TODO: these might be enhanced to include the C++ loc for the frame
-  // so that a debugger could stitch together a hybrid cross-language call stack
+  // e.g "(native)"- Functions implemented on the native side.
+  // TODO: Might be enhanced to include the native (C++/Java/etc) loc
+  // for the frame so that a debugger could stitch together a
+  // hybrid cross-language call stack
   'native' |
-  // frames with empty url
-  // TODO: these seem to be happening due to a bug that needs to be investigated
+  // e.g "(eval:1:2)"- Seem to be reported when there's a
+  // ReferenceError or TypeError during the initial bundle code execution.
+  // TODO: Understand exactly where these originate from and what further work
+  // should be done in regards to them
+  'eval' |
+  // e.g "(:3:4)"- Frames with empty url
+  // TODO: Seems to be happening due to a bug that needs to be investigated
   // and produce an actual script URL instead
   'address at' |
-  // frames pointing to a bytecode locations
-  // TODO: these could be symbolicated and link to source files with the help of
+  // e.g "(address at InternalBytecode.js:5:6)"- Frames pointing to bytecode locations
+  // TODO: Could be symbolicated and link to source files with the help of
   // a bytecode source maps once they are available.
   'empty url' |
-  // frames collepsed in the middle of a stack trace for very long stack traces
-  'skipping x frames';
+  // e.g " ... skipping 7 frames" - Frames collapsed in the middle of a stack trace
+  // for very long stack traces
+  'skipping x frames' ;
 
 function getSpecialHermesStackTraceFrameType({
     url,
@@ -46,6 +53,10 @@ function getSpecialHermesStackTraceFrameType({
     return 'native';
   }
 
+  if (url === 'eval') {
+    return 'eval';
+  }
+
   if (url === '') {
     return 'empty url';
   }