Index: extensions/renderer/module_system.cc |
diff --git a/extensions/renderer/module_system.cc b/extensions/renderer/module_system.cc |
index 6a120749f4107499b322fc42f998c9f83528a703..0b75e96edd460a2dd4dd55641127e9c64fdc17db 100644 |
--- a/extensions/renderer/module_system.cc |
+++ b/extensions/renderer/module_system.cc |
@@ -31,16 +31,13 @@ const char* kModuleName = "module_name"; |
const char* kModuleField = "module_field"; |
const char* kModulesField = "modules"; |
-// Logs a fatal error for the calling context, with some added metadata about |
-// the context: |
+// Logs an error for the calling context in preparation for potentially |
+// crashing the renderer, with some added metadata about the context: |
// - Its type (blessed, unblessed, etc). |
// - Whether it's valid. |
// - The extension ID, if one exists. |
-// |
-// This will only actual be fatal in in dev/canary, since in too many cases |
-// we're at the mercy of the extension or web page's environment. They can mess |
-// up our JS in unexpected ways. Hopefully dev/canary channel will pick up such |
-// problems, but given the wider variety on stable/beta it's impossible to know. |
+// Crashing won't happen in stable/beta releases, but is encouraged to happen |
+// in the less stable released to catch errors early. |
void Fatal(ScriptContext* context, const std::string& message) { |
// Prepend some context metadata. |
std::string full_message = "("; |