Java VideoRenderer.Callbacks: Make renderFrame() interface asynchronous

webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java

 /*
  *  Copyright 2014 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 77 matching lines @@
       if (!renderFrameCalled) {
         if (rendererName != null) {
           Log.d(TAG, rendererName + " render frame: "
               + frame.rotatedWidth() + " x " + frame.rotatedHeight());
         } else {
           Log.d(TAG, "Render frame: " + frame.rotatedWidth() + " x " + frame.rotatedHeight());
         }
       }
       renderFrameCalled = true;
       doneRendering.countDown();
+      VideoRenderer.renderFrameDone(frame);

[AlexG, 2015/08/26 23:27:25]

nit: move this above doneRendering.countDown();

[magjed_webrtc, 2015/08/27 15:52:20]

Done.

     }
 
 
     // This method shouldn't hold any locks or touch member variables since it
     // blocks.
     public boolean waitForFramesRendered(int timeoutMs)
         throws InterruptedException {
       doneRendering.await(timeoutMs, TimeUnit.MILLISECONDS);
       return (doneRendering.getCount() <= 0);
     }
@@ skipping 316 matching lines @@
           localRenderer.waitForFramesRendered(WAIT_TIMEOUT));
       assertTrue("Remote video frames were not rendered after video restart.",
           remoteRenderer.waitForFramesRendered(WAIT_TIMEOUT));
     }
     pcClient.close();
     assertTrue(waitForPeerConnectionClosed(WAIT_TIMEOUT));
     Log.d(TAG, "testVideoSourceRestart done.");
   }
 
 }