From 2533c83b4ed8e1ca5b259d59373f941c8f0e9635 Mon Sep 17 00:00:00 2001 From: Andy McFadden Date: Wed, 14 Aug 2013 11:23:06 -0700 Subject: Use destroyDisplay() Use the new SurfaceComposerClient::destroyDisplay call to shut the virtual display down. Change-Id: I0225a44594abf427d386456622b23b01fdd1690f --- cmds/screenrecord/screenrecord.cpp | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'cmds') diff --git a/cmds/screenrecord/screenrecord.cpp b/cmds/screenrecord/screenrecord.cpp index 28fc00f..c904b03 100644 --- a/cmds/screenrecord/screenrecord.cpp +++ b/cmds/screenrecord/screenrecord.cpp @@ -460,17 +460,9 @@ static status_t recordScreen(const char* fileName) { printf("Stopping encoder and muxer\n"); } - // Shut everything down. - // - // The virtual display will continue to produce frames until "dpy" - // goes out of scope (and something causes the Binder traffic to transmit; - // can be forced with IPCThreadState::self()->flushCommands()). This - // could cause SurfaceFlinger to get stuck trying to feed us, so we want - // to set a NULL Surface to make the virtual display "dormant". + // Shut everything down, starting with the producer side. bufferProducer = NULL; - SurfaceComposerClient::openGlobalTransaction(); - SurfaceComposerClient::setDisplaySurface(dpy, bufferProducer); - SurfaceComposerClient::closeGlobalTransaction(); + SurfaceComposerClient::destroyDisplay(dpy); encoder->stop(); muxer->stop(); -- cgit v1.1