chore: remove logs

Author: AndrewAminInstabug

lib/src/utils/instabug_navigator_observer.dart

@@ -1,5 +1,4 @@
 import 'dart:async';
-import 'dart:developer';
 
 import 'package:flutter/material.dart';
 import 'package:instabug_flutter/instabug_flutter.dart';
@@ -68,7 +67,6 @@ class InstabugNavigatorObserver extends NavigatorObserver {
 
   FutureOr<void> _startScreenRenderCollector(int? uiTraceId) async {
     final isScreenRender = await FlagsConfig.screenRendering.isEnabled();
-    log("isScreenRender: $isScreenRender", name: 'Andrew');
     _checkForScreenRenderInitialization(isScreenRender);
     if (uiTraceId != null && isScreenRender) {
       InstabugScreenRenderManager.I

lib/src/utils/screen_rendering/instabug_screen_render_manager.dart

@@ -1,5 +1,4 @@
 import 'dart:async';
-import 'dart:developer';
 import 'dart:ui';
 
 import 'package:flutter/widgets.dart';
@@ -10,8 +9,6 @@ import 'package:instabug_flutter/src/utils/instabug_logger.dart';
 import 'package:instabug_flutter/src/utils/screen_rendering/instabug_widget_binding_observer.dart';
 import 'package:meta/meta.dart';
 
-//todo: remove logs
-
 @internal
 enum UiTraceType {
   auto,
@@ -65,7 +62,6 @@ class InstabugScreenRenderManager {
     try {
       // passing WidgetsBinding? (nullable) for flutter versions prior than 3.x
       if (!screenRenderEnabled && widgetBinding != null) {
-        log("$tag: init", name: 'andrew');
         _widgetsBinding = widgetBinding;
         _addWidgetBindingObserver();
         await _initStaticValues();
@@ -84,7 +80,6 @@ class InstabugScreenRenderManager {
     _rasterTime = frameTiming.rasterDuration.inMilliseconds;
     _totalTime = frameTiming.totalSpan.inMilliseconds;
 
-    _displayFrameTimingDetails(frameTiming);
     if (_isUiFrozen) {
       _frozenFramesTotalDurationMs += _buildTime;
     } else if (_isRasterFrozen) {
@@ -126,16 +121,8 @@ class InstabugScreenRenderManager {
     try {
       // Return if frameTimingListener not attached
       if (!screenRenderEnabled || !_isTimingsListenerAttached) {
-        log(
-          "$tag: startScreenRenderCollectorForTraceId returned",
-          name: 'andrew',
-        );
         return;
       }
-      log(
-        "$tag: startScreenRenderCollectorForTraceId $traceId (${type.name})",
-        name: 'andrew',
-      );
 
       //Save the memory cached data to be sent to native side
       if (_delayedFrames.isNotEmpty) {
@@ -188,8 +175,6 @@ class InstabugScreenRenderManager {
           _screenRenderForAutoUiTrace.isNotEmpty) {
         _reportScreenRenderForAutoUiTrace(_screenRenderForAutoUiTrace);
       }
-
-      log("$tag: stopScreenRenderCollector", name: 'andrew');
     } catch (error, stackTrace) {
       _logExceptionErrorAndStackTrace(error, stackTrace);
     }
@@ -210,7 +195,6 @@ class InstabugScreenRenderManager {
       _reportScreenRenderForCustomUiTrace(_screenRenderForCustomUiTrace);
 
       _screenRenderForCustomUiTrace.clear();
-      log("$tag: endCustom", name: 'andrew');
     } catch (error, stackTrace) {
       _logExceptionErrorAndStackTrace(error, stackTrace);
     }
@@ -222,13 +206,10 @@ class InstabugScreenRenderManager {
     _removeFrameTimings();
     _widgetsBinding = null;
     screenRenderEnabled = false;
-    log("$tag: dispose", name: 'andrew');
   }
 
   /// --------------------------- private methods ---------------------
 
-  bool get _isSlow => _isUiSlow || _isRasterSlow;
-
   bool get _isUiDelayed => _isUiSlow || _isUiFrozen;
 
   bool get _isRasterDelayed => _isRasterSlow || _isRasterFrozen;
@@ -241,8 +222,6 @@ class InstabugScreenRenderManager {
       _rasterTime > _slowFrameThresholdMs &&
       _rasterTime < _frozenFrameThresholdMs;
 
-  bool get _isFrozen => _isUiFrozen || _isRasterFrozen || _isTotalTimeLarge;
-
   bool get _isTotalTimeLarge => _totalTime >= _frozenFrameThresholdMs;
 
   bool get _isUiFrozen => _buildTime >= _frozenFrameThresholdMs;
@@ -272,7 +251,6 @@ class InstabugScreenRenderManager {
     _slowFrameThresholdMs = _targetMsPerFrame(_deviceRefreshRate);
     _screenRenderForAutoUiTrace = InstabugScreenRenderData(frameData: []);
     _screenRenderForCustomUiTrace = InstabugScreenRenderData(frameData: []);
-    log("$tag: _initStaticValues ", name: 'andrew');
   }
 
   /// Add a frame observer by calling [WidgetsBinding.instance.addTimingsCallback]
@@ -282,29 +260,20 @@ class InstabugScreenRenderManager {
     }
     _widgetsBinding?.addTimingsCallback(_timingsCallback);
     _isTimingsListenerAttached = true;
-    log("$tag: _initFrameTimings ", name: 'andrew');
   }
 
   /// Remove the running frame observer by calling [_widgetsBinding.removeTimingsCallback]
   void _removeFrameTimings() {
     if (!_isTimingsListenerAttached) return; // No timings callback attached.
     _widgetsBinding?.removeTimingsCallback(_timingsCallback);
     _isTimingsListenerAttached = false;
-    log(
-      "$tag: _removeFrameTimings ",
-      name: 'andrew',
-    );
   }
 
   /// Reset the memory cashed data
   void _resetCachedFrameData() {
     _slowFramesTotalDurationMs = 0;
     _frozenFramesTotalDurationMs = 0;
     _delayedFrames.clear();
-    log(
-      "$tag: _resetCachedFrameData ",
-      name: 'andrew',
-    );
   }
 
   /// Save Slow/Frozen Frames data
@@ -318,45 +287,6 @@ class InstabugScreenRenderManager {
     );
   }
 
-  //todo: will be removed (is used for debugging)
-  void _displayFrameTimingDetails(FrameTiming frameTiming) {
-    if (_isSlow) {
-      debugPrint(
-        '========================= Slow frame detected ⚠️ =========================',
-      );
-    }
-    if (_isFrozen) {
-      debugPrint(
-        '========================= Frozen frame detected 🚨 =========================',
-      );
-    }
-
-    if (_isFrozen || _isSlow) {
-      debugPrint(
-        "{\n\t$frameTiming\n\t"
-        "Timestamps(${frameTiming.timestampInMicroseconds(
-          FramePhase.buildStart,
-        )}, ${frameTiming.timestampInMicroseconds(
-          FramePhase.buildFinish,
-        )}, ${frameTiming.timestampInMicroseconds(
-          FramePhase.rasterStart,
-        )}, ${frameTiming.timestampInMicroseconds(
-          FramePhase.rasterFinish,
-        )}, ${frameTiming.timestampInMicroseconds(
-          FramePhase.vsyncStart,
-        )}, ${frameTiming.timestampInMicroseconds(
-          FramePhase.rasterFinishWallTime,
-        )}"
-        ")\n}\n",
-      );
-      debugPrint("Device refresh rate: $_deviceRefreshRate FPS");
-      debugPrint(
-        "Threshold: $_slowFrameThresholdMs ms\n"
-        "===============================================================================",
-      );
-    }
-  }
-
   /// Ends custom ui trace with the screen render data that has been collected for it.
   /// params:
   ///   [InstabugScreenRenderData] screenRenderData.
@@ -398,10 +328,6 @@ class InstabugScreenRenderManager {
     if (_screenRenderForCustomUiTrace.isActive) {
       _updateCustomUiData();
     }
-    log(
-      "$tag: Captured data is saved ",
-      name: 'andrew',
-    );
   }
 
   /// Updates the custom UI trace screen render data with the currently collected

lib/src/utils/screen_rendering/instabug_widget_binding_observer.dart

@@ -1,12 +1,9 @@
-import 'dart:developer';
-
 import 'package:flutter/widgets.dart';
 import 'package:instabug_flutter/src/utils/screen_loading/screen_loading_manager.dart';
 import 'package:instabug_flutter/src/utils/screen_name_masker.dart';
 import 'package:instabug_flutter/src/utils/screen_rendering/instabug_screen_render_manager.dart';
 import 'package:meta/meta.dart';
 
-//todo: remove logs
 class InstabugWidgetsBindingObserver extends WidgetsBindingObserver {
   InstabugWidgetsBindingObserver._();
 
@@ -30,7 +27,6 @@ class InstabugWidgetsBindingObserver extends WidgetsBindingObserver {
   }
 
   void _handleResumedState() {
-    log('Performing resume actions...', name: 'andrew');
     final lastUiTrace = ScreenLoadingManager.I.currentUiTrace;
     if (lastUiTrace == null) {
       return;
@@ -48,23 +44,20 @@ class InstabugWidgetsBindingObserver extends WidgetsBindingObserver {
   }
 
   void _handlePausedState() {
-    log('Performing pause actions...', name: 'andrew');
-
     if (InstabugScreenRenderManager.I.screenRenderEnabled) {
       InstabugScreenRenderManager.I.stopScreenRenderCollector();
     }
   }
 
   void _handleDetachedState() {
-    log('Performing detached actions...', name: 'andrew');
     if (InstabugScreenRenderManager.I.screenRenderEnabled) {
       InstabugScreenRenderManager.I.stopScreenRenderCollector();
     }
     dispose();
   }
 
   void _handleDefaultState() {
-    log("handle default state", name: 'andrew');
+    // Added for lint warnings
   }
 
   @override