Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: src/objects.h

Issue 2203803002: [debug] Don't notify listener of exceptions internal to a desugaring. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebase Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/isolate.cc ('k') | src/parsing/parser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 337cb3f33f1a43081d2fbc4cceb09d8a4a9ba93f..38c154a4d6c01324430973af06861c41d862d87c 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -4466,9 +4466,13 @@ class HandlerTable : public FixedArray {
// exception or cause a re-throw to outside the code boundary. Since this is
// undecidable it is merely an approximation (e.g. useful for debugger).
enum CatchPrediction {
- UNCAUGHT, // the handler will (likely) rethrow the exception.
- CAUGHT, // the exception will be caught by the handler.
- PROMISE // the exception will be caught and cause a promise rejection.
+ UNCAUGHT, // The handler will (likely) rethrow the exception.
+ CAUGHT, // The exception will be caught by the handler.
+ PROMISE, // The exception will be caught and cause a promise rejection.
+ DESUGARING, // The exception will be caught, but both the exception and the
+ // catching are part of a desugaring and should therefore not
+ // be visible to the user (we won't notify the debugger of such
+ // exceptions).
};
// Getters for handler table based on ranges.
« no previous file with comments | « src/isolate.cc ('k') | src/parsing/parser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698