Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 11a337c

Browse files
committed
Remove the Android-drawn surface rendering from scenario_app.
1 parent 622b372 commit 11a337c

File tree

1 file changed

+2
-28
lines changed

1 file changed

+2
-28
lines changed

testing/scenario_app/android/app/src/main/java/dev/flutter/scenarios/ExternalTextureFlutterActivity.java

Lines changed: 2 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ public class ExternalTextureFlutterActivity extends TestActivity {
4848
private static final int SURFACE_WIDTH = 192;
4949
private static final int SURFACE_HEIGHT = 256;
5050

51-
private SurfaceRenderer surfaceViewRenderer, flutterRenderer;
51+
private SurfaceRenderer flutterRenderer;
5252

5353
// Latch used to ensure both SurfaceRenderers produce a frame before taking a screenshot.
54-
private final CountDownLatch firstFrameLatch = new CountDownLatch(2);
54+
private final CountDownLatch firstFrameLatch = new CountDownLatch(1);
5555

5656
private long textureId = 0;
5757
private TextureRegistry.SurfaceProducer surfaceProducer;
@@ -62,7 +62,6 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
6262

6363
String surfaceRenderer = getIntent().getStringExtra("surface_renderer");
6464
assert surfaceRenderer != null;
65-
surfaceViewRenderer = selectSurfaceRenderer(surfaceRenderer, getIntent().getExtras());
6665
flutterRenderer = selectSurfaceRenderer(surfaceRenderer, getIntent().getExtras());
6766

6867
// Create and place a SurfaceView above the Flutter content.
@@ -86,7 +85,6 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
8685

8786
SurfaceHolder surfaceHolder = surfaceView.getHolder();
8887
surfaceHolder.setFixedSize(SURFACE_WIDTH, SURFACE_HEIGHT);
89-
surfaceHolder.addCallback(new SurfaceRendererCallback(surfaceViewRenderer, firstFrameLatch));
9088
}
9189

9290
@Override
@@ -137,7 +135,6 @@ private MediaExtractor createMediaExtractor() {
137135

138136
@Override
139137
public void onPause() {
140-
surfaceViewRenderer.destroy();
141138
flutterRenderer.destroy();
142139
surfaceProducer.release();
143140
super.onPause();
@@ -461,27 +458,4 @@ private void destroyReaderWriter() {
461458
handlerThread.quitSafely();
462459
}
463460
}
464-
465-
private static class SurfaceRendererCallback implements SurfaceHolder.Callback {
466-
final SurfaceRenderer surfaceRenderer;
467-
final CountDownLatch onFirstFrame;
468-
469-
public SurfaceRendererCallback(SurfaceRenderer surfaceRenderer, CountDownLatch onFirstFrame) {
470-
this.surfaceRenderer = surfaceRenderer;
471-
this.onFirstFrame = onFirstFrame;
472-
}
473-
474-
@Override
475-
public void surfaceCreated(@NonNull SurfaceHolder holder) {
476-
surfaceRenderer.attach(holder.getSurface(), onFirstFrame);
477-
}
478-
479-
@Override
480-
public void surfaceChanged(@NonNull SurfaceHolder holder, int format, int width, int height) {
481-
surfaceRenderer.repaint();
482-
}
483-
484-
@Override
485-
public void surfaceDestroyed(@NonNull SurfaceHolder holder) {}
486-
}
487461
}

0 commit comments

Comments
 (0)