chore: add store API in Flutter side

Author: AndrewAminInstabug

android/src/main/java/com/instabug/flutter/modules/ApmApi.java

@@ -16,15 +16,15 @@
 import com.instabug.flutter.util.Reflection;
 import com.instabug.flutter.util.ThreadManager;
 
-import io.flutter.plugin.common.BinaryMessenger;
-
 import org.json.JSONObject;
 
 import java.lang.reflect.Method;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.concurrent.Callable;
 
+import io.flutter.plugin.common.BinaryMessenger;
+
 public class ApmApi implements ApmPigeon.ApmHostApi {
     private final String TAG = ApmApi.class.getName();
     private final HashMap<String, ExecutionTrace> traces = new HashMap<>();
@@ -40,14 +40,14 @@ public static void init(BinaryMessenger messenger, Callable<Float> refreshRatePr
         ApmPigeon.ApmHostApi.setup(messenger, api);
     }
 
-  /**
-   * The function sets the enabled status of APM.
-   * 
-   * @param isEnabled The `setEnabled` method in the code snippet is used to enable or disable a
-   * feature, and it takes a `Boolean` parameter named `isEnabled`. When this method is called with
-   * `true`, it enables the feature, and when called with `false`, it disables the feature. The method
-   * internally calls
-   */
+    /**
+     * The function sets the enabled status of APM.
+     *
+     * @param isEnabled The `setEnabled` method in the code snippet is used to enable or disable a
+     *                  feature, and it takes a `Boolean` parameter named `isEnabled`. When this method is called with
+     *                  `true`, it enables the feature, and when called with `false`, it disables the feature. The method
+     *                  internally calls
+     */
     @Override
     public void setEnabled(@NonNull Boolean isEnabled) {
         try {
@@ -58,12 +58,12 @@ public void setEnabled(@NonNull Boolean isEnabled) {
     }
 
     /**
-    * Sets the cold app launch enabled status using the APM library.
-    * 
-    * @param isEnabled The `isEnabled` parameter is a Boolean value that indicates whether cold app launch
-    * is enabled or not. When `isEnabled` is set to `true`, cold app launch is enabled, and when it is set
-    * to `false`, cold app launch is disabled.
-    */
+     * Sets the cold app launch enabled status using the APM library.
+     *
+     * @param isEnabled The `isEnabled` parameter is a Boolean value that indicates whether cold app launch
+     *                  is enabled or not. When `isEnabled` is set to `true`, cold app launch is enabled, and when it is set
+     *                  to `false`, cold app launch is disabled.
+     */
     @Override
     public void setColdAppLaunchEnabled(@NonNull Boolean isEnabled) {
         try {
@@ -73,14 +73,14 @@ public void setColdAppLaunchEnabled(@NonNull Boolean isEnabled) {
         }
     }
 
-   /**
-    * The function sets the auto UI trace enabled status in an APM system, handling any exceptions that
-    * may occur.
-    * 
-    * @param isEnabled The `isEnabled` parameter is a Boolean value that indicates whether the Auto UI
-    * trace feature should be enabled or disabled. When `isEnabled` is set to `true`, the Auto UI trace
-    * feature is enabled, and when it is set to `false`, the feature is disabled.
-    */
+    /**
+     * The function sets the auto UI trace enabled status in an APM system, handling any exceptions that
+     * may occur.
+     *
+     * @param isEnabled The `isEnabled` parameter is a Boolean value that indicates whether the Auto UI
+     *                  trace feature should be enabled or disabled. When `isEnabled` is set to `true`, the Auto UI trace
+     *                  feature is enabled, and when it is set to `false`, the feature is disabled.
+     */
     @Override
     public void setAutoUITraceEnabled(@NonNull Boolean isEnabled) {
         try {
@@ -90,21 +90,20 @@ public void setAutoUITraceEnabled(@NonNull Boolean isEnabled) {
         }
     }
 
-  /**
-   * Starts an execution trace and handles the result
-   * using callbacks.
-   * 
-   * @param id The `id` parameter is a non-null String that represents the identifier of the execution
-   * trace.
-   * @param name The `name` parameter in the `startExecutionTrace` method represents the name of the
-   * execution trace that will be started. It is used as a reference to identify the trace during
-   * execution monitoring.
-   * @param result The `result` parameter in the `startExecutionTrace` method is an instance of
-   * `ApmPigeon.Result<String>`. This parameter is used to provide the result of the execution trace
-   * operation back to the caller. The `success` method of the `result` object is called with the
-   * 
-   * @deprecated see {@link #startFlow}
-   */
+    /**
+     * Starts an execution trace and handles the result
+     * using callbacks.
+     *
+     * @param id     The `id` parameter is a non-null String that represents the identifier of the execution
+     *               trace.
+     * @param name   The `name` parameter in the `startExecutionTrace` method represents the name of the
+     *               execution trace that will be started. It is used as a reference to identify the trace during
+     *               execution monitoring.
+     * @param result The `result` parameter in the `startExecutionTrace` method is an instance of
+     *               `ApmPigeon.Result<String>`. This parameter is used to provide the result of the execution trace
+     *               operation back to the caller. The `success` method of the `result` object is called with the
+     * @deprecated see {@link #startFlow}
+     */
     @Override
     public void startExecutionTrace(@NonNull String id, @NonNull String name, ApmPigeon.Result<String> result) {
         ThreadManager.runOnBackground(
@@ -165,7 +164,7 @@ public void startFlow(@NonNull String name) {
         }
     }
 
-   /**
+    /**
      * Sets custom attributes for AppFlow with a given name.
      * <br/>
      * Setting an attribute value to null will remove its corresponding key if it already exists.
@@ -194,7 +193,7 @@ public void setFlowAttribute(@NonNull String name, @NonNull String key, @Nullabl
         }
     }
 
-   /**
+    /**
      * Ends AppFlow with a given name.
      *
      * @param name AppFlow name to be ended. It can not be empty string or null
@@ -208,13 +207,12 @@ public void endFlow(@NonNull String name) {
         }
     }
 
-     /**
+    /**
      * Adds a new attribute to trace
      *
      * @param id    String id of the trace.
      * @param key   attribute key
      * @param value attribute value. Null to remove attribute
-     *
      * @deprecated see {@link #setFlowAttribute}
      */
     @Override
@@ -230,7 +228,6 @@ public void setExecutionTraceAttribute(@NonNull String id, @NonNull String key,
      * Ends a trace
      *
      * @param id string id of the trace.
-     *
      * @deprecated see {@link #endFlow}
      */
     @Override
@@ -283,7 +280,7 @@ public void endAppLaunch() {
 
     /**
      * logs network-related information
-     * 
+     *
      * @param data Map of network data object.
      */
     @Override
@@ -349,43 +346,44 @@ public void networkLogAndroid(@NonNull Map<String, Object> data) {
 
 
             }
-                if (data.containsKey("w3CCaughtHeader")) {
-                    w3CCaughtHeader = (String) data.get("w3CCaughtHeader");
-
-                }
+            if (data.containsKey("w3CCaughtHeader")) {
+                w3CCaughtHeader = (String) data.get("w3CCaughtHeader");
 
+            }
 
-                APMCPNetworkLog.W3CExternalTraceAttributes w3cExternalTraceAttributes =
-                        null;
-                if (isW3cHeaderFound != null) {
-                    w3cExternalTraceAttributes = new APMCPNetworkLog.W3CExternalTraceAttributes(
-                            isW3cHeaderFound, partialId == null ? null : partialId.longValue(),
-                            networkStartTimeInSeconds == null ? null : networkStartTimeInSeconds.longValue(),
-                            w3CGeneratedHeader, w3CCaughtHeader
 
-                    );
-                }
+            APMCPNetworkLog.W3CExternalTraceAttributes w3cExternalTraceAttributes =
+                    null;
+            if (isW3cHeaderFound != null) {
+                w3cExternalTraceAttributes = new APMCPNetworkLog.W3CExternalTraceAttributes(
+                        isW3cHeaderFound, partialId == null ? null : partialId.longValue(),
+                        networkStartTimeInSeconds == null ? null : networkStartTimeInSeconds.longValue(),
+                        w3CGeneratedHeader, w3CCaughtHeader
 
-                Method method = Reflection.getMethod(Class.forName("com.instabug.apm.networking.APMNetworkLogger"), "log", long.class, long.class, String.class, String.class, long.class, String.class, String.class, String.class, String.class, String.class, long.class, int.class, String.class, String.class, String.class, String.class, APMCPNetworkLog.W3CExternalTraceAttributes.class);
-                if (method != null) {
-                    method.invoke(apmNetworkLogger, requestStartTime, requestDuration, requestHeaders, requestBody, requestBodySize, requestMethod, requestUrl, requestContentType, responseHeaders, responseBody, responseBodySize, statusCode, responseContentType, errorMessage, gqlQueryName, serverErrorMessage, w3cExternalTraceAttributes);
-                } else {
-                    Log.e(TAG, "APMNetworkLogger.log was not found by reflection");
-                }
+                );
+            }
 
-            } catch(Exception e){
-                e.printStackTrace();
+            Method method = Reflection.getMethod(Class.forName("com.instabug.apm.networking.APMNetworkLogger"), "log", long.class, long.class, String.class, String.class, long.class, String.class, String.class, String.class, String.class, String.class, long.class, int.class, String.class, String.class, String.class, String.class, APMCPNetworkLog.W3CExternalTraceAttributes.class);
+            if (method != null) {
+                method.invoke(apmNetworkLogger, requestStartTime, requestDuration, requestHeaders, requestBody, requestBodySize, requestMethod, requestUrl, requestContentType, responseHeaders, responseBody, responseBodySize, statusCode, responseContentType, errorMessage, gqlQueryName, serverErrorMessage, w3cExternalTraceAttributes);
+            } else {
+                Log.e(TAG, "APMNetworkLogger.log was not found by reflection");
             }
+
+        } catch (Exception e) {
+            e.printStackTrace();
         }
+    }
 
 
-    /** 
-    * This method is responsible for initiating a custom performance UI trace
-    * in the APM module. It takes three parameters:
-    * @param screenName: A string representing the name of the screen or UI element being traced.
-    * @param microTimeStamp: A number representing the timestamp in microseconds when the trace is started.
-    * @param traceId: A number representing the unique identifier for the trace.
-    */
+    /**
+     * This method is responsible for initiating a custom performance UI trace
+     * in the APM module. It takes three parameters:
+     *
+     * @param screenName:     A string representing the name of the screen or UI element being traced.
+     * @param microTimeStamp: A number representing the timestamp in microseconds when the trace is started.
+     * @param traceId:        A number representing the unique identifier for the trace.
+     */
     @Override
     public void startCpUiTrace(@NonNull String screenName, @NonNull Long microTimeStamp, @NonNull Long traceId) {
         try {
@@ -396,16 +394,17 @@ public void startCpUiTrace(@NonNull String screenName, @NonNull Long microTimeSt
     }
 
 
-    /** 
-    * This method is responsible for reporting the screen
-    * loading data from Dart side to Android side. It takes three parameters:
-    * @param startTimeStampMicro: A number representing the start timestamp in microseconds of the screen
-    * loading custom performance data.
-    * @param durationMicro: A number representing the duration in microseconds of the screen loading custom
-    * performance data.
-    * @param uiTraceId: A number representing the unique identifier for the UI trace associated with the
-    * screen loading.
-    */
+    /**
+     * This method is responsible for reporting the screen
+     * loading data from Dart side to Android side. It takes three parameters:
+     *
+     * @param startTimeStampMicro: A number representing the start timestamp in microseconds of the screen
+     *                             loading custom performance data.
+     * @param durationMicro:       A number representing the duration in microseconds of the screen loading custom
+     *                             performance data.
+     * @param uiTraceId:           A number representing the unique identifier for the UI trace associated with the
+     *                             screen loading.
+     */
     @Override
     public void reportScreenLoadingCP(@NonNull Long startTimeStampMicro, @NonNull Long durationMicro, @NonNull Long uiTraceId) {
         try {
@@ -417,13 +416,14 @@ public void reportScreenLoadingCP(@NonNull Long startTimeStampMicro, @NonNull Lo
 
 
     /**
-    *  This method is responsible for extend the end time if the screen loading custom
-    * trace. It takes two parameters:
-    * @param timeStampMicro: A number representing the timestamp in microseconds when the screen loading
-    * custom trace is ending.
-    * @param uiTraceId: A number representing the unique identifier for the UI trace associated with the
-    * screen loading.
-    */
+     * This method is responsible for extend the end time if the screen loading custom
+     * trace. It takes two parameters:
+     *
+     * @param timeStampMicro: A number representing the timestamp in microseconds when the screen loading
+     *                        custom trace is ending.
+     * @param uiTraceId:      A number representing the unique identifier for the UI trace associated with the
+     *                        screen loading.
+     */
     @Override
     public void endScreenLoadingCP(@NonNull Long timeStampMicro, @NonNull Long uiTraceId) {
         try {
@@ -435,7 +435,7 @@ public void endScreenLoadingCP(@NonNull Long timeStampMicro, @NonNull Long uiTra
 
 
     /**
-     *  This method is used to check whether the end screen loading feature is enabled or not.
+     * This method is used to check whether the end screen loading feature is enabled or not.
      */
     @Override
     public void isEndScreenLoadingEnabled(@NonNull ApmPigeon.Result<Boolean> result) {
@@ -453,9 +453,9 @@ public void isEnabled(@NonNull ApmPigeon.Result<Boolean> result) {
         }
     }
 
-    /** 
-     * checks whether the screen loading feature is enabled. 
-     * */    
+    /**
+     * checks whether the screen loading feature is enabled.
+     */
     @Override
     public void isScreenLoadingEnabled(@NonNull ApmPigeon.Result<Boolean> result) {
         try {
@@ -514,4 +514,14 @@ public void deviceRefreshRate(@NonNull ApmPigeon.Result<Double> result) {
         }
     }
 
+    @Override
+    public void endScreenRenderForAutoUiTrace(@NonNull Map<String, Object> data) {
+
+    }
+
+    @Override
+    public void endScreenRenderForCustomUiTrace(@NonNull Map<String, Object> data) {
+
+    }
+
 }

ios/Classes/Modules/ApmApi.m

@@ -3,7 +3,6 @@
 #import "ArgsRegistry.h"
 #import "IBGAPM+PrivateAPIs.h"
 #import "IBGTimeIntervalUnits.h"
-#import "IBGFrameInfo.h"
 
 void InitApmApi(id<FlutterBinaryMessenger> messenger) {
     ApmApi *api = [[ApmApi alloc] init];
@@ -219,17 +218,23 @@ - (void)deviceRefreshRateWithCompletion:(void (^)(NSNumber * _Nullable, FlutterE
     }
 }
 
-- (void)endScreenRenderForAutoUiTraceData:(nonnull NSDictionary<NSString *,id> *)data error:(FlutterError * _Nullable __autoreleasing * _Nonnull)error { 
+- (void)endScreenRenderForAutoUiTraceData:(nonnull NSDictionary<NSString *,id> *)data error:(FlutterError * _Nullable __autoreleasing * _Nonnull)error {
     int traceId = [data[@"traceId"] intValue];
-      int slowFrames = [data[@"slowFramesTotalDuration"] intValue];
-      int frozenFrames = [data[@"frozenFramesTotalDuration"] intValue];
-
-      NSArray<NSArray<NSNumber *> *> *rawFrames = data[@"frameData"];
-
-    //complete from here
+    int slowFrames = [data[@"slowFramesTotalDuration"] intValue];
+    int frozenFrames = [data[@"frozenFramesTotalDuration"] intValue];
 
-    
-//    [IBGAPM endAutoUITraceCPWithFrames:/*<#(nullable NSArray *)#>*/]
+    NSArray<NSArray<NSNumber *> *> *rawFrames = data[@"frameData"];
+    NSLog(@"traceID%d" , traceId);
+    NSLog(@"slowFrames%d" , slowFrames);
+    NSLog(@"frozenFrames%d" , frozenFrames);
+    if (rawFrames && [rawFrames isKindOfClass:[NSArray class]]) {
+        for (NSArray<NSNumber *> *frameValues in rawFrames) {
+            if ([frameValues count] == 2) {
+                NSLog(@"startTime%lld" , [frameValues[0] longLongValue]);
+                NSLog(@"duration%lld" , [frameValues[1] longLongValue]);
+            }
+        }
+    }
 }
 
 

ios/Classes/Util/IBGAPM+PrivateAPIs.h

@@ -8,7 +8,7 @@
 
 #import <InstabugSDK/IBGAPM.h>
 #import "IBGTimeIntervalUnits.h"
-#import "IBGFrameInfo.h"
+//#import "IBGFrameInfo.h"
 
 @interface IBGAPM (PrivateAPIs)
 
@@ -25,8 +25,8 @@
 
 + (BOOL)isScreenRenderingOperational;
 
-+ (void)endAutoUITraceCPWithFrames:(nullable NSArray<IBGFrameInfo *> *)frames;
-
-+ (void)endCustomUITraceCPWithFrames:(nullable NSArray<IBGFrameInfo *> *)frames;
+//+ (void)endAutoUITraceCPWithFrames:(nullable NSArray<IBGFrameInfo *> *)frames;
+//
+//+ (void)endCustomUITraceCPWithFrames:(nullable NSArray<IBGFrameInfo *> *)frames;
 
 @end