Reverts "Add SurfaceProducer#onSurfaceAvailable, deprecate onSurfaceCreated. (#55418)" (#55450)

Reverts: #55418
Initiated by: bdero
Reason for reverting: [Engine->Framework roll breakage](flutter/flutter#155727 (comment))
Original PR Author: matanlurey

Reviewed By: {jonahwilliams}

This change reverts the following previous change:
Closes flutter/flutter#155131.

Not only did I rename the method, but I also changed the contract slightly - now `onSurfaceAvailable` is _only_ invoked _after_ `onSurfaceDestroyed` has been called. The cost is a single `boolean`, and it honestly makes the API make a lot more sense than someone having to track this themselves.

/cc @johnmccutchan (OOO), and @flutter/android-reviewers.

Author: auto-submit[bot]

shell/platform/android/io/flutter/embedding/engine/renderer/FlutterRenderer.java

@@ -108,9 +108,8 @@ public FlutterRenderer(@NonNull FlutterJNI flutterJNI) {
               public void onResume(@NonNull LifecycleOwner owner) {
                 Log.v(TAG, "onResume called; notifying SurfaceProducers");
                 for (ImageReaderSurfaceProducer producer : imageReaderProducers) {
-                  if (producer.callback != null && producer.notifiedDestroy) {
-                    producer.notifiedDestroy = false;
-                    producer.callback.onSurfaceAvailable();
+                  if (producer.callback != null) {
+                    producer.callback.onSurfaceCreated();
                   }
                 }
               }
@@ -463,13 +462,6 @@ final class ImageReaderSurfaceProducer
     // will be produced at that size.
     private boolean createNewReader = true;
 
-    /**
-     * Stores whether {@link Callback#onSurfaceDestroyed()} was previously invoked.
-     *
-     * <p>Used to avoid signaling {@link Callback#onSurfaceAvailable()} unnecessarily.
-     */
-    private boolean notifiedDestroy = false;
-
     // State held to track latency of various stages.
     private long lastDequeueTime = 0;
     private long lastQueueTime = 0;
@@ -697,7 +689,6 @@ public void onTrimMemory(int level) {
       cleanup();
       createNewReader = true;
       if (this.callback != null) {
-        notifiedDestroy = true;
         this.callback.onSurfaceDestroyed();
       }
     }

shell/platform/android/io/flutter/view/TextureRegistry.java

@@ -96,8 +96,8 @@ interface SurfaceProducer extends TextureEntry {
 
     /**
      * Sets a callback that is notified when a previously created {@link Surface} returned by {@link
-     * SurfaceProducer#getSurface()} is no longer valid due to being destroyed, or a new surface is
-     * now available (after the previous one was destroyed) for rendering.
+     * SurfaceProducer#getSurface()} is no longer valid, either due to being destroyed or being
+     * changed.
      *
      * @param callback The callback to notify, or null to remove the callback.
      */
@@ -106,65 +106,18 @@ interface SurfaceProducer extends TextureEntry {
     /** Callback invoked by {@link #setCallback(Callback)}. */
     interface Callback {
       /**
-       * An alias for {@link Callback#onSurfaceAvailable()} with a less accurate name.
-       *
-       * @deprecated Override and use {@link Callback#onSurfaceAvailable()} instead.
-       */
-      @Deprecated(since = "Flutter 3.27", forRemoval = true)
-      default void onSurfaceCreated() {}
-
-      /**
-       * Invoked when an Android application is resumed after {@link Callback#onSurfaceDestroyed()}.
+       * Invoked when a previous surface is now invalid and a new surface is now available.
        *
-       * <p>Applications should now call {@link SurfaceProducer#getSurface()} to get a new
-       * {@link Surface}, as the previous one was destroyed and released as a result of a low memory
-       * event from the Android OS.
-       *
-       * <pre>
-       * {@code
-       * void example(SurfaceProducer producer) {
-       *   producer.setCallback(new SurfaceProducer.Callback() {
-       *     @override
-       *     public void onSurfaceAvailable() {
-       *       Surface surface = producer.getSurface();
-       *       redrawOrUse(surface);
-       *     }
-       *
-       *     // ...
-       *   });
-       * }
-       * }
-       * </pre>
+       * <p>Typically plugins will use this callback as a signal to redraw, such as due to the
+       * texture being resized, the format being changed, or the application being resumed after
+       * being suspended in the background.
        */
-      default void onSurfaceAvailable() {
-        this.onSurfaceCreated();
-      }
+      void onSurfaceCreated();
 
       /**
-       * Invoked when a {@link Surface} returned by {@link SurfaceProducer#getSurface()} is invalid.
-       *
-       * <p>In a low memory environment, the Android OS will signal to Flutter to release resources,
-       * such as surfaces, that are not currently in use, such as when the application is in the
-       * background, and this method is subsequently called to notify a plugin author to stop
-       * using or rendering to the last surface.
-       *
-       * <p>Use {@link Callback#onSurfaceAvailable()} to be notified to resume rendering.
-       *
-       * <pre>
-       * {@code
-       * void example(SurfaceProducer producer) {
-       *   producer.setCallback(new SurfaceProducer.Callback() {
-       *     @override
-       *     public void onSurfaceDestroyed() {
-       *       // Store information about the last frame, if necessary.
-       *       // Potentially release other dependent resources.
-       *     }
+       * Invoked when a previous surface is now invalid.
        *
-       *     // ...
-       *   });
-       * }
-       * }
-       * </pre>
+       * <p>Typically plugins will use this callback as a signal to release resources.
        */
       void onSurfaceDestroyed();
     }

shell/platform/android/test/io/flutter/embedding/engine/renderer/FlutterRendererTest.java

@@ -785,7 +785,7 @@ public void ImageReaderSurfaceProducerIsCreatedOnLifecycleResume() throws Except
     TextureRegistry.SurfaceProducer.Callback callback =
         new TextureRegistry.SurfaceProducer.Callback() {
           @Override
-          public void onSurfaceAvailable() {
+          public void onSurfaceCreated() {
             latch.countDown();
           }
 
@@ -794,9 +794,6 @@ public void onSurfaceDestroyed() {}
         };
     producer.setCallback(callback);
 
-    // Trim memory.
-    ((FlutterRenderer.ImageReaderSurfaceProducer) producer).onTrimMemory(40);
-
     // Trigger a resume.
     ((LifecycleRegistry) ProcessLifecycleOwner.get().getLifecycle())
         .setCurrentState(Lifecycle.State.RESUMED);