[Android] Add ChromiumNetTestSupport.apk for the java EmbeddedTestServer.

net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.net.test;
 
-import org.chromium.base.annotations.CalledByNative;
-import org.chromium.base.annotations.JNINamespace;
+import android.content.ComponentName;
+import android.content.Context;
+import android.content.Intent;
+import android.content.ServiceConnection;
+import android.os.IBinder;
+import android.os.RemoteException;
+
+import org.chromium.base.Log;
 
 import java.io.File;
 
 /**
  * Java bindings for the native embedded test server.

[mef, 2015/11/25 17:20:22]

This comment is no longer true.

[jbudorick, 2015/12/01 15:38:23]

Yeah, this first sentence is no longer accurate.

  *
  * An example use:
  *   EmbeddedTestServer s = new EmbeddedTestServer();
  *   s.initializeNative();
  *   s.serveFilesFromDirectory("/path/to/my/directory");
  *   if (!s.start()) {
  *       throw new SomeKindOfException("Unable to initialize EmbeddedTestServer.");
  *   }
  *
  *   // serve requests...
  *
  *   s.shutdownAndWait();
  *   s.destroy();
  */
-@JNINamespace("net::test_server")
 public class EmbeddedTestServer {

[mef, 2015/11/25 17:20:22]

Maybe call it LocalTestServer instead? It isn't embedded, is it?

[jbudorick, 2015/12/01 15:38:23]

It's not, but it is still a java endpoint for embedded_test_server.h/.cc and is
named to reflect that.

-    private long mNativeEmbeddedTestServer;
+    private static final String TAG = "cr_TestServer";
 
-    /** Create an uninitialized EmbeddedTestServer. */
-    public EmbeddedTestServer() {}
+    private static final String EMBEDDED_TEST_SERVER_SERVICE =
+            "org.chromium.net.test.EMBEDDED_TEST_SERVER_SERVICE";
+    private static final long SERVICE_CONNECTION_WAIT_INTERVAL_MS = 5000;
 
-    /** Initialize the native EmbeddedTestServer object. */
-    public void initializeNative() {
-        if (mNativeEmbeddedTestServer == 0) nativeInit();
-        assert mNativeEmbeddedTestServer != 0;
+    private IEmbeddedTestServerImpl mImpl;
+    private ServiceConnection mConn = new ServiceConnection() {
+        @Override
+        public void onServiceConnected(ComponentName name, IBinder service) {
+            synchronized (mImplMonitor) {
+                mImpl = IEmbeddedTestServerImpl.Stub.asInterface(service);
+                mImplMonitor.notify();
+            }
+        }
+
+        @Override
+        public void onServiceDisconnected(ComponentName name) {
+            synchronized (mImplMonitor) {
+                mImpl = null;
+                mImplMonitor.notify();
+            }
+        }
+    };
+
+    private final Object mImplMonitor = new Object();
+
+    /**
+     * Exception class raised on failure in the EmbeddedTestServer.
+     */
+    public static final class EmbeddedTestServerFailure extends Error {
+        public EmbeddedTestServerFailure(String errorDesc) {
+            super(errorDesc);
+        }
+    }
+
+    public void initializeNative(Context context) throws InterruptedException {
+        Intent intent = new Intent(EMBEDDED_TEST_SERVER_SERVICE);
+        intent.setClassName(
+                "org.chromium.net.test.support", "org.chromium.net.test.EmbeddedTestServerService");
+        if (!context.bindService(intent, mConn, Context.BIND_AUTO_CREATE)) {
+            throw new EmbeddedTestServerFailure(
+                    "Unable to bind to the EmbeddedTestServer service.");
+        }
+        synchronized (mImplMonitor) {
+            Log.i(TAG, "Waiting for EmbeddedTestServer service connection.");
+            while (mImpl == null) {
+                mImplMonitor.wait(SERVICE_CONNECTION_WAIT_INTERVAL_MS);
+                Log.i(TAG, "Still waiting for EmbeddedTestServer service connection.");
+            }
+            Log.i(TAG, "EmbeddedTestServer service connected.");
+        }
+        try {
+            mImpl.initializeNative();
+        } catch (RemoteException e) {
+            Log.e(TAG, "Failed to initialize native server.", e);
+            throw new EmbeddedTestServerFailure("Failed to initialize native server.");
+        }
     }
 
     /** Serve files from the provided directory.
      *
      *  @param directory The directory from which files should be served.
      */
     public void serveFilesFromDirectory(File directory) {
-        nativeServeFilesFromDirectory(mNativeEmbeddedTestServer, directory.getPath());
+        serveFilesFromDirectory(directory.getPath());
     }
 
     /** Serve files from the provided directory.
      *
      *  @param directoryPath The path of the directory from which files should be served.
      */
     public void serveFilesFromDirectory(String directoryPath) {
-        nativeServeFilesFromDirectory(mNativeEmbeddedTestServer, directoryPath);
+        try {
+            mImpl.serveFilesFromDirectory(directoryPath);
+        } catch (RemoteException e) {
+            throw new EmbeddedTestServerFailure(
+                    "Failed to start serving files from " + directoryPath + ": " + e.toString());
+        }
     }
 
     // TODO(svaldez): Remove once all consumers have switched to start().
     /** Wrapper for start()
      *
      *  start() should be used instead of this.
      *
      *  @return Whether the server was successfully initialized.
      */
     public boolean initializeAndWaitUntilReady() {
         return start();
     }
 
     /** Starts the server.
      *
      *  Note that this should be called after handlers are set up, including any relevant calls
      *  serveFilesFromDirectory.
      *
      *  @return Whether the server was successfully initialized.
      */
     public boolean start() {
-        return nativeStart(mNativeEmbeddedTestServer);
+        try {
+            return mImpl.start();
+        } catch (RemoteException e) {
+            throw new EmbeddedTestServerFailure("Failed to start server: " + e.toString());

[mef, 2015/11/25 17:20:22]

I think it makes sense to include |e| as the |cause| for new exception.

[jbudorick, 2015/12/01 15:38:23]

Done here and in similar cases below.

+        }
+    }
+
+    /** Create and initialize a server that serves files from the provided directory.
+     *
+     *  This handles native object initialization, server configuration, and server initialization.
+     *  On returning, the server is ready for use.
+     *
+     *  @param context The context in which the server is being started.
+     *  @param directory The directory from which files should be served.
+     *  @return The created server.
+     */
+    public static EmbeddedTestServer createAndStartFileServer(Context context, File directory)
+            throws InterruptedException {
+        EmbeddedTestServer server = new EmbeddedTestServer();
+        server.initializeNative(context);
+        server.serveFilesFromDirectory(directory);
+        if (!server.start()) {
+            throw new EmbeddedTestServerFailure(
+                    "Failed to start serving files from " + directory.getPath());
+        }
+        return server;
     }
 
     /** Get the full URL for the given relative URL.
      *
      *  @param relativeUrl The relative URL for which a full URL will be obtained.
      *  @return The URL as a String.
      */
     public String getURL(String relativeUrl) {
-        return nativeGetURL(mNativeEmbeddedTestServer, relativeUrl);
+        try {
+            return mImpl.getURL(relativeUrl);
+        } catch (RemoteException e) {
+            throw new EmbeddedTestServerFailure(
+                    "Failed to get URL for " + relativeUrl + ": " + e.toString());
+        }
     }
 
     /** Shutdown the server.
      *
      *  @return Whether the server was successfully shut down.
      */
     public boolean shutdownAndWaitUntilComplete() {
-        return nativeShutdownAndWaitUntilComplete(mNativeEmbeddedTestServer);
+        try {
+            return mImpl.shutdownAndWaitUntilComplete();
+        } catch (RemoteException e) {
+            throw new EmbeddedTestServerFailure("Failed to shut down: " + e.toString());
+        }
     }
 
     /** Destroy the native EmbeddedTestServer object. */
     public void destroy() {
-        assert mNativeEmbeddedTestServer != 0;
-        nativeDestroy(mNativeEmbeddedTestServer);
-        assert mNativeEmbeddedTestServer == 0;
+        try {
+            mImpl.destroy();
+        } catch (RemoteException e) {
+            throw new EmbeddedTestServerFailure("Failed to destroy native server: " + e.toString());
+        }
     }
 
-    @CalledByNative
-    private void setNativePtr(long nativePtr) {
-        assert mNativeEmbeddedTestServer == 0;
-        mNativeEmbeddedTestServer = nativePtr;
+    /** Stop and destroy the provided server.
+     *
+     *  This handles stopping the server and destroying the native object.
+     *
+     *  @param server The server to stop and destroy.
+     */
+    public static void stopAndDestroyServer(EmbeddedTestServer server) {
+        if (!server.shutdownAndWaitUntilComplete()) {
+            throw new EmbeddedTestServerFailure("Failed to stop server.");
+        }
+        server.destroy();
     }
-
-    @CalledByNative
-    private void clearNativePtr() {
-        assert mNativeEmbeddedTestServer != 0;
-        mNativeEmbeddedTestServer = 0;
-    }
-
-    private native void nativeInit();
-    private native void nativeDestroy(long nativeEmbeddedTestServerAndroid);
-    private native boolean nativeStart(long nativeEmbeddedTestServerAndroid);
-    private native boolean nativeShutdownAndWaitUntilComplete(long nativeEmbeddedTestServerAndroid);
-    private native String nativeGetURL(long nativeEmbeddedTestServerAndroid, String relativeUrl);
-    private native void nativeServeFilesFromDirectory(
-            long nativeEmbeddedTestServerAndroid, String directoryPath);
 }