CPLAT-7721 Add error logging to ErrorBoundary

lib/src/component/error_boundary.dart

@@ -167,8 +167,19 @@ class _$ErrorBoundaryProps extends UiProps {
 
   /// The name to use when the component's logger logs an error via [ErrorBoundaryComponent.componentDidCatch].
   ///
+  /// Not used if a custom [logger] is specified.
+  ///
   /// > Default: 'over_react.ErrorBoundary'
   String loggerName;
+
+  /// Whether errors caught by this [ErrorBoundary] should be logged using a [Logger].
+  ///
+  /// > Default: `true`
+  bool shouldLogErrors;
+
+  /// An optional custom logger instance that will be used to log errors caught by
+  /// this [ErrorBoundary] when [shouldLogErrors] is true.
+  Logger logger;
 }
 
 @State()
@@ -199,6 +210,7 @@ class ErrorBoundaryComponent<T extends ErrorBoundaryProps, S extends ErrorBounda
   Map getDefaultProps() => (newProps()
     ..identicalErrorFrequencyTolerance = Duration(seconds: 5)
     ..loggerName = defaultErrorBoundaryLoggerName
+    ..shouldLogErrors = true
   );
 
   @override
@@ -207,33 +219,6 @@ class ErrorBoundaryComponent<T extends ErrorBoundaryProps, S extends ErrorBounda
     ..showFallbackUIOnError = props.fallbackUIRenderer != null
   );
 
-  @mustCallSuper
-  @override
-  void componentWillMount() {
-    super.componentWillMount();
-
-    _logger = Logger(_getLoggerName());
-  }
-
-  @mustCallSuper
-  @override
-  void componentWillUnmount() {
-    super.componentWillUnmount();
-
-    _logger.clearListeners();
-  }
-
-  @mustCallSuper
-  @override
-  void componentWillReceiveProps(Map nextProps) {
-    super.componentWillReceiveProps(nextProps);
-
-    final tNextProps = typedPropsFactory(nextProps);
-    if (tNextProps.loggerName != props.loggerName) {
-      _logger = Logger(_getLoggerName(tNextProps));
-    }
-  }
-
   @mustCallSuper
   /*@override*/
   S getDerivedStateFromError(_) {
@@ -334,7 +319,7 @@ class ErrorBoundaryComponent<T extends ErrorBoundaryProps, S extends ErrorBounda
   //                   wrap in a try catch just in case `findDomNode` throws as a result of the
   //                   wrapped react tree rendering a string instead of a composite or dom component.
   //
-  // [3] Log the caught error using the instance Logger.
+  // [3] Log the caught error using a logger if `props.shouldLogErrors` is true.
   // ---------------------------------------------- /\ ----------------------------------------------
 
   String _domAtTimeOfError;
@@ -428,27 +413,24 @@ class ErrorBoundaryComponent<T extends ErrorBoundaryProps, S extends ErrorBounda
   String _getReadableErrorInfo(/*NativeJavascriptObject*/dynamic jsErrorInfo) =>
       getProperty(jsErrorInfo, 'componentStack');
 
-  /// The logger that logs errors when a component somewhere within the React tree wrapped by
-  /// this [ErrorBoundary] instance throws within the React lifecycle.
-  Logger get logger => _logger;
-  Logger _logger;
+  /// The value that will be used when creating a [Logger] to log errors from this component.
+  String get loggerName {
+    if (props.logger != null) return props.logger.name;
 
-  String _getLoggerName([T propsMap]) {
-    propsMap ??= props;
-    if (propsMap.loggerName != null && propsMap.loggerName.isNotEmpty) return propsMap.loggerName;
-
-    return defaultErrorBoundaryLoggerName;
+    return props.loggerName ?? defaultErrorBoundaryLoggerName;
   }
 
   // ----- [3] ----- //
   void _logErrorCaughtByErrorBoundary(
     /*Error|Exception*/ dynamic error,
-    /*ReactErrorInfo*/ dynamic info, {
+    /*ReactErrorInfo*/ String info, {
     bool isRecoverable = true,
   }) {
+    if (!props.shouldLogErrors) return;
+
     String message = isRecoverable
-        ? 'An error was caught by an ErrorBoundary'
-        : 'An unrecoverable error was caught by an ErrorBoundary (the entire react tree had to be unmounted)';
+        ? 'An error was caught by an ErrorBoundary: \nInfo: $info'
+        : 'An unrecoverable error was caught by an ErrorBoundary (attempting to remount it was unsuccessful): \nInfo: $info';
 
     dynamic stackTrace;
     try {
@@ -457,6 +439,6 @@ class ErrorBoundaryComponent<T extends ErrorBoundaryProps, S extends ErrorBounda
       // The error / exception doesn't extend from Error or Exception
     }
 
-    _logger.severe(message, error, stackTrace);
+    (props.logger ?? Logger(loggerName)).severe(message, error, stackTrace);
   }
 }