Improve low memory breadcrumb capturing

Author: romtsn

sentry-android-core/src/main/java/io/sentry/android/core/AppComponentsBreadcrumbsIntegration.java

@@ -91,40 +91,36 @@ public void onConfigurationChanged(@NotNull Configuration newConfig) {
 
   @Override
   public void onLowMemory() {
-    final long now = System.currentTimeMillis();
-    executeInBackground(() -> captureLowMemoryBreadcrumb(now, null));
+    // we do this in onTrimMemory below already, this is legacy API (14 or below)
   }
 
   @Override
   public void onTrimMemory(final int level) {
+    if (level < TRIM_MEMORY_BACKGROUND) {
+      // only add breadcrumb if TRIM_MEMORY_BACKGROUND, TRIM_MEMORY_MODERATE or
+      // TRIM_MEMORY_COMPLETE.
+      // Release as much memory as the process can.
+
+      // TRIM_MEMORY_UI_HIDDEN, TRIM_MEMORY_RUNNING_MODERATE, TRIM_MEMORY_RUNNING_LOW and
+      // TRIM_MEMORY_RUNNING_CRITICAL.
+      // Release any memory that your app doesn't need to run.
+      // So they are still not so critical at the point of killing the process.
+      // https://developer.android.com/topic/performance/memory
+      return;
+    }
+
     final long now = System.currentTimeMillis();
     executeInBackground(() -> captureLowMemoryBreadcrumb(now, level));
   }
 
-  private void captureLowMemoryBreadcrumb(final long timeMs, final @Nullable Integer level) {
+  private void captureLowMemoryBreadcrumb(final long timeMs, final int level) {
     if (scopes != null) {
       final Breadcrumb breadcrumb = new Breadcrumb(timeMs);
-      if (level != null) {
-        // only add breadcrumb if TRIM_MEMORY_BACKGROUND, TRIM_MEMORY_MODERATE or
-        // TRIM_MEMORY_COMPLETE.
-        // Release as much memory as the process can.
-
-        // TRIM_MEMORY_UI_HIDDEN, TRIM_MEMORY_RUNNING_MODERATE, TRIM_MEMORY_RUNNING_LOW and
-        // TRIM_MEMORY_RUNNING_CRITICAL.
-        // Release any memory that your app doesn't need to run.
-        // So they are still not so critical at the point of killing the process.
-        // https://developer.android.com/topic/performance/memory
-
-        if (level < TRIM_MEMORY_BACKGROUND) {
-          return;
-        }
-        breadcrumb.setData("level", level);
-      }
-
       breadcrumb.setType("system");
       breadcrumb.setCategory("device.event");
       breadcrumb.setMessage("Low memory");
       breadcrumb.setData("action", "LOW_MEMORY");
+      breadcrumb.setData("level", level);
       breadcrumb.setLevel(SentryLevel.WARNING);
       scopes.addBreadcrumb(breadcrumb);
     }

sentry-android-core/src/test/java/io/sentry/android/core/AppComponentsBreadcrumbsIntegrationTest.kt

@@ -95,24 +95,6 @@ class AppComponentsBreadcrumbsIntegrationTest {
         sut.close()
     }
 
-    @Test
-    fun `When low memory event, a breadcrumb with type, category and level should be set`() {
-        val sut = fixture.getSut()
-        val options = SentryAndroidOptions().apply {
-            executorService = ImmediateExecutorService()
-        }
-        val scopes = mock<IScopes>()
-        sut.register(scopes, options)
-        sut.onLowMemory()
-        verify(scopes).addBreadcrumb(
-            check<Breadcrumb> {
-                assertEquals("device.event", it.category)
-                assertEquals("system", it.type)
-                assertEquals(SentryLevel.WARNING, it.level)
-            }
-        )
-    }
-
     @Test
     fun `When trim memory event with level, a breadcrumb with type, category and level should be set`() {
         val sut = fixture.getSut()