Android VideoRendererGui: Add dispose function

talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java

Index: talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java
diff --git a/talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java b/talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java
index 401640b463bfdfff7f832ebf75ce73100a0b40fb..9a7e292e62e4dd189d71d1b1e2c7e53c83ef663f 100644
--- a/talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java
+++ b/talk/app/webrtc/java/src/org/webrtc/VideoRenderer.java
@@ -179,7 +179,7 @@ public class VideoRenderer {
   // |this| either wraps a native (GUI) renderer or a client-supplied Callbacks
   // (Java) implementation; so exactly one of these will be non-0/null.
   final long nativeVideoRenderer;
-  private final Callbacks callbacks;
+  private Callbacks callbacks;
 
   public static VideoRenderer createGui(int x, int y) {
     long nativeVideoRenderer = nativeCreateGuiVideoRenderer(x, y);
@@ -203,6 +203,7 @@ public class VideoRenderer {
     if (callbacks == null) {
       freeGuiVideoRenderer(nativeVideoRenderer);
     } else {
+      callbacks = null;
       freeWrappedVideoRenderer(nativeVideoRenderer);
     }

[AlexG, 2015/08/05 18:19:23]

set nativeVideoRenderer to 0 and add a check at the beginning of dispose() to
exit if nativeVideoRenderer == 0 - just to be safe if someone will call
dispose() twice?

[magjed_webrtc, 2015/08/11 10:38:16]

Done. I also changed |callbacks| to a boolean instead.

   }