Add network tester client [android]

webrtc/tools/network_tester/androidapp/src/com/google/media/networktester/Tester.java

Index: webrtc/tools/network_tester/androidapp/src/com/google/media/networktester/Tester.java
diff --git a/webrtc/tools/network_tester/androidapp/src/com/google/media/networktester/Tester.java b/webrtc/tools/network_tester/androidapp/src/com/google/media/networktester/Tester.java
new file mode 100644
index 0000000000000000000000000000000000000000..a33a066be9e8e936e994d9e254b283b047ae8393
--- /dev/null
+++ b/webrtc/tools/network_tester/androidapp/src/com/google/media/networktester/Tester.java
@@ -0,0 +1,55 @@
+/*
+ *  Copyright 2017 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.
+ */
+
+package com.google.media.networktester;
+
+public class Tester extends Thread {

[magjed_webrtc, 2017/04/04 09:05:59]

Make the class name more descriptive, or consider moving this to a nested class
in MainActivity, or even just an anonymous 'new Thread(new Runnable() {...'.

[michaelt, 2017/04/05 15:36:16]

I'd like to capsulate the c functions in this class so i would prefer to keep
it.
Renamed it.

+  private native static long CreateTestController();
+  private native static void TestControllerConnect(long testController);
+  private native static void TestControllerRun(long testController);
+  private native static boolean TestControllerIsDone(long testController);
+  private native static void DestroyTestController(long testController);
+  static {
+    System.loadLibrary("network_tester_so");
+  }
+  private static final String TAG = "NetworkTester";

[magjed_webrtc, 2017/04/04 09:05:59]

This is not used.

[michaelt, 2017/04/05 15:36:16]

Acknowledged.

+  private long testController = 0;
+  boolean startTest = false;
+  boolean stopTest = false;
+
+  public synchronized void startTest() {

[magjed_webrtc, 2017/04/04 09:05:59]

Remove this and launch a new thread for every test run instead.

[michaelt, 2017/04/05 15:36:15]

Done.

+    startTest = true;
+  }
+
+  public synchronized void stopTest() {
+    stopTest = true;
+  }
+
+  public void run() {
+    testController = CreateTestController();
+    while (true) {

[magjed_webrtc, 2017/04/04 09:05:59]

Do this instead:

public void run() {
  final long testController = CreateTestController();
  TestControllerConnect(testController);
  while (!Thread.interrupted() && !TestControllerIsDone(testController)) {
    TestControllerRun(testController);
  }
  DestroyTestController(testController);
}

and interrupt the thread from MainActivity to stop it.

[michaelt, 2017/04/05 15:36:16]

Done.

+      if (startTest) {
+        startTest = false;
+        TestControllerConnect(testController);
+        while (!TestControllerIsDone(testController)) {
+          TestControllerRun(testController);
+          if (stopTest) {
+            stopTest = false;
+            break;
+          }
+        };
+      }
+      if (Thread.currentThread().isInterrupted()) {
+        break;
+      }
+    }
+    DestroyTestController(testController);
+  }
+}