Merge pull request #604 from Instabug/feat/screen-render-local-store

update logic in the sample app

Author: AndrewAminInstabug

example/lib/src/screens/screen_render_page.dart

@@ -1,9 +1,14 @@
 part of '../../main.dart';
 
-class ScreenRenderPage extends StatelessWidget {
+class ScreenRenderPage extends StatefulWidget {
   const ScreenRenderPage({Key? key}) : super(key: key);
   static const String screenName = "/screenRenderPageRoute";
 
+  @override
+  State<ScreenRenderPage> createState() => _ScreenRenderPageState();
+}
+
+class _ScreenRenderPageState extends State<ScreenRenderPage> {
   @override
   Widget build(BuildContext context) {
     return Page(title: 'Screen Render', children: [
@@ -16,11 +21,11 @@ class ScreenRenderPage extends StatelessWidget {
       SizedBox.fromSize(size: const Size.fromHeight(16.0)),
       InstabugButton(
         text: 'Trigger Slow Frame',
-        onPressed: () => _simulateHeavyComputationForSlowFrames(),
+        onPressed: () => _simulateHeavyComputation(200),
       ),
       InstabugButton(
         text: 'Trigger Frozen Frame',
-        onPressed: () => _simulateHeavyComputationForFrozenFrames(),
+        onPressed: () => _simulateHeavyComputation(1000),
       ),
       InstabugButton(
         text: 'Monitored Complex Page',
@@ -42,19 +47,14 @@ class ScreenRenderPage extends StatelessWidget {
   }
 
   // Simulates a computationally expensive task
-  void _simulateHeavyComputationForSlowFrames() {
-    Random random = Random();
-    for (int i = 0; i < 1000000; i++) {
-      random.nextInt(100) * random.nextInt(100);
-    }
-  }
-
-  _simulateHeavyComputationForFrozenFrames() {
-    final startTime = DateTime.now();
-    const pauseTime = 1000;
-    // Block the UI thread for ~1000ms
-    while (DateTime.now().difference(startTime).inMilliseconds <= pauseTime) {
-      // Busy waiting
-    }
+  _simulateHeavyComputation(int delayInMilliseconds) {
+    setState(() {
+      final startTime = DateTime.now();
+      final pauseTime = delayInMilliseconds;
+      // Block the UI thread for ~delayInMilliseconds
+      while (DateTime.now().difference(startTime).inMilliseconds <= pauseTime) {
+        // Busy waiting
+      }
+    });
   }
 }

lib/src/utils/screen_rendering/instabug_screen_render_manager.dart

@@ -250,7 +250,7 @@ class InstabugScreenRenderManager {
   /// Initialize the static variables
   Future<void> _initStaticValues() async {
     _timingsCallback = (timings) {
-      // 1. Establish the offset on the first available timing.
+      // Establish the offset on the first available timing.
       _epochOffset ??= _getEpochOffset(timings.first);
 
       for (final frameTiming in timings) {
@@ -384,6 +384,7 @@ class InstabugScreenRenderManager {
     );
   }
 
+  /// @nodoc
   int _getMicrosecondsSinceEpoch(int timeInMicroseconds) =>
       timeInMicroseconds + (_epochOffset ?? 0);
 

test/utils/screen_render/instabug_screen_render_manager_test.dart

@@ -260,7 +260,7 @@ void main() {
     });
 
     test('should not save data if no custom UI trace is started', () {
-      final frameTestdata = InstabugScreenRenderData(
+      final frameTestData = InstabugScreenRenderData(
         traceId: 123,
         frameData: [
           InstabugFrameData(10000, 200),
@@ -270,18 +270,18 @@ void main() {
         slowFramesTotalDurationMicro: 200,
       );
 
-      manager.setFrameData(frameTestdata);
+      manager.setFrameData(frameTestData);
 
       manager.endScreenRenderCollectorForCustomUiTrace();
 
       expect(manager.screenRenderForCustomUiTrace.isActive, false);
-      expect(manager.screenRenderForCustomUiTrace == frameTestdata, false);
+      expect(manager.screenRenderForCustomUiTrace == frameTestData, false);
     });
 
     test(
         'should save data to  screenRenderForCustomUiTrace if custom UI trace is started',
         () {
-      final frameTestdata = InstabugScreenRenderData(
+      final frameTestData = InstabugScreenRenderData(
         traceId: 123,
         frameData: [
           InstabugFrameData(10000, 200),
@@ -292,11 +292,11 @@ void main() {
       );
 
       manager.startScreenRenderCollectorForTraceId(
-        frameTestdata.traceId,
+        frameTestData.traceId,
         UiTraceType.custom,
       );
 
-      manager.setFrameData(frameTestdata);
+      manager.setFrameData(frameTestData);
 
       manager.endScreenRenderCollectorForCustomUiTrace();
     });
@@ -352,7 +352,7 @@ void main() {
       expect(
         manager.screenRenderForAutoUiTrace.slowFramesTotalDurationMicro,
         buildDuration * 1000,
-      ); // * 1000 to convert from milli to micro
+      ); // * 1000 to convert from milliseconds to microseconds
       expect(
         manager.screenRenderForAutoUiTrace.frozenFramesTotalDurationMicro,
         0,
@@ -372,7 +372,7 @@ void main() {
       expect(
         manager.screenRenderForAutoUiTrace.slowFramesTotalDurationMicro,
         rasterDuration * 1000,
-      ); // * 1000 to convert from milli to micro
+      ); // * 1000 to convert from milliseconds to microseconds
       expect(
         manager.screenRenderForAutoUiTrace.frozenFramesTotalDurationMicro,
         0,
@@ -393,7 +393,7 @@ void main() {
       expect(
         manager.screenRenderForAutoUiTrace.frozenFramesTotalDurationMicro,
         buildDuration * 1000,
-      ); // * 1000 to convert from milli to micro
+      ); // * 1000 to convert from milliseconds to microseconds
       expect(
         manager.screenRenderForAutoUiTrace.slowFramesTotalDurationMicro,
         0,
@@ -414,7 +414,7 @@ void main() {
       expect(
         manager.screenRenderForAutoUiTrace.frozenFramesTotalDurationMicro,
         rasterBuild * 1000,
-      ); // * 1000 to convert from milli to micro
+      ); // * 1000 to convert from milliseconds to microseconds
       expect(
         manager.screenRenderForAutoUiTrace.slowFramesTotalDurationMicro,
         0,
@@ -433,7 +433,7 @@ void main() {
       expect(
         manager.screenRenderForAutoUiTrace.frozenFramesTotalDurationMicro,
         0,
-      ); // * 1000 to convert from milli to micro
+      ); // * 1000 to convert from milliseconds to microseconds
       expect(
         manager.screenRenderForAutoUiTrace.slowFramesTotalDurationMicro,
         0,