Update to 3.1.2 release

This reverts commit 7b12fd1ad2.

README.md

@@ -8,6 +8,8 @@ whether in your own home or over the internet.
 
 [Limelight-pc](https://github.com/limelight-stream/limelight-pc) is also currently in development for Windows, OS X and Linux. Versions for [iOS](https://github.com/limelight-stream/limelight-ios) and [Windows and Windows Phone](https://github.com/limelight-stream/limelight-windows) are also in development.
 
+Check our [wiki](https://github.com/limelight-stream/limelight-android/wiki) for more detailed information or a troubleshooting guide. 
+
 ##Features
 
 * Streams any of your games from your PC to your Android device

app/app.iml

@@ -9,11 +9,12 @@
     <facet type="android" name="Android">
       <configuration>
         <option name="SELECTED_BUILD_VARIANT" value="nonRootDebug" />
+        <option name="SELECTED_TEST_ARTIFACT" value="_android_test_" />
         <option name="ASSEMBLE_TASK_NAME" value="assembleNonRootDebug" />
         <option name="COMPILE_JAVA_TASK_NAME" value="compileNonRootDebugSources" />
-        <option name="ASSEMBLE_TEST_TASK_NAME" value="assembleNonRootDebugTest" />
+        <option name="ASSEMBLE_TEST_TASK_NAME" value="assembleNonRootDebugAndroidTest" />
         <option name="SOURCE_GEN_TASK_NAME" value="generateNonRootDebugSources" />
-        <option name="TEST_SOURCE_GEN_TASK_NAME" value="generateNonRootDebugTestSources" />
+        <option name="TEST_SOURCE_GEN_TASK_NAME" value="generateNonRootDebugAndroidTestSources" />
         <option name="ALLOW_USER_CONFIGURATION" value="false" />
         <option name="MANIFEST_FILE_RELATIVE_PATH" value="/src/main/AndroidManifest.xml" />
         <option name="RES_FOLDER_RELATIVE_PATH" value="/src/main/res" />
@@ -24,6 +25,7 @@
   </component>
   <component name="NewModuleRootManager" inherit-compiler-output="false">
     <output url="file://$MODULE_DIR$/build/intermediates/classes/nonRoot/debug" />
+    <output-test url="file://$MODULE_DIR$/build/intermediates/classes/androidTest/nonRoot/debug" />
     <exclude-output />
     <content url="file://$MODULE_DIR$">
       <sourceFolder url="file://$MODULE_DIR$/build/generated/source/r/nonRoot/debug" isTestSource="false" generated="true" />
@@ -31,6 +33,7 @@
       <sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/nonRoot/debug" isTestSource="false" generated="true" />
       <sourceFolder url="file://$MODULE_DIR$/build/generated/source/rs/nonRoot/debug" isTestSource="false" generated="true" />
       <sourceFolder url="file://$MODULE_DIR$/build/generated/res/rs/nonRoot/debug" type="java-resource" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/res/generated/nonRoot/debug" type="java-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/res" type="java-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/resources" type="java-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/assets" type="java-resource" />
@@ -38,11 +41,12 @@
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/java" isTestSource="false" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/jni" isTestSource="false" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRootDebug/rs" isTestSource="false" />
-      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/r/test/nonRoot/debug" isTestSource="true" generated="true" />
-      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/aidl/test/nonRoot/debug" isTestSource="true" generated="true" />
-      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/test/nonRoot/debug" isTestSource="true" generated="true" />
-      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/rs/test/nonRoot/debug" isTestSource="true" generated="true" />
-      <sourceFolder url="file://$MODULE_DIR$/build/generated/res/rs/test/nonRoot/debug" type="java-test-resource" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/r/androidTest/nonRoot/debug" isTestSource="true" generated="true" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/aidl/androidTest/nonRoot/debug" isTestSource="true" generated="true" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/buildConfig/androidTest/nonRoot/debug" isTestSource="true" generated="true" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/source/rs/androidTest/nonRoot/debug" isTestSource="true" generated="true" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/res/rs/androidTest/nonRoot/debug" type="java-test-resource" />
+      <sourceFolder url="file://$MODULE_DIR$/build/generated/res/generated/androidTest/nonRoot/debug" type="java-test-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRoot/res" type="java-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRoot/resources" type="java-resource" />
       <sourceFolder url="file://$MODULE_DIR$/src/nonRoot/assets" type="java-resource" />
@@ -101,18 +105,14 @@
     </content>
     <orderEntry type="jdk" jdkName="Android API 21 Platform" jdkType="Android SDK" />
     <orderEntry type="sourceFolder" forTests="false" />
-    <orderEntry type="library" exported="" name="okhttp-2.1.0-RC1" level="project" />
     <orderEntry type="library" exported="" name="bcprov-jdk15on-1.51" level="project" />
-    <orderEntry type="library" exported="" name="gson-2.3.1" level="project" />
-    <orderEntry type="library" exported="" name="androidasync-1.3.7" level="project" />
     <orderEntry type="library" exported="" name="jmdns-fixed" level="project" />
-    <orderEntry type="library" exported="" name="jcodec-0.1.6-3" level="project" />
     <orderEntry type="library" exported="" name="bcpkix-jdk15on-1.51" level="project" />
     <orderEntry type="library" exported="" name="tinyrtsp" level="project" />
     <orderEntry type="library" exported="" name="limelight-common" level="project" />
-    <orderEntry type="library" exported="" name="support-v4-r6" level="project" />
-    <orderEntry type="library" exported="" name="okio-1.0.1" level="project" />
-    <orderEntry type="library" exported="" name="ion-1.3.7" level="project" />
+    <orderEntry type="library" exported="" name="okhttp-2.2.0" level="project" />
+    <orderEntry type="library" exported="" name="jcodec-0.1.9" level="project" />
+    <orderEntry type="library" exported="" name="okio-1.2.0" level="project" />
   </component>
 </module>
 

app/build.gradle

@@ -5,14 +5,14 @@ apply plugin: 'com.android.application'
 
 android {
     compileSdkVersion 21
-    buildToolsVersion "21.0.2"
+    buildToolsVersion "21.1.2"
 
     defaultConfig {
         minSdkVersion 16
         targetSdkVersion 21
 
-        versionName "3.0"
-        versionCode = 45
+        versionName "3.1.2"
+        versionCode = 56
     }
 
     productFlavors {
@@ -27,7 +27,7 @@ android {
 
     buildTypes {
         release {
-            runProguard false
+            minifyEnabled false
             proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.txt'
         }
     }
@@ -62,13 +62,14 @@ android {
 }
 
 dependencies {
-    compile group: 'org.jcodec', name: 'jcodec', version: '0.1.6-3'
+    compile group: 'org.jcodec', name: 'jcodec', version: '0.1.9'
+
     compile group: 'org.bouncycastle', name: 'bcprov-jdk15on', version: '1.51'
     compile group: 'org.bouncycastle', name: 'bcpkix-jdk15on', version: '1.51'
-    compile group: 'com.google.android', name: 'support-v4', version:'r6'
-    compile group: 'com.koushikdutta.ion', name: 'ion', version:'1.3.7'
-    compile group: 'com.squareup.okhttp', name: 'okhttp', version:'2.1.0-RC1'
-    compile group: 'com.squareup.okio', name:'okio', version:'1.0.1'
+
+    compile group: 'com.squareup.okhttp', name: 'okhttp', version:'2.2.0'
+    compile group: 'com.squareup.okio', name:'okio', version:'1.2.0'
+
     compile files('libs/jmdns-fixed.jar')
     compile files('libs/limelight-common.jar')
     compile files('libs/tinyrtsp.jar')

app/src/main/AndroidManifest.xml

@@ -68,7 +68,7 @@
             android:configChanges="mcc|mnc|locale|touchscreen|keyboard|keyboardHidden|navigation|screenLayout|fontScale|uiMode|orientation|screenSize|smallestScreenSize|layoutDirection" >
             <meta-data
                 android:name="android.support.PARENT_ACTIVITY"
-                android:value="com.limelight.Connection" />
+                android:value="com.limelight.AppView" />
         </activity>
         <service
             android:name=".discovery.DiscoveryService"

app/src/main/java/com/limelight/AppView.java

@@ -1,185 +1,351 @@
 package com.limelight;
 
 import java.io.FileNotFoundException;
-import java.io.IOException;
+import java.io.StringReader;
 import java.net.InetAddress;
 import java.net.UnknownHostException;
 import java.util.List;
-
-import org.xmlpull.v1.XmlPullParserException;
+import java.util.Locale;
+import java.util.UUID;
 
 import com.limelight.binding.PlatformBinding;
+import com.limelight.computers.ComputerManagerListener;
+import com.limelight.computers.ComputerManagerService;
 import com.limelight.grid.AppGridAdapter;
+import com.limelight.nvstream.http.ComputerDetails;
 import com.limelight.nvstream.http.GfeHttpResponseException;
 import com.limelight.nvstream.http.NvApp;
 import com.limelight.nvstream.http.NvHTTP;
+import com.limelight.preferences.PreferenceConfiguration;
+import com.limelight.ui.AdapterFragment;
+import com.limelight.ui.AdapterFragmentCallbacks;
+import com.limelight.utils.CacheHelper;
 import com.limelight.utils.Dialog;
 import com.limelight.utils.SpinnerDialog;
 import com.limelight.utils.UiHelper;
 
 import android.app.Activity;
+import android.app.AlertDialog;
+import android.app.Service;
+import android.content.ComponentName;
+import android.content.DialogInterface;
 import android.content.Intent;
+import android.content.ServiceConnection;
+import android.content.res.Configuration;
 import android.os.Bundle;
+import android.os.IBinder;
 import android.view.ContextMenu;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.View;
 import android.view.ContextMenu.ContextMenuInfo;
+import android.widget.AbsListView;
 import android.widget.AdapterView;
 import android.widget.AdapterView.OnItemClickListener;
-import android.widget.GridView;
 import android.widget.TextView;
 import android.widget.Toast;
 import android.widget.AdapterView.AdapterContextMenuInfo;
 
-public class AppView extends Activity {
+public class AppView extends Activity implements AdapterFragmentCallbacks {
     private AppGridAdapter appGridAdapter;
-	private InetAddress ipAddress;
-	private String uniqueId;
-	private boolean remote;
-    private boolean firstLoad = true;
+    private String uuidString;
 
-	private final static int RESUME_ID = 1;
-	private final static int QUIT_ID = 2;
-	private final static int CANCEL_ID = 3;
+    private ComputerDetails computer;
+    private ComputerManagerService.ApplistPoller poller;
+    private SpinnerDialog blockingLoadSpinner;
+    private String lastRawApplist;
 
-	public final static String ADDRESS_EXTRA = "Address";
-	public final static String UNIQUEID_EXTRA = "UniqueId";
-	public final static String NAME_EXTRA = "Name";
-	public final static String REMOTE_EXTRA = "Remote";
+    private final static int START_OR_RESUME_ID = 1;
+    private final static int QUIT_ID = 2;
+    private final static int CANCEL_ID = 3;
+    private final static int START_WTIH_QUIT = 4;
 
-	@Override
-	protected void onCreate(Bundle savedInstanceState) {
-		super.onCreate(savedInstanceState);
-		setContentView(R.layout.activity_app_view);
+    public final static String NAME_EXTRA = "Name";
+    public final static String UUID_EXTRA = "UUID";
 
-        UiHelper.notifyNewRootView(this);
+    private ComputerManagerService.ComputerManagerBinder managerBinder;
+    private final ServiceConnection serviceConnection = new ServiceConnection() {
+        public void onServiceConnected(ComponentName className, IBinder binder) {
+            final ComputerManagerService.ComputerManagerBinder localBinder =
+                    ((ComputerManagerService.ComputerManagerBinder)binder);
 
-		byte[] address = getIntent().getByteArrayExtra(ADDRESS_EXTRA);
-		uniqueId = getIntent().getStringExtra(UNIQUEID_EXTRA);
-		remote = getIntent().getBooleanExtra(REMOTE_EXTRA, false);
-		if (address == null || uniqueId == null) {
-            finish();
-			return;
-		}
+            // Wait in a separate thread to avoid stalling the UI
+            new Thread() {
+                @Override
+                public void run() {
+                    // Wait for the binder to be ready
+                    localBinder.waitForReady();
 
-		String labelText = getResources().getString(R.string.title_applist)+" "+getIntent().getStringExtra(NAME_EXTRA);
-		TextView label = (TextView) findViewById(R.id.appListText);
-		setTitle(labelText);
-		label.setText(labelText);
+                    // Now make the binder visible
+                    managerBinder = localBinder;
 
-		try {
-			ipAddress = InetAddress.getByAddress(address);
-		} catch (UnknownHostException e) {
-            e.printStackTrace();
-            finish();
-            return;
-		}
+                    // Get the computer object
+                    computer = managerBinder.getComputer(UUID.fromString(uuidString));
 
-		// Setup the list view
-        GridView appGrid = (GridView) findViewById(R.id.appGridView);
-        try {
-            appGridAdapter = new AppGridAdapter(this, ipAddress, uniqueId);
-        } catch (Exception e) {
-            e.printStackTrace();
-            finish();
+                    try {
+                        appGridAdapter = new AppGridAdapter(AppView.this,
+                                PreferenceConfiguration.readPreferences(AppView.this).listMode,
+                                PreferenceConfiguration.readPreferences(AppView.this).smallIconMode,
+                                computer, managerBinder.getUniqueId());
+                    } catch (Exception e) {
+                        e.printStackTrace();
+                        finish();
+                        return;
+                    }
+
+                    // Start updates
+                    startComputerUpdates();
+
+                    // Load the app grid with cached data (if possible)
+                    populateAppGridWithCache();
+
+                    getFragmentManager().beginTransaction()
+                            .replace(R.id.appFragmentContainer, new AdapterFragment())
+                            .commitAllowingStateLoss();
+                }
+            }.start();
+        }
+
+        public void onServiceDisconnected(ComponentName className) {
+            managerBinder = null;
+        }
+    };
+
+    private InetAddress getAddress() {
+        return computer.reachability == ComputerDetails.Reachability.LOCAL ?
+        computer.localIp : computer.remoteIp;
+    }
+
+    private void startComputerUpdates() {
+        if (managerBinder == null) {
             return;
         }
-        appGrid.setAdapter(appGridAdapter);
-        appGrid.setOnItemClickListener(new OnItemClickListener() {
+
+        managerBinder.startPolling(new ComputerManagerListener() {
             @Override
-            public void onItemClick(AdapterView<?> arg0, View arg1, int pos,
-                                    long id) {
-                AppObject app = (AppObject) appGridAdapter.getItem(pos);
-                if (app == null || app.app == null) {
+            public void notifyComputerUpdated(ComputerDetails details) {
+                // Don't care about other computers
+                if (!details.uuid.toString().equalsIgnoreCase(uuidString)) {
                     return;
                 }
 
-                // Only open the context menu if something is running, otherwise start it
-                if (getRunningAppId() != -1) {
-                    openContextMenu(arg1);
-                } else {
-                    doStart(app.app);
+                if (details.state == ComputerDetails.State.OFFLINE) {
+                    // The PC is unreachable now
+                    AppView.this.runOnUiThread(new Runnable() {
+                        @Override
+                        public void run() {
+                            // Display a toast to the user and quit the activity
+                            Toast.makeText(AppView.this, getResources().getText(R.string.lost_connection), Toast.LENGTH_SHORT).show();
+                            finish();
+                        }
+                    });
+
+                    return;
                 }
+
+                // App list is the same or empty; nothing to do
+                if (details.rawAppList == null || details.rawAppList.equals(lastRawApplist)) {
+                    return;
+                }
+
+                try {
+                    lastRawApplist = details.rawAppList;
+                    updateUiWithAppList(NvHTTP.getAppListByReader(new StringReader(details.rawAppList)));
+
+                    if (blockingLoadSpinner != null) {
+                        blockingLoadSpinner.dismiss();
+                        blockingLoadSpinner = null;
+                    }
+                } catch (Exception ignored) {}
             }
         });
-        registerForContextMenu(appGrid);
-	}
 
-	@Override
-	protected void onDestroy() {
-		super.onDestroy();
-		
-		SpinnerDialog.closeDialogs(this);
-		Dialog.closeDialogs();
-	}
-	
-	@Override
-	protected void onResume() {
-		super.onResume();
-
-        // Display the error message if it was the
-        // first load, but just kill the activity
-        // on subsequent errors
-        updateAppList(firstLoad);
-        firstLoad = false;
-	}
-	
-	private int getRunningAppId() {
-        int runningAppId = -1;
-        for (int i = 0; i < appGridAdapter.getCount(); i++) {
-        	AppObject app = (AppObject) appGridAdapter.getItem(i);
-        	if (app.app == null) {
-        		continue;
-        	}
-        	
-        	if (app.app.getIsRunning()) {
-        		runningAppId = app.app.getAppId();
-        		break;
-        	}
+        if (poller == null) {
+            poller = managerBinder.createAppListPoller(computer);
         }
-        return runningAppId;
-	}
+        poller.start();
+    }
 
-	@Override
-	public void onCreateContextMenu(ContextMenu menu, View v, ContextMenuInfo menuInfo) {
-		super.onCreateContextMenu(menu, v, menuInfo);
-        
-        AdapterContextMenuInfo info = (AdapterContextMenuInfo) menuInfo;
-        AppObject selectedApp = (AppObject) appGridAdapter.getItem(info.position);
-        if (selectedApp == null || selectedApp.app == null) {
-        	return;
+    private void stopComputerUpdates() {
+        if (poller != null) {
+            poller.stop();
         }
 
-        int runningAppId = getRunningAppId();
-        if (runningAppId != -1) {
-        	if (runningAppId == selectedApp.app.getAppId()) {
-                menu.add(Menu.NONE, RESUME_ID, 1, getResources().getString(R.string.applist_menu_resume));
-                menu.add(Menu.NONE, QUIT_ID, 2, getResources().getString(R.string.applist_menu_quit));
-        	}
-        	else {
-                menu.add(Menu.NONE, RESUME_ID, 1, getResources().getString(R.string.applist_menu_quit_and_start));
-                menu.add(Menu.NONE, CANCEL_ID, 2, getResources().getString(R.string.applist_menu_cancel));
-        	}
+        if (managerBinder != null) {
+            managerBinder.stopPolling();
+        }
+
+        if (appGridAdapter != null) {
+            appGridAdapter.cancelQueuedOperations();
         }
     }
 
-	@Override
-	public void onContextMenuClosed(Menu menu) {
-	}
+    @Override
+    protected void onCreate(Bundle savedInstanceState) {
+        super.onCreate(savedInstanceState);
+
+        String locale = PreferenceConfiguration.readPreferences(this).language;
+        if (!locale.equals(PreferenceConfiguration.DEFAULT_LANGUAGE)) {
+            Configuration config = new Configuration(getResources().getConfiguration());
+            config.locale = new Locale(locale);
+            getResources().updateConfiguration(config, getResources().getDisplayMetrics());
+        }
+
+        setContentView(R.layout.activity_app_view);
+
+        UiHelper.notifyNewRootView(this);
+
+        uuidString = getIntent().getStringExtra(UUID_EXTRA);
+
+        String labelText = getResources().getString(R.string.title_applist)+" "+getIntent().getStringExtra(NAME_EXTRA);
+        TextView label = (TextView) findViewById(R.id.appListText);
+        setTitle(labelText);
+        label.setText(labelText);
+
+        // Bind to the computer manager service
+        bindService(new Intent(this, ComputerManagerService.class), serviceConnection,
+                Service.BIND_AUTO_CREATE);
+    }
+
+    private void populateAppGridWithCache() {
+        try {
+            // Try to load from cache
+            lastRawApplist = CacheHelper.readInputStreamToString(CacheHelper.openCacheFileForInput(getCacheDir(), "applist", uuidString));
+            List<NvApp> applist = NvHTTP.getAppListByReader(new StringReader(lastRawApplist));
+            updateUiWithAppList(applist);
+            LimeLog.info("Loaded applist from cache");
+        } catch (Exception e) {
+            if (lastRawApplist != null) {
+                LimeLog.warning("Saved applist corrupted: "+lastRawApplist);
+                e.printStackTrace();
+            }
+            LimeLog.info("Loading applist from the network");
+            // We'll need to load from the network
+            loadAppsBlocking();
+        }
+    }
+
+    private void loadAppsBlocking() {
+        blockingLoadSpinner = SpinnerDialog.displayDialog(this, getResources().getString(R.string.applist_refresh_title),
+                getResources().getString(R.string.applist_refresh_msg), true);
+    }
+
+    @Override
+    protected void onDestroy() {
+        super.onDestroy();
+
+        SpinnerDialog.closeDialogs(this);
+        Dialog.closeDialogs();
+
+        if (managerBinder != null) {
+            unbindService(serviceConnection);
+        }
+    }
+
+    @Override
+    protected void onResume() {
+        super.onResume();
+
+        startComputerUpdates();
+    }
+
+    @Override
+    protected void onPause() {
+        super.onPause();
+
+        stopComputerUpdates();
+    }
+
+    private int getRunningAppId() {
+        int runningAppId = -1;
+        for (int i = 0; i < appGridAdapter.getCount(); i++) {
+            AppObject app = (AppObject) appGridAdapter.getItem(i);
+            if (app.app.getIsRunning()) {
+                runningAppId = app.app.getAppId();
+                break;
+            }
+        }
+        return runningAppId;
+    }
+
+    @Override
+    public void onCreateContextMenu(ContextMenu menu, View v, ContextMenuInfo menuInfo) {
+        super.onCreateContextMenu(menu, v, menuInfo);
+        
+        AdapterContextMenuInfo info = (AdapterContextMenuInfo) menuInfo;
+        AppObject selectedApp = (AppObject) appGridAdapter.getItem(info.position);
+        int runningAppId = getRunningAppId();
+        if (runningAppId != -1) {
+            if (runningAppId == selectedApp.app.getAppId()) {
+                menu.add(Menu.NONE, START_OR_RESUME_ID, 1, getResources().getString(R.string.applist_menu_resume));
+                menu.add(Menu.NONE, QUIT_ID, 2, getResources().getString(R.string.applist_menu_quit));
+            }
+            else {
+                menu.add(Menu.NONE, START_WTIH_QUIT, 1, getResources().getString(R.string.applist_menu_quit_and_start));
+                menu.add(Menu.NONE, CANCEL_ID, 2, getResources().getString(R.string.applist_menu_cancel));
+            }
+        }
+    }
+
+    @Override
+    public void onContextMenuClosed(Menu menu) {
+    }
+
+    private void displayQuitConfirmationDialog(final Runnable onYes, final Runnable onNo) {
+        DialogInterface.OnClickListener dialogClickListener = new DialogInterface.OnClickListener() {
+            @Override
+            public void onClick(DialogInterface dialog, int which) {
+                switch (which){
+                    case DialogInterface.BUTTON_POSITIVE:
+                        if (onYes != null) {
+                            onYes.run();
+                        }
+                        break;
+
+                    case DialogInterface.BUTTON_NEGATIVE:
+                        if (onNo != null) {
+                            onNo.run();
+                        }
+                        break;
+                }
+            }
+        };
+
+        AlertDialog.Builder builder = new AlertDialog.Builder(this);
+        builder.setMessage(getResources().getString(R.string.applist_quit_confirmation))
+                .setPositiveButton(getResources().getString(R.string.yes), dialogClickListener)
+                .setNegativeButton(getResources().getString(R.string.no), dialogClickListener)
+                .show();
+    }
 
     @Override
     public boolean onContextItemSelected(MenuItem item) {
         AdapterContextMenuInfo info = (AdapterContextMenuInfo) item.getMenuInfo();
-        AppObject app = (AppObject) appGridAdapter.getItem(info.position);
+        final AppObject app = (AppObject) appGridAdapter.getItem(info.position);
         switch (item.getItemId()) {
-            case RESUME_ID:
+            case START_WTIH_QUIT:
+                // Display a confirmation dialog first
+                displayQuitConfirmationDialog(new Runnable() {
+                    @Override
+                    public void run() {
+                        doStart(app.app);
+                    }
+                }, null);
+                return true;
+
+            case START_OR_RESUME_ID:
                 // Resume is the same as start for us
                 doStart(app.app);
                 return true;
 
             case QUIT_ID:
-                doQuit(app.app);
+                // Display a confirmation dialog first
+                displayQuitConfirmationDialog(new Runnable() {
+                    @Override
+                    public void run() {
+                        doQuit(app.app);
+                    }
+                }, null);
                 return true;
 
             case CANCEL_ID:
@@ -190,109 +356,179 @@ public class AppView extends Activity {
         }
     }
 
-    private void updateAppList(final boolean displayError) {
-		final SpinnerDialog spinner = SpinnerDialog.displayDialog(this, getResources().getString(R.string.applist_refresh_title),
-				getResources().getString(R.string.applist_refresh_msg), true);
-		new Thread() {
-			@Override
-			public void run() {
-				NvHTTP httpConn = new NvHTTP(ipAddress, uniqueId, null,  PlatformBinding.getCryptoProvider(AppView.this));
+    private void updateUiWithAppList(final List<NvApp> appList) {
+        AppView.this.runOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                boolean updated = false;
 
-				try {
-					final List<NvApp> appList = httpConn.getAppList();
+                // First handle app updates and additions
+                for (NvApp app : appList) {
+                    boolean foundExistingApp = false;
 
-					AppView.this.runOnUiThread(new Runnable() {
-						@Override
-						public void run() {
-                            appGridAdapter.clear();
-                            for (NvApp app : appList) {
-                                appGridAdapter.addApp(new AppObject(app));
+                    // Try to update an existing app in the list first
+                    for (int i = 0; i < appGridAdapter.getCount(); i++) {
+                        AppObject existingApp = (AppObject) appGridAdapter.getItem(i);
+                        if (existingApp.app.getAppId() == app.getAppId()) {
+                            // Found the app; update its properties
+                            if (existingApp.app.getIsRunning() != app.getIsRunning()) {
+                                existingApp.app.setIsRunning(app.getIsRunning());
+                                updated = true;
+                            }
+                            if (!existingApp.app.getAppName().equals(app.getAppName())) {
+                                existingApp.app.setAppName(app.getAppName());
+                                updated = true;
                             }
 
-                            appGridAdapter.notifyDataSetChanged();
-						}
-					});
-					
-					// Success case
-					return;
-				} catch (GfeHttpResponseException ignored) {
-				} catch (IOException ignored) {
-				} catch (XmlPullParserException ignored) {
-				} finally {
-					spinner.dismiss();
-				}
-
-                if (displayError) {
-                    Dialog.displayDialog(AppView.this, getResources().getString(R.string.applist_refresh_error_title),
-                            getResources().getString(R.string.applist_refresh_error_msg), true);
-                }
-                else {
-                    // Just finish the activity immediately
-                    AppView.this.runOnUiThread(new Runnable() {
-                        @Override
-                        public void run() {
-                            finish();
+                            foundExistingApp = true;
+                            break;
                         }
-                    });
+                    }
+
+                    if (!foundExistingApp) {
+                        // This app must be new
+                        appGridAdapter.addApp(new AppObject(app));
+                        updated = true;
+                    }
                 }
-			}
-		}.start();
+
+                // Next handle app removals
+                int i = 0;
+                while (i < appGridAdapter.getCount()) {
+                    boolean foundExistingApp = false;
+                    AppObject existingApp = (AppObject) appGridAdapter.getItem(i);
+
+                    // Check if this app is in the latest list
+                    for (NvApp app : appList) {
+                        if (existingApp.app.getAppId() == app.getAppId()) {
+                            foundExistingApp = true;
+                            break;
+                        }
+                    }
+
+                    // This app was removed in the latest app list
+                    if (!foundExistingApp) {
+                        appGridAdapter.removeApp(existingApp);
+                        updated = true;
+
+                        // Check this same index again because the item at i+1 is now at i after
+                        // the removal
+                        continue;
+                    }
+
+                    // Move on to the next item
+                    i++;
+                }
+
+                if (updated) {
+                    appGridAdapter.notifyDataSetChanged();
+                }
+            }
+        });
     }
 
-	private void doStart(NvApp app) {
-		Intent intent = new Intent(this, Game.class);
-		intent.putExtra(Game.EXTRA_HOST, ipAddress.getHostAddress());
-		intent.putExtra(Game.EXTRA_APP, app.getAppName());
-		intent.putExtra(Game.EXTRA_UNIQUEID, uniqueId);
-		intent.putExtra(Game.EXTRA_STREAMING_REMOTE, remote);
-		startActivity(intent);
-	}
+    private void doStart(NvApp app) {
+        Intent intent = new Intent(this, Game.class);
+        intent.putExtra(Game.EXTRA_HOST,
+                computer.reachability == ComputerDetails.Reachability.LOCAL ?
+                computer.localIp.getHostAddress() : computer.remoteIp.getHostAddress());
+        intent.putExtra(Game.EXTRA_APP_NAME, app.getAppName());
+        intent.putExtra(Game.EXTRA_APP_ID, app.getAppId());
+        intent.putExtra(Game.EXTRA_UNIQUEID, managerBinder.getUniqueId());
+        intent.putExtra(Game.EXTRA_STREAMING_REMOTE,
+                computer.reachability != ComputerDetails.Reachability.LOCAL);
+        startActivity(intent);
+    }
 
-	private void doQuit(final NvApp app) {
-		Toast.makeText(AppView.this, getResources().getString(R.string.applist_quit_app)+" "+app.getAppName()+"...", Toast.LENGTH_SHORT).show();
-		new Thread(new Runnable() {
-			@Override
-			public void run() {
-				NvHTTP httpConn;
-				String message;
-				try {
-					httpConn = new NvHTTP(ipAddress, uniqueId, null,  PlatformBinding.getCryptoProvider(AppView.this));
-					if (httpConn.quitApp()) {
-						message = getResources().getString(R.string.applist_quit_success)+" "+app.getAppName();
-					}
-					else {
-						message = getResources().getString(R.string.applist_quit_fail)+" "+app.getAppName();
-					}
-					updateAppList(true);
-				} catch (UnknownHostException e) {
-					message = getResources().getString(R.string.error_unknown_host);
-				} catch (FileNotFoundException e) {
-					message = getResources().getString(R.string.error_404);
-				} catch (Exception e) {
-					message = e.getMessage();
-				}
+    private void doQuit(final NvApp app) {
+        Toast.makeText(AppView.this, getResources().getString(R.string.applist_quit_app)+" "+app.getAppName()+"...", Toast.LENGTH_SHORT).show();
+        new Thread(new Runnable() {
+            @Override
+            public void run() {
+                NvHTTP httpConn;
+                String message;
+                try {
+                    httpConn = new NvHTTP(getAddress(),
+                            managerBinder.getUniqueId(), null, PlatformBinding.getCryptoProvider(AppView.this));
+                    if (httpConn.quitApp()) {
+                        message = getResources().getString(R.string.applist_quit_success) + " " + app.getAppName();
+                    } else {
+                        message = getResources().getString(R.string.applist_quit_fail) + " " + app.getAppName();
+                    }
+                } catch (GfeHttpResponseException e) {
+                    if (e.getErrorCode() == 599) {
+                        message = "This session wasn't started by this device," +
+                                " so it cannot be quit. End streaming on the original " +
+                                "device or the PC itself. (Error code: "+e.getErrorCode()+")";
+                    }
+                    else {
+                        message = e.getMessage();
+                    }
+                } catch (UnknownHostException e) {
+                    message = getResources().getString(R.string.error_unknown_host);
+                } catch (FileNotFoundException e) {
+                    message = getResources().getString(R.string.error_404);
+                } catch (Exception e) {
+                    message = e.getMessage();
+                } finally {
+                    // Trigger a poll immediately
+                    if (poller != null) {
+                        poller.pollNow();
+                    }
+                }
 
-				final String toastMessage = message;
-				runOnUiThread(new Runnable() {
-					@Override
-					public void run() {
-						Toast.makeText(AppView.this, toastMessage, Toast.LENGTH_LONG).show();
-					}
-				});
-			}
-		}).start();
-	}
+                final String toastMessage = message;
+                runOnUiThread(new Runnable() {
+                    @Override
+                    public void run() {
+                        Toast.makeText(AppView.this, toastMessage, Toast.LENGTH_LONG).show();
+                    }
+                });
+            }
+        }).start();
+    }
 
-	public class AppObject {
-		public NvApp app;
+    @Override
+    public int getAdapterFragmentLayoutId() {
+        return PreferenceConfiguration.readPreferences(this).listMode ?
+                R.layout.list_view : (PreferenceConfiguration.readPreferences(AppView.this).smallIconMode ?
+                    R.layout.app_grid_view_small : R.layout.app_grid_view);
+    }
 
-		public AppObject(NvApp app) {
-			this.app = app;
-		}
+    @Override
+    public void receiveAbsListView(AbsListView listView) {
+        listView.setAdapter(appGridAdapter);
+        listView.setOnItemClickListener(new OnItemClickListener() {
+            @Override
+            public void onItemClick(AdapterView<?> arg0, View arg1, int pos,
+                                    long id) {
+                AppObject app = (AppObject) appGridAdapter.getItem(pos);
 
-		@Override
-		public String toString() {
-			return app.getAppName();
-		}
-	}
+                // Only open the context menu if something is running, otherwise start it
+                if (getRunningAppId() != -1) {
+                    openContextMenu(arg1);
+                } else {
+                    doStart(app.app);
+                }
+            }
+        });
+        registerForContextMenu(listView);
+        listView.requestFocus();
+    }
+
+    public class AppObject {
+        public final NvApp app;
+
+        public AppObject(NvApp app) {
+            if (app == null) {
+                throw new IllegalArgumentException("app must not be null");
+            }
+            this.app = app;
+        }
+
+        @Override
+        public String toString() {
+            return app.getAppName();
+        }
+    }
 }

app/src/main/java/com/limelight/binding/PlatformBinding.java

@@ -8,17 +8,17 @@ import com.limelight.nvstream.av.audio.AudioRenderer;
 import com.limelight.nvstream.http.LimelightCryptoProvider;
 
 public class PlatformBinding {
-	public static String getDeviceName() {
-		String deviceName = android.os.Build.MODEL;
+    public static String getDeviceName() {
+        String deviceName = android.os.Build.MODEL;
         deviceName = deviceName.replace(" ", "");
         return deviceName;
-	}
+    }
 
-	public static AudioRenderer getAudioRenderer() {
-		return new AndroidAudioRenderer();
-	}
+    public static AudioRenderer getAudioRenderer() {
+        return new AndroidAudioRenderer();
+    }
 
-	public static LimelightCryptoProvider getCryptoProvider(Context c) {
-		return new AndroidCryptoProvider(c);
-	}
+    public static LimelightCryptoProvider getCryptoProvider(Context c) {
+        return new AndroidCryptoProvider(c);
+    }
 }

app/src/main/java/com/limelight/binding/audio/AndroidAudioRenderer.java

@@ -9,27 +9,27 @@ import com.limelight.nvstream.av.audio.AudioRenderer;
 
 public class AndroidAudioRenderer implements AudioRenderer {
 
-	public static final int FRAME_SIZE = 960;
+    private static final int FRAME_SIZE = 960;
 
-	private AudioTrack track;
+    private AudioTrack track;
 
-	@Override
-	public boolean streamInitialized(int channelCount, int sampleRate) {
-		int channelConfig;
-		int bufferSize;
+    @Override
+    public boolean streamInitialized(int channelCount, int sampleRate) {
+        int channelConfig;
+        int bufferSize;
 
-		switch (channelCount)
-		{
-		case 1:
-			channelConfig = AudioFormat.CHANNEL_OUT_MONO;
-			break;
-		case 2:
-			channelConfig = AudioFormat.CHANNEL_OUT_STEREO;
-			break;
-		default:
-			LimeLog.severe("Decoder returned unhandled channel count");
-			return false;
-		}
+        switch (channelCount)
+        {
+        case 1:
+            channelConfig = AudioFormat.CHANNEL_OUT_MONO;
+            break;
+        case 2:
+            channelConfig = AudioFormat.CHANNEL_OUT_STEREO;
+            break;
+        default:
+            LimeLog.severe("Decoder returned unhandled channel count");
+            return false;
+        }
 
         // We're not supposed to request less than the minimum
         // buffer size for our buffer, but it appears that we can
@@ -73,25 +73,25 @@ public class AndroidAudioRenderer implements AudioRenderer {
             track.play();
         }
 
-		LimeLog.info("Audio track buffer size: "+bufferSize);
+        LimeLog.info("Audio track buffer size: "+bufferSize);
 
-		return true;
-	}
+        return true;
+    }
 
-	@Override
-	public void playDecodedAudio(byte[] audioData, int offset, int length) {
-		track.write(audioData, offset, length);
-	}
+    @Override
+    public void playDecodedAudio(byte[] audioData, int offset, int length) {
+        track.write(audioData, offset, length);
+    }
 
-	@Override
-	public void streamClosing() {
-		if (track != null) {
-			track.release();
-		}
-	}
+    @Override
+    public void streamClosing() {
+        if (track != null) {
+            track.release();
+        }
+    }
 
-	@Override
-	public int getCapabilities() {
-		return 0;
-	}
+    @Override
+    public int getCapabilities() {
+        return 0;
+    }
 }

app/src/main/java/com/limelight/binding/crypto/AndroidCryptoProvider.java

@@ -45,239 +45,239 @@ import com.limelight.nvstream.http.LimelightCryptoProvider;
 
 public class AndroidCryptoProvider implements LimelightCryptoProvider {
 
-	private File certFile;
-	private File keyFile;
+    private final File certFile;
+    private final File keyFile;
 
-	private X509Certificate cert;
-	private RSAPrivateKey key;
-	private byte[] pemCertBytes;
+    private X509Certificate cert;
+    private RSAPrivateKey key;
+    private byte[] pemCertBytes;
 
-	private static final Object globalCryptoLock = new Object();
+    private static final Object globalCryptoLock = new Object();
 
-	static {
-		// Install the Bouncy Castle provider
-		Security.addProvider(new BouncyCastleProvider());
-	}
+    static {
+        // Install the Bouncy Castle provider
+        Security.addProvider(new BouncyCastleProvider());
+    }
 
-	public AndroidCryptoProvider(Context c) {
-		String dataPath = c.getFilesDir().getAbsolutePath();
+    public AndroidCryptoProvider(Context c) {
+        String dataPath = c.getFilesDir().getAbsolutePath();
 
-		certFile = new File(dataPath + File.separator + "client.crt");
-		keyFile = new File(dataPath + File.separator + "client.key");
-	}
+        certFile = new File(dataPath + File.separator + "client.crt");
+        keyFile = new File(dataPath + File.separator + "client.key");
+    }
 
-	private byte[] loadFileToBytes(File f) {
-		if (!f.exists()) {
-			return null;
-		}
+    private byte[] loadFileToBytes(File f) {
+        if (!f.exists()) {
+            return null;
+        }
 
-		try {
-			FileInputStream fin = new FileInputStream(f);
-			byte[] fileData = new byte[(int) f.length()];
-			if (fin.read(fileData) != f.length()) {
+        try {
+            FileInputStream fin = new FileInputStream(f);
+            byte[] fileData = new byte[(int) f.length()];
+            if (fin.read(fileData) != f.length()) {
                 // Failed to read
                 fileData = null;
             }
-			fin.close();
-			return fileData;
-		} catch (IOException e) {
-			return null;
-		}
-	}
+            fin.close();
+            return fileData;
+        } catch (IOException e) {
+            return null;
+        }
+    }
 
-	private boolean loadCertKeyPair() {
-		byte[] certBytes = loadFileToBytes(certFile);
-		byte[] keyBytes = loadFileToBytes(keyFile);
+    private boolean loadCertKeyPair() {
+        byte[] certBytes = loadFileToBytes(certFile);
+        byte[] keyBytes = loadFileToBytes(keyFile);
 
-		// If either file was missing, we definitely can't succeed
-		if (certBytes == null || keyBytes == null) {
-			LimeLog.info("Missing cert or key; need to generate a new one");
-			return false;
-		}
+        // If either file was missing, we definitely can't succeed
+        if (certBytes == null || keyBytes == null) {
+            LimeLog.info("Missing cert or key; need to generate a new one");
+            return false;
+        }
 
-		try {
-			CertificateFactory certFactory = CertificateFactory.getInstance("X.509", "BC");
-			cert = (X509Certificate) certFactory.generateCertificate(new ByteArrayInputStream(certBytes));
-			pemCertBytes = certBytes;
-			KeyFactory keyFactory = KeyFactory.getInstance("RSA", "BC");
-			key = (RSAPrivateKey) keyFactory.generatePrivate(new PKCS8EncodedKeySpec(keyBytes));
-		} catch (CertificateException e) {
-			// May happen if the cert is corrupt
-			LimeLog.warning("Corrupted certificate");
-			return false;
-		} catch (NoSuchAlgorithmException e) {
-			// Should never happen
-			e.printStackTrace();
-			return false;
-		} catch (InvalidKeySpecException e) {
-			// May happen if the key is corrupt
-			LimeLog.warning("Corrupted key");
-			return false;
-		} catch (NoSuchProviderException e) {
-			// Should never happen
-			e.printStackTrace();
-			return false;
-		}
+        try {
+            CertificateFactory certFactory = CertificateFactory.getInstance("X.509", "BC");
+            cert = (X509Certificate) certFactory.generateCertificate(new ByteArrayInputStream(certBytes));
+            pemCertBytes = certBytes;
+            KeyFactory keyFactory = KeyFactory.getInstance("RSA", "BC");
+            key = (RSAPrivateKey) keyFactory.generatePrivate(new PKCS8EncodedKeySpec(keyBytes));
+        } catch (CertificateException e) {
+            // May happen if the cert is corrupt
+            LimeLog.warning("Corrupted certificate");
+            return false;
+        } catch (NoSuchAlgorithmException e) {
+            // Should never happen
+            e.printStackTrace();
+            return false;
+        } catch (InvalidKeySpecException e) {
+            // May happen if the key is corrupt
+            LimeLog.warning("Corrupted key");
+            return false;
+        } catch (NoSuchProviderException e) {
+            // Should never happen
+            e.printStackTrace();
+            return false;
+        }
 
-		return true;
-	}
+        return true;
+    }
 
-	@SuppressLint("TrulyRandom")
-	private boolean generateCertKeyPair() {
-		byte[] snBytes = new byte[8];
-		new SecureRandom().nextBytes(snBytes);
+    @SuppressLint("TrulyRandom")
+    private boolean generateCertKeyPair() {
+        byte[] snBytes = new byte[8];
+        new SecureRandom().nextBytes(snBytes);
 
-		KeyPair keyPair;
-		try {
-			KeyPairGenerator keyPairGenerator = KeyPairGenerator.getInstance("RSA", "BC");
-			keyPairGenerator.initialize(2048);
-			keyPair = keyPairGenerator.generateKeyPair();
-		} catch (NoSuchAlgorithmException e1) {
-			// Should never happen
-			e1.printStackTrace();
-			return false;
-		} catch (NoSuchProviderException e) {
-			// Should never happen
-			e.printStackTrace();
-			return false;
-		}
+        KeyPair keyPair;
+        try {
+            KeyPairGenerator keyPairGenerator = KeyPairGenerator.getInstance("RSA", "BC");
+            keyPairGenerator.initialize(2048);
+            keyPair = keyPairGenerator.generateKeyPair();
+        } catch (NoSuchAlgorithmException e1) {
+            // Should never happen
+            e1.printStackTrace();
+            return false;
+        } catch (NoSuchProviderException e) {
+            // Should never happen
+            e.printStackTrace();
+            return false;
+        }
 
-		Date now = new Date();
+        Date now = new Date();
 
-		// Expires in 20 years
-		Calendar calendar = Calendar.getInstance();
-		calendar.setTime(now);
-		calendar.add(Calendar.YEAR, 20);
-		Date expirationDate = calendar.getTime();
+        // Expires in 20 years
+        Calendar calendar = Calendar.getInstance();
+        calendar.setTime(now);
+        calendar.add(Calendar.YEAR, 20);
+        Date expirationDate = calendar.getTime();
 
-		BigInteger serial = new BigInteger(snBytes).abs();
+        BigInteger serial = new BigInteger(snBytes).abs();
 
-		X500NameBuilder nameBuilder = new X500NameBuilder(BCStyle.INSTANCE);
-		nameBuilder.addRDN(BCStyle.CN, "NVIDIA GameStream Client");
-		X500Name name = nameBuilder.build();
+        X500NameBuilder nameBuilder = new X500NameBuilder(BCStyle.INSTANCE);
+        nameBuilder.addRDN(BCStyle.CN, "NVIDIA GameStream Client");
+        X500Name name = nameBuilder.build();
 
-		X509v3CertificateBuilder certBuilder = new X509v3CertificateBuilder(name, serial, now, expirationDate, Locale.ENGLISH, name,
-			SubjectPublicKeyInfo.getInstance(keyPair.getPublic().getEncoded()));
+        X509v3CertificateBuilder certBuilder = new X509v3CertificateBuilder(name, serial, now, expirationDate, Locale.ENGLISH, name,
+            SubjectPublicKeyInfo.getInstance(keyPair.getPublic().getEncoded()));
 
-		try {
-			ContentSigner sigGen = new JcaContentSignerBuilder("SHA1withRSA").setProvider(BouncyCastleProvider.PROVIDER_NAME).build(keyPair.getPrivate());
-			cert = new JcaX509CertificateConverter().setProvider(BouncyCastleProvider.PROVIDER_NAME).getCertificate(certBuilder.build(sigGen));
-			key = (RSAPrivateKey) keyPair.getPrivate();
-		} catch (Exception e) {
-			// Nothing should go wrong here
-			e.printStackTrace();
-			return false;
-		}
+        try {
+            ContentSigner sigGen = new JcaContentSignerBuilder("SHA1withRSA").setProvider(BouncyCastleProvider.PROVIDER_NAME).build(keyPair.getPrivate());
+            cert = new JcaX509CertificateConverter().setProvider(BouncyCastleProvider.PROVIDER_NAME).getCertificate(certBuilder.build(sigGen));
+            key = (RSAPrivateKey) keyPair.getPrivate();
+        } catch (Exception e) {
+            // Nothing should go wrong here
+            e.printStackTrace();
+            return false;
+        }
 
-		LimeLog.info("Generated a new key pair");
+        LimeLog.info("Generated a new key pair");
 
-		// Save the resulting pair
-		saveCertKeyPair();
+        // Save the resulting pair
+        saveCertKeyPair();
 
-		return true;
-	}
+        return true;
+    }
 
-	private void saveCertKeyPair() {
-		try {
-			FileOutputStream certOut = new FileOutputStream(certFile);
-			FileOutputStream keyOut = new FileOutputStream(keyFile);
+    private void saveCertKeyPair() {
+        try {
+            FileOutputStream certOut = new FileOutputStream(certFile);
+            FileOutputStream keyOut = new FileOutputStream(keyFile);
 
-			// Write the certificate in OpenSSL PEM format (important for the server)
-			StringWriter strWriter = new StringWriter();
-			JcaPEMWriter pemWriter = new JcaPEMWriter(strWriter);
-			pemWriter.writeObject(cert);
-			pemWriter.close();
+            // Write the certificate in OpenSSL PEM format (important for the server)
+            StringWriter strWriter = new StringWriter();
+            JcaPEMWriter pemWriter = new JcaPEMWriter(strWriter);
+            pemWriter.writeObject(cert);
+            pemWriter.close();
 
-			// Line endings MUST be UNIX for the PC to accept the cert properly
-			OutputStreamWriter certWriter = new OutputStreamWriter(certOut);
-			String pemStr = strWriter.getBuffer().toString();
-			for (int i = 0; i < pemStr.length(); i++) {
-				char c = pemStr.charAt(i);
-				if (c != '\r')
-					certWriter.append(c);
-			}
-			certWriter.close();
+            // Line endings MUST be UNIX for the PC to accept the cert properly
+            OutputStreamWriter certWriter = new OutputStreamWriter(certOut);
+            String pemStr = strWriter.getBuffer().toString();
+            for (int i = 0; i < pemStr.length(); i++) {
+                char c = pemStr.charAt(i);
+                if (c != '\r')
+                    certWriter.append(c);
+            }
+            certWriter.close();
 
-			// Write the private out in PKCS8 format
-			keyOut.write(key.getEncoded());
+            // Write the private out in PKCS8 format
+            keyOut.write(key.getEncoded());
 
-			certOut.close();
-			keyOut.close();
+            certOut.close();
+            keyOut.close();
 
-			LimeLog.info("Saved generated key pair to disk");
-		} catch (IOException e) {
-			// This isn't good because it means we'll have
-			// to re-pair next time
-			e.printStackTrace();
-		}
-	}
+            LimeLog.info("Saved generated key pair to disk");
+        } catch (IOException e) {
+            // This isn't good because it means we'll have
+            // to re-pair next time
+            e.printStackTrace();
+        }
+    }
 
-	public X509Certificate getClientCertificate() {
-		// Use a lock here to ensure only one guy will be generating or loading
-		// the certificate and key at a time
-		synchronized (globalCryptoLock) {
-			// Return a loaded cert if we have one
-			if (cert != null) {
-				return cert;
-			}
+    public X509Certificate getClientCertificate() {
+        // Use a lock here to ensure only one guy will be generating or loading
+        // the certificate and key at a time
+        synchronized (globalCryptoLock) {
+            // Return a loaded cert if we have one
+            if (cert != null) {
+                return cert;
+            }
 
-			// No loaded cert yet, let's see if we have one on disk
-			if (loadCertKeyPair()) {
-				// Got one
-				return cert;
-			}
+            // No loaded cert yet, let's see if we have one on disk
+            if (loadCertKeyPair()) {
+                // Got one
+                return cert;
+            }
 
-			// Try to generate a new key pair
-			if (!generateCertKeyPair()) {
-				// Failed
-				return null;
-			}
+            // Try to generate a new key pair
+            if (!generateCertKeyPair()) {
+                // Failed
+                return null;
+            }
 
-			// Load the generated pair
-			loadCertKeyPair();
-			return cert;
-		}
-	}
+            // Load the generated pair
+            loadCertKeyPair();
+            return cert;
+        }
+    }
 
-	public RSAPrivateKey getClientPrivateKey() {
-		// Use a lock here to ensure only one guy will be generating or loading
-		// the certificate and key at a time
-		synchronized (globalCryptoLock) {
-			// Return a loaded key if we have one
-			if (key != null) {
-				return key;
-			}
+    public RSAPrivateKey getClientPrivateKey() {
+        // Use a lock here to ensure only one guy will be generating or loading
+        // the certificate and key at a time
+        synchronized (globalCryptoLock) {
+            // Return a loaded key if we have one
+            if (key != null) {
+                return key;
+            }
 
-			// No loaded key yet, let's see if we have one on disk
-			if (loadCertKeyPair()) {
-				// Got one
-				return key;
-			}
+            // No loaded key yet, let's see if we have one on disk
+            if (loadCertKeyPair()) {
+                // Got one
+                return key;
+            }
 
-			// Try to generate a new key pair
-			if (!generateCertKeyPair()) {
-				// Failed
-				return null;
-			}
+            // Try to generate a new key pair
+            if (!generateCertKeyPair()) {
+                // Failed
+                return null;
+            }
 
-			// Load the generated pair
-			loadCertKeyPair();
-			return key;
-		}
-	}
+            // Load the generated pair
+            loadCertKeyPair();
+            return key;
+        }
+    }
 
-	public byte[] getPemEncodedClientCertificate() {
-		synchronized (globalCryptoLock) {
-			// Call our helper function to do the cert loading/generation for us
-			getClientCertificate();
+    public byte[] getPemEncodedClientCertificate() {
+        synchronized (globalCryptoLock) {
+            // Call our helper function to do the cert loading/generation for us
+            getClientCertificate();
 
-			// Return a cached value if we have it
-			return pemCertBytes;
-		}
-	}
+            // Return a cached value if we have it
+            return pemCertBytes;
+        }
+    }
 
-	@Override
-	public String encodeBase64String(byte[] data) {
-		return Base64.encodeToString(data, Base64.NO_WRAP);
-	}
+    @Override
+    public String encodeBase64String(byte[] data) {
+        return Base64.encodeToString(data, Base64.NO_WRAP);
+    }
 }

app/src/main/java/com/limelight/binding/input/KeyboardTranslator.java

@@ -15,7 +15,7 @@ public class KeyboardTranslator extends KeycodeTranslator {
 	/**
 	 * GFE's prefix for every key code
 	 */
-	public static final short KEY_PREFIX = (short) 0x80;
+	private static final short KEY_PREFIX = (short) 0x80;
 	
 	public static final int VK_0 = 48;
 	public static final int VK_9 = 57;
@@ -23,8 +23,8 @@ public class KeyboardTranslator extends KeycodeTranslator {
 	public static final int VK_Z = 90;
 	public static final int VK_ALT = 18;
 	public static final int VK_NUMPAD0 = 96;
-	public static final int VK_BACK_SLASH = 92;
-	public static final int VK_CAPS_LOCK = 20;
+    public static final int VK_BACK_SLASH = 92;
+    public static final int VK_CAPS_LOCK = 20;
 	public static final int VK_CLEAR = 12;
 	public static final int VK_COMMA = 44;
 	public static final int VK_CONTROL = 17;

app/src/main/java/com/limelight/binding/input/TouchContext.java

@@ -4,88 +4,95 @@ import com.limelight.nvstream.NvConnection;
 import com.limelight.nvstream.input.MouseButtonPacket;
 
 public class TouchContext {
-	private int lastTouchX = 0;
-	private int lastTouchY = 0;
-	private int originalTouchX = 0;
-	private int originalTouchY = 0;
-	private long originalTouchTime = 0;
+    private int lastTouchX = 0;
+    private int lastTouchY = 0;
+    private int originalTouchX = 0;
+    private int originalTouchY = 0;
+    private long originalTouchTime = 0;
+    private boolean cancelled;
 
-	private NvConnection conn;
-	private int actionIndex;
-    private double xFactor, yFactor;
+    private final NvConnection conn;
+    private final int actionIndex;
+    private final double xFactor;
+    private final double yFactor;
 
-	private static final int TAP_MOVEMENT_THRESHOLD = 10;
-	private static final int TAP_TIME_THRESHOLD = 250;
+    private static final int TAP_MOVEMENT_THRESHOLD = 10;
+    private static final int TAP_TIME_THRESHOLD = 250;
 
-	public TouchContext(NvConnection conn, int actionIndex, double xFactor, double yFactor)
-	{
-		this.conn = conn;
-		this.actionIndex = actionIndex;
+    public TouchContext(NvConnection conn, int actionIndex, double xFactor, double yFactor)
+    {
+        this.conn = conn;
+        this.actionIndex = actionIndex;
         this.xFactor = xFactor;
         this.yFactor = yFactor;
-	}
+    }
 
     public int getActionIndex()
     {
         return actionIndex;
     }
 
-	private boolean isTap()
-	{
-		int xDelta = Math.abs(lastTouchX - originalTouchX);
-		int yDelta = Math.abs(lastTouchY - originalTouchY);
-		long timeDelta = System.currentTimeMillis() - originalTouchTime;
+    private boolean isTap()
+    {
+        int xDelta = Math.abs(lastTouchX - originalTouchX);
+        int yDelta = Math.abs(lastTouchY - originalTouchY);
+        long timeDelta = System.currentTimeMillis() - originalTouchTime;
 
-		return xDelta <= TAP_MOVEMENT_THRESHOLD &&
-				yDelta <= TAP_MOVEMENT_THRESHOLD &&
-				timeDelta <= TAP_TIME_THRESHOLD;
-	}
+        return xDelta <= TAP_MOVEMENT_THRESHOLD &&
+                yDelta <= TAP_MOVEMENT_THRESHOLD &&
+                timeDelta <= TAP_TIME_THRESHOLD;
+    }
 
-	private byte getMouseButtonIndex()
-	{
-		if (actionIndex == 1) {
-			return MouseButtonPacket.BUTTON_RIGHT;
-		}
-		else {
-			return MouseButtonPacket.BUTTON_LEFT;
-		}
-	}
+    private byte getMouseButtonIndex()
+    {
+        if (actionIndex == 1) {
+            return MouseButtonPacket.BUTTON_RIGHT;
+        }
+        else {
+            return MouseButtonPacket.BUTTON_LEFT;
+        }
+    }
 
-	public boolean touchDownEvent(int eventX, int eventY)
-	{
-		originalTouchX = lastTouchX = eventX;
-		originalTouchY = lastTouchY = eventY;
-		originalTouchTime = System.currentTimeMillis();
+    public boolean touchDownEvent(int eventX, int eventY)
+    {
+        originalTouchX = lastTouchX = eventX;
+        originalTouchY = lastTouchY = eventY;
+        originalTouchTime = System.currentTimeMillis();
+        cancelled = false;
 
-		return true;
-	}
+        return true;
+    }
 
-	public void touchUpEvent(int eventX, int eventY)
-	{
-		if (isTap())
-		{
-			byte buttonIndex = getMouseButtonIndex();
+    public void touchUpEvent(int eventX, int eventY)
+    {
+        if (cancelled) {
+            return;
+        }
 
-			// Lower the mouse button
-			conn.sendMouseButtonDown(buttonIndex);
+        if (isTap())
+        {
+            byte buttonIndex = getMouseButtonIndex();
 
-			// We need to sleep a bit here because some games
-			// do input detection by polling
-			try {
-				Thread.sleep(100);
-			} catch (InterruptedException ignored) {}
+            // Lower the mouse button
+            conn.sendMouseButtonDown(buttonIndex);
 
-			// Raise the mouse button
-			conn.sendMouseButtonUp(buttonIndex);
-		}
-	}
+            // We need to sleep a bit here because some games
+            // do input detection by polling
+            try {
+                Thread.sleep(100);
+            } catch (InterruptedException ignored) {}
 
-	public boolean touchMoveEvent(int eventX, int eventY)
-	{	
-		if (eventX != lastTouchX || eventY != lastTouchY)
-		{
-			// We only send moves for the primary touch point
-			if (actionIndex == 0) {
+            // Raise the mouse button
+            conn.sendMouseButtonUp(buttonIndex);
+        }
+    }
+
+    public boolean touchMoveEvent(int eventX, int eventY)
+    {
+        if (eventX != lastTouchX || eventY != lastTouchY)
+        {
+            // We only send moves for the primary touch point
+            if (actionIndex == 0) {
                 int deltaX = eventX - lastTouchX;
                 int deltaY = eventY - lastTouchY;
 
@@ -93,13 +100,21 @@ public class TouchContext {
                 deltaX = (int)Math.round((double)deltaX * xFactor);
                 deltaY = (int)Math.round((double)deltaY * yFactor);
 
-				conn.sendMouseMove((short)deltaX, (short)deltaY);
-			}
+                conn.sendMouseMove((short)deltaX, (short)deltaY);
+            }
 
-			lastTouchX = eventX;
-			lastTouchY = eventY;
-		}
+            lastTouchX = eventX;
+            lastTouchY = eventY;
+        }
 
-		return true;
-	}
+        return true;
+    }
+
+    public void cancelTouch() {
+        cancelled = true;
+    }
+
+    public boolean isCancelled() {
+        return cancelled;
+    }
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevEvent.java

@@ -1,41 +1,41 @@
 package com.limelight.binding.input.evdev;
 
 public class EvdevEvent {
-	public static final int EVDEV_MIN_EVENT_SIZE = 16;
-	public static final int EVDEV_MAX_EVENT_SIZE = 24;
+    public static final int EVDEV_MIN_EVENT_SIZE = 16;
+    public static final int EVDEV_MAX_EVENT_SIZE = 24;
 
-	/* Event types */
-	public static final short EV_SYN = 0x00;
-	public static final short EV_KEY = 0x01;
-	public static final short EV_REL = 0x02;
-	public static final short EV_MSC = 0x04;
+    /* Event types */
+    public static final short EV_SYN = 0x00;
+    public static final short EV_KEY = 0x01;
+    public static final short EV_REL = 0x02;
+    public static final short EV_MSC = 0x04;
 
-	/* Relative axes */
-	public static final short REL_X = 0x00;
-	public static final short REL_Y = 0x01;
-	public static final short REL_WHEEL = 0x08;
+    /* Relative axes */
+    public static final short REL_X = 0x00;
+    public static final short REL_Y = 0x01;
+    public static final short REL_WHEEL = 0x08;
 
-	/* Buttons */
-	public static final short BTN_LEFT = 0x110;
-	public static final short BTN_RIGHT = 0x111;
-	public static final short BTN_MIDDLE = 0x112;
-	public static final short BTN_SIDE = 0x113;
-	public static final short BTN_EXTRA = 0x114;
-	public static final short BTN_FORWARD = 0x115;
-	public static final short BTN_BACK = 0x116;
-	public static final short BTN_TASK = 0x117;
-	public static final short BTN_GAMEPAD = 0x130;
+    /* Buttons */
+    public static final short BTN_LEFT = 0x110;
+    public static final short BTN_RIGHT = 0x111;
+    public static final short BTN_MIDDLE = 0x112;
+    public static final short BTN_SIDE = 0x113;
+    public static final short BTN_EXTRA = 0x114;
+    public static final short BTN_FORWARD = 0x115;
+    public static final short BTN_BACK = 0x116;
+    public static final short BTN_TASK = 0x117;
+    public static final short BTN_GAMEPAD = 0x130;
 
-	/* Keys */
-	public static final short KEY_Q = 16;
+    /* Keys */
+    public static final short KEY_Q = 16;
 
-	public short type;
-	public short code;
-	public int value;
+    public final short type;
+    public final short code;
+    public final int value;
 
-	public EvdevEvent(short type, short code, int value) {
-		this.type = type;
-		this.code = code;
-		this.value = value;
-	}
+    public EvdevEvent(short type, short code, int value) {
+        this.type = type;
+        this.code = code;
+        this.value = value;
+    }
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevHandler.java

@@ -7,161 +7,161 @@ import com.limelight.LimeLog;
 
 public class EvdevHandler {
 
-	private String absolutePath;
-	private EvdevListener listener;
-	private boolean shutdown = false;
-	private int fd = -1;
+    private final String absolutePath;
+    private final EvdevListener listener;
+    private boolean shutdown = false;
+    private int fd = -1;
 
-	private Thread handlerThread = new Thread() {
-		@Override
-		public void run() {
-			// All the finally blocks here make this code look like a mess
-			// but it's important that we get this right to avoid causing
-			// system-wide input problems.
+    private final Thread handlerThread = new Thread() {
+        @Override
+        public void run() {
+            // All the finally blocks here make this code look like a mess
+            // but it's important that we get this right to avoid causing
+            // system-wide input problems.
 
-			// Open the /dev/input/eventX file
-			fd = EvdevReader.open(absolutePath);
-			if (fd == -1) {
-				LimeLog.warning("Unable to open "+absolutePath);
-				return;
-			}
+            // Open the /dev/input/eventX file
+            fd = EvdevReader.open(absolutePath);
+            if (fd == -1) {
+                LimeLog.warning("Unable to open "+absolutePath);
+                return;
+            }
 
-			try {
-				// Check if it's a mouse or keyboard, but not a gamepad
-				if ((!EvdevReader.isMouse(fd) && !EvdevReader.isAlphaKeyboard(fd)) ||
-					EvdevReader.isGamepad(fd)) {
-					// We only handle keyboards and mice
-					return;
-				}
+            try {
+                // Check if it's a mouse or keyboard, but not a gamepad
+                if ((!EvdevReader.isMouse(fd) && !EvdevReader.isAlphaKeyboard(fd)) ||
+                    EvdevReader.isGamepad(fd)) {
+                    // We only handle keyboards and mice
+                    return;
+                }
 
-				// Grab it for ourselves
-				if (!EvdevReader.grab(fd)) {
-					LimeLog.warning("Unable to grab "+absolutePath);
-					return;
-				}
+                // Grab it for ourselves
+                if (!EvdevReader.grab(fd)) {
+                    LimeLog.warning("Unable to grab "+absolutePath);
+                    return;
+                }
 
-				LimeLog.info("Grabbed device for raw keyboard/mouse input: "+absolutePath);
+                LimeLog.info("Grabbed device for raw keyboard/mouse input: "+absolutePath);
 
-				ByteBuffer buffer = ByteBuffer.allocate(EvdevEvent.EVDEV_MAX_EVENT_SIZE).order(ByteOrder.nativeOrder());
+                ByteBuffer buffer = ByteBuffer.allocate(EvdevEvent.EVDEV_MAX_EVENT_SIZE).order(ByteOrder.nativeOrder());
 
-				try {
-					int deltaX = 0;
-					int deltaY = 0;
-					byte deltaScroll = 0;
+                try {
+                    int deltaX = 0;
+                    int deltaY = 0;
+                    byte deltaScroll = 0;
 
-					while (!isInterrupted() && !shutdown) {
-						EvdevEvent event = EvdevReader.read(fd, buffer);
-						if (event == null) {
-							return;
-						}
+                    while (!isInterrupted() && !shutdown) {
+                        EvdevEvent event = EvdevReader.read(fd, buffer);
+                        if (event == null) {
+                            return;
+                        }
 
-						switch (event.type)
-						{
-						case EvdevEvent.EV_SYN:
-							if (deltaX != 0 || deltaY != 0) {
-								listener.mouseMove(deltaX, deltaY);
-								deltaX = deltaY = 0;
-							}
-							if (deltaScroll != 0) {
-								listener.mouseScroll(deltaScroll);
-								deltaScroll = 0;
-							}
-							break;
+                        switch (event.type)
+                        {
+                        case EvdevEvent.EV_SYN:
+                            if (deltaX != 0 || deltaY != 0) {
+                                listener.mouseMove(deltaX, deltaY);
+                                deltaX = deltaY = 0;
+                            }
+                            if (deltaScroll != 0) {
+                                listener.mouseScroll(deltaScroll);
+                                deltaScroll = 0;
+                            }
+                            break;
 
-						case EvdevEvent.EV_REL:
-							switch (event.code)
-							{
-							case EvdevEvent.REL_X:
-								deltaX = event.value;
-								break;
-							case EvdevEvent.REL_Y:
-								deltaY = event.value;
-								break;
-							case EvdevEvent.REL_WHEEL:
-								deltaScroll = (byte) event.value;
-								break;
-							}
-							break;
+                        case EvdevEvent.EV_REL:
+                            switch (event.code)
+                            {
+                            case EvdevEvent.REL_X:
+                                deltaX = event.value;
+                                break;
+                            case EvdevEvent.REL_Y:
+                                deltaY = event.value;
+                                break;
+                            case EvdevEvent.REL_WHEEL:
+                                deltaScroll = (byte) event.value;
+                                break;
+                            }
+                            break;
 
-						case EvdevEvent.EV_KEY:
-							switch (event.code)
-							{
-							case EvdevEvent.BTN_LEFT:
-								listener.mouseButtonEvent(EvdevListener.BUTTON_LEFT,
-										event.value != 0);
-								break;
-							case EvdevEvent.BTN_MIDDLE:
-								listener.mouseButtonEvent(EvdevListener.BUTTON_MIDDLE,
-										event.value != 0);
-								break;
-							case EvdevEvent.BTN_RIGHT:
-								listener.mouseButtonEvent(EvdevListener.BUTTON_RIGHT,
-										event.value != 0);
-								break;
+                        case EvdevEvent.EV_KEY:
+                            switch (event.code)
+                            {
+                            case EvdevEvent.BTN_LEFT:
+                                listener.mouseButtonEvent(EvdevListener.BUTTON_LEFT,
+                                        event.value != 0);
+                                break;
+                            case EvdevEvent.BTN_MIDDLE:
+                                listener.mouseButtonEvent(EvdevListener.BUTTON_MIDDLE,
+                                        event.value != 0);
+                                break;
+                            case EvdevEvent.BTN_RIGHT:
+                                listener.mouseButtonEvent(EvdevListener.BUTTON_RIGHT,
+                                        event.value != 0);
+                                break;
 
-							case EvdevEvent.BTN_SIDE:
-							case EvdevEvent.BTN_EXTRA:
-							case EvdevEvent.BTN_FORWARD:
-							case EvdevEvent.BTN_BACK:
-							case EvdevEvent.BTN_TASK:
-								// Other unhandled mouse buttons
-								break;
+                            case EvdevEvent.BTN_SIDE:
+                            case EvdevEvent.BTN_EXTRA:
+                            case EvdevEvent.BTN_FORWARD:
+                            case EvdevEvent.BTN_BACK:
+                            case EvdevEvent.BTN_TASK:
+                                // Other unhandled mouse buttons
+                                break;
 
-							default:
-								// We got some unrecognized button. This means
-								// someone is trying to use the other device in this
-								// "combination" input device. We'll try to handle
-								// it via keyboard, but we're not going to disconnect
-								// if we can't
-								short keyCode = EvdevTranslator.translateEvdevKeyCode(event.code);
-								if (keyCode != 0) {
-									listener.keyboardEvent(event.value != 0, keyCode);
-								}
-								break;
-							}
-							break;
+                            default:
+                                // We got some unrecognized button. This means
+                                // someone is trying to use the other device in this
+                                // "combination" input device. We'll try to handle
+                                // it via keyboard, but we're not going to disconnect
+                                // if we can't
+                                short keyCode = EvdevTranslator.translateEvdevKeyCode(event.code);
+                                if (keyCode != 0) {
+                                    listener.keyboardEvent(event.value != 0, keyCode);
+                                }
+                                break;
+                            }
+                            break;
 
-						case EvdevEvent.EV_MSC:
-							break;
-						}
-					}
-				} finally {
-					// Release our grab
-					EvdevReader.ungrab(fd);
-				}
-			} finally {
-				// Close the file
-				EvdevReader.close(fd);
-			}
-		}
-	};
+                        case EvdevEvent.EV_MSC:
+                            break;
+                        }
+                    }
+                } finally {
+                    // Release our grab
+                    EvdevReader.ungrab(fd);
+                }
+            } finally {
+                // Close the file
+                EvdevReader.close(fd);
+            }
+        }
+    };
 
-	public EvdevHandler(String absolutePath, EvdevListener listener) {
-		this.absolutePath = absolutePath;
-		this.listener = listener;
-	}
+    public EvdevHandler(String absolutePath, EvdevListener listener) {
+        this.absolutePath = absolutePath;
+        this.listener = listener;
+    }
 
-	public void start() {
-		handlerThread.start();
-	}
+    public void start() {
+        handlerThread.start();
+    }
 
-	public void stop() {
-		// Close the fd. It doesn't matter if this races
-		// with the handler thread. We'll close this out from
-		// under the thread to wake it up
-		if (fd != -1) {
-			EvdevReader.close(fd);
-		}
+    public void stop() {
+        // Close the fd. It doesn't matter if this races
+        // with the handler thread. We'll close this out from
+        // under the thread to wake it up
+        if (fd != -1) {
+            EvdevReader.close(fd);
+        }
 
-		shutdown = true;
-		handlerThread.interrupt();
+        shutdown = true;
+        handlerThread.interrupt();
 
-		try {
-			handlerThread.join();
-		} catch (InterruptedException ignored) {}
-	}
+        try {
+            handlerThread.join();
+        } catch (InterruptedException ignored) {}
+    }
 
-	public void notifyDeleted() {
-		stop();
-	}
+    public void notifyDeleted() {
+        stop();
+    }
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevListener.java

@@ -1,12 +1,12 @@
 package com.limelight.binding.input.evdev;
 
 public interface EvdevListener {
-	public static final int BUTTON_LEFT = 1;
-	public static final int BUTTON_MIDDLE = 2;
-	public static final int BUTTON_RIGHT = 3;
+    public static final int BUTTON_LEFT = 1;
+    public static final int BUTTON_MIDDLE = 2;
+    public static final int BUTTON_RIGHT = 3;
 
-	public void mouseMove(int deltaX, int deltaY);
-	public void mouseButtonEvent(int buttonId, boolean down);
-	public void mouseScroll(byte amount);
-	public void keyboardEvent(boolean buttonDown, short keyCode);
+    public void mouseMove(int deltaX, int deltaY);
+    public void mouseButtonEvent(int buttonId, boolean down);
+    public void mouseScroll(byte amount);
+    public void keyboardEvent(boolean buttonDown, short keyCode);
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevReader.java

@@ -1,103 +1,105 @@
 package com.limelight.binding.input.evdev;
 
-import java.io.IOException;
-import java.io.OutputStream;
+import android.os.Build;
+
 import java.nio.ByteBuffer;
 import java.util.Locale;
 
 import com.limelight.LimeLog;
 
 public class EvdevReader {
-	static {
-		System.loadLibrary("evdev_reader");
-	}
+    static {
+        System.loadLibrary("evdev_reader");
+    }
 
-	// Requires root to chmod /dev/input/eventX
-	public static boolean setPermissions(String[] files, int octalPermissions) {
-		ProcessBuilder builder = new ProcessBuilder("su");
+    public static void patchSeLinuxPolicies() {
+        //
+        // FIXME: We REALLY shouldn't being changing permissions on the input devices like this.
+        // We should probably do something clever with a separate daemon and talk via a localhost
+        // socket. We don't return the SELinux policies back to default after we're done which I feel
+        // bad about, but we do chmod the input devices back so I don't think any additional attack surface
+        // remains opened after streaming other than listing the /dev/input directory which you wouldn't
+        // normally be able to do with SELinux enforcing on Lollipop.
+        //
+        // We need to modify SELinux policies to allow us to capture input devices on Lollipop and possibly other
+        // more restrictive ROMs. Per Chainfire's SuperSU documentation, the supolicy binary is provided on
+        // 4.4 and later to do live SELinux policy changes.
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
+            EvdevShell shell = EvdevShell.getInstance();
+            shell.runCommand("supolicy --live \"allow untrusted_app input_device dir { open getattr read search }\" " +
+                    "\"allow untrusted_app input_device chr_file { open read write ioctl }\"");
+        }
+    }
 
-		try {
-			Process p = builder.start();
+    // Requires root to chmod /dev/input/eventX
+    public static void setPermissions(String[] files, int octalPermissions) {
+        EvdevShell shell = EvdevShell.getInstance();
 
-			OutputStream stdin = p.getOutputStream();
-			for (String file : files) {
-				stdin.write(String.format((Locale)null, "chmod %o %s\n", octalPermissions, file).getBytes("UTF-8"));
-			}
-			stdin.write("exit\n".getBytes("UTF-8"));
-			stdin.flush();
+        for (String file : files) {
+            shell.runCommand(String.format((Locale)null, "chmod %o %s", octalPermissions, file));
+        }
+    }
 
-			p.waitFor();
-			p.destroy();
-            return true;
-		} catch (IOException e) {
-			e.printStackTrace();
-		} catch (InterruptedException e) {
-			e.printStackTrace();
-		}
+    // Returns the fd to be passed to other function or -1 on error
+    public static native int open(String fileName);
 
-		return false;
-	}
+    // Prevent other apps (including Android itself) from using the device while "grabbed"
+    public static native boolean grab(int fd);
+    public static native boolean ungrab(int fd);
 
-	// Returns the fd to be passed to other function or -1 on error
-	public static native int open(String fileName);
+    // Used for checking device capabilities
+    public static native boolean hasRelAxis(int fd, short axis);
+    public static native boolean hasAbsAxis(int fd, short axis);
+    public static native boolean hasKey(int fd, short key);
 
-	// Prevent other apps (including Android itself) from using the device while "grabbed"
-	public static native boolean grab(int fd);
-	public static native boolean ungrab(int fd);
+    public static boolean isMouse(int fd) {
+        // This is the same check that Android does in EventHub.cpp
+        return hasRelAxis(fd, EvdevEvent.REL_X) &&
+                hasRelAxis(fd, EvdevEvent.REL_Y) &&
+                hasKey(fd, EvdevEvent.BTN_LEFT);
+    }
 
-	// Used for checking device capabilities
-	public static native boolean hasRelAxis(int fd, short axis);
-	public static native boolean hasAbsAxis(int fd, short axis);
-	public static native boolean hasKey(int fd, short key);
+    public static boolean isAlphaKeyboard(int fd) {
+        // This is the same check that Android does in EventHub.cpp
+        return hasKey(fd, EvdevEvent.KEY_Q);
+    }
 
-	public static boolean isMouse(int fd) {
-		// This is the same check that Android does in EventHub.cpp
-		return hasRelAxis(fd, EvdevEvent.REL_X) &&
-				hasRelAxis(fd, EvdevEvent.REL_Y) &&
-				hasKey(fd, EvdevEvent.BTN_LEFT);
-	}
+    public static boolean isGamepad(int fd) {
+        return hasKey(fd, EvdevEvent.BTN_GAMEPAD);
+    }
 
-	public static boolean isAlphaKeyboard(int fd) {
-		// This is the same check that Android does in EventHub.cpp
-		return hasKey(fd, EvdevEvent.KEY_Q);
-	}
+    // Returns the bytes read or -1 on error
+    private static native int read(int fd, byte[] buffer);
 
-	public static boolean isGamepad(int fd) {
-		return hasKey(fd, EvdevEvent.BTN_GAMEPAD);
-	}
+    // Takes a byte buffer to use to read the output into.
+    // This buffer MUST be in native byte order and at least
+    // EVDEV_MAX_EVENT_SIZE bytes long.
+    public static EvdevEvent read(int fd, ByteBuffer buffer) {
+        int bytesRead = read(fd, buffer.array());
+        if (bytesRead < 0) {
+            LimeLog.warning("Failed to read: "+bytesRead);
+            return null;
+        }
+        else if (bytesRead < EvdevEvent.EVDEV_MIN_EVENT_SIZE) {
+            LimeLog.warning("Short read: "+bytesRead);
+            return null;
+        }
 
-	// Returns the bytes read or -1 on error
-	private static native int read(int fd, byte[] buffer);
+        buffer.limit(bytesRead);
+        buffer.rewind();
 
-	// Takes a byte buffer to use to read the output into.
-	// This buffer MUST be in native byte order and at least
-	// EVDEV_MAX_EVENT_SIZE bytes long.
-	public static EvdevEvent read(int fd, ByteBuffer buffer) {
-		int bytesRead = read(fd, buffer.array());
-		if (bytesRead < 0) {
-			LimeLog.warning("Failed to read: "+bytesRead);
-			return null;
-		}
-		else if (bytesRead < EvdevEvent.EVDEV_MIN_EVENT_SIZE) {
-			LimeLog.warning("Short read: "+bytesRead);
-			return null;
-		}
+        // Throw away the time stamp
+        if (bytesRead == EvdevEvent.EVDEV_MAX_EVENT_SIZE) {
+            buffer.getLong();
+            buffer.getLong();
+        } else {
+            buffer.getInt();
+            buffer.getInt();
+        }
 
-		buffer.limit(bytesRead);
-		buffer.rewind();
+        return new EvdevEvent(buffer.getShort(), buffer.getShort(), buffer.getInt());
+    }
 
-		// Throw away the time stamp
-		if (bytesRead == EvdevEvent.EVDEV_MAX_EVENT_SIZE) {
-			buffer.getLong();
-			buffer.getLong();
-		} else {
-			buffer.getInt();
-			buffer.getInt();
-		}
-		
-		return new EvdevEvent(buffer.getShort(), buffer.getShort(), buffer.getInt());
-	}
-	
-	// Closes the fd from open()
-	public static native int close(int fd);
+    // Closes the fd from open()
+    public static native int close(int fd);
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevShell.java

@@ -0,0 +1,116 @@
+package com.limelight.binding.input.evdev;
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.OutputStream;
+import java.util.Scanner;
+import java.util.UUID;
+
+public class EvdevShell {
+    private OutputStream stdin;
+    private InputStream stdout;
+    private Process shell;
+    private final String uuidString = UUID.randomUUID().toString();
+
+    private static final EvdevShell globalShell = new EvdevShell();
+
+    public static EvdevShell getInstance() {
+        return globalShell;
+    }
+
+    public void startShell() {
+        ProcessBuilder builder = new ProcessBuilder("su");
+
+        try {
+            // Redirect stderr to stdout
+            builder.redirectErrorStream(true);
+            shell = builder.start();
+
+            stdin = shell.getOutputStream();
+            stdout = shell.getInputStream();
+        } catch (IOException e) {
+            // This is unexpected
+            e.printStackTrace();
+
+            // Kill the shell if it spawned
+            if (stdin != null) {
+                try {
+                    stdin.close();
+                } catch (IOException e1) {
+                    e1.printStackTrace();
+                } finally {
+                    stdin = null;
+                }
+            }
+            if (stdout != null) {
+                try {
+                    stdout.close();
+                } catch (IOException e1) {
+                    e1.printStackTrace();
+                } finally {
+                    stdout = null;
+                }
+            }
+            if (shell != null) {
+                shell.destroy();
+                shell = null;
+            }
+        }
+    }
+
+    public void runCommand(String command) {
+        if (shell == null) {
+            // Shell never started
+            return;
+        }
+
+        try {
+            // Write the command followed by an echo with our UUID
+            stdin.write((command+'\n').getBytes("UTF-8"));
+            stdin.write(("echo "+uuidString+'\n').getBytes("UTF-8"));
+            stdin.flush();
+
+            // This is the only command in flight so we can use a scanner
+            // without worrying about it eating too many characters
+            Scanner scanner = new Scanner(stdout);
+            while (scanner.hasNext()) {
+                if (scanner.next().contains(uuidString)) {
+                    // Our command ran
+                    return;
+                }
+            }
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+    }
+
+    public void stopShell() throws InterruptedException {
+        boolean exitWritten = false;
+
+        if (shell == null) {
+            // Shell never started
+            return;
+        }
+
+        try {
+            stdin.write("exit\n".getBytes("UTF-8"));
+            exitWritten = true;
+        } catch (IOException e) {
+            // We'll destroy the process without
+            // waiting for it to terminate since
+            // we don't know whether our exit command made it
+            e.printStackTrace();
+        }
+
+        if (exitWritten) {
+            try {
+                shell.waitFor();
+            } finally {
+                shell.destroy();
+            }
+        }
+        else {
+            shell.destroy();
+        }
+    }
+}

app/src/main/java/com/limelight/binding/input/evdev/EvdevTranslator.java

@@ -4,136 +4,136 @@ import android.view.KeyEvent;
 
 public class EvdevTranslator {
 
-	public static final short EVDEV_KEY_CODES[] = {
-		0, //KeyEvent.VK_RESERVED
-		KeyEvent.KEYCODE_ESCAPE,
-		KeyEvent.KEYCODE_1,
-		KeyEvent.KEYCODE_2,
-		KeyEvent.KEYCODE_3,
-		KeyEvent.KEYCODE_4,
-		KeyEvent.KEYCODE_5,
-		KeyEvent.KEYCODE_6,
-		KeyEvent.KEYCODE_7,
-		KeyEvent.KEYCODE_8,
-		KeyEvent.KEYCODE_9,
-		KeyEvent.KEYCODE_0,
-		KeyEvent.KEYCODE_MINUS,
-		KeyEvent.KEYCODE_EQUALS,
-		KeyEvent.KEYCODE_DEL,
-		KeyEvent.KEYCODE_TAB,
-		KeyEvent.KEYCODE_Q,
-		KeyEvent.KEYCODE_W,
-		KeyEvent.KEYCODE_E,
-		KeyEvent.KEYCODE_R,
-		KeyEvent.KEYCODE_T,
-		KeyEvent.KEYCODE_Y,
-		KeyEvent.KEYCODE_U,
-		KeyEvent.KEYCODE_I,
-		KeyEvent.KEYCODE_O,
-		KeyEvent.KEYCODE_P,
-		KeyEvent.KEYCODE_LEFT_BRACKET,
-		KeyEvent.KEYCODE_RIGHT_BRACKET,
-		KeyEvent.KEYCODE_ENTER,
-		KeyEvent.KEYCODE_CTRL_LEFT,
-		KeyEvent.KEYCODE_A,
-		KeyEvent.KEYCODE_S,
-		KeyEvent.KEYCODE_D,
-		KeyEvent.KEYCODE_F,
-		KeyEvent.KEYCODE_G,
-		KeyEvent.KEYCODE_H,
-		KeyEvent.KEYCODE_J,
-		KeyEvent.KEYCODE_K,
-		KeyEvent.KEYCODE_L,
-		KeyEvent.KEYCODE_SEMICOLON,
-		KeyEvent.KEYCODE_APOSTROPHE,
-		KeyEvent.KEYCODE_GRAVE,
-		KeyEvent.KEYCODE_SHIFT_LEFT,
-		KeyEvent.KEYCODE_BACKSLASH,
-		KeyEvent.KEYCODE_Z,
-		KeyEvent.KEYCODE_X,
-		KeyEvent.KEYCODE_C,
-		KeyEvent.KEYCODE_V,
-		KeyEvent.KEYCODE_B,
-		KeyEvent.KEYCODE_N,
-		KeyEvent.KEYCODE_M,
-		KeyEvent.KEYCODE_COMMA,
-		KeyEvent.KEYCODE_PERIOD,
-		KeyEvent.KEYCODE_SLASH,
-		KeyEvent.KEYCODE_SHIFT_RIGHT,
-		KeyEvent.KEYCODE_NUMPAD_MULTIPLY,
-		KeyEvent.KEYCODE_ALT_LEFT,
-		KeyEvent.KEYCODE_SPACE,
-		KeyEvent.KEYCODE_CAPS_LOCK,
-		KeyEvent.KEYCODE_F1,
-		KeyEvent.KEYCODE_F2,
-		KeyEvent.KEYCODE_F3,
-		KeyEvent.KEYCODE_F4,
-		KeyEvent.KEYCODE_F5,
-		KeyEvent.KEYCODE_F6,
-		KeyEvent.KEYCODE_F7,
-		KeyEvent.KEYCODE_F8,
-		KeyEvent.KEYCODE_F9,
-		KeyEvent.KEYCODE_F10,
-		KeyEvent.KEYCODE_NUM_LOCK,
-		KeyEvent.KEYCODE_SCROLL_LOCK,
-		KeyEvent.KEYCODE_NUMPAD_7,
-		KeyEvent.KEYCODE_NUMPAD_8,
-		KeyEvent.KEYCODE_NUMPAD_9,
-		KeyEvent.KEYCODE_NUMPAD_SUBTRACT,
-		KeyEvent.KEYCODE_NUMPAD_4,
-		KeyEvent.KEYCODE_NUMPAD_5,
-		KeyEvent.KEYCODE_NUMPAD_6,
-		KeyEvent.KEYCODE_NUMPAD_ADD,
-		KeyEvent.KEYCODE_NUMPAD_1,
-		KeyEvent.KEYCODE_NUMPAD_2,
-		KeyEvent.KEYCODE_NUMPAD_3,
-		KeyEvent.KEYCODE_NUMPAD_0,
-		KeyEvent.KEYCODE_NUMPAD_DOT,
-		0,
-		0, //KeyEvent.VK_ZENKAKUHANKAKU,
-		0, //KeyEvent.VK_102ND,
-		KeyEvent.KEYCODE_F11,
-		KeyEvent.KEYCODE_F12,
-		0, //KeyEvent.VK_RO,
-		0, //KeyEvent.VK_KATAKANA,
-		0, //KeyEvent.VK_HIRAGANA,
-		0, //KeyEvent.VK_HENKAN,
-		0, //KeyEvent.VK_KATAKANAHIRAGANA,
-		0, //KeyEvent.VK_MUHENKAN,
-		0, //KeyEvent.VK_KPJPCOMMA,
-		KeyEvent.KEYCODE_NUMPAD_ENTER,
-		KeyEvent.KEYCODE_CTRL_RIGHT,
-		KeyEvent.KEYCODE_NUMPAD_DIVIDE,
-		KeyEvent.KEYCODE_SYSRQ,
-		KeyEvent.KEYCODE_ALT_RIGHT,
-		0, //KeyEvent.VK_LINEFEED,
-		KeyEvent.KEYCODE_HOME,
-		KeyEvent.KEYCODE_DPAD_UP,
-		KeyEvent.KEYCODE_PAGE_UP,
-		KeyEvent.KEYCODE_DPAD_LEFT,
-		KeyEvent.KEYCODE_DPAD_RIGHT,
-		KeyEvent.KEYCODE_MOVE_END,
-		KeyEvent.KEYCODE_DPAD_DOWN,
-		KeyEvent.KEYCODE_PAGE_DOWN,
-		KeyEvent.KEYCODE_INSERT,
-		KeyEvent.KEYCODE_FORWARD_DEL,
-		0, //KeyEvent.VK_MACRO,
-		0, //KeyEvent.VK_MUTE,
-		0, //KeyEvent.VK_VOLUMEDOWN,
-		0, //KeyEvent.VK_VOLUMEUP,
-		0, //KeyEvent.VK_POWER, /* SC System Power Down */
-		KeyEvent.KEYCODE_NUMPAD_EQUALS,
-		0, //KeyEvent.VK_KPPLUSMINUS,
-		KeyEvent.KEYCODE_BREAK,
-		0, //KeyEvent.VK_SCALE, /* AL Compiz Scale (Expose) */
-	};
+    private static final short[] EVDEV_KEY_CODES = {
+        0, //KeyEvent.VK_RESERVED
+        KeyEvent.KEYCODE_ESCAPE,
+        KeyEvent.KEYCODE_1,
+        KeyEvent.KEYCODE_2,
+        KeyEvent.KEYCODE_3,
+        KeyEvent.KEYCODE_4,
+        KeyEvent.KEYCODE_5,
+        KeyEvent.KEYCODE_6,
+        KeyEvent.KEYCODE_7,
+        KeyEvent.KEYCODE_8,
+        KeyEvent.KEYCODE_9,
+        KeyEvent.KEYCODE_0,
+        KeyEvent.KEYCODE_MINUS,
+        KeyEvent.KEYCODE_EQUALS,
+        KeyEvent.KEYCODE_DEL,
+        KeyEvent.KEYCODE_TAB,
+        KeyEvent.KEYCODE_Q,
+        KeyEvent.KEYCODE_W,
+        KeyEvent.KEYCODE_E,
+        KeyEvent.KEYCODE_R,
+        KeyEvent.KEYCODE_T,
+        KeyEvent.KEYCODE_Y,
+        KeyEvent.KEYCODE_U,
+        KeyEvent.KEYCODE_I,
+        KeyEvent.KEYCODE_O,
+        KeyEvent.KEYCODE_P,
+        KeyEvent.KEYCODE_LEFT_BRACKET,
+        KeyEvent.KEYCODE_RIGHT_BRACKET,
+        KeyEvent.KEYCODE_ENTER,
+        KeyEvent.KEYCODE_CTRL_LEFT,
+        KeyEvent.KEYCODE_A,
+        KeyEvent.KEYCODE_S,
+        KeyEvent.KEYCODE_D,
+        KeyEvent.KEYCODE_F,
+        KeyEvent.KEYCODE_G,
+        KeyEvent.KEYCODE_H,
+        KeyEvent.KEYCODE_J,
+        KeyEvent.KEYCODE_K,
+        KeyEvent.KEYCODE_L,
+        KeyEvent.KEYCODE_SEMICOLON,
+        KeyEvent.KEYCODE_APOSTROPHE,
+        KeyEvent.KEYCODE_GRAVE,
+        KeyEvent.KEYCODE_SHIFT_LEFT,
+        KeyEvent.KEYCODE_BACKSLASH,
+        KeyEvent.KEYCODE_Z,
+        KeyEvent.KEYCODE_X,
+        KeyEvent.KEYCODE_C,
+        KeyEvent.KEYCODE_V,
+        KeyEvent.KEYCODE_B,
+        KeyEvent.KEYCODE_N,
+        KeyEvent.KEYCODE_M,
+        KeyEvent.KEYCODE_COMMA,
+        KeyEvent.KEYCODE_PERIOD,
+        KeyEvent.KEYCODE_SLASH,
+        KeyEvent.KEYCODE_SHIFT_RIGHT,
+        KeyEvent.KEYCODE_NUMPAD_MULTIPLY,
+        KeyEvent.KEYCODE_ALT_LEFT,
+        KeyEvent.KEYCODE_SPACE,
+        KeyEvent.KEYCODE_CAPS_LOCK,
+        KeyEvent.KEYCODE_F1,
+        KeyEvent.KEYCODE_F2,
+        KeyEvent.KEYCODE_F3,
+        KeyEvent.KEYCODE_F4,
+        KeyEvent.KEYCODE_F5,
+        KeyEvent.KEYCODE_F6,
+        KeyEvent.KEYCODE_F7,
+        KeyEvent.KEYCODE_F8,
+        KeyEvent.KEYCODE_F9,
+        KeyEvent.KEYCODE_F10,
+        KeyEvent.KEYCODE_NUM_LOCK,
+        KeyEvent.KEYCODE_SCROLL_LOCK,
+        KeyEvent.KEYCODE_NUMPAD_7,
+        KeyEvent.KEYCODE_NUMPAD_8,
+        KeyEvent.KEYCODE_NUMPAD_9,
+        KeyEvent.KEYCODE_NUMPAD_SUBTRACT,
+        KeyEvent.KEYCODE_NUMPAD_4,
+        KeyEvent.KEYCODE_NUMPAD_5,
+        KeyEvent.KEYCODE_NUMPAD_6,
+        KeyEvent.KEYCODE_NUMPAD_ADD,
+        KeyEvent.KEYCODE_NUMPAD_1,
+        KeyEvent.KEYCODE_NUMPAD_2,
+        KeyEvent.KEYCODE_NUMPAD_3,
+        KeyEvent.KEYCODE_NUMPAD_0,
+        KeyEvent.KEYCODE_NUMPAD_DOT,
+        0,
+        0, //KeyEvent.VK_ZENKAKUHANKAKU,
+        0, //KeyEvent.VK_102ND,
+        KeyEvent.KEYCODE_F11,
+        KeyEvent.KEYCODE_F12,
+        0, //KeyEvent.VK_RO,
+        0, //KeyEvent.VK_KATAKANA,
+        0, //KeyEvent.VK_HIRAGANA,
+        0, //KeyEvent.VK_HENKAN,
+        0, //KeyEvent.VK_KATAKANAHIRAGANA,
+        0, //KeyEvent.VK_MUHENKAN,
+        0, //KeyEvent.VK_KPJPCOMMA,
+        KeyEvent.KEYCODE_NUMPAD_ENTER,
+        KeyEvent.KEYCODE_CTRL_RIGHT,
+        KeyEvent.KEYCODE_NUMPAD_DIVIDE,
+        KeyEvent.KEYCODE_SYSRQ,
+        KeyEvent.KEYCODE_ALT_RIGHT,
+        0, //KeyEvent.VK_LINEFEED,
+        KeyEvent.KEYCODE_HOME,
+        KeyEvent.KEYCODE_DPAD_UP,
+        KeyEvent.KEYCODE_PAGE_UP,
+        KeyEvent.KEYCODE_DPAD_LEFT,
+        KeyEvent.KEYCODE_DPAD_RIGHT,
+        KeyEvent.KEYCODE_MOVE_END,
+        KeyEvent.KEYCODE_DPAD_DOWN,
+        KeyEvent.KEYCODE_PAGE_DOWN,
+        KeyEvent.KEYCODE_INSERT,
+        KeyEvent.KEYCODE_FORWARD_DEL,
+        0, //KeyEvent.VK_MACRO,
+        0, //KeyEvent.VK_MUTE,
+        0, //KeyEvent.VK_VOLUMEDOWN,
+        0, //KeyEvent.VK_VOLUMEUP,
+        0, //KeyEvent.VK_POWER, /* SC System Power Down */
+        KeyEvent.KEYCODE_NUMPAD_EQUALS,
+        0, //KeyEvent.VK_KPPLUSMINUS,
+        KeyEvent.KEYCODE_BREAK,
+        0, //KeyEvent.VK_SCALE, /* AL Compiz Scale (Expose) */
+    };
 
-	public static short translateEvdevKeyCode(short evdevKeyCode) {
-		if (evdevKeyCode < EVDEV_KEY_CODES.length) {
-			return EVDEV_KEY_CODES[evdevKeyCode];
-		}
+    public static short translateEvdevKeyCode(short evdevKeyCode) {
+        if (evdevKeyCode < EVDEV_KEY_CODES.length) {
+            return EVDEV_KEY_CODES[evdevKeyCode];
+        }
 
-		return 0;
-	}
+        return 0;
+    }
 
 }

app/src/main/java/com/limelight/binding/input/evdev/EvdevWatcher.java

@@ -10,163 +10,179 @@ import android.os.FileObserver;
 
 @SuppressWarnings("ALL")
 public class EvdevWatcher {
-	private static final String PATH = "/dev/input";
-	private static final String REQUIRED_FILE_PREFIX = "event";
+    private static final String PATH = "/dev/input";
+    private static final String REQUIRED_FILE_PREFIX = "event";
 
-	private final HashMap<String, EvdevHandler> handlers = new HashMap<String, EvdevHandler>();
-	private boolean shutdown = false;
-	private boolean init = false;
-	private boolean ungrabbed = false;
-	private EvdevListener listener;
-	private Thread startThread;
+    private final HashMap<String, EvdevHandler> handlers = new HashMap<String, EvdevHandler>();
+    private boolean shutdown = false;
+    private boolean init = false;
+    private boolean ungrabbed = false;
+    private EvdevListener listener;
+    private Thread startThread;
 
-	private FileObserver observer = new FileObserver(PATH, FileObserver.CREATE | FileObserver.DELETE) {
-		@Override
-		public void onEvent(int event, String fileName) {
-			if (fileName == null) {
-				return;
-			}
+    private static boolean patchedSeLinuxPolicies = false;
 
-			if (!fileName.startsWith(REQUIRED_FILE_PREFIX)) {
-				return;
-			}
+    private FileObserver observer = new FileObserver(PATH, FileObserver.CREATE | FileObserver.DELETE) {
+        @Override
+        public void onEvent(int event, String fileName) {
+            if (fileName == null) {
+                return;
+            }
 
-			synchronized (handlers) {
-				if (shutdown) {
-					return;
-				}
+            if (!fileName.startsWith(REQUIRED_FILE_PREFIX)) {
+                return;
+            }
 
-				if ((event & FileObserver.CREATE) != 0) {
-					LimeLog.info("Starting evdev handler for "+fileName);
+            synchronized (handlers) {
+                if (shutdown) {
+                    return;
+                }
 
-					if (!init) {
-						// If this a real new device, update permissions again so we can read it
-						EvdevReader.setPermissions(new String[]{PATH + "/" + fileName}, 0666);
-					}
+                if ((event & FileObserver.CREATE) != 0) {
+                    LimeLog.info("Starting evdev handler for "+fileName);
 
-					EvdevHandler handler = new EvdevHandler(PATH + "/" + fileName, listener);
+                    if (!init) {
+                        // If this a real new device, update permissions again so we can read it
+                        EvdevReader.setPermissions(new String[]{PATH + "/" + fileName}, 0666);
+                    }
 
-					// If we're ungrabbed now, don't start the handler
-					if (!ungrabbed) {
-						handler.start();
-					}
+                    EvdevHandler handler = new EvdevHandler(PATH + "/" + fileName, listener);
 
-					handlers.put(fileName, handler);
-				}
+                    // If we're ungrabbed now, don't start the handler
+                    if (!ungrabbed) {
+                        handler.start();
+                    }
 
-				if ((event & FileObserver.DELETE) != 0) {
-					LimeLog.info("Halting evdev handler for "+fileName);
+                    handlers.put(fileName, handler);
+                }
 
-					EvdevHandler handler = handlers.remove(fileName);
-					if (handler != null) {
-						handler.notifyDeleted();
-					}
-				}
-			}
-		}
-	};
+                if ((event & FileObserver.DELETE) != 0) {
+                    LimeLog.info("Halting evdev handler for "+fileName);
 
-	public EvdevWatcher(EvdevListener listener) {
-		this.listener = listener;
-	}
+                    EvdevHandler handler = handlers.remove(fileName);
+                    if (handler != null) {
+                        handler.notifyDeleted();
+                    }
+                }
+            }
+        }
+    };
 
-	private File[] rundownWithPermissionsChange(int newPermissions) {
-		// Rundown existing files
-		File devInputDir = new File(PATH);
-		File[] files = devInputDir.listFiles();
-		if (files == null) {
-			return new File[0];
-		}
+    public EvdevWatcher(EvdevListener listener) {
+        this.listener = listener;
+    }
 
-		// Set desired permissions
-		String[] filePaths = new String[files.length];
-		for (int i = 0; i < files.length; i++) {
-			filePaths[i] = files[i].getAbsolutePath();
-		}
-		EvdevReader.setPermissions(filePaths, newPermissions);
+    private File[] rundownWithPermissionsChange(int newPermissions) {
+        // Rundown existing files
+        File devInputDir = new File(PATH);
+        File[] files = devInputDir.listFiles();
+        if (files == null) {
+            return new File[0];
+        }
 
-		return files;
-	}
+        // Set desired permissions
+        String[] filePaths = new String[files.length];
+        for (int i = 0; i < files.length; i++) {
+            filePaths[i] = files[i].getAbsolutePath();
+        }
+        EvdevReader.setPermissions(filePaths, newPermissions);
 
-	public void ungrabAll() {
-		synchronized (handlers) {
-			// Note that we're ungrabbed for now
-			ungrabbed = true;
+        return files;
+    }
 
-			// Stop all handlers
-			for (EvdevHandler handler : handlers.values()) {
-				handler.stop();
-			}
-		}
-	}
+    public void ungrabAll() {
+        synchronized (handlers) {
+            // Note that we're ungrabbed for now
+            ungrabbed = true;
 
-	public void regrabAll() {
-		synchronized (handlers) {
-			// We're regrabbing everything now
-			ungrabbed = false;
+            // Stop all handlers
+            for (EvdevHandler handler : handlers.values()) {
+                handler.stop();
+            }
+        }
+    }
 
-			for (Map.Entry<String, EvdevHandler> entry : handlers.entrySet()) {
-				// We need to recreate each entry since we can't reuse a stopped one
-				entry.setValue(new EvdevHandler(PATH + "/" + entry.getKey(), listener));
-				entry.getValue().start();
-			}
-		}
-	}
+    public void regrabAll() {
+        synchronized (handlers) {
+            // We're regrabbing everything now
+            ungrabbed = false;
 
-	public void start() {
-		startThread = new Thread() {
-			@Override
-			public void run() {
-				// List all files and allow us access
-				File[] files = rundownWithPermissionsChange(0666);
+            for (Map.Entry<String, EvdevHandler> entry : handlers.entrySet()) {
+                // We need to recreate each entry since we can't reuse a stopped one
+                entry.setValue(new EvdevHandler(PATH + "/" + entry.getKey(), listener));
+                entry.getValue().start();
+            }
+        }
+    }
 
-				init = true;
-				for (File f : files) {
-					observer.onEvent(FileObserver.CREATE, f.getName());
-				}
+    public void start() {
+        startThread = new Thread() {
+            @Override
+            public void run() {
+                // Initialize the root shell
+                EvdevShell.getInstance().startShell();
 
-				// Done with initial onEvent calls
-				init = false;
+                // Patch SELinux policies (if needed)
+                if (!patchedSeLinuxPolicies) {
+                    EvdevReader.patchSeLinuxPolicies();
+                    patchedSeLinuxPolicies = true;
+                }
 
-				// Start watching for new files
-				observer.startWatching();
+                // List all files and allow us access
+                File[] files = rundownWithPermissionsChange(0666);
 
-				synchronized (startThread) {
-					// Wait to be awoken again by shutdown()
-					try {
-						startThread.wait();
-					} catch (InterruptedException e) {}
-				}
+                init = true;
+                for (File f : files) {
+                    observer.onEvent(FileObserver.CREATE, f.getName());
+                }
 
-				// Giveup eventX permissions
-				rundownWithPermissionsChange(066);
-			}
-		};
-		startThread.start();
-	}
+                // Done with initial onEvent calls
+                init = false;
 
-	public void shutdown() {
-		// Let start thread cleanup on it's own sweet time
-		synchronized (startThread) {
-			startThread.notify();
-		}
+                // Start watching for new files
+                observer.startWatching();
 
-		// Stop the observer
-		observer.stopWatching();
+                synchronized (startThread) {
+                    // Wait to be awoken again by shutdown()
+                    try {
+                        startThread.wait();
+                    } catch (InterruptedException e) {}
+                }
 
-		synchronized (handlers) {			
-			// Stop creating new handlers
-			shutdown = true;
+                // Giveup eventX permissions
+                rundownWithPermissionsChange(0660);
 
-			// If we've already ungrabbed, there's nothing else to do
-			if (ungrabbed) {
-				return;
-			}
+                // Kill the root shell
+                try {
+                    EvdevShell.getInstance().stopShell();
+                } catch (InterruptedException e) {}
+            }
+        };
+        startThread.start();
+    }
 
-			// Stop all handlers
-			for (EvdevHandler handler : handlers.values()) {
-				handler.stop();
-			}
-		}
-	}
+    public void shutdown() {
+        // Let start thread cleanup on it's own sweet time
+        synchronized (startThread) {
+            startThread.notify();
+        }
+
+        // Stop the observer
+        observer.stopWatching();
+
+        synchronized (handlers) {
+            // Stop creating new handlers
+            shutdown = true;
+
+            // If we've already ungrabbed, there's nothing else to do
+            if (ungrabbed) {
+                return;
+            }
+
+            // Stop all handlers
+            for (EvdevHandler handler : handlers.values()) {
+                handler.stop();
+            }
+        }
+    }
 }

app/src/main/java/com/limelight/binding/video/AndroidCpuDecoderRenderer.java

@@ -20,138 +20,140 @@ import com.limelight.nvstream.av.video.cpu.AvcDecoder;
 @SuppressWarnings("EmptyCatchBlock")
 public class AndroidCpuDecoderRenderer extends EnhancedDecoderRenderer {
 
-	private Thread rendererThread, decoderThread;
-	private int targetFps;
+    private Thread rendererThread, decoderThread;
+    private int targetFps;
 
-	private static final int DECODER_BUFFER_SIZE = 92*1024;
-	private ByteBuffer decoderBuffer;
+    private static final int DECODER_BUFFER_SIZE = 92*1024;
+    private ByteBuffer decoderBuffer;
 
-	// Only sleep if the difference is above this value
-	private static final int WAIT_CEILING_MS = 5;
+    // Only sleep if the difference is above this value
+    private static final int WAIT_CEILING_MS = 5;
 
-	private static final int LOW_PERF = 1;
-	private static final int MED_PERF = 2;
-	private static final int HIGH_PERF = 3;
+    private static final int LOW_PERF = 1;
+    private static final int MED_PERF = 2;
+    private static final int HIGH_PERF = 3;
 
-	private int totalFrames;
-	private long totalTimeMs;
+    private int totalFrames;
+    private long totalTimeMs;
 
-	private int cpuCount = Runtime.getRuntime().availableProcessors();
+    private final int cpuCount = Runtime.getRuntime().availableProcessors();
 
-	@SuppressWarnings("unused")
-	private int findOptimalPerformanceLevel() {
-		StringBuilder cpuInfo = new StringBuilder();
-		BufferedReader br = null;
-		try {
-			br = new BufferedReader(new FileReader(new File("/proc/cpuinfo")));
-			for (;;) {
-				int ch = br.read();
-				if (ch == -1)
-					break;
-				cpuInfo.append((char)ch);
-			}
+    @SuppressWarnings("unused")
+    private int findOptimalPerformanceLevel() {
+        StringBuilder cpuInfo = new StringBuilder();
+        BufferedReader br = null;
+        try {
+            br = new BufferedReader(new FileReader(new File("/proc/cpuinfo")));
+            for (;;) {
+                int ch = br.read();
+                if (ch == -1)
+                    break;
+                cpuInfo.append((char)ch);
+            }
 
-			// Here we're doing very simple heuristics based on CPU model
-			String cpuInfoStr = cpuInfo.toString();
+            // Here we're doing very simple heuristics based on CPU model
+            String cpuInfoStr = cpuInfo.toString();
 
-			// We order them from greatest to least for proper detection
-			// of devices with multiple sets of cores (like Exynos 5 Octa)
-			// TODO Make this better (only even kind of works on ARM)
-			if (Build.FINGERPRINT.contains("generic")) {
-				// Emulator
-				return LOW_PERF;
-			}
-			else if (cpuInfoStr.contains("0xc0f")) {
-				// Cortex-A15
-				return MED_PERF;
-			}
-			else if (cpuInfoStr.contains("0xc09")) {
-				// Cortex-A9
-				return LOW_PERF;
-			}
-			else if (cpuInfoStr.contains("0xc07")) {
-				// Cortex-A7
-				return LOW_PERF;
-			}
-			else {
-				// Didn't have anything we're looking for
-				return MED_PERF;
-			}
-		} catch (IOException e) {
-		} finally {
-			if (br != null) {
-				try {
-					br.close();
-				} catch (IOException e) {}
-			}
-		}
+            // We order them from greatest to least for proper detection
+            // of devices with multiple sets of cores (like Exynos 5 Octa)
+            // TODO Make this better (only even kind of works on ARM)
+            if (Build.FINGERPRINT.contains("generic")) {
+                // Emulator
+                return LOW_PERF;
+            }
+            else if (cpuInfoStr.contains("0xc0f")) {
+                // Cortex-A15
+                return MED_PERF;
+            }
+            else if (cpuInfoStr.contains("0xc09")) {
+                // Cortex-A9
+                return LOW_PERF;
+            }
+            else if (cpuInfoStr.contains("0xc07")) {
+                // Cortex-A7
+                return LOW_PERF;
+            }
+            else {
+                // Didn't have anything we're looking for
+                return MED_PERF;
+            }
+        } catch (IOException e) {
+        } finally {
+            if (br != null) {
+                try {
+                    br.close();
+                } catch (IOException e) {}
+            }
+        }
 
-		// Couldn't read cpuinfo, so assume medium
-		return MED_PERF;
-	}
+        // Couldn't read cpuinfo, so assume medium
+        return MED_PERF;
+    }
 
-	@Override
-	public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
-		this.targetFps = redrawRate;
+    @Override
+    public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
+        this.targetFps = redrawRate;
 
-		int perfLevel = LOW_PERF; //findOptimalPerformanceLevel();
-		int threadCount;
+        int perfLevel = LOW_PERF; //findOptimalPerformanceLevel();
+        int threadCount;
 
-		int avcFlags = 0;
-		switch (perfLevel) {
-		case HIGH_PERF:
-			// Single threaded low latency decode is ideal but hard to acheive
-			avcFlags = AvcDecoder.LOW_LATENCY_DECODE;
-			threadCount = 1;
-			break;
+        int avcFlags = 0;
+        switch (perfLevel) {
+        case HIGH_PERF:
+            // Single threaded low latency decode is ideal but hard to acheive
+            avcFlags = AvcDecoder.LOW_LATENCY_DECODE;
+            threadCount = 1;
+            break;
 
-		case LOW_PERF:
-			// Disable the loop filter for performance reasons
-			avcFlags = AvcDecoder.FAST_BILINEAR_FILTERING;
+        case LOW_PERF:
+            // Disable the loop filter for performance reasons
+            avcFlags = AvcDecoder.FAST_BILINEAR_FILTERING;
 
-			// Use plenty of threads to try to utilize the CPU as best we can
-			threadCount = cpuCount - 1;
-			break;
+            // Use plenty of threads to try to utilize the CPU as best we can
+            threadCount = cpuCount - 1;
+            break;
 
-		default:
-		case MED_PERF:
-			avcFlags = AvcDecoder.BILINEAR_FILTERING;
+        default:
+        case MED_PERF:
+            avcFlags = AvcDecoder.BILINEAR_FILTERING;
 
-			// Only use 2 threads to minimize frame processing latency
-			threadCount = 2;
-			break;
-		}
+            // Only use 2 threads to minimize frame processing latency
+            threadCount = 2;
+            break;
+        }
 
-		// If the user wants quality, we'll remove the low IQ flags
-		if ((drFlags & VideoDecoderRenderer.FLAG_PREFER_QUALITY) != 0) {
-			// Make sure the loop filter is enabled
-			avcFlags &= ~AvcDecoder.DISABLE_LOOP_FILTER;
+        // If the user wants quality, we'll remove the low IQ flags
+        if ((drFlags & VideoDecoderRenderer.FLAG_PREFER_QUALITY) != 0) {
+            // Make sure the loop filter is enabled
+            avcFlags &= ~AvcDecoder.DISABLE_LOOP_FILTER;
 
-			// Disable the non-compliant speed optimizations
-			avcFlags &= ~AvcDecoder.FAST_DECODE;
+            // Disable the non-compliant speed optimizations
+            avcFlags &= ~AvcDecoder.FAST_DECODE;
 
-			LimeLog.info("Using high quality decoding");
-		}
+            LimeLog.info("Using high quality decoding");
+        }
 
-		SurfaceHolder sh = (SurfaceHolder)renderTarget;
-		sh.setFormat(PixelFormat.RGBX_8888);
+        SurfaceHolder sh = (SurfaceHolder)renderTarget;
+        sh.setFormat(PixelFormat.RGBX_8888);
 
-		int err = AvcDecoder.init(width, height, avcFlags, threadCount);
-		if (err != 0) {
-			throw new IllegalStateException("AVC decoder initialization failure: "+err);
-		}
+        int err = AvcDecoder.init(width, height, avcFlags, threadCount);
+        if (err != 0) {
+            throw new IllegalStateException("AVC decoder initialization failure: "+err);
+        }
 
-		AvcDecoder.setRenderTarget(sh.getSurface());
+        if (!AvcDecoder.setRenderTarget(sh.getSurface())) {
+            return false;
+        }
 
-		decoderBuffer = ByteBuffer.allocate(DECODER_BUFFER_SIZE + AvcDecoder.getInputPaddingSize());
+        decoderBuffer = ByteBuffer.allocate(DECODER_BUFFER_SIZE + AvcDecoder.getInputPaddingSize());
 
-		LimeLog.info("Using software decoding (performance level: "+perfLevel+")");
+        LimeLog.info("Using software decoding (performance level: "+perfLevel+")");
 
-		return true;
-	}
+        return true;
+    }
 
-	@Override
-	public boolean start(final VideoDepacketizer depacketizer) {
+    @Override
+    public boolean start(final VideoDepacketizer depacketizer) {
         decoderThread = new Thread() {
             @Override
             public void run() {
@@ -172,112 +174,111 @@ public class AndroidCpuDecoderRenderer extends EnhancedDecoderRenderer {
         decoderThread.setPriority(Thread.MAX_PRIORITY - 1);
         decoderThread.start();
 
-		rendererThread = new Thread() {
-			@Override
-			public void run() {
-				long nextFrameTime = System.currentTimeMillis();
-				DecodeUnit du;
-				while (!isInterrupted())
-				{
-					long diff = nextFrameTime - System.currentTimeMillis();
+        rendererThread = new Thread() {
+            @Override
+            public void run() {
+                long nextFrameTime = System.currentTimeMillis();
+                while (!isInterrupted())
+                {
+                    long diff = nextFrameTime - System.currentTimeMillis();
 
-					if (diff > WAIT_CEILING_MS) {
+                    if (diff > WAIT_CEILING_MS) {
                         try {
                             Thread.sleep(diff - WAIT_CEILING_MS);
                         } catch (InterruptedException e) {
                             return;
                         }
                         continue;
-					}
+                    }
 
-					nextFrameTime = computePresentationTimeMs(targetFps);
-					AvcDecoder.redraw();
-				}
-			}
-		};
-		rendererThread.setName("Video - Renderer (CPU)");
-		rendererThread.setPriority(Thread.MAX_PRIORITY);
-		rendererThread.start();
-		return true;
-	}
+                    nextFrameTime = computePresentationTimeMs(targetFps);
+                    AvcDecoder.redraw();
+                }
+            }
+        };
+        rendererThread.setName("Video - Renderer (CPU)");
+        rendererThread.setPriority(Thread.MAX_PRIORITY);
+        rendererThread.start();
+        return true;
+    }
 
-	private long computePresentationTimeMs(int frameRate) {
-		return System.currentTimeMillis() + (1000 / frameRate);
-	}
+    private long computePresentationTimeMs(int frameRate) {
+        return System.currentTimeMillis() + (1000 / frameRate);
+    }
 
-	@Override
-	public void stop() {
-		rendererThread.interrupt();
+    @Override
+    public void stop() {
+        rendererThread.interrupt();
         decoderThread.interrupt();
 
-		try {
+        try {
             rendererThread.join();
         } catch (InterruptedException e) { }
         try {
             decoderThread.join();
         } catch (InterruptedException e) { }
-	}
+    }
 
-	@Override
-	public void release() {
-		AvcDecoder.destroy();
-	}
+    @Override
+    public void release() {
+        AvcDecoder.destroy();
+    }
 
-	private boolean submitDecodeUnit(DecodeUnit decodeUnit) {
-		byte[] data;
+    private boolean submitDecodeUnit(DecodeUnit decodeUnit) {
+        byte[] data;
 
-		// Use the reserved decoder buffer if this decode unit will fit
-		if (decodeUnit.getDataLength() <= DECODER_BUFFER_SIZE) {
-			decoderBuffer.clear();
+        // Use the reserved decoder buffer if this decode unit will fit
+        if (decodeUnit.getDataLength() <= DECODER_BUFFER_SIZE) {
+            decoderBuffer.clear();
 
-			for (ByteBufferDescriptor bbd : decodeUnit.getBufferList()) {
-				decoderBuffer.put(bbd.data, bbd.offset, bbd.length);
-			}
+            for (ByteBufferDescriptor bbd : decodeUnit.getBufferList()) {
+                decoderBuffer.put(bbd.data, bbd.offset, bbd.length);
+            }
 
-			data = decoderBuffer.array();
-		}
-		else {
-			data = new byte[decodeUnit.getDataLength()+AvcDecoder.getInputPaddingSize()];
+            data = decoderBuffer.array();
+        }
+        else {
+            data = new byte[decodeUnit.getDataLength()+AvcDecoder.getInputPaddingSize()];
 
-			int offset = 0;
-			for (ByteBufferDescriptor bbd : decodeUnit.getBufferList()) {
-				System.arraycopy(bbd.data, bbd.offset, data, offset, bbd.length);
-				offset += bbd.length;
-			}
-		}
+            int offset = 0;
+            for (ByteBufferDescriptor bbd : decodeUnit.getBufferList()) {
+                System.arraycopy(bbd.data, bbd.offset, data, offset, bbd.length);
+                offset += bbd.length;
+            }
+        }
 
-		boolean success = (AvcDecoder.decode(data, 0, decodeUnit.getDataLength()) == 0);
-		if (success) {
-			long timeAfterDecode = System.currentTimeMillis();
+        boolean success = (AvcDecoder.decode(data, 0, decodeUnit.getDataLength()) == 0);
+        if (success) {
+            long timeAfterDecode = System.currentTimeMillis();
 
-		    // Add delta time to the totals (excluding probable outliers)
-		    long delta = timeAfterDecode - decodeUnit.getReceiveTimestamp();
-			if (delta >= 0 && delta < 300) {
-			    totalTimeMs += delta;
-			    totalFrames++;
-			}
-		}
+            // Add delta time to the totals (excluding probable outliers)
+            long delta = timeAfterDecode - decodeUnit.getReceiveTimestamp();
+            if (delta >= 0 && delta < 1000) {
+                totalTimeMs += delta;
+                totalFrames++;
+            }
+        }
 
-		return success;
-	}
+        return success;
+    }
 
-	@Override
-	public int getCapabilities() {
-		return 0;
-	}
+    @Override
+    public int getCapabilities() {
+        return 0;
+    }
 
-	@Override
-	public int getAverageDecoderLatency() {
-		return 0;
-	}
+    @Override
+    public int getAverageDecoderLatency() {
+        return 0;
+    }
 
-	@Override
-	public int getAverageEndToEndLatency() {
-		if (totalFrames == 0) {
-			return 0;
-		}
-		return (int)(totalTimeMs / totalFrames);
-	}
+    @Override
+    public int getAverageEndToEndLatency() {
+        if (totalFrames == 0) {
+            return 0;
+        }
+        return (int)(totalTimeMs / totalFrames);
+    }
 
     @Override
     public String getDecoderName() {

app/src/main/java/com/limelight/binding/video/ConfigurableDecoderRenderer.java

@@ -5,75 +5,75 @@ import com.limelight.nvstream.av.video.VideoDepacketizer;
 
 public class ConfigurableDecoderRenderer extends EnhancedDecoderRenderer {
 
-	private EnhancedDecoderRenderer decoderRenderer;
+    private EnhancedDecoderRenderer decoderRenderer;
 
-	@Override
-	public void release() {
-		if (decoderRenderer != null) {
-			decoderRenderer.release();
-		}
-	}
+    @Override
+    public void release() {
+        if (decoderRenderer != null) {
+            decoderRenderer.release();
+        }
+    }
 
-	@Override
-	public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
-		if (decoderRenderer == null) {
-			throw new IllegalStateException("ConfigurableDecoderRenderer not initialized");
-		}
-		return decoderRenderer.setup(width, height, redrawRate, renderTarget, drFlags);
-	}
+    @Override
+    public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
+        if (decoderRenderer == null) {
+            throw new IllegalStateException("ConfigurableDecoderRenderer not initialized");
+        }
+        return decoderRenderer.setup(width, height, redrawRate, renderTarget, drFlags);
+    }
 
-	public void initializeWithFlags(int drFlags) {
-		if ((drFlags & VideoDecoderRenderer.FLAG_FORCE_HARDWARE_DECODING) != 0 ||
-				((drFlags & VideoDecoderRenderer.FLAG_FORCE_SOFTWARE_DECODING) == 0 &&
-				MediaCodecHelper.findProbableSafeDecoder() != null)) {
-			decoderRenderer = new MediaCodecDecoderRenderer();
-		}
-		else {
-			decoderRenderer = new AndroidCpuDecoderRenderer();
-		}
-	}
+    public void initializeWithFlags(int drFlags) {
+        if ((drFlags & VideoDecoderRenderer.FLAG_FORCE_HARDWARE_DECODING) != 0 ||
+                ((drFlags & VideoDecoderRenderer.FLAG_FORCE_SOFTWARE_DECODING) == 0 &&
+                MediaCodecHelper.findProbableSafeDecoder() != null)) {
+            decoderRenderer = new MediaCodecDecoderRenderer();
+        }
+        else {
+            decoderRenderer = new AndroidCpuDecoderRenderer();
+        }
+    }
 
-	public boolean isHardwareAccelerated() {
-		if (decoderRenderer == null) {
-			throw new IllegalStateException("ConfigurableDecoderRenderer not initialized");
-		}
-		return (decoderRenderer instanceof MediaCodecDecoderRenderer);
-	}
+    public boolean isHardwareAccelerated() {
+        if (decoderRenderer == null) {
+            throw new IllegalStateException("ConfigurableDecoderRenderer not initialized");
+        }
+        return (decoderRenderer instanceof MediaCodecDecoderRenderer);
+    }
 
-	@Override
-	public boolean start(VideoDepacketizer depacketizer) {
-		return decoderRenderer.start(depacketizer);
-	}
+    @Override
+    public boolean start(VideoDepacketizer depacketizer) {
+        return decoderRenderer.start(depacketizer);
+    }
 
-	@Override
-	public void stop() {
-		decoderRenderer.stop();
-	}
+    @Override
+    public void stop() {
+        decoderRenderer.stop();
+    }
 
-	@Override
-	public int getCapabilities() {
-		return decoderRenderer.getCapabilities();
-	}
+    @Override
+    public int getCapabilities() {
+        return decoderRenderer.getCapabilities();
+    }
 
-	@Override
-	public int getAverageDecoderLatency() {
-		if (decoderRenderer != null) {
-			return decoderRenderer.getAverageDecoderLatency();
-		}
-		else {
-			return 0;
-		}
-	}
+    @Override
+    public int getAverageDecoderLatency() {
+        if (decoderRenderer != null) {
+            return decoderRenderer.getAverageDecoderLatency();
+        }
+        else {
+            return 0;
+        }
+    }
 
-	@Override
-	public int getAverageEndToEndLatency() {
-		if (decoderRenderer != null) {
-			return decoderRenderer.getAverageEndToEndLatency();
-		}
-		else {
-			return 0;
-		}
-	}
+    @Override
+    public int getAverageEndToEndLatency() {
+        if (decoderRenderer != null) {
+            return decoderRenderer.getAverageEndToEndLatency();
+        }
+        else {
+            return 0;
+        }
+    }
 
     @Override
     public String getDecoderName() {

app/src/main/java/com/limelight/binding/video/MediaCodecDecoderRenderer.java

@@ -25,415 +25,432 @@ import android.view.SurfaceHolder;
 @SuppressWarnings("unused")
 public class MediaCodecDecoderRenderer extends EnhancedDecoderRenderer {
 
-	private ByteBuffer[] videoDecoderInputBuffers;
-	private MediaCodec videoDecoder;
-	private Thread rendererThread;
-	private boolean needsSpsBitstreamFixup, isExynos4;
-	private VideoDepacketizer depacketizer;
-	private boolean adaptivePlayback;
-	private int initialWidth, initialHeight;
+    private ByteBuffer[] videoDecoderInputBuffers;
+    private MediaCodec videoDecoder;
+    private Thread rendererThread;
+    private boolean needsSpsBitstreamFixup, isExynos4;
+    private VideoDepacketizer depacketizer;
+    private boolean adaptivePlayback;
+    private int initialWidth, initialHeight;
 
     private boolean needsBaselineSpsHack;
     private SeqParameterSet savedSps;
 
-	private long lastTimestampUs;
-	private long totalTimeMs;
-	private long decoderTimeMs;
-	private int totalFrames;
+    private long lastTimestampUs;
+    private long totalTimeMs;
+    private long decoderTimeMs;
+    private int totalFrames;
 
-	private String decoderName;
-	private int numSpsIn;
-	private int numPpsIn;
-	private int numIframeIn;
+    private String decoderName;
+    private int numSpsIn;
+    private int numPpsIn;
+    private int numIframeIn;
 
-	private static final boolean ENABLE_ASYNC_RENDERER = false;
+    private static final boolean ENABLE_ASYNC_RENDERER = false;
 
-	@TargetApi(Build.VERSION_CODES.KITKAT)
-	public MediaCodecDecoderRenderer() {
-		//dumpDecoders();
+    @TargetApi(Build.VERSION_CODES.KITKAT)
+    public MediaCodecDecoderRenderer() {
+        //dumpDecoders();
 
-		MediaCodecInfo decoder = MediaCodecHelper.findProbableSafeDecoder();
-		if (decoder == null) {
-			decoder = MediaCodecHelper.findFirstDecoder();
-		}
-		if (decoder == null) {
-			// This case is handled later in setup()
-			return;
-		}
+        MediaCodecInfo decoder = MediaCodecHelper.findProbableSafeDecoder();
+        if (decoder == null) {
+            decoder = MediaCodecHelper.findFirstDecoder();
+        }
+        if (decoder == null) {
+            // This case is handled later in setup()
+            return;
+        }
 
-		decoderName = decoder.getName();
+        decoderName = decoder.getName();
 
-		// Set decoder-specific attributes
-		adaptivePlayback = MediaCodecHelper.decoderSupportsAdaptivePlayback(decoderName, decoder);
-		needsSpsBitstreamFixup = MediaCodecHelper.decoderNeedsSpsBitstreamRestrictions(decoderName, decoder);
+        // Set decoder-specific attributes
+        adaptivePlayback = MediaCodecHelper.decoderSupportsAdaptivePlayback(decoderName, decoder);
+        needsSpsBitstreamFixup = MediaCodecHelper.decoderNeedsSpsBitstreamRestrictions(decoderName, decoder);
         needsBaselineSpsHack = MediaCodecHelper.decoderNeedsBaselineSpsHack(decoderName, decoder);
         isExynos4 = MediaCodecHelper.isExynos4Device();
         if (needsSpsBitstreamFixup) {
-			LimeLog.info("Decoder "+decoderName+" needs SPS bitstream restrictions fixup");
-		}
+            LimeLog.info("Decoder "+decoderName+" needs SPS bitstream restrictions fixup");
+        }
         if (needsBaselineSpsHack) {
             LimeLog.info("Decoder "+decoderName+" needs baseline SPS hack");
         }
-		if (isExynos4) {
-			LimeLog.info("Decoder "+decoderName+" is on Exynos 4");
-		}
-	}
+        if (isExynos4) {
+            LimeLog.info("Decoder "+decoderName+" is on Exynos 4");
+        }
+    }
 
-	@TargetApi(Build.VERSION_CODES.LOLLIPOP)
-	@Override
-	public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
-		this.initialWidth = width;
-		this.initialHeight = height;
+    @TargetApi(Build.VERSION_CODES.LOLLIPOP)
+    @Override
+    public boolean setup(int width, int height, int redrawRate, Object renderTarget, int drFlags) {
+        this.initialWidth = width;
+        this.initialHeight = height;
 
-		if (decoderName == null) {
-			LimeLog.severe("No available hardware decoder!");
-			return false;
-		}
+        if (decoderName == null) {
+            LimeLog.severe("No available hardware decoder!");
+            return false;
+        }
 
-		// Codecs have been known to throw all sorts of crazy runtime exceptions
-		// due to implementation problems
-		try {
-			videoDecoder = MediaCodec.createByCodecName(decoderName);
-		} catch (Exception e) {
-			return false;
-		}
+        // Codecs have been known to throw all sorts of crazy runtime exceptions
+        // due to implementation problems
+        try {
+            videoDecoder = MediaCodec.createByCodecName(decoderName);
+        } catch (Exception e) {
+            return false;
+        }
 
-		MediaFormat videoFormat = MediaFormat.createVideoFormat("video/avc", width, height);
+        MediaFormat videoFormat = MediaFormat.createVideoFormat("video/avc", width, height);
 
-		// Adaptive playback can also be enabled by the whitelist on pre-KitKat devices
-		// so we don't fill these pre-KitKat
-		if (adaptivePlayback && Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
-			videoFormat.setInteger(MediaFormat.KEY_MAX_WIDTH, width);
-			videoFormat.setInteger(MediaFormat.KEY_MAX_HEIGHT, height);
-		}
+        // Adaptive playback can also be enabled by the whitelist on pre-KitKat devices
+        // so we don't fill these pre-KitKat
+        if (adaptivePlayback && Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
+            videoFormat.setInteger(MediaFormat.KEY_MAX_WIDTH, width);
+            videoFormat.setInteger(MediaFormat.KEY_MAX_HEIGHT, height);
+        }
 
-		// On Lollipop, we use asynchronous mode to avoid having a busy looping renderer thread
-		if (ENABLE_ASYNC_RENDERER && Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
-			videoDecoder.setCallback(new MediaCodec.Callback() {
-				@Override
-				public void onOutputFormatChanged(MediaCodec codec, MediaFormat format) {
-			    	LimeLog.info("Output format changed");
-			    	LimeLog.info("New output Format: " + format);
-				}
+        // On Lollipop, we use asynchronous mode to avoid having a busy looping renderer thread
+        if (ENABLE_ASYNC_RENDERER && Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+            videoDecoder.setCallback(new MediaCodec.Callback() {
+                @Override
+                public void onOutputFormatChanged(MediaCodec codec, MediaFormat format) {
+                    LimeLog.info("Output format changed");
+                    LimeLog.info("New output Format: " + format);
+                }
 
-				@Override
-				public void onOutputBufferAvailable(MediaCodec codec, int index,
-						BufferInfo info) {
-					try {
-						// FIXME: It looks like we can't frameskip here
-						codec.releaseOutputBuffer(index, true);
-					} catch (Exception e) {
-						handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
-					}
-				}
+                @Override
+                public void onOutputBufferAvailable(MediaCodec codec, int index,
+                        BufferInfo info) {
+                    try {
+                        // FIXME: It looks like we can't frameskip here
+                        codec.releaseOutputBuffer(index, true);
+                    } catch (Exception e) {
+                        handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
+                    }
+                }
 
-				@Override
-				public void onInputBufferAvailable(MediaCodec codec, int index) {
-					try {
-						submitDecodeUnit(depacketizer.takeNextDecodeUnit(), codec.getInputBuffer(index), index);
-					} catch (InterruptedException e) {
-						// What do we do here?
-						e.printStackTrace();
-					} catch (Exception e) {
-						handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
-					}
-				}
+                @Override
+                public void onInputBufferAvailable(MediaCodec codec, int index) {
+                    try {
+                        submitDecodeUnit(depacketizer.takeNextDecodeUnit(), codec.getInputBuffer(index), index);
+                    } catch (InterruptedException e) {
+                        // What do we do here?
+                        e.printStackTrace();
+                    } catch (Exception e) {
+                        handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
+                    }
+                }
 
-				@Override
-				public void onError(MediaCodec codec, CodecException e) {
-					if (e.isTransient()) {
-						LimeLog.warning(e.getDiagnosticInfo());
-						e.printStackTrace();
-					}
-					else {
-						LimeLog.severe(e.getDiagnosticInfo());
-						e.printStackTrace();
-					}
-				}
-			});
-		}
+                @Override
+                public void onError(MediaCodec codec, CodecException e) {
+                    if (e.isTransient()) {
+                        LimeLog.warning(e.getDiagnosticInfo());
+                        e.printStackTrace();
+                    }
+                    else {
+                        LimeLog.severe(e.getDiagnosticInfo());
+                        e.printStackTrace();
+                    }
+                }
+            });
+        }
 
-		videoDecoder.configure(videoFormat, ((SurfaceHolder)renderTarget).getSurface(), null, 0);
-		videoDecoder.setVideoScalingMode(MediaCodec.VIDEO_SCALING_MODE_SCALE_TO_FIT);
+        videoDecoder.configure(videoFormat, ((SurfaceHolder)renderTarget).getSurface(), null, 0);
+        videoDecoder.setVideoScalingMode(MediaCodec.VIDEO_SCALING_MODE_SCALE_TO_FIT);
 
-		LimeLog.info("Using hardware decoding");
+        LimeLog.info("Using hardware decoding");
 
-		return true;
-	}
+        return true;
+    }
 
-	@TargetApi(Build.VERSION_CODES.LOLLIPOP)
-	private void handleDecoderException(MediaCodecDecoderRenderer dr, Exception e, ByteBuffer buf, int codecFlags) {
-		if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
-			if (e instanceof CodecException) {
-				CodecException codecExc = (CodecException) e;
+    @TargetApi(Build.VERSION_CODES.LOLLIPOP)
+    private void handleDecoderException(MediaCodecDecoderRenderer dr, Exception e, ByteBuffer buf, int codecFlags) {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+            if (e instanceof CodecException) {
+                CodecException codecExc = (CodecException) e;
 
-				if (codecExc.isTransient()) {
-					// We'll let transient exceptions go
-					LimeLog.warning(codecExc.getDiagnosticInfo());
-					return;
-				}
+                if (codecExc.isTransient()) {
+                    // We'll let transient exceptions go
+                    LimeLog.warning(codecExc.getDiagnosticInfo());
+                    return;
+                }
 
-				LimeLog.severe(codecExc.getDiagnosticInfo());
-			}
-		}
+                LimeLog.severe(codecExc.getDiagnosticInfo());
+            }
+        }
 
-		if (buf != null || codecFlags != 0) {
-			throw new RendererException(dr, e, buf, codecFlags);
-		}
-		else {
-			throw new RendererException(dr, e);
-		}
-	}
+        if (buf != null || codecFlags != 0) {
+            throw new RendererException(dr, e, buf, codecFlags);
+        }
+        else {
+            throw new RendererException(dr, e);
+        }
+    }
 
-	private void startRendererThread()
-	{
-		rendererThread = new Thread() {
-			@SuppressWarnings("deprecation")
-			@Override
-			public void run() {
-				BufferInfo info = new BufferInfo();
-				DecodeUnit du = null;
-				int inputIndex = -1;
-				while (!isInterrupted())
-				{
-					// In order to get as much data to the decoder as early as possible,
-					// try to submit up to 5 decode units at once without blocking.
-					if (inputIndex == -1 && du == null) {
-						try {
-							for (int i = 0; i < 5; i++) {
-								inputIndex = videoDecoder.dequeueInputBuffer(0);
-								du = depacketizer.pollNextDecodeUnit();
+    private void startRendererThread()
+    {
+        rendererThread = new Thread() {
+            @SuppressWarnings("deprecation")
+            @Override
+            public void run() {
+                BufferInfo info = new BufferInfo();
+                DecodeUnit du = null;
+                int inputIndex = -1;
+                while (!isInterrupted())
+                {
+                    // In order to get as much data to the decoder as early as possible,
+                    // try to submit up to 5 decode units at once without blocking.
+                    if (inputIndex == -1 && du == null) {
+                        try {
+                            for (int i = 0; i < 5; i++) {
+                                inputIndex = videoDecoder.dequeueInputBuffer(0);
+                                du = depacketizer.pollNextDecodeUnit();
 
-								// Stop if we can't get a DU or input buffer
-								if (du == null || inputIndex == -1) {
-									break;
-								}
+                                // Stop if we can't get a DU or input buffer
+                                if (du == null || inputIndex == -1) {
+                                    break;
+                                }
 
-								submitDecodeUnit(du, videoDecoderInputBuffers[inputIndex], inputIndex);
+                                submitDecodeUnit(du, videoDecoderInputBuffers[inputIndex], inputIndex);
 
-								du = null;
-								inputIndex = -1;
-							}
-						} catch (Exception e) {
-							inputIndex = -1;
-							handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
-						}
-					}
+                                du = null;
+                                inputIndex = -1;
+                            }
+                        } catch (Exception e) {
+                            inputIndex = -1;
+                            handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
+                        }
+                    }
 
-					// Grab an input buffer if we don't have one already.
-					// This way we can have one ready hopefully by the time
-					// the depacketizer is done with this frame. It's important
-					// that this can timeout because it's possible that we could exhaust
-					// the decoder's input buffers and deadlocks because aren't pulling
-					// frames out of the other end.
-					if (inputIndex == -1) {
-						try {
-							// If we've got a DU waiting to be given to the decoder, 
-							// wait a full 3 ms for an input buffer. Otherwise
-							// just see if we can get one immediately.
-							inputIndex = videoDecoder.dequeueInputBuffer(du != null ? 3000 : 0);
-						} catch (Exception e) {
-							inputIndex = -1;
-							handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
-						}
-					}
+                    // Grab an input buffer if we don't have one already.
+                    // This way we can have one ready hopefully by the time
+                    // the depacketizer is done with this frame. It's important
+                    // that this can timeout because it's possible that we could exhaust
+                    // the decoder's input buffers and deadlocks because aren't pulling
+                    // frames out of the other end.
+                    if (inputIndex == -1) {
+                        try {
+                            // If we've got a DU waiting to be given to the decoder,
+                            // wait a full 3 ms for an input buffer. Otherwise
+                            // just see if we can get one immediately.
+                            inputIndex = videoDecoder.dequeueInputBuffer(du != null ? 3000 : 0);
+                        } catch (Exception e) {
+                            inputIndex = -1;
+                            handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
+                        }
+                    }
 
-					// Grab a decode unit if we don't have one already
-					if (du == null) {
-						du = depacketizer.pollNextDecodeUnit();
-					}
+                    // Grab a decode unit if we don't have one already
+                    if (du == null) {
+                        du = depacketizer.pollNextDecodeUnit();
+                    }
 
-					// If we've got both a decode unit and an input buffer, we'll
-					// submit now. Otherwise, we wait until we have one.
-					if (du != null && inputIndex >= 0) {
-						submitDecodeUnit(du, videoDecoderInputBuffers[inputIndex], inputIndex);
+                    // If we've got both a decode unit and an input buffer, we'll
+                    // submit now. Otherwise, we wait until we have one.
+                    if (du != null && inputIndex >= 0) {
+                        submitDecodeUnit(du, videoDecoderInputBuffers[inputIndex], inputIndex);
 
-						// DU and input buffer have both been consumed
-						du = null;
-						inputIndex = -1;
-					}
+                        // DU and input buffer have both been consumed
+                        du = null;
+                        inputIndex = -1;
+                    }
 
-					// Try to output a frame
-					try {
-						int outIndex = videoDecoder.dequeueOutputBuffer(info, 0);
+                    // Try to output a frame
+                    try {
+                        int outIndex = videoDecoder.dequeueOutputBuffer(info, 0);
 
-					    if (outIndex >= 0) {
-					    	long presentationTimeUs = info.presentationTimeUs;
-						    int lastIndex = outIndex;
+                        if (outIndex >= 0) {
+                            long presentationTimeUs = info.presentationTimeUs;
+                            int lastIndex = outIndex;
 
-						    // Get the last output buffer in the queue
-						    while ((outIndex = videoDecoder.dequeueOutputBuffer(info, 0)) >= 0) {
-						    	videoDecoder.releaseOutputBuffer(lastIndex, false);
-						    	lastIndex = outIndex;
-						    	presentationTimeUs = info.presentationTimeUs;
-						    }
+                            // Get the last output buffer in the queue
+                            while ((outIndex = videoDecoder.dequeueOutputBuffer(info, 0)) >= 0) {
+                                videoDecoder.releaseOutputBuffer(lastIndex, false);
+                                lastIndex = outIndex;
+                                presentationTimeUs = info.presentationTimeUs;
+                            }
 
-						    // Render the last buffer
-					    	videoDecoder.releaseOutputBuffer(lastIndex, true);
+                            // Render the last buffer
+                            videoDecoder.releaseOutputBuffer(lastIndex, true);
 
-						    // Add delta time to the totals (excluding probable outliers)
-						    long delta = System.currentTimeMillis()-(presentationTimeUs/1000);
-						    if (delta >= 0 && delta < 300) {
-						    	decoderTimeMs += delta;
-							    totalTimeMs += delta;
-						    }
-					    } else {
-						    switch (outIndex) {
-						    case MediaCodec.INFO_TRY_AGAIN_LATER:
-						    	// Getting an input buffer may already block
-						    	// so don't park if we still need to do that
-						    	if (inputIndex >= 0) {
-							    	LockSupport.parkNanos(1);
-						    	}
-						    	break;
-						    case MediaCodec.INFO_OUTPUT_BUFFERS_CHANGED:
-						    	LimeLog.info("Output buffers changed");
-							    break;
-						    case MediaCodec.INFO_OUTPUT_FORMAT_CHANGED:
-						    	LimeLog.info("Output format changed");
-						    	LimeLog.info("New output Format: " + videoDecoder.getOutputFormat());
-						    	break;
-						    default:
-						      break;
-						    }
-					    }
-					} catch (Exception e) {
-						handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
-					}
-				}
-			}
-		};
-		rendererThread.setName("Video - Renderer (MediaCodec)");
-		rendererThread.setPriority(Thread.MAX_PRIORITY);
-		rendererThread.start();
-	}
+                            // Add delta time to the totals (excluding probable outliers)
+                            long delta = System.currentTimeMillis()-(presentationTimeUs/1000);
+                            if (delta >= 0 && delta < 1000) {
+                                decoderTimeMs += delta;
+                                totalTimeMs += delta;
+                            }
+                        } else {
+                            switch (outIndex) {
+                            case MediaCodec.INFO_TRY_AGAIN_LATER:
+                                // Getting an input buffer may already block
+                                // so don't park if we still need to do that
+                                if (inputIndex >= 0) {
+                                    LockSupport.parkNanos(1);
+                                }
+                                break;
+                            case MediaCodec.INFO_OUTPUT_BUFFERS_CHANGED:
+                                LimeLog.info("Output buffers changed");
+                                break;
+                            case MediaCodec.INFO_OUTPUT_FORMAT_CHANGED:
+                                LimeLog.info("Output format changed");
+                                LimeLog.info("New output Format: " + videoDecoder.getOutputFormat());
+                                break;
+                            default:
+                              break;
+                            }
+                        }
+                    } catch (Exception e) {
+                        handleDecoderException(MediaCodecDecoderRenderer.this, e, null, 0);
+                    }
+                }
+            }
+        };
+        rendererThread.setName("Video - Renderer (MediaCodec)");
+        rendererThread.setPriority(Thread.MAX_PRIORITY);
+        rendererThread.start();
+    }
 
-	@SuppressWarnings("deprecation")
-	@Override
-	public boolean start(VideoDepacketizer depacketizer) {
-		this.depacketizer = depacketizer;
+    @SuppressWarnings("deprecation")
+    @Override
+    public boolean start(VideoDepacketizer depacketizer) {
+        this.depacketizer = depacketizer;
 
-		// Start the decoder
-		videoDecoder.start();
+        // Start the decoder
+        videoDecoder.start();
 
-		// On devices pre-Lollipop, we'll use a rendering thread
-		if (!ENABLE_ASYNC_RENDERER || Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) {
-			videoDecoderInputBuffers = videoDecoder.getInputBuffers();
-			startRendererThread();
-		}
-		return true;
-	}
+        // On devices pre-Lollipop, we'll use a rendering thread
+        if (!ENABLE_ASYNC_RENDERER || Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) {
+            videoDecoderInputBuffers = videoDecoder.getInputBuffers();
+            startRendererThread();
+        }
+        return true;
+    }
 
-	@Override
-	public void stop() {
-		if (rendererThread != null) {
-			// Halt the rendering thread
-			rendererThread.interrupt();
-			try {
-				rendererThread.join();
-			} catch (InterruptedException ignored) { }
-		}
+    @Override
+    public void stop() {
+        if (rendererThread != null) {
+            // Halt the rendering thread
+            rendererThread.interrupt();
+            try {
+                rendererThread.join();
+            } catch (InterruptedException ignored) { }
+        }
 
-		// Stop the decoder
-		videoDecoder.stop();
-	}
+        // Stop the decoder
+        videoDecoder.stop();
+    }
 
-	@Override
-	public void release() {
-		if (videoDecoder != null) {
-			videoDecoder.release();
-		}
-	}
+    @Override
+    public void release() {
+        if (videoDecoder != null) {
+            videoDecoder.release();
+        }
+    }
 
-	private void queueInputBuffer(int inputBufferIndex, int offset, int length, long timestampUs, int codecFlags) {
-		// Try 25 times to submit the input buffer before throwing a real exception
-		int i;
-		Exception lastException = null;
+    private void queueInputBuffer(int inputBufferIndex, int offset, int length, long timestampUs, int codecFlags) {
+        // Try 25 times to submit the input buffer before throwing a real exception
+        int i;
+        Exception lastException = null;
 
-		for (i = 0; i < 25; i++) {
-			try {
-				videoDecoder.queueInputBuffer(inputBufferIndex,
-						0, length,
-						timestampUs, codecFlags);
-				break;
-			} catch (Exception e) {
-				handleDecoderException(this, e, null, codecFlags);
-				lastException = e;
-			}
-		}
+        for (i = 0; i < 25; i++) {
+            try {
+                videoDecoder.queueInputBuffer(inputBufferIndex,
+                        0, length,
+                        timestampUs, codecFlags);
+                break;
+            } catch (Exception e) {
+                handleDecoderException(this, e, null, codecFlags);
+                lastException = e;
+            }
+        }
 
-		if (i == 25) {
-			throw new RendererException(this, lastException, null, codecFlags);
-		}
-	}
+        if (i == 25) {
+            throw new RendererException(this, lastException, null, codecFlags);
+        }
+    }
 
-	@SuppressWarnings("deprecation")
-	private void submitDecodeUnit(DecodeUnit decodeUnit, ByteBuffer buf, int inputBufferIndex) {
-		long currentTime = System.currentTimeMillis();
-		long delta = currentTime-decodeUnit.getReceiveTimestamp();
-		if (delta >= 0 && delta < 300) {
-		    totalTimeMs += currentTime-decodeUnit.getReceiveTimestamp();
-		    totalFrames++;
-		}
+    @SuppressWarnings("deprecation")
+    private void submitDecodeUnit(DecodeUnit decodeUnit, ByteBuffer buf, int inputBufferIndex) {
+        long currentTime = System.currentTimeMillis();
+        long delta = currentTime-decodeUnit.getReceiveTimestamp();
+        if (delta >= 0 && delta < 1000) {
+            totalTimeMs += currentTime-decodeUnit.getReceiveTimestamp();
+            totalFrames++;
+        }
 
-		long timestampUs = currentTime * 1000;
-		if (timestampUs <= lastTimestampUs) {
-			// We can't submit multiple buffers with the same timestamp
-			// so bump it up by one before queuing
-			timestampUs = lastTimestampUs + 1;
-		}
-		lastTimestampUs = timestampUs;
+        long timestampUs = currentTime * 1000;
+        if (timestampUs <= lastTimestampUs) {
+            // We can't submit multiple buffers with the same timestamp
+            // so bump it up by one before queuing
+            timestampUs = lastTimestampUs + 1;
+        }
+        lastTimestampUs = timestampUs;
 
-		// Clear old input data
-		buf.clear();
+        // Clear old input data
+        buf.clear();
 
-		int codecFlags = 0;
-		int decodeUnitFlags = decodeUnit.getFlags();
-		if ((decodeUnitFlags & DecodeUnit.DU_FLAG_CODEC_CONFIG) != 0) {
-			codecFlags |= MediaCodec.BUFFER_FLAG_CODEC_CONFIG;
-		}
-		if ((decodeUnitFlags & DecodeUnit.DU_FLAG_SYNC_FRAME) != 0) {
-			codecFlags |= MediaCodec.BUFFER_FLAG_SYNC_FRAME;
-			numIframeIn++;
-		}
+        int codecFlags = 0;
+        int decodeUnitFlags = decodeUnit.getFlags();
+        if ((decodeUnitFlags & DecodeUnit.DU_FLAG_CODEC_CONFIG) != 0) {
+            codecFlags |= MediaCodec.BUFFER_FLAG_CODEC_CONFIG;
+        }
+        if ((decodeUnitFlags & DecodeUnit.DU_FLAG_SYNC_FRAME) != 0) {
+            codecFlags |= MediaCodec.BUFFER_FLAG_SYNC_FRAME;
+            numIframeIn++;
+        }
 
         boolean needsSpsReplay = false;
 
-		if ((decodeUnitFlags & DecodeUnit.DU_FLAG_CODEC_CONFIG) != 0) {
-			ByteBufferDescriptor header = decodeUnit.getBufferList().get(0);
-			if (header.data[header.offset+4] == 0x67) {
-				numSpsIn++;
+        if ((decodeUnitFlags & DecodeUnit.DU_FLAG_CODEC_CONFIG) != 0) {
+            ByteBufferDescriptor header = decodeUnit.getBufferList().get(0);
+            if (header.data[header.offset+4] == 0x67) {
+                numSpsIn++;
 
-				ByteBuffer spsBuf = ByteBuffer.wrap(header.data);
+                ByteBuffer spsBuf = ByteBuffer.wrap(header.data);
 
-				// Skip to the start of the NALU data
-				spsBuf.position(header.offset+5);
+                // Skip to the start of the NALU data
+                spsBuf.position(header.offset+5);
 
-				SeqParameterSet sps = SeqParameterSet.read(spsBuf);
+                SeqParameterSet sps = SeqParameterSet.read(spsBuf);
 
-				// TI OMAP4 requires a reference frame count of 1 to decode successfully. Exynos 4
-				// also requires this fixup.
-				//
-				// I'm doing this fixup for all devices because I haven't seen any devices that
-				// this causes issues for. At worst, it seems to do nothing and at best it fixes
-				// issues with video lag, hangs, and crashes.
-				LimeLog.info("Patching num_ref_frames in SPS");
-				sps.num_ref_frames = 1;
+                // Some decoders rely on H264 level to decide how many buffers are needed
+                // Since we only need one frame buffered, we'll set the level as low as we can
+                // for known resolution combinations
+                if (initialWidth == 1280 && initialHeight == 720) {
+                    // Max 5 buffered frames at 1280x720x60
+                    LimeLog.info("Patching level_idc to 32");
+                    sps.level_idc = 32;
+                }
+                else if (initialWidth == 1920 && initialHeight == 1080) {
+                    // Max 4 buffered frames at 1920x1080x64
+                    LimeLog.info("Patching level_idc to 42");
+                    sps.level_idc = 42;
+                }
+                else {
+                    // Leave the profile alone (currently 5.0)
+                }
 
-				if (needsSpsBitstreamFixup || isExynos4) {
-					// The SPS that comes in the current H264 bytestream doesn't set bitstream_restriction_flag
-					// or max_dec_frame_buffering which increases decoding latency on Tegra.
-					LimeLog.info("Adding bitstream restrictions");
+                // TI OMAP4 requires a reference frame count of 1 to decode successfully. Exynos 4
+                // also requires this fixup.
+                //
+                // I'm doing this fixup for all devices because I haven't seen any devices that
+                // this causes issues for. At worst, it seems to do nothing and at best it fixes
+                // issues with video lag, hangs, and crashes.
+                LimeLog.info("Patching num_ref_frames in SPS");
+                sps.num_ref_frames = 1;
 
-					sps.vuiParams.bitstreamRestriction = new VUIParameters.BitstreamRestriction();
-					sps.vuiParams.bitstreamRestriction.motion_vectors_over_pic_boundaries_flag = true;
-					sps.vuiParams.bitstreamRestriction.max_bytes_per_pic_denom = 2;
-					sps.vuiParams.bitstreamRestriction.max_bits_per_mb_denom = 1;
-					sps.vuiParams.bitstreamRestriction.log2_max_mv_length_horizontal = 16;
-					sps.vuiParams.bitstreamRestriction.log2_max_mv_length_vertical = 16;
-					sps.vuiParams.bitstreamRestriction.num_reorder_frames = 0;
-					sps.vuiParams.bitstreamRestriction.max_dec_frame_buffering = 1;
-				}
+                if (needsSpsBitstreamFixup || isExynos4) {
+                    // The SPS that comes in the current H264 bytestream doesn't set bitstream_restriction_flag
+                    // or max_dec_frame_buffering which increases decoding latency on Tegra.
+                    LimeLog.info("Adding bitstream restrictions");
+
+                    sps.vuiParams.bitstreamRestriction = new VUIParameters.BitstreamRestriction();
+                    sps.vuiParams.bitstreamRestriction.motion_vectors_over_pic_boundaries_flag = true;
+                    sps.vuiParams.bitstreamRestriction.max_bytes_per_pic_denom = 2;
+                    sps.vuiParams.bitstreamRestriction.max_bits_per_mb_denom = 1;
+                    sps.vuiParams.bitstreamRestriction.log2_max_mv_length_horizontal = 16;
+                    sps.vuiParams.bitstreamRestriction.log2_max_mv_length_vertical = 16;
+                    sps.vuiParams.bitstreamRestriction.num_reorder_frames = 0;
+                    sps.vuiParams.bitstreamRestriction.max_dec_frame_buffering = 1;
+                }
 
                 // If we need to hack this SPS to say we're baseline, do so now
                 if (needsBaselineSpsHack) {
@@ -442,20 +459,20 @@ public class MediaCodecDecoderRenderer extends EnhancedDecoderRenderer {
                     savedSps = sps;
                 }
 
-				// Write the annex B header
-				buf.put(header.data, header.offset, 5);
+                // Write the annex B header
+                buf.put(header.data, header.offset, 5);
 
-				// Write the modified SPS to the input buffer
-				sps.write(buf);
+                // Write the modified SPS to the input buffer
+                sps.write(buf);
 
-				queueInputBuffer(inputBufferIndex,
-						0, buf.position(),
-						timestampUs, codecFlags);
+                queueInputBuffer(inputBufferIndex,
+                        0, buf.position(),
+                        timestampUs, codecFlags);
 
-				depacketizer.freeDecodeUnit(decodeUnit);
-				return;
-			} else if (header.data[header.offset+4] == 0x68) {
-				numPpsIn++;
+                depacketizer.freeDecodeUnit(decodeUnit);
+                return;
+            } else if (header.data[header.offset+4] == 0x68) {
+                numPpsIn++;
 
                 if (needsBaselineSpsHack) {
                     LimeLog.info("Saw PPS; disabling SPS hack");
@@ -464,25 +481,25 @@ public class MediaCodecDecoderRenderer extends EnhancedDecoderRenderer {
                     // Give the decoder the SPS again with the proper profile now
                     needsSpsReplay = true;
                 }
-			}
-		}
+            }
+        }
 
-		// Copy data from our buffer list into the input buffer
-		for (ByteBufferDescriptor desc : decodeUnit.getBufferList())
-		{
-			buf.put(desc.data, desc.offset, desc.length);
-		}
+        // Copy data from our buffer list into the input buffer
+        for (ByteBufferDescriptor desc : decodeUnit.getBufferList())
+        {
+            buf.put(desc.data, desc.offset, desc.length);
+        }
 
-		queueInputBuffer(inputBufferIndex,
-				0, decodeUnit.getDataLength(),
-				timestampUs, codecFlags);
+        queueInputBuffer(inputBufferIndex,
+                0, decodeUnit.getDataLength(),
+                timestampUs, codecFlags);
 
-		depacketizer.freeDecodeUnit(decodeUnit);
+        depacketizer.freeDecodeUnit(decodeUnit);
 
         if (needsSpsReplay) {
             replaySps();
         }
-	}
+    }
 
     private void replaySps() {
         int inputIndex = videoDecoder.dequeueInputBuffer(-1);
@@ -511,27 +528,27 @@ public class MediaCodecDecoderRenderer extends EnhancedDecoderRenderer {
         LimeLog.info("SPS replay complete");
     }
 
-	@Override
-	public int getCapabilities() {
-		return adaptivePlayback ?
-				VideoDecoderRenderer.CAPABILITY_ADAPTIVE_RESOLUTION : 0;
-	}
+    @Override
+    public int getCapabilities() {
+        return adaptivePlayback ?
+                VideoDecoderRenderer.CAPABILITY_ADAPTIVE_RESOLUTION : 0;
+    }
 
-	@Override
-	public int getAverageDecoderLatency() {
-		if (totalFrames == 0) {
-			return 0;
-		}
-		return (int)(decoderTimeMs / totalFrames);
-	}
+    @Override
+    public int getAverageDecoderLatency() {
+        if (totalFrames == 0) {
+            return 0;
+        }
+        return (int)(decoderTimeMs / totalFrames);
+    }
 
-	@Override
-	public int getAverageEndToEndLatency() {
-		if (totalFrames == 0) {
-			return 0;
-		}
-		return (int)(totalTimeMs / totalFrames);
-	}
+    @Override
+    public int getAverageEndToEndLatency() {
+        if (totalFrames == 0) {
+            return 0;
+        }
+        return (int)(totalTimeMs / totalFrames);
+    }
 
     @Override
     public String getDecoderName() {
@@ -539,62 +556,62 @@ public class MediaCodecDecoderRenderer extends EnhancedDecoderRenderer {
     }
 
     public class RendererException extends RuntimeException {
-		private static final long serialVersionUID = 8985937536997012406L;
+        private static final long serialVersionUID = 8985937536997012406L;
 
-		private Exception originalException;
-		private MediaCodecDecoderRenderer renderer;
-		private ByteBuffer currentBuffer;
-		private int currentCodecFlags;
+        private final Exception originalException;
+        private final MediaCodecDecoderRenderer renderer;
+        private ByteBuffer currentBuffer;
+        private int currentCodecFlags;
 
-		public RendererException(MediaCodecDecoderRenderer renderer, Exception e) {
-			this.renderer = renderer;
-			this.originalException = e;
-		}
+        public RendererException(MediaCodecDecoderRenderer renderer, Exception e) {
+            this.renderer = renderer;
+            this.originalException = e;
+        }
 
-		public RendererException(MediaCodecDecoderRenderer renderer, Exception e, ByteBuffer currentBuffer, int currentCodecFlags) {
-			this.renderer = renderer;
-			this.originalException = e;
-			this.currentBuffer = currentBuffer;
-			this.currentCodecFlags = currentCodecFlags;
-		}
+        public RendererException(MediaCodecDecoderRenderer renderer, Exception e, ByteBuffer currentBuffer, int currentCodecFlags) {
+            this.renderer = renderer;
+            this.originalException = e;
+            this.currentBuffer = currentBuffer;
+            this.currentCodecFlags = currentCodecFlags;
+        }
 
-		public String toString() {
-			String str = "";
+        public String toString() {
+            String str = "";
 
-			str += "Decoder: "+renderer.decoderName+"\n";
-			str += "Initial video dimensions: "+renderer.initialWidth+"x"+renderer.initialHeight+"\n";
-			str += "In stats: "+renderer.numSpsIn+", "+renderer.numPpsIn+", "+renderer.numIframeIn+"\n";
-			str += "Total frames: "+renderer.totalFrames+"\n";
+            str += "Decoder: "+renderer.decoderName+"\n";
+            str += "Initial video dimensions: "+renderer.initialWidth+"x"+renderer.initialHeight+"\n";
+            str += "In stats: "+renderer.numSpsIn+", "+renderer.numPpsIn+", "+renderer.numIframeIn+"\n";
+            str += "Total frames: "+renderer.totalFrames+"\n";
 
-			if (currentBuffer != null) {
-				str += "Current buffer: ";
-				currentBuffer.flip();
-				while (currentBuffer.hasRemaining() && currentBuffer.position() < 10) {
-					str += String.format((Locale)null, "%02x ", currentBuffer.get());
-				}
-				str += "\n";
-				str += "Buffer codec flags: "+currentCodecFlags+"\n";
-			}
+            if (currentBuffer != null) {
+                str += "Current buffer: ";
+                currentBuffer.flip();
+                while (currentBuffer.hasRemaining() && currentBuffer.position() < 10) {
+                    str += String.format((Locale)null, "%02x ", currentBuffer.get());
+                }
+                str += "\n";
+                str += "Buffer codec flags: "+currentCodecFlags+"\n";
+            }
 
-			str += "Is Exynos 4: "+renderer.isExynos4+"\n";
+            str += "Is Exynos 4: "+renderer.isExynos4+"\n";
 
-			str += "/proc/cpuinfo:\n";
-			try {
-				str += MediaCodecHelper.readCpuinfo();
-			} catch (Exception e) {
-				str += e.getMessage();
-			}
+            str += "/proc/cpuinfo:\n";
+            try {
+                str += MediaCodecHelper.readCpuinfo();
+            } catch (Exception e) {
+                str += e.getMessage();
+            }
 
-			str += "Full decoder dump:\n";
-			try {
-				str += MediaCodecHelper.dumpDecoders();
-			} catch (Exception e) {
-				str += e.getMessage();
-			}
+            str += "Full decoder dump:\n";
+            try {
+                str += MediaCodecHelper.dumpDecoders();
+            } catch (Exception e) {
+                str += e.getMessage();
+            }
 
-			str += originalException.toString();
+            str += originalException.toString();
 
-			return str;
-		}
-	}
+            return str;
+        }
+    }
 }

app/src/main/java/com/limelight/binding/video/MediaCodecHelper.java

@@ -20,12 +20,12 @@ import com.limelight.LimeLog;
 
 public class MediaCodecHelper {
 	
-	public static final List<String> preferredDecoders;
+	private static final List<String> preferredDecoders;
 
-	public static final List<String> blacklistedDecoderPrefixes;
-	public static final List<String> spsFixupBitstreamFixupDecoderPrefixes;
-	public static final List<String> whitelistedAdaptiveResolutionPrefixes;
-    public static final List<String> baselineProfileHackPrefixes;
+	private static final List<String> blacklistedDecoderPrefixes;
+	private static final List<String> spsFixupBitstreamFixupDecoderPrefixes;
+	private static final List<String> whitelistedAdaptiveResolutionPrefixes;
+    private static final List<String> baselineProfileHackPrefixes;
 	
 	static {
 		preferredDecoders = new LinkedList<String>();
@@ -44,6 +44,7 @@ public class MediaCodecHelper {
 		spsFixupBitstreamFixupDecoderPrefixes.add("omx.nvidia");
 		spsFixupBitstreamFixupDecoderPrefixes.add("omx.qcom");
 		spsFixupBitstreamFixupDecoderPrefixes.add("omx.mtk");
+        spsFixupBitstreamFixupDecoderPrefixes.add("omx.brcm");
 
         baselineProfileHackPrefixes = new LinkedList<String>();
         baselineProfileHackPrefixes.add("omx.intel");
@@ -71,8 +72,8 @@ public class MediaCodecHelper {
 	@TargetApi(Build.VERSION_CODES.KITKAT)
 	public static boolean decoderSupportsAdaptivePlayback(String decoderName, MediaCodecInfo decoderInfo) {
 		/*
-		FIXME: Intel's decoder on Nexus Player forces the high latency path if adaptive playback is enabled
-		so we'll keep it off for now, since we don't know whether other devices also do the same
+        FIXME: Intel's decoder on Nexus Player forces the high latency path if adaptive playback is enabled
+        so we'll keep it off for now, since we don't know whether other devices also do the same
 
 		if (isDecoderInList(whitelistedAdaptiveResolutionPrefixes, decoderName)) {
 			LimeLog.info("Adaptive playback supported (whitelist)");
@@ -145,7 +146,7 @@ public class MediaCodecHelper {
 		return str;
 	}
 	
-	public static MediaCodecInfo findPreferredDecoder() {
+	private static MediaCodecInfo findPreferredDecoder() {
 		// This is a different algorithm than the other findXXXDecoder functions,
 		// because we want to evaluate the decoders in our list's order
 		// rather than MediaCodecList's order
@@ -216,7 +217,7 @@ public class MediaCodecHelper {
 	// since some bad decoders can throw IllegalArgumentExceptions unexpectedly
 	// and we want to be sure all callers are handling this possibility
 	@SuppressWarnings("RedundantThrows")
-    public static MediaCodecInfo findKnownSafeDecoder() throws Exception {
+    private static MediaCodecInfo findKnownSafeDecoder() throws Exception {
 		for (MediaCodecInfo codecInfo : getMediaCodecList()) {		
 			// Skip encoders
 			if (codecInfo.isEncoder()) {

app/src/main/java/com/limelight/computers/ComputerDatabaseManager.java

@@ -17,153 +17,153 @@ import android.database.sqlite.SQLiteDatabase;
 import android.database.sqlite.SQLiteException;
 
 public class ComputerDatabaseManager {
-	private static final String COMPUTER_DB_NAME = "computers.db";
-	private static final String COMPUTER_TABLE_NAME = "Computers";
-	private static final String COMPUTER_NAME_COLUMN_NAME = "ComputerName";
-	private static final String COMPUTER_UUID_COLUMN_NAME = "UUID";
-	private static final String LOCAL_IP_COLUMN_NAME = "LocalIp";
-	private static final String REMOTE_IP_COLUMN_NAME = "RemoteIp";
-	private static final String MAC_COLUMN_NAME = "Mac";
+    private static final String COMPUTER_DB_NAME = "computers.db";
+    private static final String COMPUTER_TABLE_NAME = "Computers";
+    private static final String COMPUTER_NAME_COLUMN_NAME = "ComputerName";
+    private static final String COMPUTER_UUID_COLUMN_NAME = "UUID";
+    private static final String LOCAL_IP_COLUMN_NAME = "LocalIp";
+    private static final String REMOTE_IP_COLUMN_NAME = "RemoteIp";
+    private static final String MAC_COLUMN_NAME = "Mac";
 
-	private SQLiteDatabase computerDb;
+    private SQLiteDatabase computerDb;
 
-	public ComputerDatabaseManager(Context c) {
-		try {
-			// Create or open an existing DB
-			computerDb = c.openOrCreateDatabase(COMPUTER_DB_NAME, 0, null);
-		} catch (SQLiteException e) {
-			// Delete the DB and try again
-			c.deleteDatabase(COMPUTER_DB_NAME);
-			computerDb = c.openOrCreateDatabase(COMPUTER_DB_NAME, 0, null);
-		}
-		initializeDb();
-	}
+    public ComputerDatabaseManager(Context c) {
+        try {
+            // Create or open an existing DB
+            computerDb = c.openOrCreateDatabase(COMPUTER_DB_NAME, 0, null);
+        } catch (SQLiteException e) {
+            // Delete the DB and try again
+            c.deleteDatabase(COMPUTER_DB_NAME);
+            computerDb = c.openOrCreateDatabase(COMPUTER_DB_NAME, 0, null);
+        }
+        initializeDb();
+    }
 
-	public void close() {
-		computerDb.close();
-	}
+    public void close() {
+        computerDb.close();
+    }
 
-	private void initializeDb() {
-		// Create tables if they aren't already there
-		computerDb.execSQL(String.format((Locale)null, "CREATE TABLE IF NOT EXISTS %s(%s TEXT PRIMARY KEY," +
-				" %s TEXT NOT NULL, %s TEXT NOT NULL, %s TEXT NOT NULL, %s TEXT NOT NULL)",
-				COMPUTER_TABLE_NAME,
-				COMPUTER_NAME_COLUMN_NAME, COMPUTER_UUID_COLUMN_NAME, LOCAL_IP_COLUMN_NAME,
-				REMOTE_IP_COLUMN_NAME, MAC_COLUMN_NAME));
-	}
+    private void initializeDb() {
+        // Create tables if they aren't already there
+        computerDb.execSQL(String.format((Locale)null, "CREATE TABLE IF NOT EXISTS %s(%s TEXT PRIMARY KEY," +
+                " %s TEXT NOT NULL, %s TEXT NOT NULL, %s TEXT NOT NULL, %s TEXT NOT NULL)",
+                COMPUTER_TABLE_NAME,
+                COMPUTER_NAME_COLUMN_NAME, COMPUTER_UUID_COLUMN_NAME, LOCAL_IP_COLUMN_NAME,
+                REMOTE_IP_COLUMN_NAME, MAC_COLUMN_NAME));
+    }
 
-	public void deleteComputer(String name) {
-		computerDb.delete(COMPUTER_TABLE_NAME, COMPUTER_NAME_COLUMN_NAME+"='"+name+"'", null);
-	}
+    public void deleteComputer(String name) {
+        computerDb.delete(COMPUTER_TABLE_NAME, COMPUTER_NAME_COLUMN_NAME+"='"+name+"'", null);
+    }
 
-	public boolean updateComputer(ComputerDetails details) {
-		ContentValues values = new ContentValues();
-		values.put(COMPUTER_NAME_COLUMN_NAME, details.name);
-		values.put(COMPUTER_UUID_COLUMN_NAME, details.uuid.toString());
-		values.put(LOCAL_IP_COLUMN_NAME, details.localIp.getAddress());
-		values.put(REMOTE_IP_COLUMN_NAME, details.remoteIp.getAddress());
-		values.put(MAC_COLUMN_NAME, details.macAddress);
-		return -1 != computerDb.insertWithOnConflict(COMPUTER_TABLE_NAME, null, values, SQLiteDatabase.CONFLICT_REPLACE);
-	}
+    public boolean updateComputer(ComputerDetails details) {
+        ContentValues values = new ContentValues();
+        values.put(COMPUTER_NAME_COLUMN_NAME, details.name);
+        values.put(COMPUTER_UUID_COLUMN_NAME, details.uuid.toString());
+        values.put(LOCAL_IP_COLUMN_NAME, details.localIp.getAddress());
+        values.put(REMOTE_IP_COLUMN_NAME, details.remoteIp.getAddress());
+        values.put(MAC_COLUMN_NAME, details.macAddress);
+        return -1 != computerDb.insertWithOnConflict(COMPUTER_TABLE_NAME, null, values, SQLiteDatabase.CONFLICT_REPLACE);
+    }
 
-	public List<ComputerDetails> getAllComputers() {
-		Cursor c = computerDb.rawQuery("SELECT * FROM "+COMPUTER_TABLE_NAME, null);
-		LinkedList<ComputerDetails> computerList = new LinkedList<ComputerDetails>();
-		while (c.moveToNext()) {
-			ComputerDetails details = new ComputerDetails();
+    public List<ComputerDetails> getAllComputers() {
+        Cursor c = computerDb.rawQuery("SELECT * FROM "+COMPUTER_TABLE_NAME, null);
+        LinkedList<ComputerDetails> computerList = new LinkedList<ComputerDetails>();
+        while (c.moveToNext()) {
+            ComputerDetails details = new ComputerDetails();
 
-			details.name = c.getString(0);
+            details.name = c.getString(0);
 
-			String uuidStr = c.getString(1);
-			try {
-				details.uuid = UUID.fromString(uuidStr);
-			} catch (IllegalArgumentException e) {
-				// We'll delete this entry
-				LimeLog.severe("DB: Corrupted UUID for "+details.name);
-			}
+            String uuidStr = c.getString(1);
+            try {
+                details.uuid = UUID.fromString(uuidStr);
+            } catch (IllegalArgumentException e) {
+                // We'll delete this entry
+                LimeLog.severe("DB: Corrupted UUID for "+details.name);
+            }
 
-			try {
-				details.localIp = InetAddress.getByAddress(c.getBlob(2));
-			} catch (UnknownHostException e) {
-				// We'll delete this entry
-				LimeLog.severe("DB: Corrupted local IP for "+details.name);
-			}
+            try {
+                details.localIp = InetAddress.getByAddress(c.getBlob(2));
+            } catch (UnknownHostException e) {
+                // We'll delete this entry
+                LimeLog.severe("DB: Corrupted local IP for "+details.name);
+            }
 
-			try {
-				details.remoteIp = InetAddress.getByAddress(c.getBlob(3));
-			} catch (UnknownHostException e) {
-				// We'll delete this entry
-				LimeLog.severe("DB: Corrupted remote IP for "+details.name);
-			}
+            try {
+                details.remoteIp = InetAddress.getByAddress(c.getBlob(3));
+            } catch (UnknownHostException e) {
+                // We'll delete this entry
+                LimeLog.severe("DB: Corrupted remote IP for "+details.name);
+            }
 
-			details.macAddress = c.getString(4);
+            details.macAddress = c.getString(4);
 
-			// This signifies we don't have dynamic state (like pair state)
+            // This signifies we don't have dynamic state (like pair state)
             details.state = ComputerDetails.State.UNKNOWN;
             details.reachability = ComputerDetails.Reachability.UNKNOWN;
 
-			// If a field is corrupt or missing, skip the database entry
-			if (details.uuid == null || details.localIp == null || details.remoteIp == null ||
-					details.macAddress == null) {
-				continue;
-			}
+            // If a field is corrupt or missing, skip the database entry
+            if (details.uuid == null || details.localIp == null || details.remoteIp == null ||
+                    details.macAddress == null) {
+                continue;
+            }
 
 
             computerList.add(details);
-		}
+        }
 
-		c.close();
+        c.close();
 
-		return computerList;
-	}
+        return computerList;
+    }
 
-	public ComputerDetails getComputerByName(String name) {
-		Cursor c = computerDb.rawQuery("SELECT * FROM "+COMPUTER_TABLE_NAME+" WHERE "+COMPUTER_NAME_COLUMN_NAME+"='"+name+"'", null);
-		ComputerDetails details = new ComputerDetails();
-		if (!c.moveToFirst()) {
-			// No matching computer
-			c.close();
-			return null;
-		}
+    public ComputerDetails getComputerByName(String name) {
+        Cursor c = computerDb.rawQuery("SELECT * FROM "+COMPUTER_TABLE_NAME+" WHERE "+COMPUTER_NAME_COLUMN_NAME+"='"+name+"'", null);
+        ComputerDetails details = new ComputerDetails();
+        if (!c.moveToFirst()) {
+            // No matching computer
+            c.close();
+            return null;
+        }
 
-		details.name = c.getString(0);
+        details.name = c.getString(0);
 
-		String uuidStr = c.getString(1);
-		try {
-			details.uuid = UUID.fromString(uuidStr);
-		} catch (IllegalArgumentException e) {
-			// We'll delete this entry
-			LimeLog.severe("DB: Corrupted UUID for "+details.name);
-		}
+        String uuidStr = c.getString(1);
+        try {
+            details.uuid = UUID.fromString(uuidStr);
+        } catch (IllegalArgumentException e) {
+            // We'll delete this entry
+            LimeLog.severe("DB: Corrupted UUID for "+details.name);
+        }
 
-		try {
-			details.localIp = InetAddress.getByAddress(c.getBlob(2));
-		} catch (UnknownHostException e) {
-			// We'll delete this entry
-			LimeLog.severe("DB: Corrupted local IP for "+details.name);
-		}
+        try {
+            details.localIp = InetAddress.getByAddress(c.getBlob(2));
+        } catch (UnknownHostException e) {
+            // We'll delete this entry
+            LimeLog.severe("DB: Corrupted local IP for "+details.name);
+        }
 
-		try {
-			details.remoteIp = InetAddress.getByAddress(c.getBlob(3));
-		} catch (UnknownHostException e) {
-			// We'll delete this entry
-			LimeLog.severe("DB: Corrupted remote IP for "+details.name);
-		}
+        try {
+            details.remoteIp = InetAddress.getByAddress(c.getBlob(3));
+        } catch (UnknownHostException e) {
+            // We'll delete this entry
+            LimeLog.severe("DB: Corrupted remote IP for "+details.name);
+        }
 
-		details.macAddress = c.getString(4);
+        details.macAddress = c.getString(4);
 
-		c.close();
+        c.close();
 
         details.state = ComputerDetails.State.UNKNOWN;
         details.reachability = ComputerDetails.Reachability.UNKNOWN;
 
-		// If a field is corrupt or missing, delete the database entry
-		if (details.uuid == null || details.localIp == null || details.remoteIp == null ||
-				details.macAddress == null) {
-			deleteComputer(details.name);
-			return null;
-		}
+        // If a field is corrupt or missing, delete the database entry
+        if (details.uuid == null || details.localIp == null || details.remoteIp == null ||
+                details.macAddress == null) {
+            deleteComputer(details.name);
+            return null;
+        }
 
-		return details;
-	}
+        return details;
+    }
 }

app/src/main/java/com/limelight/computers/ComputerManagerListener.java

@@ -3,5 +3,5 @@ package com.limelight.computers;
 import com.limelight.nvstream.http.ComputerDetails;
 
 public interface ComputerManagerListener {
-	public void notifyComputerUpdated(ComputerDetails details);
+    public void notifyComputerUpdated(ComputerDetails details);
 }

app/src/main/java/com/limelight/computers/ComputerManagerService.java

@@ -1,16 +1,25 @@
 package com.limelight.computers;
 
+import java.io.IOException;
+import java.io.OutputStream;
+import java.io.StringReader;
 import java.net.InetAddress;
+import java.net.InetSocketAddress;
+import java.net.Socket;
 import java.util.LinkedList;
+import java.util.List;
+import java.util.UUID;
 import java.util.concurrent.atomic.AtomicInteger;
 
 import com.limelight.LimeLog;
 import com.limelight.binding.PlatformBinding;
 import com.limelight.discovery.DiscoveryService;
 import com.limelight.nvstream.http.ComputerDetails;
+import com.limelight.nvstream.http.NvApp;
 import com.limelight.nvstream.http.NvHTTP;
 import com.limelight.nvstream.mdns.MdnsComputer;
 import com.limelight.nvstream.mdns.MdnsDiscoveryListener;
+import com.limelight.utils.CacheHelper;
 
 import android.app.Service;
 import android.content.ComponentName;
@@ -19,46 +28,47 @@ import android.content.ServiceConnection;
 import android.os.Binder;
 import android.os.IBinder;
 
+import org.xmlpull.v1.XmlPullParserException;
+
 public class ComputerManagerService extends Service {
-	private static final int POLLING_PERIOD_MS = 3000;
-	private static final int MDNS_QUERY_PERIOD_MS = 1000;
+    private static final int POLLING_PERIOD_MS = 3000;
+    private static final int MDNS_QUERY_PERIOD_MS = 1000;
+    private static final int FAST_POLL_TIMEOUT = 500;
+    private static final int OFFLINE_POLL_TRIES = 3;
 
-	private ComputerManagerBinder binder = new ComputerManagerBinder();
+    private final ComputerManagerBinder binder = new ComputerManagerBinder();
 
-	private ComputerDatabaseManager dbManager;
-	private AtomicInteger dbRefCount = new AtomicInteger(0);
+    private ComputerDatabaseManager dbManager;
+    private final AtomicInteger dbRefCount = new AtomicInteger(0);
 
-	private IdentityManager idManager;
-	private final LinkedList<PollingTuple> pollingTuples = new LinkedList<PollingTuple>();
-	private ComputerManagerListener listener = null;
-	private AtomicInteger activePolls = new AtomicInteger(0);
+    private IdentityManager idManager;
+    private final LinkedList<PollingTuple> pollingTuples = new LinkedList<PollingTuple>();
+    private ComputerManagerListener listener = null;
+    private final AtomicInteger activePolls = new AtomicInteger(0);
     private boolean pollingActive = false;
 
-	private DiscoveryService.DiscoveryBinder discoveryBinder;
-	private final ServiceConnection discoveryServiceConnection = new ServiceConnection() {
-		public void onServiceConnected(ComponentName className, IBinder binder) {
-			synchronized (discoveryServiceConnection) {
-				DiscoveryService.DiscoveryBinder privateBinder = ((DiscoveryService.DiscoveryBinder)binder);
+    private DiscoveryService.DiscoveryBinder discoveryBinder;
+    private final ServiceConnection discoveryServiceConnection = new ServiceConnection() {
+        public void onServiceConnected(ComponentName className, IBinder binder) {
+            synchronized (discoveryServiceConnection) {
+                DiscoveryService.DiscoveryBinder privateBinder = ((DiscoveryService.DiscoveryBinder)binder);
 
-				// Set us as the event listener
-				privateBinder.setListener(createDiscoveryListener());
+                // Set us as the event listener
+                privateBinder.setListener(createDiscoveryListener());
 
-				// Signal a possible waiter that we're all setup
-				discoveryBinder = privateBinder;
-				discoveryServiceConnection.notifyAll();
-			}
-		}
+                // Signal a possible waiter that we're all setup
+                discoveryBinder = privateBinder;
+                discoveryServiceConnection.notifyAll();
+            }
+        }
 
-		public void onServiceDisconnected(ComponentName className) {
-			discoveryBinder = null;
-		}
-	};
+        public void onServiceDisconnected(ComponentName className) {
+            discoveryBinder = null;
+        }
+    };
 
     // Returns true if the details object was modified
-    private boolean runPoll(ComputerDetails details)
-    {
-        boolean newPc = details.name.isEmpty();
-
+    private boolean runPoll(ComputerDetails details, boolean newPc, int offlineCount) throws InterruptedException {
         if (!getLocalDatabaseReference()) {
             return false;
         }
@@ -66,12 +76,22 @@ public class ComputerManagerService extends Service {
         activePolls.incrementAndGet();
 
         // Poll the machine
-        if (!doPollMachine(details)) {
-            details.state = ComputerDetails.State.OFFLINE;
-            details.reachability = ComputerDetails.Reachability.OFFLINE;
-        }
+        try {
+            if (!pollComputer(details)) {
+                if (!newPc && offlineCount < OFFLINE_POLL_TRIES) {
+                    // Return without calling the listener
+                    return false;
+                }
 
-        activePolls.decrementAndGet();
+                details.state = ComputerDetails.State.OFFLINE;
+                details.reachability = ComputerDetails.Reachability.OFFLINE;
+            }
+        } catch (InterruptedException e) {
+            releaseLocalDatabaseReference();
+            throw e;
+        } finally {
+            activePolls.decrementAndGet();
+        }
 
         // If it's online, update our persistent state
         if (details.state == ComputerDetails.State.ONLINE) {
@@ -81,7 +101,7 @@ public class ComputerManagerService extends Service {
                 if (dbManager.getComputerByName(details.name) == null) {
                     // It's gone
                     releaseLocalDatabaseReference();
-                    return true;
+                    return false;
                 }
             }
 
@@ -101,13 +121,21 @@ public class ComputerManagerService extends Service {
         Thread t = new Thread() {
             @Override
             public void run() {
-                while (!isInterrupted() && pollingActive) {
-                    // Check if this poll has modified the details
-                    runPoll(details);
 
-                    // Wait until the next polling interval
+                int offlineCount = 0;
+                while (!isInterrupted() && pollingActive) {
                     try {
-                        Thread.sleep(POLLING_PERIOD_MS);
+                        // Check if this poll has modified the details
+                        if (!runPoll(details, false, offlineCount)) {
+                            LimeLog.warning(details.name + " is offline (try " + offlineCount + ")");
+                            offlineCount++;
+                        }
+                        else {
+                            offlineCount = 0;
+                        }
+
+                        // Wait until the next polling interval
+                        Thread.sleep(POLLING_PERIOD_MS / ((offlineCount > 0) ? 2 : 1));
                     } catch (InterruptedException e) {
                         break;
                     }
@@ -118,16 +146,16 @@ public class ComputerManagerService extends Service {
         return t;
     }
 
-	public class ComputerManagerBinder extends Binder {
-		public void startPolling(ComputerManagerListener listener) {
+    public class ComputerManagerBinder extends Binder {
+        public void startPolling(ComputerManagerListener listener) {
             // Polling is active
             pollingActive = true;
 
-			// Set the listener
-			ComputerManagerService.this.listener = listener;
+            // Set the listener
+            ComputerManagerService.this.listener = listener;
 
-			// Start mDNS autodiscovery too
-			discoveryBinder.startDiscovery(MDNS_QUERY_PERIOD_MS);
+            // Start mDNS autodiscovery too
+            discoveryBinder.startDiscovery(MDNS_QUERY_PERIOD_MS);
 
             synchronized (pollingTuples) {
                 for (PollingTuple tuple : pollingTuples) {
@@ -141,56 +169,68 @@ public class ComputerManagerService extends Service {
                     }
                 }
             }
-		}
+        }
 
-		public void waitForReady() {
-			synchronized (discoveryServiceConnection) {
-				try {
-					while (discoveryBinder == null) {
-						// Wait for the bind notification
-						discoveryServiceConnection.wait(1000);
-					}
-				} catch (InterruptedException ignored) {
-				}
-			}
-		}
+        public void waitForReady() {
+            synchronized (discoveryServiceConnection) {
+                try {
+                    while (discoveryBinder == null) {
+                        // Wait for the bind notification
+                        discoveryServiceConnection.wait(1000);
+                    }
+                } catch (InterruptedException ignored) {
+                }
+            }
+        }
 
-		public void waitForPollingStopped() {
-			while (activePolls.get() != 0) {
-				try {
-					Thread.sleep(250);
-				} catch (InterruptedException ignored) {}
-			}
-		}
+        public void waitForPollingStopped() {
+            while (activePolls.get() != 0) {
+                try {
+                    Thread.sleep(250);
+                } catch (InterruptedException ignored) {}
+            }
+        }
 
-		public boolean addComputerBlocking(InetAddress addr) {
-			return ComputerManagerService.this.addComputerBlocking(addr);
-		}
+        public boolean addComputerBlocking(InetAddress addr) {
+            return ComputerManagerService.this.addComputerBlocking(addr);
+        }
 
-		public void addComputer(InetAddress addr) {
-			ComputerManagerService.this.addComputer(addr);
-		}
+        public void removeComputer(String name) {
+            ComputerManagerService.this.removeComputer(name);
+        }
 
-		public void removeComputer(String name) {
-			ComputerManagerService.this.removeComputer(name);
-		}
+        public void stopPolling() {
+            // Just call the unbind handler to cleanup
+            ComputerManagerService.this.onUnbind(null);
+        }
 
-		public void stopPolling() {
-			// Just call the unbind handler to cleanup
-			ComputerManagerService.this.onUnbind(null);
-		}
+        public ApplistPoller createAppListPoller(ComputerDetails computer) {
+            return new ApplistPoller(computer);
+        }
 
-		public String getUniqueId() {
-			return idManager.getUniqueId();
-		}
-	}
+        public String getUniqueId() {
+            return idManager.getUniqueId();
+        }
 
-	@Override
-	public boolean onUnbind(Intent intent) {
-		// Stop mDNS autodiscovery
-		discoveryBinder.stopDiscovery();
+        public ComputerDetails getComputer(UUID uuid) {
+            synchronized (pollingTuples) {
+                for (PollingTuple tuple : pollingTuples) {
+                    if (uuid.equals(tuple.computer.uuid)) {
+                        return tuple.computer;
+                    }
+                }
+            }
 
-		// Stop polling
+            return null;
+        }
+    }
+
+    @Override
+    public boolean onUnbind(Intent intent) {
+        // Stop mDNS autodiscovery
+        discoveryBinder.stopDiscovery();
+
+        // Stop polling
         pollingActive = false;
         synchronized (pollingTuples) {
             for (PollingTuple tuple : pollingTuples) {
@@ -202,51 +242,42 @@ public class ComputerManagerService extends Service {
             }
         }
 
-		// Remove the listener
-		listener = null;
+        // Remove the listener
+        listener = null;
 
-		return false;
-	}
+        return false;
+    }
 
-	private MdnsDiscoveryListener createDiscoveryListener() {
-		return new MdnsDiscoveryListener() {
-			@Override
-			public void notifyComputerAdded(MdnsComputer computer) {
-				// Kick off a serverinfo poll on this machine
-				addComputer(computer.getAddress());
-			}
+    private MdnsDiscoveryListener createDiscoveryListener() {
+        return new MdnsDiscoveryListener() {
+            @Override
+            public void notifyComputerAdded(MdnsComputer computer) {
+                // Kick off a serverinfo poll on this machine
+                addComputerBlocking(computer.getAddress());
+            }
 
-			@Override
-			public void notifyComputerRemoved(MdnsComputer computer) {
-				// Nothing to do here
-			}
+            @Override
+            public void notifyComputerRemoved(MdnsComputer computer) {
+                // Nothing to do here
+            }
 
-			@Override
-			public void notifyDiscoveryFailure(Exception e) {
-				LimeLog.severe("mDNS discovery failed");
-				e.printStackTrace();
-			}
-		};
-	}
-
-	public void addComputer(InetAddress addr) {
-		// Setup a placeholder
-		ComputerDetails fakeDetails = new ComputerDetails();
-		fakeDetails.localIp = addr;
-		fakeDetails.remoteIp = addr;
-        fakeDetails.name = "";
-
-        addTuple(fakeDetails);
-	}
+            @Override
+            public void notifyDiscoveryFailure(Exception e) {
+                LimeLog.severe("mDNS discovery failed");
+                e.printStackTrace();
+            }
+        };
+    }
 
     private void addTuple(ComputerDetails details) {
         synchronized (pollingTuples) {
             for (PollingTuple tuple : pollingTuples) {
                 // Check if this is the same computer
-                if (tuple.computer == details ||
-                        tuple.computer.localIp.equals(details.localIp) ||
-                        tuple.computer.remoteIp.equals(details.remoteIp) ||
-                        tuple.computer.name.equals(details.name)) {
+                if (tuple.computer.uuid.equals(details.uuid)) {
+                    // Update details anyway in case this machine has been re-added by IP
+                    // after not being reachable by our existing information
+                    tuple.computer.localIp = details.localIp;
+                    tuple.computer.remoteIp = details.remoteIp;
 
                     // Start a polling thread if polling is active
                     if (pollingActive && tuple.thread == null) {
@@ -268,18 +299,23 @@ public class ComputerManagerService extends Service {
         }
     }
 
-	public boolean addComputerBlocking(InetAddress addr) {
-		// Setup a placeholder
-		ComputerDetails fakeDetails = new ComputerDetails();
-		fakeDetails.localIp = addr;
-		fakeDetails.remoteIp = addr;
-        fakeDetails.name = "";
+    public boolean addComputerBlocking(InetAddress addr) {
+        // Setup a placeholder
+        ComputerDetails fakeDetails = new ComputerDetails();
+        fakeDetails.localIp = addr;
+        fakeDetails.remoteIp = addr;
 
-		// Block while we try to fill the details
-        runPoll(fakeDetails);
+        // Block while we try to fill the details
+        try {
+            runPoll(fakeDetails, true, 0);
+        } catch (InterruptedException e) {
+            return false;
+        }
+
+        // If the machine is reachable, it was successful
+        if (fakeDetails.state == ComputerDetails.State.ONLINE) {
+            LimeLog.info("New PC ("+fakeDetails.name+") is UUID "+fakeDetails.uuid);
 
-		// If the machine is reachable, it was successful
-		if (fakeDetails.state == ComputerDetails.State.ONLINE) {
             // Start a polling thread for this machine
             addTuple(fakeDetails);
             return true;
@@ -287,15 +323,15 @@ public class ComputerManagerService extends Service {
         else {
             return false;
         }
-	}
+    }
 
-	public void removeComputer(String name) {
-		if (!getLocalDatabaseReference()) {
-			return;
-		}
+    public void removeComputer(String name) {
+        if (!getLocalDatabaseReference()) {
+            return;
+        }
 
-		// Remove it from the database
-		dbManager.deleteComputer(name);
+        // Remove it from the database
+        dbManager.deleteComputer(name);
 
         synchronized (pollingTuples) {
             // Remove the computer from the computer list
@@ -311,102 +347,190 @@ public class ComputerManagerService extends Service {
             }
         }
 
-		releaseLocalDatabaseReference();
-	}
+        releaseLocalDatabaseReference();
+    }
 
-	private boolean getLocalDatabaseReference() {
-		if (dbRefCount.get() == 0) {
-			return false;
-		}
+    private boolean getLocalDatabaseReference() {
+        if (dbRefCount.get() == 0) {
+            return false;
+        }
 
-		dbRefCount.incrementAndGet();
-		return true;
-	}
+        dbRefCount.incrementAndGet();
+        return true;
+    }
 
-	private void releaseLocalDatabaseReference() {
-		if (dbRefCount.decrementAndGet() == 0) {
-			dbManager.close();
-		}
-	}
+    private void releaseLocalDatabaseReference() {
+        if (dbRefCount.decrementAndGet() == 0) {
+            dbManager.close();
+        }
+    }
 
-	private ComputerDetails tryPollIp(InetAddress ipAddr) {
-		try {
-			NvHTTP http = new NvHTTP(ipAddr, idManager.getUniqueId(),
-					null, PlatformBinding.getCryptoProvider(ComputerManagerService.this));
+    private ComputerDetails tryPollIp(ComputerDetails details, InetAddress ipAddr) {
+        try {
+            NvHTTP http = new NvHTTP(ipAddr, idManager.getUniqueId(),
+                    null, PlatformBinding.getCryptoProvider(ComputerManagerService.this));
 
-			return http.getComputerDetails();
-		} catch (Exception e) {
-			return null;
-		}
-	}
+            ComputerDetails newDetails = http.getComputerDetails();
 
-	private boolean pollComputer(ComputerDetails details, boolean localFirst) {
-		ComputerDetails polledDetails;
+            // Check if this is the PC we expected
+            if (details.uuid != null && newDetails.uuid != null &&
+                    !details.uuid.equals(newDetails.uuid)) {
+                // We got the wrong PC!
+                LimeLog.info("Polling returned the wrong PC!");
+                return null;
+            }
 
-		if (localFirst) {
-			polledDetails = tryPollIp(details.localIp);
-		}
-		else {
-			polledDetails = tryPollIp(details.remoteIp);
-		}
+            return newDetails;
+        } catch (Exception e) {
+            return null;
+        }
+    }
 
-		if (polledDetails == null && !details.localIp.equals(details.remoteIp)) {
-			// Failed, so let's try the fallback
-			if (!localFirst) {
-				polledDetails = tryPollIp(details.localIp);
-			}
-			else {
-				polledDetails = tryPollIp(details.remoteIp);
-			}
+    // Just try to establish a TCP connection to speculatively detect a running
+    // GFE server
+    private boolean fastPollIp(InetAddress addr) {
+        Socket s = new Socket();
+        try {
+            s.connect(new InetSocketAddress(addr, NvHTTP.PORT), FAST_POLL_TIMEOUT);
+            s.close();
+            return true;
+        } catch (IOException e) {
+            return false;
+        }
+    }
 
-			// The fallback poll worked
-			if (polledDetails != null) {
-				polledDetails.reachability = !localFirst ? ComputerDetails.Reachability.LOCAL :
-					ComputerDetails.Reachability.REMOTE;
-			}
-		}
-		else if (polledDetails != null) {
-			polledDetails.reachability = localFirst ? ComputerDetails.Reachability.LOCAL :
-				ComputerDetails.Reachability.REMOTE;
-		}
+    private void startFastPollThread(final InetAddress addr, final boolean[] info) {
+        Thread t = new Thread() {
+            @Override
+            public void run() {
+                boolean pollRes = fastPollIp(addr);
 
-		// Machine was unreachable both tries
-		if (polledDetails == null) {
-			return false;
-		}
+                synchronized (info) {
+                    info[0] = true; // Done
+                    info[1] = pollRes; // Polling result
 
-		// If we got here, it's reachable
-		details.update(polledDetails);
-		return true;
-	}
+                    info.notify();
+                }
+            }
+        };
+        t.setName("Fast Poll - "+addr.getHostAddress());
+        t.start();
+    }
 
-	private boolean doPollMachine(ComputerDetails details) {
-        if (details.reachability == ComputerDetails.Reachability.UNKNOWN ||
-            details.reachability == ComputerDetails.Reachability.OFFLINE) {
-            // Always try local first to avoid potential UDP issues when
-            // attempting to stream via the router's external IP address
-            // behind its NAT
-            return pollComputer(details, true);
+    private ComputerDetails.Reachability fastPollPc(final InetAddress local, final InetAddress remote) throws InterruptedException {
+        final boolean[] remoteInfo = new boolean[2];
+        final boolean[] localInfo = new boolean[2];
+
+        startFastPollThread(local, localInfo);
+        startFastPollThread(remote, remoteInfo);
+
+        // Check local first
+        synchronized (localInfo) {
+            while (!localInfo[0]) {
+                localInfo.wait(500);
+            }
+
+            if (localInfo[1]) {
+                return ComputerDetails.Reachability.LOCAL;
+            }
+        }
+
+        // Now remote
+        synchronized (remoteInfo) {
+            while (!remoteInfo[0]) {
+                remoteInfo.wait(500);
+            }
+
+            if (remoteInfo[1]) {
+                return ComputerDetails.Reachability.REMOTE;
+            }
+        }
+
+        return ComputerDetails.Reachability.OFFLINE;
+    }
+
+    private boolean pollComputer(ComputerDetails details) throws InterruptedException {
+        ComputerDetails polledDetails;
+        ComputerDetails.Reachability reachability;
+
+        // If the local address is routable across the Internet,
+        // always consider this PC remote to be conservative
+        if (details.localIp.equals(details.remoteIp)) {
+            reachability = ComputerDetails.Reachability.REMOTE;
         }
         else {
-            // If we're already reached a machine via a particular IP address,
-            // always try that one first
-            return pollComputer(details, details.reachability == ComputerDetails.Reachability.LOCAL);
+            // Do a TCP-level connection to the HTTP server to see if it's listening
+            LimeLog.info("Starting fast poll for "+details.name+" ("+details.localIp+", "+details.remoteIp+")");
+            reachability = fastPollPc(details.localIp, details.remoteIp);
+            LimeLog.info("Fast poll for "+details.name+" returned "+reachability.toString());
         }
-	}
 
-	@Override
-	public void onCreate() {
-		// Bind to the discovery service
-		bindService(new Intent(this, DiscoveryService.class),
-				discoveryServiceConnection, Service.BIND_AUTO_CREATE);
+        // If no connection could be established to either IP address, there's nothing we can do
+        if (reachability == ComputerDetails.Reachability.OFFLINE) {
+            return false;
+        }
 
-		// Lookup or generate this device's UID
-		idManager = new IdentityManager(this);
+        boolean localFirst = (reachability == ComputerDetails.Reachability.LOCAL);
 
-		// Initialize the DB
-		dbManager = new ComputerDatabaseManager(this);
-		dbRefCount.set(1);
+        if (localFirst) {
+            polledDetails = tryPollIp(details, details.localIp);
+        }
+        else {
+            polledDetails = tryPollIp(details, details.remoteIp);
+        }
+
+        if (polledDetails == null && !details.localIp.equals(details.remoteIp)) {
+            // Failed, so let's try the fallback
+            if (!localFirst) {
+                polledDetails = tryPollIp(details, details.localIp);
+            }
+            else {
+                polledDetails = tryPollIp(details, details.remoteIp);
+            }
+
+            // The fallback poll worked
+            if (polledDetails != null) {
+                polledDetails.reachability = !localFirst ? ComputerDetails.Reachability.LOCAL :
+                    ComputerDetails.Reachability.REMOTE;
+            }
+        }
+        else if (polledDetails != null) {
+            polledDetails.reachability = localFirst ? ComputerDetails.Reachability.LOCAL :
+                ComputerDetails.Reachability.REMOTE;
+        }
+
+        // Machine was unreachable both tries
+        if (polledDetails == null) {
+            return false;
+        }
+
+        // Save the old MAC address
+        String savedMacAddress = details.macAddress;
+
+        // If we got here, it's reachable
+        details.update(polledDetails);
+
+        // If the new MAC address is empty, restore the old one (workaround for GFE bug)
+        if (details.macAddress.equals("00:00:00:00:00:00") && savedMacAddress != null) {
+            LimeLog.info("MAC address was empty; using existing value: "+savedMacAddress);
+            details.macAddress = savedMacAddress;
+        }
+
+        return true;
+    }
+
+    @Override
+    public void onCreate() {
+        // Bind to the discovery service
+        bindService(new Intent(this, DiscoveryService.class),
+                discoveryServiceConnection, Service.BIND_AUTO_CREATE);
+
+        // Lookup or generate this device's UID
+        idManager = new IdentityManager(this);
+
+        // Initialize the DB
+        dbManager = new ComputerDatabaseManager(this);
+        dbRefCount.set(1);
 
         // Grab known machines into our computer list
         if (!getLocalDatabaseReference()) {
@@ -414,35 +538,146 @@ public class ComputerManagerService extends Service {
         }
 
         for (ComputerDetails computer : dbManager.getAllComputers()) {
-            // Add this computer without a thread
-            pollingTuples.add(new PollingTuple(computer, null));
+            // Add tuples for each computer
+            addTuple(computer);
         }
 
         releaseLocalDatabaseReference();
-	}
+    }
 
-	@Override
-	public void onDestroy() {
-		if (discoveryBinder != null) {
-			// Unbind from the discovery service
-			unbindService(discoveryServiceConnection);
-		}
+    @Override
+    public void onDestroy() {
+        if (discoveryBinder != null) {
+            // Unbind from the discovery service
+            unbindService(discoveryServiceConnection);
+        }
 
-		// FIXME: Should await termination here but we have timeout issues in HttpURLConnection
+        // FIXME: Should await termination here but we have timeout issues in HttpURLConnection
 
-		// Remove the initial DB reference
-		releaseLocalDatabaseReference();
-	}
+        // Remove the initial DB reference
+        releaseLocalDatabaseReference();
+    }
 
-	@Override
-	public IBinder onBind(Intent intent) {
-		return binder;
-	}
+    @Override
+    public IBinder onBind(Intent intent) {
+        return binder;
+    }
+
+    public class ApplistPoller {
+        private Thread thread;
+        private final ComputerDetails computer;
+        private final Object pollEvent = new Object();
+
+        public ApplistPoller(ComputerDetails computer) {
+            this.computer = computer;
+        }
+
+        public void pollNow() {
+            synchronized (pollEvent) {
+                pollEvent.notify();
+            }
+        }
+
+        private boolean waitPollingDelay() {
+            try {
+                synchronized (pollEvent) {
+                    pollEvent.wait(POLLING_PERIOD_MS);
+                }
+            } catch (InterruptedException e) {
+                return false;
+            }
+
+            return thread != null && !thread.isInterrupted();
+        }
+
+        public void start() {
+            thread = new Thread() {
+                @Override
+                public void run() {
+                    do {
+                        InetAddress selectedAddr;
+
+                        // Can't poll if it's not online
+                        if (computer.state != ComputerDetails.State.ONLINE) {
+                            if (listener != null) {
+                                listener.notifyComputerUpdated(computer);
+                            }
+                            continue;
+                        }
+
+                        // Can't poll if there's no UUID yet
+                        if (computer.uuid == null) {
+                            continue;
+                        }
+
+                        if (computer.reachability == ComputerDetails.Reachability.LOCAL) {
+                            selectedAddr = computer.localIp;
+                        }
+                        else {
+                            selectedAddr = computer.remoteIp;
+                        }
+
+                        NvHTTP http = new NvHTTP(selectedAddr, idManager.getUniqueId(),
+                                null, PlatformBinding.getCryptoProvider(ComputerManagerService.this));
+
+                        try {
+                            // Query the app list from the server
+                            String appList = http.getAppListRaw();
+                            List<NvApp> list = NvHTTP.getAppListByReader(new StringReader(appList));
+                            if (appList != null && !appList.isEmpty() && !list.isEmpty()) {
+                                // Open the cache file
+                                OutputStream cacheOut = null;
+                                try {
+                                    cacheOut = CacheHelper.openCacheFileForOutput(getCacheDir(), "applist", computer.uuid.toString());
+                                    CacheHelper.writeStringToOutputStream(cacheOut, appList);
+                                } catch (IOException e) {
+                                    e.printStackTrace();
+                                } finally {
+                                    try {
+                                        if (cacheOut != null) {
+                                            cacheOut.close();
+                                        }
+                                    } catch (IOException e) {}
+                                }
+
+                                // Update the computer
+                                computer.rawAppList = appList;
+
+                                // Notify that the app list has been updated
+                                // and ensure that the thread is still active
+                                if (listener != null && thread != null) {
+                                    listener.notifyComputerUpdated(computer);
+                                }
+                            }
+                            else {
+                                LimeLog.warning("Empty app list received from "+computer.uuid);
+                            }
+                        } catch (IOException e) {
+                            e.printStackTrace();
+                        } catch (XmlPullParserException e) {
+                            e.printStackTrace();
+                        }
+                    } while (waitPollingDelay());
+                }
+            };
+            thread.start();
+        }
+
+        public void stop() {
+            if (thread != null) {
+                thread.interrupt();
+
+                // Don't join here because we might be blocked on network I/O
+
+                thread = null;
+            }
+        }
+    }
 }
 
 class PollingTuple {
     public Thread thread;
-    public ComputerDetails computer;
+    public final ComputerDetails computer;
 
     public PollingTuple(ComputerDetails computer, Thread thread) {
         this.computer = computer;

app/src/main/java/com/limelight/computers/IdentityManager.java

@@ -12,75 +12,75 @@ import com.limelight.LimeLog;
 import android.content.Context;
 
 public class IdentityManager {
-	private static final String UNIQUE_ID_FILE_NAME = "uniqueid";
-	private static final int UID_SIZE_IN_BYTES = 8;
+    private static final String UNIQUE_ID_FILE_NAME = "uniqueid";
+    private static final int UID_SIZE_IN_BYTES = 8;
 
-	private String uniqueId;
+    private String uniqueId;
 
-	public IdentityManager(Context c) {
-		uniqueId = loadUniqueId(c);
-		if (uniqueId == null) {
-			uniqueId = generateNewUniqueId(c);
-		}
+    public IdentityManager(Context c) {
+        uniqueId = loadUniqueId(c);
+        if (uniqueId == null) {
+            uniqueId = generateNewUniqueId(c);
+        }
 
-		LimeLog.info("UID is now: "+uniqueId);
-	}
+        LimeLog.info("UID is now: "+uniqueId);
+    }
 
-	public String getUniqueId() {
-		return uniqueId;
-	}
+    public String getUniqueId() {
+        return uniqueId;
+    }
 
-	private static String loadUniqueId(Context c) {
-		// 2 Hex digits per byte
-		char[] uid = new char[UID_SIZE_IN_BYTES * 2];
-		InputStreamReader reader = null;
-		LimeLog.info("Reading UID from disk");
-		try {
-			reader = new InputStreamReader(c.openFileInput(UNIQUE_ID_FILE_NAME));
-			if (reader.read(uid) != UID_SIZE_IN_BYTES * 2)
-			{
-				LimeLog.severe("UID file data is truncated");
-				return null;
-			}
-			return new String(uid);
-		} catch (FileNotFoundException e) {
-			LimeLog.info("No UID file found");
-			return null;
-		} catch (IOException e) {
-			LimeLog.severe("Error while reading UID file");
-			e.printStackTrace();
-			return null;
-		} finally {
-			if (reader != null) {
-				try {
-					reader.close();
-				} catch (IOException ignored) {}
-			}
-		}
-	}
+    private static String loadUniqueId(Context c) {
+        // 2 Hex digits per byte
+        char[] uid = new char[UID_SIZE_IN_BYTES * 2];
+        InputStreamReader reader = null;
+        LimeLog.info("Reading UID from disk");
+        try {
+            reader = new InputStreamReader(c.openFileInput(UNIQUE_ID_FILE_NAME));
+            if (reader.read(uid) != UID_SIZE_IN_BYTES * 2)
+            {
+                LimeLog.severe("UID file data is truncated");
+                return null;
+            }
+            return new String(uid);
+        } catch (FileNotFoundException e) {
+            LimeLog.info("No UID file found");
+            return null;
+        } catch (IOException e) {
+            LimeLog.severe("Error while reading UID file");
+            e.printStackTrace();
+            return null;
+        } finally {
+            if (reader != null) {
+                try {
+                    reader.close();
+                } catch (IOException ignored) {}
+            }
+        }
+    }
 
-	private static String generateNewUniqueId(Context c) {
-		// Generate a new UID hex string
-		LimeLog.info("Generating new UID");
-		String uidStr = String.format((Locale)null, "%016x", new Random().nextLong());
+    private static String generateNewUniqueId(Context c) {
+        // Generate a new UID hex string
+        LimeLog.info("Generating new UID");
+        String uidStr = String.format((Locale)null, "%016x", new Random().nextLong());
 
-		OutputStreamWriter writer = null;
-		try {
-			writer = new OutputStreamWriter(c.openFileOutput(UNIQUE_ID_FILE_NAME, 0));
-			writer.write(uidStr);
-			LimeLog.info("UID written to disk");
-		} catch (IOException e) {
-			LimeLog.severe("Error while writing UID file");
-			e.printStackTrace();
-		} finally {
-			if (writer != null) {
-				try {
-					writer.close();
-				} catch (IOException ignored) {}
-			}
-		}
+        OutputStreamWriter writer = null;
+        try {
+            writer = new OutputStreamWriter(c.openFileOutput(UNIQUE_ID_FILE_NAME, 0));
+            writer.write(uidStr);
+            LimeLog.info("UID written to disk");
+        } catch (IOException e) {
+            LimeLog.severe("Error while writing UID file");
+            e.printStackTrace();
+        } finally {
+            if (writer != null) {
+                try {
+                    writer.close();
+                } catch (IOException ignored) {}
+            }
+        }
 
-		// We can return a UID even if I/O fails
-		return uidStr;
-	}
+        // We can return a UID even if I/O fails
+        return uidStr;
+    }
 }

app/src/main/java/com/limelight/discovery/DiscoveryService.java

@@ -16,75 +16,75 @@ import android.os.IBinder;
 
 public class DiscoveryService extends Service {
 
-	private MdnsDiscoveryAgent discoveryAgent;
-	private MdnsDiscoveryListener boundListener;
-	private MulticastLock multicastLock;
+    private MdnsDiscoveryAgent discoveryAgent;
+    private MdnsDiscoveryListener boundListener;
+    private MulticastLock multicastLock;
 
-	public class DiscoveryBinder extends Binder {
-		public void setListener(MdnsDiscoveryListener listener) {
-			boundListener = listener;
-		}
+    public class DiscoveryBinder extends Binder {
+        public void setListener(MdnsDiscoveryListener listener) {
+            boundListener = listener;
+        }
 
-		public void startDiscovery(int queryIntervalMs) {
-			multicastLock.acquire();
-			discoveryAgent.startDiscovery(queryIntervalMs);
-		}
+        public void startDiscovery(int queryIntervalMs) {
+            multicastLock.acquire();
+            discoveryAgent.startDiscovery(queryIntervalMs);
+        }
 
-		public void stopDiscovery() {
-			discoveryAgent.stopDiscovery();
-			multicastLock.release();
-		}
+        public void stopDiscovery() {
+            discoveryAgent.stopDiscovery();
+            multicastLock.release();
+        }
 
-		public List<MdnsComputer> getComputerSet() {
-			return discoveryAgent.getComputerSet();
-		}
-	}
+        public List<MdnsComputer> getComputerSet() {
+            return discoveryAgent.getComputerSet();
+        }
+    }
 
-	@Override
-	public void onCreate() {
-		WifiManager wifiMgr = (WifiManager) getSystemService(Context.WIFI_SERVICE);
-		multicastLock = wifiMgr.createMulticastLock("Limelight mDNS");
-		multicastLock.setReferenceCounted(false);
+    @Override
+    public void onCreate() {
+        WifiManager wifiMgr = (WifiManager) getSystemService(Context.WIFI_SERVICE);
+        multicastLock = wifiMgr.createMulticastLock("Limelight mDNS");
+        multicastLock.setReferenceCounted(false);
 
-		discoveryAgent = new MdnsDiscoveryAgent(new MdnsDiscoveryListener() {
-			@Override
-			public void notifyComputerAdded(MdnsComputer computer) {
-				if (boundListener != null) {
-					boundListener.notifyComputerAdded(computer);
-				}
-			}
+        discoveryAgent = new MdnsDiscoveryAgent(new MdnsDiscoveryListener() {
+            @Override
+            public void notifyComputerAdded(MdnsComputer computer) {
+                if (boundListener != null) {
+                    boundListener.notifyComputerAdded(computer);
+                }
+            }
 
-			@Override
-			public void notifyComputerRemoved(MdnsComputer computer) {
-				if (boundListener != null) {
-					boundListener.notifyComputerRemoved(computer);
-				}
-			}
+            @Override
+            public void notifyComputerRemoved(MdnsComputer computer) {
+                if (boundListener != null) {
+                    boundListener.notifyComputerRemoved(computer);
+                }
+            }
 
-			@Override
-			public void notifyDiscoveryFailure(Exception e) {
-				if (boundListener != null) {
-					boundListener.notifyDiscoveryFailure(e);
-				}
-			}
-		});
-	}
+            @Override
+            public void notifyDiscoveryFailure(Exception e) {
+                if (boundListener != null) {
+                    boundListener.notifyDiscoveryFailure(e);
+                }
+            }
+        });
+    }
 
-	private DiscoveryBinder binder = new DiscoveryBinder();
+    private final DiscoveryBinder binder = new DiscoveryBinder();
 
-	@Override
-	public IBinder onBind(Intent intent) {
-		return binder;
-	}
+    @Override
+    public IBinder onBind(Intent intent) {
+        return binder;
+    }
 
-	@Override
-	public boolean onUnbind(Intent intent) {
-		// Stop any discovery session
-		discoveryAgent.stopDiscovery();
-		multicastLock.release();
+    @Override
+    public boolean onUnbind(Intent intent) {
+        // Stop any discovery session
+        discoveryAgent.stopDiscovery();
+        multicastLock.release();
 
-		// Unbind the listener
-		boundListener = null;
-		return false;
-	}
+        // Unbind the listener
+        boundListener = null;
+        return false;
+    }
 }

app/src/main/java/com/limelight/grid/AppGridAdapter.java

@@ -1,148 +1,204 @@
 package com.limelight.grid;
 
-import android.content.Context;
+import android.app.Activity;
+import android.graphics.Bitmap;
 import android.widget.ImageView;
 import android.widget.TextView;
 
-import com.koushikdutta.async.future.FutureCallback;
-import com.koushikdutta.ion.Ion;
 import com.limelight.AppView;
+import com.limelight.LimeLog;
 import com.limelight.R;
-import com.limelight.binding.PlatformBinding;
-import com.limelight.nvstream.http.LimelightCryptoProvider;
+import com.limelight.grid.assets.CachedAppAssetLoader;
+import com.limelight.grid.assets.DiskAssetLoader;
+import com.limelight.grid.assets.MemoryAssetLoader;
+import com.limelight.grid.assets.NetworkAssetLoader;
+import com.limelight.nvstream.http.ComputerDetails;
 
-import java.net.InetAddress;
-import java.net.Socket;
+import java.lang.ref.WeakReference;
 import java.security.KeyManagementException;
 import java.security.NoSuchAlgorithmException;
-import java.security.Principal;
-import java.security.PrivateKey;
-import java.security.SecureRandom;
-import java.util.HashMap;
-import java.util.concurrent.Future;
-
-import javax.net.ssl.HostnameVerifier;
-import javax.net.ssl.KeyManager;
-import javax.net.ssl.SSLContext;
-import javax.net.ssl.SSLSession;
-import javax.net.ssl.TrustManager;
-import javax.net.ssl.X509KeyManager;
-import javax.net.ssl.X509TrustManager;
-import java.security.cert.X509Certificate;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.Comparator;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
+@SuppressWarnings("unchecked")
 public class AppGridAdapter extends GenericGridAdapter<AppView.AppObject> {
+    private final Activity activity;
 
-    private InetAddress address;
-    private String uniqueId;
-    private LimelightCryptoProvider cryptoProvider;
-    private SSLContext sslContext;
-    private final HashMap<ImageView, Future> pendingRequests = new HashMap<ImageView, Future>();
+    private static final int ART_WIDTH_PX = 300;
+    private static final int SMALL_WIDTH_DP = 100;
+    private static final int LARGE_WIDTH_DP = 150;
 
-    public AppGridAdapter(Context context, InetAddress address, String uniqueId) throws NoSuchAlgorithmException, KeyManagementException {
-        super(context, R.layout.app_grid_item, R.drawable.image_loading);
+    private final CachedAppAssetLoader loader;
+    private final ConcurrentHashMap<WeakReference<ImageView>, CachedAppAssetLoader.LoaderTuple> loadingTuples = new ConcurrentHashMap<>();
+    private final ConcurrentHashMap<Object, CachedAppAssetLoader.LoaderTuple> backgroundLoadingTuples = new ConcurrentHashMap<>();
 
-        this.address = address;
-        this.uniqueId = uniqueId;
+    public AppGridAdapter(Activity activity, boolean listMode, boolean small, ComputerDetails computer, String uniqueId) throws KeyManagementException, NoSuchAlgorithmException {
+        super(activity, listMode ? R.layout.simple_row : (small ? R.layout.app_grid_item_small : R.layout.app_grid_item), R.drawable.image_loading);
 
-        cryptoProvider = PlatformBinding.getCryptoProvider(context);
+        int dpi = activity.getResources().getDisplayMetrics().densityDpi;
+        int dp;
 
-        sslContext = SSLContext.getInstance("SSL");
-        sslContext.init(ourKeyman, trustAllCerts, new SecureRandom());
+        if (small) {
+            dp = SMALL_WIDTH_DP;
+        }
+        else {
+            dp = LARGE_WIDTH_DP;
+        }
+
+        double scalingDivisor = ART_WIDTH_PX / (dp * (dpi / 160));
+        if (scalingDivisor < 1.0) {
+            // We don't want to make them bigger before draw-time
+            scalingDivisor = 1.0;
+        }
+        LimeLog.info("Art scaling divisor: " + scalingDivisor);
+
+        this.activity = activity;
+        this.loader = new CachedAppAssetLoader(computer, scalingDivisor,
+                new NetworkAssetLoader(context, uniqueId),
+                new MemoryAssetLoader(), new DiskAssetLoader(context.getCacheDir()));
     }
 
-    TrustManager[] trustAllCerts = new TrustManager[] {
-            new X509TrustManager() {
-                public X509Certificate[] getAcceptedIssuers() {
-                    return new X509Certificate[0];
-                }
-                public void checkClientTrusted(X509Certificate[] certs, String authType) {}
-                public void checkServerTrusted(X509Certificate[] certs, String authType) {}
-            }};
+    private static void cancelTuples(ConcurrentHashMap<?, CachedAppAssetLoader.LoaderTuple> map) {
+        Collection<CachedAppAssetLoader.LoaderTuple> tuples = map.values();
 
-    KeyManager[] ourKeyman = new KeyManager[] {
-            new X509KeyManager() {
-                public String chooseClientAlias(String[] keyTypes,
-                                                Principal[] issuers, Socket socket) {
-                    return "Limelight-RSA";
-                }
+        for (CachedAppAssetLoader.LoaderTuple tuple : tuples) {
+            tuple.cancel();
+        }
 
-                public String chooseServerAlias(String keyType, Principal[] issuers,
-                                                Socket socket) {
-                    return null;
-                }
+        map.clear();
+    }
 
-                public X509Certificate[] getCertificateChain(String alias) {
-                    return new X509Certificate[] {cryptoProvider.getClientCertificate()};
-                }
+    public void cancelQueuedOperations() {
+        cancelTuples(loadingTuples);
+        cancelTuples(backgroundLoadingTuples);
 
-                public String[] getClientAliases(String keyType, Principal[] issuers) {
-                    return null;
-                }
+        loader.freeCacheMemory();
+    }
 
-                public PrivateKey getPrivateKey(String alias) {
-                    return cryptoProvider.getClientPrivateKey();
-                }
-
-                public String[] getServerAliases(String keyType, Principal[] issuers) {
-                    return null;
-                }
+    private void sortList() {
+        Collections.sort(itemList, new Comparator<AppView.AppObject>() {
+            @Override
+            public int compare(AppView.AppObject lhs, AppView.AppObject rhs) {
+                return lhs.app.getAppName().compareTo(rhs.app.getAppName());
             }
-    };
-
-    // Ignore differences between given hostname and certificate hostname
-    HostnameVerifier hv = new HostnameVerifier() {
-        public boolean verify(String hostname, SSLSession session) { return true; }
-    };
+        });
+    }
 
     public void addApp(AppView.AppObject app) {
+        // Queue a request to fetch this bitmap in the background
+        Object tupleKey = new Object();
+        CachedAppAssetLoader.LoaderTuple tuple =
+                loader.loadBitmapWithContextInBackground(app.app, tupleKey, backgroundLoadListener);
+        if (tuple != null) {
+            backgroundLoadingTuples.put(tupleKey, tuple);
+        }
+
         itemList.add(app);
+        sortList();
     }
 
-    public void abortPendingRequests() {
-        HashMap<ImageView, Future> tempMap;
+    public void removeApp(AppView.AppObject app) {
+        itemList.remove(app);
+    }
 
-        synchronized (pendingRequests) {
-            // Copy the pending requests under a lock
-            tempMap = new HashMap<ImageView, Future>(pendingRequests);
+    private final CachedAppAssetLoader.LoadListener imageViewLoadListener = new CachedAppAssetLoader.LoadListener() {
+        @Override
+        public void notifyLongLoad(Object object) {
+            final WeakReference<ImageView> viewRef = (WeakReference<ImageView>) object;
+
+            // If the view isn't there anymore, don't bother scheduling on the UI thread
+            if (viewRef.get() == null) {
+                return;
+            }
+
+            activity.runOnUiThread(new Runnable() {
+                @Override
+                public void run() {
+                    ImageView view = viewRef.get();
+                    if (view != null) {
+                        view.setImageResource(R.drawable.image_loading);
+                        fadeInImage(view);
+                    }
+                }
+            });
         }
 
-        for (Future f : tempMap.values()) {
-            f.cancel(true);
-        }
+        @Override
+        public void notifyLoadComplete(Object object, final Bitmap bitmap) {
+            final WeakReference<ImageView> viewRef = (WeakReference<ImageView>) object;
 
-        synchronized (pendingRequests) {
-            // Remove cancelled requests
-            for (ImageView v : tempMap.keySet()) {
-                pendingRequests.remove(v);
+            loadingTuples.remove(viewRef);
+
+            // Just leave the loading icon in place
+            if (bitmap == null) {
+                return;
+            }
+
+            // If the view isn't there anymore, don't bother scheduling on the UI thread
+            if (viewRef.get() == null) {
+                return;
+            }
+
+            activity.runOnUiThread(new Runnable() {
+                @Override
+                public void run() {
+                    ImageView view = viewRef.get();
+                    if (view != null) {
+                        view.setImageBitmap(bitmap);
+                        fadeInImage(view);
+                    }
+                }
+            });
+        }
+    };
+
+    private final CachedAppAssetLoader.LoadListener backgroundLoadListener = new CachedAppAssetLoader.LoadListener() {
+        @Override
+        public void notifyLongLoad(Object object) {}
+
+        @Override
+        public void notifyLoadComplete(Object object, final Bitmap bitmap) {
+            backgroundLoadingTuples.remove(object);
+        }
+    };
+
+    private void reapLoaderTuples(ImageView view) {
+        // Poor HashMap doesn't deserve this...
+        Iterator<Map.Entry<WeakReference<ImageView>, CachedAppAssetLoader.LoaderTuple>> i = loadingTuples.entrySet().iterator();
+        while (i.hasNext()) {
+            Map.Entry<WeakReference<ImageView>, CachedAppAssetLoader.LoaderTuple> entry = i.next();
+            ImageView imageView = entry.getKey().get();
+
+            // Remove tuples that refer to this view or no view
+            if (imageView == null || imageView == view) {
+                // FIXME: There's a small chance that this can race if we've already gone down
+                // the path to notification but haven't been notified yet
+                entry.getValue().cancel();
+
+                // Remove it from the tuple list
+                i.remove();
             }
         }
     }
 
-    @Override
-    public boolean populateImageView(final ImageView imgView, AppView.AppObject obj) {
+    public boolean populateImageView(final ImageView imgView, final AppView.AppObject obj) {
+        // Cancel pending loads on this image view
+        reapLoaderTuples(imgView);
 
-        // Set SSL contexts correctly to allow us to authenticate
-        Ion.getDefault(imgView.getContext()).getHttpClient().getSSLSocketMiddleware().setTrustManagers(trustAllCerts);
-        Ion.getDefault(imgView.getContext()).getHttpClient().getSSLSocketMiddleware().setSSLContext(sslContext);
+        // Clear existing contents of the image view
+        imgView.setAlpha(0.0f);
 
-        // Set off the deferred image load
-        synchronized (pendingRequests) {
-            Future f = Ion.with(imgView)
-                    .placeholder(defaultImageRes)
-                    .error(defaultImageRes)
-                    .load("https://" + address.getHostAddress() + ":47984/appasset?uniqueid=" + uniqueId + "&appid=" +
-                            obj.app.getAppId() + "&AssetType=2&AssetIdx=0")
-                    .setCallback(new FutureCallback<ImageView>() {
-                        @Override
-                        public void onCompleted(Exception e, ImageView result) {
-                            synchronized (pendingRequests) {
-                                pendingRequests.remove(imgView);
-                            }
-                        }
-                    });
-            pendingRequests.put(imgView, f);
+        // Start loading the bitmap
+        WeakReference<ImageView> viewRef = new WeakReference<>(imgView);
+        CachedAppAssetLoader.LoaderTuple tuple = loader.loadBitmapWithContext(obj.app, viewRef, imageViewLoadListener);
+        if (tuple != null) {
+            // The load was issued asynchronously
+            loadingTuples.put(viewRef, tuple);
         }
-
         return true;
     }
 
@@ -166,4 +222,8 @@ public class AppGridAdapter extends GenericGridAdapter<AppView.AppObject> {
         // No overlay
         return false;
     }
+
+    private static void fadeInImage(ImageView view) {
+        view.animate().alpha(1.0f).setDuration(100).start();
+    }
 }

app/src/main/java/com/limelight/grid/GenericGridAdapter.java

@@ -13,11 +13,11 @@ import com.limelight.R;
 import java.util.ArrayList;
 
 public abstract class GenericGridAdapter<T> extends BaseAdapter {
-    protected Context context;
-    protected int defaultImageRes;
-    protected int layoutId;
-    protected ArrayList<T> itemList = new ArrayList<T>();
-    protected LayoutInflater inflater;
+    protected final Context context;
+    protected final int defaultImageRes;
+    protected final int layoutId;
+    protected final ArrayList<T> itemList = new ArrayList<T>();
+    protected final LayoutInflater inflater;
 
     public GenericGridAdapter(Context context, int layoutId, int defaultImageRes) {
         this.context = context;
@@ -60,17 +60,21 @@ public abstract class GenericGridAdapter<T> extends BaseAdapter {
         ImageView overlayView = (ImageView) convertView.findViewById(R.id.grid_overlay);
         TextView txtView = (TextView) convertView.findViewById(R.id.grid_text);
 
-        if (!populateImageView(imgView, itemList.get(i))) {
-            imgView.setImageResource(defaultImageRes);
+        if (imgView != null) {
+            if (!populateImageView(imgView, itemList.get(i))) {
+                imgView.setImageResource(defaultImageRes);
+            }
         }
         if (!populateTextView(txtView, itemList.get(i))) {
             txtView.setText(itemList.get(i).toString());
         }
-        if (!populateOverlayView(overlayView, itemList.get(i))) {
-            overlayView.setVisibility(View.INVISIBLE);
-        }
-        else {
-            overlayView.setVisibility(View.VISIBLE);
+        if (overlayView != null) {
+            if (!populateOverlayView(overlayView, itemList.get(i))) {
+                overlayView.setVisibility(View.INVISIBLE);
+            }
+            else {
+                overlayView.setVisibility(View.VISIBLE);
+            }
         }
 
         return convertView;

app/src/main/java/com/limelight/grid/PcGridAdapter.java

@@ -8,14 +8,27 @@ import com.limelight.PcView;
 import com.limelight.R;
 import com.limelight.nvstream.http.ComputerDetails;
 
+import java.util.Collections;
+import java.util.Comparator;
+
 public class PcGridAdapter extends GenericGridAdapter<PcView.ComputerObject> {
 
-    public PcGridAdapter(Context context) {
-        super(context, R.layout.pc_grid_item, R.drawable.computer);
+    public PcGridAdapter(Context context, boolean listMode, boolean small) {
+        super(context, listMode ? R.layout.simple_row : (small ? R.layout.pc_grid_item_small : R.layout.pc_grid_item), R.drawable.computer);
     }
 
     public void addComputer(PcView.ComputerObject computer) {
         itemList.add(computer);
+        sortList();
+    }
+
+    private void sortList() {
+        Collections.sort(itemList, new Comparator<PcView.ComputerObject>() {
+            @Override
+            public int compare(PcView.ComputerObject lhs, PcView.ComputerObject rhs) {
+                return lhs.details.name.compareTo(rhs.details.name);
+            }
+        });
     }
 
     public boolean removeComputer(PcView.ComputerObject computer) {

app/src/main/java/com/limelight/grid/assets/CachedAppAssetLoader.java

@@ -0,0 +1,162 @@
+package com.limelight.grid.assets;
+
+import android.graphics.Bitmap;
+
+import com.limelight.nvstream.http.ComputerDetails;
+import com.limelight.nvstream.http.NvApp;
+
+import java.io.InputStream;
+import java.util.concurrent.LinkedBlockingQueue;
+import java.util.concurrent.ThreadPoolExecutor;
+import java.util.concurrent.TimeUnit;
+
+public class CachedAppAssetLoader {
+    private final ComputerDetails computer;
+    private final double scalingDivider;
+    private final ThreadPoolExecutor foregroundExecutor = new ThreadPoolExecutor(8, 8, Long.MAX_VALUE, TimeUnit.DAYS, new LinkedBlockingQueue<Runnable>());
+    private final ThreadPoolExecutor backgroundExecutor = new ThreadPoolExecutor(2, 2, Long.MAX_VALUE, TimeUnit.DAYS, new LinkedBlockingQueue<Runnable>());
+    private final NetworkAssetLoader networkLoader;
+    private final MemoryAssetLoader memoryLoader;
+    private final DiskAssetLoader diskLoader;
+
+    public CachedAppAssetLoader(ComputerDetails computer, double scalingDivider,
+                                NetworkAssetLoader networkLoader, MemoryAssetLoader memoryLoader,
+                                DiskAssetLoader diskLoader) {
+        this.computer = computer;
+        this.scalingDivider = scalingDivider;
+
+        this.networkLoader = networkLoader;
+        this.memoryLoader = memoryLoader;
+        this.diskLoader = diskLoader;
+    }
+
+    public void freeCacheMemory() {
+        memoryLoader.clearCache();
+    }
+
+    private Runnable createLoaderRunnable(final LoaderTuple tuple, final Object context, final LoadListener listener) {
+        return new Runnable() {
+            @Override
+            public void run() {
+                // Abort if we've been cancelled
+                if (tuple.cancelled) {
+                    return;
+                }
+
+                Bitmap bmp = diskLoader.loadBitmapFromCache(tuple, (int) scalingDivider);
+                if (bmp == null) {
+                    // Notify the listener that this may take a while
+                    listener.notifyLongLoad(context);
+
+                    // Try 5 times maximum
+                    for (int i = 0; i < 5; i++) {
+                        // Check again whether we've been cancelled
+                        if (tuple.cancelled) {
+                            return;
+                        }
+
+                        InputStream in = networkLoader.getBitmapStream(tuple);
+                        if (in != null) {
+                            // Write the stream straight to disk
+                            diskLoader.populateCacheWithStream(tuple, in);
+
+                            // Read it back scaled
+                            bmp = diskLoader.loadBitmapFromCache(tuple, (int) scalingDivider);
+                            if (bmp != null) {
+                                break;
+                            }
+                        }
+
+                        // Wait 1 second with a bit of fuzz
+                        try {
+                            Thread.sleep((int) (1000 + (Math.random() * 500)));
+                        } catch (InterruptedException e) {
+                            break;
+                        }
+                    }
+                }
+
+                if (bmp != null) {
+                    // Populate the memory cache
+                    memoryLoader.populateCache(tuple, bmp);
+                }
+
+                // Check one last time whether we've been cancelled
+                synchronized (tuple) {
+                    if (tuple.cancelled) {
+                        return;
+                    }
+                    else {
+                        tuple.notified = true;
+                    }
+                }
+
+                // Call the load complete callback (possible with a null bitmap)
+                listener.notifyLoadComplete(context, bmp);
+            }
+        };
+    }
+
+    public LoaderTuple loadBitmapWithContext(NvApp app, Object context, LoadListener listener) {
+        return loadBitmapWithContext(app, context, listener, false);
+    }
+
+    public LoaderTuple loadBitmapWithContextInBackground(NvApp app, Object context, LoadListener listener) {
+        return loadBitmapWithContext(app, context, listener, true);
+    }
+
+    private LoaderTuple loadBitmapWithContext(NvApp app, Object context, LoadListener listener, boolean background) {
+        LoaderTuple tuple = new LoaderTuple(computer, app);
+
+        // First, try the memory cache in the current context
+        Bitmap bmp = memoryLoader.loadBitmapFromCache(tuple);
+        if (bmp != null) {
+            // The caller never sees our tuple in this case
+            listener.notifyLoadComplete(context, bmp);
+            return null;
+        }
+
+        // If it's not in memory, throw this in our executor
+        if (background) {
+            backgroundExecutor.execute(createLoaderRunnable(tuple, context, listener));
+        }
+        else {
+            foregroundExecutor.execute(createLoaderRunnable(tuple, context, listener));
+        }
+        return tuple;
+    }
+
+    public class LoaderTuple {
+        public final ComputerDetails computer;
+        public final NvApp app;
+
+        public boolean notified;
+        public boolean cancelled;
+
+        public LoaderTuple(ComputerDetails computer, NvApp app) {
+            this.computer = computer;
+            this.app = app;
+        }
+
+        public boolean cancel() {
+            synchronized (this) {
+                cancelled = true;
+                return !notified;
+            }
+        }
+
+        @Override
+        public String toString() {
+            return "("+computer.uuid+", "+app.getAppId()+")";
+        }
+    }
+
+    public interface LoadListener {
+        // Notifies that the load didn't hit any cache and is about to be dispatched
+        // over the network
+        public void notifyLongLoad(Object context);
+
+        // Bitmap may be null if the load failed
+        public void notifyLoadComplete(Object context, Bitmap bitmap);
+    }
+}

app/src/main/java/com/limelight/grid/assets/DiskAssetLoader.java

@@ -0,0 +1,61 @@
+package com.limelight.grid.assets;
+
+import android.graphics.Bitmap;
+import android.graphics.BitmapFactory;
+
+import com.limelight.LimeLog;
+import com.limelight.utils.CacheHelper;
+
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.OutputStream;
+
+public class DiskAssetLoader {
+    private final File cacheDir;
+
+    public DiskAssetLoader(File cacheDir) {
+        this.cacheDir = cacheDir;
+    }
+
+    public Bitmap loadBitmapFromCache(CachedAppAssetLoader.LoaderTuple tuple, int sampleSize) {
+        InputStream in = null;
+        Bitmap bmp = null;
+        try {
+            in = CacheHelper.openCacheFileForInput(cacheDir, "boxart", tuple.computer.uuid.toString(), tuple.app.getAppId() + ".png");
+            BitmapFactory.Options options = new BitmapFactory.Options();
+            options.inSampleSize = sampleSize;
+            bmp = BitmapFactory.decodeStream(in, null, options);
+        } catch (IOException e) {
+            e.printStackTrace();
+        } finally {
+            if (in != null) {
+                try {
+                    in.close();
+                } catch (IOException ignored) {}
+            }
+        }
+
+        if (bmp != null) {
+            LimeLog.info("Disk cache hit for tuple: "+tuple);
+        }
+
+        return bmp;
+    }
+
+    public void populateCacheWithStream(CachedAppAssetLoader.LoaderTuple tuple, InputStream input) {
+        OutputStream out = null;
+        try {
+            out = CacheHelper.openCacheFileForOutput(cacheDir, "boxart", tuple.computer.uuid.toString(), tuple.app.getAppId() + ".png");
+            CacheHelper.writeInputStreamToOutputStream(input, out);
+        } catch (IOException e) {
+            e.printStackTrace();
+        } finally {
+            if (out != null) {
+                try {
+                    out.close();
+                } catch (IOException ignored) {}
+            }
+        }
+    }
+}

app/src/main/java/com/limelight/grid/assets/MemoryAssetLoader.java

@@ -0,0 +1,37 @@
+package com.limelight.grid.assets;
+
+import android.graphics.Bitmap;
+import android.util.LruCache;
+
+import com.limelight.LimeLog;
+
+public class MemoryAssetLoader {
+    private static final int maxMemory = (int) (Runtime.getRuntime().maxMemory() / 1024);
+    private static final LruCache<String, Bitmap> memoryCache = new LruCache<String, Bitmap>(maxMemory / 12) {
+        @Override
+        protected int sizeOf(String key, Bitmap bitmap) {
+            // Sizeof returns kilobytes
+            return bitmap.getByteCount() / 1024;
+        }
+    };
+
+    private static String constructKey(CachedAppAssetLoader.LoaderTuple tuple) {
+        return tuple.computer.uuid.toString()+"-"+tuple.app.getAppId();
+    }
+
+    public Bitmap loadBitmapFromCache(CachedAppAssetLoader.LoaderTuple tuple) {
+        Bitmap bmp = memoryCache.get(constructKey(tuple));
+        if (bmp != null) {
+            LimeLog.info("Memory cache hit for tuple: "+tuple);
+        }
+        return bmp;
+    }
+
+    public void populateCache(CachedAppAssetLoader.LoaderTuple tuple, Bitmap bitmap) {
+        memoryCache.put(constructKey(tuple), bitmap);
+    }
+
+    public void clearCache() {
+        memoryCache.evictAll();
+    }
+}

app/src/main/java/com/limelight/grid/assets/NetworkAssetLoader.java

@@ -0,0 +1,51 @@
+package com.limelight.grid.assets;
+
+import android.content.Context;
+
+import com.limelight.LimeLog;
+import com.limelight.binding.PlatformBinding;
+import com.limelight.nvstream.http.ComputerDetails;
+import com.limelight.nvstream.http.NvHTTP;
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.net.InetAddress;
+import java.security.KeyManagementException;
+import java.security.NoSuchAlgorithmException;
+
+public class NetworkAssetLoader {
+    private final Context context;
+    private final String uniqueId;
+
+    public NetworkAssetLoader(Context context, String uniqueId) {
+        this.context = context;
+        this.uniqueId = uniqueId;
+    }
+
+    public InputStream getBitmapStream(CachedAppAssetLoader.LoaderTuple tuple) {
+        NvHTTP http = new NvHTTP(getCurrentAddress(tuple.computer), uniqueId, null, PlatformBinding.getCryptoProvider(context));
+
+        InputStream in = null;
+        try {
+            in = http.getBoxArt(tuple.app);
+        } catch (IOException e) {}
+
+        if (in != null) {
+            LimeLog.info("Network asset load complete: " + tuple);
+        }
+        else {
+            LimeLog.info("Network asset load failed: " + tuple);
+        }
+
+        return in;
+    }
+
+    private static InetAddress getCurrentAddress(ComputerDetails computer) {
+        if (computer.reachability == ComputerDetails.Reachability.LOCAL) {
+            return computer.localIp;
+        }
+        else {
+            return computer.remoteIp;
+        }
+    }
+}

app/src/main/java/com/limelight/nvstream/av/video/cpu/AvcDecoder.java

@@ -1,44 +1,44 @@
 package com.limelight.nvstream.av.video.cpu;
 
 public class AvcDecoder {
-	static {
-		// FFMPEG dependencies
-		System.loadLibrary("avutil-52");
-		System.loadLibrary("swresample-0");
-		System.loadLibrary("swscale-2");
-		System.loadLibrary("avcodec-55");
-		System.loadLibrary("avformat-55");
+    static {
+        // FFMPEG dependencies
+        System.loadLibrary("avutil-52");
+        System.loadLibrary("swresample-0");
+        System.loadLibrary("swscale-2");
+        System.loadLibrary("avcodec-55");
+        System.loadLibrary("avformat-55");
 
-		System.loadLibrary("nv_avc_dec");
-	}
+        System.loadLibrary("nv_avc_dec");
+    }
 
-	/** Disables the deblocking filter at the cost of image quality */
-	public static final int DISABLE_LOOP_FILTER = 0x1;
-	/** Uses the low latency decode flag (disables multithreading) */
-	public static final int LOW_LATENCY_DECODE = 0x2;
-	/** Threads process each slice, rather than each frame */
-	public static final int SLICE_THREADING = 0x4;
-	/** Uses nonstandard speedup tricks */
-	public static final int FAST_DECODE = 0x8;
-	/** Uses bilinear filtering instead of bicubic */
-	public static final int BILINEAR_FILTERING = 0x10;
-	/** Uses a faster bilinear filtering with lower image quality */
-	public static final int FAST_BILINEAR_FILTERING = 0x20;
-	/** Disables color conversion (output is NV21) */
-	public static final int NO_COLOR_CONVERSION = 0x40;
+    /** Disables the deblocking filter at the cost of image quality */
+    public static final int DISABLE_LOOP_FILTER = 0x1;
+    /** Uses the low latency decode flag (disables multithreading) */
+    public static final int LOW_LATENCY_DECODE = 0x2;
+    /** Threads process each slice, rather than each frame */
+    public static final int SLICE_THREADING = 0x4;
+    /** Uses nonstandard speedup tricks */
+    public static final int FAST_DECODE = 0x8;
+    /** Uses bilinear filtering instead of bicubic */
+    public static final int BILINEAR_FILTERING = 0x10;
+    /** Uses a faster bilinear filtering with lower image quality */
+    public static final int FAST_BILINEAR_FILTERING = 0x20;
+    /** Disables color conversion (output is NV21) */
+    public static final int NO_COLOR_CONVERSION = 0x40;
 
-	public static native int init(int width, int height, int perflvl, int threadcount);
-	public static native void destroy();
+    public static native int init(int width, int height, int perflvl, int threadcount);
+    public static native void destroy();
 
-	// Rendering API when NO_COLOR_CONVERSION == 0
-	public static native boolean setRenderTarget(Object androidSurface);
-	public static native boolean getRgbFrameInt(int[] rgbFrame, int bufferSize);
-	public static native boolean getRgbFrame(byte[] rgbFrame, int bufferSize);
-	public static native boolean redraw();
+    // Rendering API when NO_COLOR_CONVERSION == 0
+    public static native boolean setRenderTarget(Object androidSurface);
+    public static native boolean getRgbFrameInt(int[] rgbFrame, int bufferSize);
+    public static native boolean getRgbFrame(byte[] rgbFrame, int bufferSize);
+    public static native boolean redraw();
 
-	// Rendering API when NO_COLOR_CONVERSION == 1
-	public static native boolean getRawFrame(byte[] yuvFrame, int bufferSize);
+    // Rendering API when NO_COLOR_CONVERSION == 1
+    public static native boolean getRawFrame(byte[] yuvFrame, int bufferSize);
 
-	public static native int getInputPaddingSize();
-	public static native int decode(byte[] indata, int inoff, int inlen);
+    public static native int getInputPaddingSize();
+    public static native int decode(byte[] indata, int inoff, int inlen);
 }

app/src/main/java/com/limelight/preferences/AddComputerManually.java

@@ -2,6 +2,7 @@ package com.limelight.preferences;
 
 import java.net.InetAddress;
 import java.net.UnknownHostException;
+import java.util.Locale;
 import java.util.concurrent.LinkedBlockingQueue;
 
 import com.limelight.computers.ComputerManagerService;
@@ -13,151 +14,168 @@ import com.limelight.utils.UiHelper;
 import android.app.Activity;
 import android.app.Service;
 import android.content.ComponentName;
+import android.content.Context;
 import android.content.Intent;
 import android.content.ServiceConnection;
+import android.content.res.Configuration;
 import android.os.Bundle;
 import android.os.IBinder;
 import android.view.KeyEvent;
 import android.view.inputmethod.EditorInfo;
+import android.view.inputmethod.InputMethodManager;
 import android.widget.TextView;
 import android.widget.Toast;
 
 public class AddComputerManually extends Activity {
-	private TextView hostText;
-	private ComputerManagerService.ComputerManagerBinder managerBinder;
-	private LinkedBlockingQueue<String> computersToAdd = new LinkedBlockingQueue<String>();
-	private Thread addThread;
-	private ServiceConnection serviceConnection = new ServiceConnection() {
-		public void onServiceConnected(ComponentName className, final IBinder binder) {
-			managerBinder = ((ComputerManagerService.ComputerManagerBinder)binder);
-			startAddThread();
-		}
+    private TextView hostText;
+    private ComputerManagerService.ComputerManagerBinder managerBinder;
+    private final LinkedBlockingQueue<String> computersToAdd = new LinkedBlockingQueue<String>();
+    private Thread addThread;
+    private final ServiceConnection serviceConnection = new ServiceConnection() {
+        public void onServiceConnected(ComponentName className, final IBinder binder) {
+            managerBinder = ((ComputerManagerService.ComputerManagerBinder)binder);
+            startAddThread();
+        }
 
-		public void onServiceDisconnected(ComponentName className) {
-			joinAddThread();
-			managerBinder = null;
-		}
-	};
+        public void onServiceDisconnected(ComponentName className) {
+            joinAddThread();
+            managerBinder = null;
+        }
+    };
 
-	private void doAddPc(String host) {
-		String msg;
-		boolean finish = false;
+    private void doAddPc(String host) {
+        String msg;
+        boolean finish = false;
 
         SpinnerDialog dialog = SpinnerDialog.displayDialog(this, getResources().getString(R.string.title_add_pc),
-			getResources().getString(R.string.msg_add_pc), false);
+            getResources().getString(R.string.msg_add_pc), false);
 
-		try {
-			InetAddress addr = InetAddress.getByName(host);
+        try {
+            InetAddress addr = InetAddress.getByName(host);
 
-			if (!managerBinder.addComputerBlocking(addr)){
-				msg = getResources().getString(R.string.addpc_fail);
-			}
-			else {
-				msg = getResources().getString(R.string.addpc_success);
-				finish = true;
-			}
-		} catch (UnknownHostException e) {
-			msg = getResources().getString(R.string.addpc_unknown_host);
-		}
+            if (!managerBinder.addComputerBlocking(addr)){
+                msg = getResources().getString(R.string.addpc_fail);
+            }
+            else {
+                msg = getResources().getString(R.string.addpc_success);
+                finish = true;
+            }
+        } catch (UnknownHostException e) {
+            msg = getResources().getString(R.string.addpc_unknown_host);
+        }
 
         dialog.dismiss();
 
         final boolean toastFinish = finish;
-		final String toastMsg = msg;
-		AddComputerManually.this.runOnUiThread(new Runnable() {
-			@Override
-			public void run() {
-				Toast.makeText(AddComputerManually.this, toastMsg, Toast.LENGTH_LONG).show();
+        final String toastMsg = msg;
+        AddComputerManually.this.runOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                Toast.makeText(AddComputerManually.this, toastMsg, Toast.LENGTH_LONG).show();
 
-				if (toastFinish && !isFinishing()) {
-					// Close the activity
-					AddComputerManually.this.finish();
-				}
-			}
-		});
-	}
+                if (toastFinish && !isFinishing()) {
+                    // Close the activity
+                    AddComputerManually.this.finish();
+                }
+            }
+        });
+    }
 
-	private void startAddThread() {
-		addThread = new Thread() {
-			@Override
-			public void run() {
-				while (!isInterrupted()) {
-					String computer;
+    private void startAddThread() {
+        addThread = new Thread() {
+            @Override
+            public void run() {
+                while (!isInterrupted()) {
+                    String computer;
 
-					try {
-						computer = computersToAdd.take();
-					} catch (InterruptedException e) {
-						return;
-					}
+                    try {
+                        computer = computersToAdd.take();
+                    } catch (InterruptedException e) {
+                        return;
+                    }
 
-					doAddPc(computer);
-				}
-			}
-		};
-		addThread.setName("UI - AddComputerManually");
-		addThread.start();
-	}
+                    doAddPc(computer);
+                }
+            }
+        };
+        addThread.setName("UI - AddComputerManually");
+        addThread.start();
+    }
 
-	private void joinAddThread() {
-		if (addThread != null) {
-			addThread.interrupt();
+    private void joinAddThread() {
+        if (addThread != null) {
+            addThread.interrupt();
 
-			try {
-				addThread.join();
-			} catch (InterruptedException ignored) {}
+            try {
+                addThread.join();
+            } catch (InterruptedException ignored) {}
 
-			addThread = null;
-		}
-	}
+            addThread = null;
+        }
+    }
 
-	@Override
-	protected void onStop() {
-		super.onStop();
+    @Override
+    protected void onStop() {
+        super.onStop();
 
-		Dialog.closeDialogs();
+        Dialog.closeDialogs();
         SpinnerDialog.closeDialogs(this);
-	}
+    }
 
-	@Override
-	protected void onDestroy() {
-		super.onDestroy();
+    @Override
+    protected void onDestroy() {
+        super.onDestroy();
 
-		if (managerBinder != null) {
-			joinAddThread();
-			unbindService(serviceConnection);
-		}
-	}
+        if (managerBinder != null) {
+            joinAddThread();
+            unbindService(serviceConnection);
+        }
+    }
 
-	@Override
-	protected void onCreate(Bundle savedInstanceState) {
-		super.onCreate(savedInstanceState);
+    @Override
+    protected void onCreate(Bundle savedInstanceState) {
+        super.onCreate(savedInstanceState);
 
-		setContentView(R.layout.activity_add_computer_manually);
+        String locale = PreferenceConfiguration.readPreferences(this).language;
+        if (!locale.equals(PreferenceConfiguration.DEFAULT_LANGUAGE)) {
+            Configuration config = new Configuration(getResources().getConfiguration());
+            config.locale = new Locale(locale);
+            getResources().updateConfiguration(config, getResources().getDisplayMetrics());
+        }
+
+        setContentView(R.layout.activity_add_computer_manually);
 
         UiHelper.notifyNewRootView(this);
 
-		this.hostText = (TextView) findViewById(R.id.hostTextView);
+        this.hostText = (TextView) findViewById(R.id.hostTextView);
         hostText.setImeOptions(EditorInfo.IME_ACTION_DONE);
         hostText.setOnEditorActionListener(new TextView.OnEditorActionListener() {
             @Override
             public boolean onEditorAction(TextView textView, int actionId, KeyEvent keyEvent) {
                 if (actionId == EditorInfo.IME_ACTION_DONE ||
-                        keyEvent.getAction() == KeyEvent.ACTION_DOWN &&
-                                keyEvent.getKeyCode() == KeyEvent.KEYCODE_ENTER) {
+                        (keyEvent != null &&
+                                keyEvent.getAction() == KeyEvent.ACTION_DOWN &&
+                                keyEvent.getKeyCode() == KeyEvent.KEYCODE_ENTER)) {
                     if (hostText.getText().length() == 0) {
                         Toast.makeText(AddComputerManually.this, getResources().getString(R.string.addpc_enter_ip), Toast.LENGTH_LONG).show();
                         return true;
                     }
 
-                    computersToAdd.add(hostText.getText().toString());
+                    computersToAdd.add(hostText.getText().toString().trim());
+                }
+                else if (actionId == EditorInfo.IME_ACTION_PREVIOUS) {
+                    // This is how the Fire TV dismisses the keyboard
+                    InputMethodManager imm = (InputMethodManager)getSystemService(Context.INPUT_METHOD_SERVICE);
+                    imm.hideSoftInputFromWindow(hostText.getWindowToken(), 0);
+                    return false;
                 }
 
                 return false;
             }
         });
 
-		// Bind to the ComputerManager service
-		bindService(new Intent(AddComputerManually.this,
-					ComputerManagerService.class), serviceConnection, Service.BIND_AUTO_CREATE);
-	}
+        // Bind to the ComputerManager service
+        bindService(new Intent(AddComputerManually.this,
+                    ComputerManagerService.class), serviceConnection, Service.BIND_AUTO_CREATE);
+    }
 }

app/src/main/java/com/limelight/preferences/PreferenceConfiguration.java

@@ -2,6 +2,7 @@ package com.limelight.preferences;
 
 import android.content.Context;
 import android.content.SharedPreferences;
+import android.content.pm.PackageManager;
 import android.preference.PreferenceManager;
 
 public class PreferenceConfiguration {
@@ -13,11 +14,15 @@ public class PreferenceConfiguration {
     private static final String DISABLE_TOASTS_PREF_STRING = "checkbox_disable_warnings";
     private static final String HOST_AUDIO_PREF_STRING = "checkbox_host_audio";
     private static final String DEADZONE_PREF_STRING = "seekbar_deadzone";
+    private static final String LANGUAGE_PREF_STRING = "list_languages";
+    private static final String LIST_MODE_PREF_STRING = "checkbox_list_mode";
+    private static final String SMALL_ICONS_PREF_STRING = "checkbox_small_icon_mode";
+    private static final String MULTI_CONTROLLER_PREF_STRING = "checkbox_multi_controller";
 
     private static final int BITRATE_DEFAULT_720_30 = 5;
     private static final int BITRATE_DEFAULT_720_60 = 10;
     private static final int BITRATE_DEFAULT_1080_30 = 10;
-    private static final int BITRATE_DEFAULT_1080_60 = 30;
+    private static final int BITRATE_DEFAULT_1080_60 = 20;
 
     private static final String DEFAULT_RES_FPS = "720p60";
     private static final String DEFAULT_DECODER = "auto";
@@ -27,6 +32,9 @@ public class PreferenceConfiguration {
     private static final boolean DEFAULT_DISABLE_TOASTS = false;
     private static final boolean DEFAULT_HOST_AUDIO = false;
     private static final int DEFAULT_DEADZONE = 15;
+    public static final String DEFAULT_LANGUAGE = "default";
+    private static final boolean DEFAULT_LIST_MODE = false;
+    private static final boolean DEFAULT_MULTI_CONTROLLER = true;
 
     public static final int FORCE_HARDWARE_DECODER = -1;
     public static final int AUTOSELECT_DECODER = 0;
@@ -37,6 +45,8 @@ public class PreferenceConfiguration {
     public int decoder;
     public int deadzonePercentage;
     public boolean stretchVideo, enableSops, playHostAudio, disableWarnings;
+    public String language;
+    public boolean listMode, smallIconMode, multiController;
 
     public static int getDefaultBitrate(String resFpsString) {
         if (resFpsString.equals("720p30")) {
@@ -57,6 +67,17 @@ public class PreferenceConfiguration {
         }
     }
 
+    public static boolean getDefaultSmallMode(Context context) {
+        PackageManager manager = context.getPackageManager();
+        if (manager != null && manager.hasSystemFeature(PackageManager.FEATURE_TELEVISION)) {
+            // TVs shouldn't use small mode by default
+            return false;
+        }
+
+        // Use small mode on anything smaller than a 7" tablet
+        return context.getResources().getConfiguration().smallestScreenWidthDp < 600;
+    }
+
     public static int getDefaultBitrate(Context context) {
         SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(context);
 
@@ -135,11 +156,16 @@ public class PreferenceConfiguration {
 
         config.deadzonePercentage = prefs.getInt(DEADZONE_PREF_STRING, DEFAULT_DEADZONE);
 
+        config.language = prefs.getString(LANGUAGE_PREF_STRING, DEFAULT_LANGUAGE);
+
         // Checkbox preferences
         config.disableWarnings = prefs.getBoolean(DISABLE_TOASTS_PREF_STRING, DEFAULT_DISABLE_TOASTS);
         config.enableSops = prefs.getBoolean(SOPS_PREF_STRING, DEFAULT_SOPS);
         config.stretchVideo = prefs.getBoolean(STRETCH_PREF_STRING, DEFAULT_STRETCH);
         config.playHostAudio = prefs.getBoolean(HOST_AUDIO_PREF_STRING, DEFAULT_HOST_AUDIO);
+        config.listMode = prefs.getBoolean(LIST_MODE_PREF_STRING, DEFAULT_LIST_MODE);
+        config.smallIconMode = prefs.getBoolean(SMALL_ICONS_PREF_STRING, getDefaultSmallMode(context));
+        config.multiController = prefs.getBoolean(MULTI_CONTROLLER_PREF_STRING, DEFAULT_MULTI_CONTROLLER);
 
         return config;
     }

app/src/main/java/com/limelight/preferences/SeekBarPreference.java

@@ -20,10 +20,14 @@ public class SeekBarPreference extends DialogPreference
 
     private SeekBar seekBar;
     private TextView valueText;
-    private Context context;
+    private final Context context;
 
-    private String dialogMessage, suffix;
-    private int defaultValue, maxValue, minValue, currentValue;
+    private final String dialogMessage;
+    private final String suffix;
+    private final int defaultValue;
+    private final int maxValue;
+    private final int minValue;
+    private int currentValue;
 
     public SeekBarPreference(Context context, AttributeSet attrs) {
         super(context, attrs);
@@ -127,13 +131,6 @@ public class SeekBarPreference extends DialogPreference
         }
     }
 
-    public void setMax(int max) {
-        this.maxValue = max;
-    }
-    public int getMax() {
-        return this.maxValue;
-    }
-
     public void setProgress(int progress) {
         this.currentValue = progress;
         if (seekBar != null) {

app/src/main/java/com/limelight/preferences/SmallIconCheckboxPreference.java

@@ -0,0 +1,21 @@
+package com.limelight.preferences;
+
+import android.content.Context;
+import android.content.res.TypedArray;
+import android.preference.CheckBoxPreference;
+import android.util.AttributeSet;
+
+public class SmallIconCheckboxPreference extends CheckBoxPreference {
+    public SmallIconCheckboxPreference(Context context, AttributeSet attrs, int defStyleAttr) {
+        super(context, attrs, defStyleAttr);
+    }
+
+    public SmallIconCheckboxPreference(Context context, AttributeSet attrs) {
+        super(context, attrs);
+    }
+
+    @Override
+    protected Object onGetDefaultValue(TypedArray a, int index) {
+        return PreferenceConfiguration.getDefaultSmallMode(getContext());
+    }
+}

app/src/main/java/com/limelight/preferences/StreamSettings.java

@@ -1,20 +1,32 @@
 package com.limelight.preferences;
 
+import android.content.Intent;
 import android.content.SharedPreferences;
+import android.content.res.Configuration;
 import android.os.Bundle;
 import android.app.Activity;
 import android.preference.Preference;
 import android.preference.PreferenceFragment;
 import android.preference.PreferenceManager;
 
+import com.limelight.PcView;
 import com.limelight.R;
 import com.limelight.utils.UiHelper;
 
+import java.util.Locale;
+
 public class StreamSettings extends Activity {
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
+        String locale = PreferenceConfiguration.readPreferences(this).language;
+        if (!locale.equals(PreferenceConfiguration.DEFAULT_LANGUAGE)) {
+            Configuration config = new Configuration(getResources().getConfiguration());
+            config.locale = new Locale(locale);
+            getResources().updateConfiguration(config, getResources().getDisplayMetrics());
+        }
+
         setContentView(R.layout.activity_stream_settings);
         getFragmentManager().beginTransaction().replace(
                 R.id.stream_settings, new SettingsFragment()
@@ -23,6 +35,16 @@ public class StreamSettings extends Activity {
         UiHelper.notifyNewRootView(this);
     }
 
+    @Override
+    public void onBackPressed() {
+        finish();
+
+        // Restart the PC view to apply UI changes
+        Intent intent = new Intent(this, PcView.class);
+        intent.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TASK | Intent.FLAG_ACTIVITY_NEW_TASK);
+        startActivity(intent, null);
+    }
+
     public static class SettingsFragment extends PreferenceFragment {
         @Override
         public void onCreate(Bundle savedInstanceState) {

app/src/main/java/com/limelight/ui/AdapterFragment.java

@@ -0,0 +1,35 @@
+package com.limelight.ui;
+
+
+import android.app.Activity;
+import android.app.Fragment;
+import android.os.Bundle;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.view.ViewGroup;
+import android.widget.AbsListView;
+
+import com.limelight.R;
+
+public class AdapterFragment extends Fragment {
+    private AdapterFragmentCallbacks callbacks;
+
+    @Override
+    public void onAttach(Activity activity) {
+        super.onAttach(activity);
+
+        callbacks = (AdapterFragmentCallbacks) activity;
+    }
+
+    @Override
+    public View onCreateView(LayoutInflater inflater, ViewGroup container,
+                             Bundle savedInstanceState) {
+        return inflater.inflate(callbacks.getAdapterFragmentLayoutId(), container, false);
+    }
+
+    @Override
+    public void onActivityCreated(Bundle savedInstanceState) {
+        super.onActivityCreated(savedInstanceState);
+        callbacks.receiveAbsListView((AbsListView) getView().findViewById(R.id.fragmentView));
+    }
+}

app/src/main/java/com/limelight/ui/AdapterFragmentCallbacks.java

@@ -0,0 +1,8 @@
+package com.limelight.ui;
+
+import android.widget.AbsListView;
+
+public interface AdapterFragmentCallbacks {
+    public int getAdapterFragmentLayoutId();
+    public void receiveAbsListView(AbsListView gridView);
+}

app/src/main/java/com/limelight/ui/GameGestures.java

@@ -0,0 +1,5 @@
+package com.limelight.ui;
+
+public interface GameGestures {
+    public void showKeyboard();
+}

app/src/main/java/com/limelight/utils/CacheHelper.java

@@ -0,0 +1,66 @@
+package com.limelight.utils;
+
+import java.io.BufferedInputStream;
+import java.io.BufferedOutputStream;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.FileNotFoundException;
+import java.io.FileOutputStream;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
+import java.io.OutputStream;
+import java.io.Reader;
+
+public class CacheHelper {
+    private static File openPath(boolean createPath, File root, String... path) {
+        File f = root;
+        for (int i = 0; i < path.length; i++) {
+            String component = path[i];
+
+            if (i == path.length - 1) {
+                // This is the file component so now we create parent directories
+                if (createPath) {
+                    f.mkdirs();
+                }
+            }
+
+            f = new File(f, component);
+        }
+        return f;
+    }
+
+    public static InputStream openCacheFileForInput(File root, String... path) throws FileNotFoundException {
+        return new BufferedInputStream(new FileInputStream(openPath(false, root, path)));
+    }
+
+    public static OutputStream openCacheFileForOutput(File root, String... path) throws FileNotFoundException {
+        return new BufferedOutputStream(new FileOutputStream(openPath(true, root, path)));
+    }
+
+    public static void writeInputStreamToOutputStream(InputStream in, OutputStream out) throws IOException {
+        byte[] buf = new byte[4096];
+        int bytesRead;
+
+        while ((bytesRead = in.read(buf)) != -1) {
+            out.write(buf, 0, bytesRead);
+        }
+    }
+
+    public static String readInputStreamToString(InputStream in) throws IOException {
+        Reader r = new InputStreamReader(in);
+
+        StringBuilder sb = new StringBuilder();
+        char[] buf = new char[256];
+        int bytesRead;
+        while ((bytesRead = r.read(buf)) != -1) {
+            sb.append(buf, 0, bytesRead);
+        }
+
+        return sb.toString();
+    }
+
+    public static void writeStringToOutputStream(OutputStream out, String str) throws IOException {
+        out.write(str.getBytes("UTF-8"));
+    }
+}

app/src/main/java/com/limelight/utils/Dialog.java

@@ -7,70 +7,71 @@ import android.app.AlertDialog;
 import android.content.DialogInterface;
 
 public class Dialog implements Runnable {
-	private String title, message;
-	private Activity activity;
-	private boolean endAfterDismiss;
+    private final String title;
+    private final String message;
+    private final Activity activity;
+    private final boolean endAfterDismiss;
 
-	private AlertDialog alert;
+    private AlertDialog alert;
 
-	private static final ArrayList<Dialog> rundownDialogs = new ArrayList<Dialog>();
+    private static final ArrayList<Dialog> rundownDialogs = new ArrayList<Dialog>();
 
-	public Dialog(Activity activity, String title, String message, boolean endAfterDismiss)
-	{
-		this.activity = activity;
-		this.title = title;
-		this.message = message;
-		this.endAfterDismiss = endAfterDismiss;
-	}
+    private Dialog(Activity activity, String title, String message, boolean endAfterDismiss)
+    {
+        this.activity = activity;
+        this.title = title;
+        this.message = message;
+        this.endAfterDismiss = endAfterDismiss;
+    }
 
-	public static void closeDialogs()
-	{
-		synchronized (rundownDialogs) {
-			for (Dialog d : rundownDialogs) {
-				if (d.alert.isShowing()) {
-					d.alert.dismiss();
-				}
-			}
+    public static void closeDialogs()
+    {
+        synchronized (rundownDialogs) {
+            for (Dialog d : rundownDialogs) {
+                if (d.alert.isShowing()) {
+                    d.alert.dismiss();
+                }
+            }
 
-			rundownDialogs.clear();
-		}
-	}
+            rundownDialogs.clear();
+        }
+    }
 
-	public static void displayDialog(Activity activity, String title, String message, boolean endAfterDismiss)
-	{
-		activity.runOnUiThread(new Dialog(activity, title, message, endAfterDismiss));
-	}
+    public static void displayDialog(Activity activity, String title, String message, boolean endAfterDismiss)
+    {
+        activity.runOnUiThread(new Dialog(activity, title, message, endAfterDismiss));
+    }
 
-	@Override
-	public void run() {
-		// If we're dying, don't bother creating a dialog
-		if (activity.isFinishing())
-			return;
+    @Override
+    public void run() {
+        // If we're dying, don't bother creating a dialog
+        if (activity.isFinishing())
+            return;
 
-		alert = new AlertDialog.Builder(activity).create();
+        alert = new AlertDialog.Builder(activity).create();
 
-    	alert.setTitle(title);
-    	alert.setMessage(message);
-    	alert.setCancelable(false);
-    	alert.setCanceledOnTouchOutside(false);
+        alert.setTitle(title);
+        alert.setMessage(message);
+        alert.setCancelable(false);
+        alert.setCanceledOnTouchOutside(false);
  
-    	alert.setButton(AlertDialog.BUTTON_NEUTRAL, "OK", new DialogInterface.OnClickListener() {
-    	      public void onClick(DialogInterface dialog, int which) {
-    	    	  synchronized (rundownDialogs) {
-        	    	  rundownDialogs.remove(Dialog.this);
-        	    	  alert.dismiss();
-    	    	  }
+        alert.setButton(AlertDialog.BUTTON_NEUTRAL, "OK", new DialogInterface.OnClickListener() {
+              public void onClick(DialogInterface dialog, int which) {
+                  synchronized (rundownDialogs) {
+                      rundownDialogs.remove(Dialog.this);
+                      alert.dismiss();
+                  }
 
-    	    	  if (endAfterDismiss) {
-    	    		  activity.finish();
-    	    	  }
-    	      }
-    	});
+                  if (endAfterDismiss) {
+                      activity.finish();
+                  }
+              }
+        });
 
-    	synchronized (rundownDialogs) {
-        	rundownDialogs.add(this);
-        	alert.show();	
-    	}
-	}
+        synchronized (rundownDialogs) {
+            rundownDialogs.add(this);
+            alert.show();
+        }
+    }
 
 }

app/src/main/java/com/limelight/utils/SpinnerDialog.java

@@ -9,111 +9,112 @@ import android.content.DialogInterface;
 import android.content.DialogInterface.OnCancelListener;
 
 public class SpinnerDialog implements Runnable,OnCancelListener {
-	private String title, message;
-	private Activity activity;
-	private ProgressDialog progress;
-	private boolean finish;
+    private final String title;
+    private final String message;
+    private final Activity activity;
+    private ProgressDialog progress;
+    private final boolean finish;
 
-	private static final ArrayList<SpinnerDialog> rundownDialogs = new ArrayList<SpinnerDialog>();
+    private static final ArrayList<SpinnerDialog> rundownDialogs = new ArrayList<SpinnerDialog>();
 
-	public SpinnerDialog(Activity activity, String title, String message, boolean finish)
-	{
-		this.activity = activity;
-		this.title = title;
-		this.message = message;
-		this.progress = null;
-		this.finish = finish;
-	}
+    private SpinnerDialog(Activity activity, String title, String message, boolean finish)
+    {
+        this.activity = activity;
+        this.title = title;
+        this.message = message;
+        this.progress = null;
+        this.finish = finish;
+    }
 
-	public static SpinnerDialog displayDialog(Activity activity, String title, String message, boolean finish)
-	{
-		SpinnerDialog spinner = new SpinnerDialog(activity, title, message, finish);
-		activity.runOnUiThread(spinner);
-		return spinner;
-	}
+    public static SpinnerDialog displayDialog(Activity activity, String title, String message, boolean finish)
+    {
+        SpinnerDialog spinner = new SpinnerDialog(activity, title, message, finish);
+        activity.runOnUiThread(spinner);
+        return spinner;
+    }
 
-	public static void closeDialogs(Activity activity)
-	{
-		synchronized (rundownDialogs) {
-			Iterator<SpinnerDialog> i = rundownDialogs.iterator();
-			while (i.hasNext()) {
-				SpinnerDialog dialog = i.next();
-				if (dialog.activity == activity) {
-					i.remove();
-					if (dialog.progress.isShowing()) {
-						dialog.progress.dismiss();
-					}
-				}
-			}
-		}
-	}
+    public static void closeDialogs(Activity activity)
+    {
+        synchronized (rundownDialogs) {
+            Iterator<SpinnerDialog> i = rundownDialogs.iterator();
+            while (i.hasNext()) {
+                SpinnerDialog dialog = i.next();
+                if (dialog.activity == activity) {
+                    i.remove();
+                    if (dialog.progress.isShowing()) {
+                        dialog.progress.dismiss();
+                    }
+                }
+            }
+        }
+    }
 
-	public void dismiss()
-	{
-		// Running again with progress != null will destroy it
-		activity.runOnUiThread(this);
-	}
+    public void dismiss()
+    {
+        // Running again with progress != null will destroy it
+        activity.runOnUiThread(this);
+    }
 
-	public void setMessage(final String message)
-	{
-		activity.runOnUiThread(new Runnable() {
-			@Override
-			public void run() {
-				progress.setMessage(message);
-			}
-		});
-	}
+    public void setMessage(final String message)
+    {
+        activity.runOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                progress.setMessage(message);
+            }
+        });
+    }
 
-	@Override
-	public void run() {
+    @Override
+    public void run() {
 
-		// If we're dying, don't bother doing anything
-		if (activity.isFinishing()) {
-			return;
-		}
+        // If we're dying, don't bother doing anything
+        if (activity.isFinishing()) {
+            return;
+        }
 
-		if (progress == null)
-		{
-	    	progress = new ProgressDialog(activity);
+        if (progress == null)
+        {
+            progress = new ProgressDialog(activity);
 
-	    	progress.setTitle(title);
-	    	progress.setMessage(message);
-	    	progress.setProgressStyle(ProgressDialog.STYLE_SPINNER);
-	    	progress.setOnCancelListener(this);
+            progress.setTitle(title);
+            progress.setMessage(message);
+            progress.setProgressStyle(ProgressDialog.STYLE_SPINNER);
+            progress.setOnCancelListener(this);
 
-	    	// If we want to finish the activity when this is killed, make it cancellable
-	    	if (finish)
-	    	{
-	    		progress.setCancelable(true);
-	    		progress.setCanceledOnTouchOutside(false);
-	    	}
-	    	else
-	    	{
-	    		progress.setCancelable(false);
-	    	}
+            // If we want to finish the activity when this is killed, make it cancellable
+            if (finish)
+            {
+                progress.setCancelable(true);
+                progress.setCanceledOnTouchOutside(false);
+            }
+            else
+            {
+                progress.setCancelable(false);
+            }
 
-	    	synchronized (rundownDialogs) {
-		    	rundownDialogs.add(this);
-		    	progress.show();
-	    	}
-		}
-		else
-		{
-			synchronized (rundownDialogs) {
-				if (rundownDialogs.remove(this) && progress.isShowing()) {
-					progress.dismiss();
-				}
-			}
-		}
-	}
+            synchronized (rundownDialogs) {
+                rundownDialogs.add(this);
+                progress.show();
+            }
+        }
+        else
+        {
+            synchronized (rundownDialogs) {
+                if (rundownDialogs.remove(this) && progress.isShowing()) {
+                    progress.dismiss();
+                }
+            }
+        }
+    }
 
-	@Override
-	public void onCancel(DialogInterface dialog) {
-		synchronized (rundownDialogs) {
-			rundownDialogs.remove(this);
-		}
+    @Override
+    public void onCancel(DialogInterface dialog) {
+        synchronized (rundownDialogs) {
+            rundownDialogs.remove(this);
+        }
 
-		// This will only be called if finish was true, so we don't need to check again
-		activity.finish();
-	}
+        // This will only be called if finish was true, so we don't need to check again
+        activity.finish();
+    }
 }

app/src/main/jni/nv_avc_dec/nv_avc_dec.c

@@ -69,9 +69,6 @@ int nv_avc_init(int width, int height, int perf_lvl, int thread_count) {
 		return -1;
 	}
 
-	// Show frames even before a reference frame
-	decoder_ctx->flags2 |= CODEC_FLAG2_SHOW_ALL;
-
 	if (perf_lvl & DISABLE_LOOP_FILTER) {
 		// Skip the loop filter for performance reasons
 		decoder_ctx->skip_loop_filter = AVDISCARD_ALL;
@@ -370,17 +367,20 @@ int nv_avc_decode(unsigned char* indata, int inlen) {
 	
 	// Only copy the picture at the end of decoding the packet
 	if (got_pic) {
+	    // Clone the current decode frame outside of the mutex
+	    AVFrame* new_frame = av_frame_clone(dec_frame);
+	    AVFrame* old_frame;
+
+	    // Swap it in under lock
 		pthread_mutex_lock(&mutex);
-
-		// Only clone this frame if the last frame was taken.
-		// This saves on extra copies for frames that don't get
-		// rendered.
-		if (yuv_frame == NULL) {
-			// Clone a new frame
-			yuv_frame = av_frame_clone(dec_frame);
-		}
-
+		old_frame = yuv_frame;
+		yuv_frame = new_frame;
 		pthread_mutex_unlock(&mutex);
+
+        // Free the old frame outside of the mutex
+        if (old_frame != NULL) {
+            av_frame_free(&old_frame);
+        }
 	}
 
 	return err < 0 ? err : 0;

app/src/main/res/drawable/list_view_unselected.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle" >
+
+    <stroke android:width="1dip" android:color="#ffffff"/>
+
+</shape>

app/src/main/res/layout-land/activity_pc_view.xml

@@ -33,12 +33,10 @@
             android:text="@string/searching_pc"/>
     </RelativeLayout>
 
-    <GridView
-        android:id="@+id/pcGridView"
+    <FrameLayout
+        android:id="@+id/pcFragmentContainer"
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
-        android:numColumns="auto_fit"
-        android:columnWidth="160dp"
         android:gravity="center"
         android:layout_alignParentBottom="true"
         android:layout_alignParentTop="true"

app/src/main/res/layout-port/activity_pc_view.xml

@@ -33,12 +33,10 @@
             android:text="@string/searching_pc"/>
     </RelativeLayout>
 
-    <GridView
-        android:id="@+id/pcGridView"
+    <FrameLayout
+        android:id="@+id/pcFragmentContainer"
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
-        android:numColumns="auto_fit"
-        android:columnWidth="160dp"
         android:gravity="center"
         android:layout_toLeftOf="@+id/manuallyAddPc"
         android:layout_toStartOf="@+id/manuallyAddPc"

app/src/main/res/layout/activity_app_view.xml

@@ -8,21 +8,17 @@
     android:paddingTop="@dimen/activity_vertical_margin"
     tools:context=".AppView" >
 
-    <GridView
-        android:id="@+id/appGridView"
+    <FrameLayout
+        android:id="@+id/appFragmentContainer"
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
-        android:numColumns="auto_fit"
-        android:columnWidth="160dp"
-        android:stretchMode="spacingWidth"
         android:gravity="center"
         android:layout_alignParentLeft="true"
         android:layout_alignParentStart="true"
         android:layout_alignParentRight="true"
         android:layout_alignParentEnd="true"
         android:layout_alignParentBottom="true"
-        android:layout_below="@+id/appListText">
-    </GridView>
+        android:layout_below="@+id/appListText"/>
     
     <TextView
         android:id="@+id/appListText"
@@ -30,6 +26,7 @@
         android:layout_height="wrap_content"
         android:layout_centerHorizontal="true"
         android:layout_alignParentTop="true"
+        android:gravity="center"
         android:paddingTop="0dp"
         android:paddingBottom="10dp"
         android:textSize="28sp"/>

app/src/main/res/layout/activity_game.xml

@@ -1,4 +1,4 @@
-<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<merge xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
@@ -10,4 +10,4 @@
         android:layout_height="match_parent"
         android:layout_gravity="center" />
 
-</FrameLayout>
+</merge>

app/src/main/res/layout/app_grid_item_small.xml

@@ -0,0 +1,41 @@
+<?xml version="1.0" encoding="utf-8"?>
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:padding="10dp">
+    <RelativeLayout
+        android:id="@+id/grid_image_layout"
+        android:layout_centerHorizontal="true"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content">
+        <ImageView
+            android:id="@+id/grid_image"
+            android:cropToPadding="false"
+            android:scaleType="fitXY"
+            android:layout_centerHorizontal="true"
+            android:layout_width="100dp"
+            android:layout_height="117dp">
+        </ImageView>
+        <ImageView
+            android:id="@+id/grid_overlay"
+            android:layout_centerHorizontal="true"
+            android:layout_centerVertical="true"
+            android:layout_width="33dp"
+            android:layout_height="33dp">
+        </ImageView>
+    </RelativeLayout>
+    <TextView
+        android:id="@+id/grid_text"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_below="@id/grid_image_layout"
+        android:layout_marginTop="10dp"
+        android:layout_centerHorizontal="true"
+        android:gravity="center"
+        android:singleLine="true"
+        android:ellipsize="marquee"
+        android:marqueeRepeatLimit="marquee_forever"
+        android:scrollHorizontally="true"
+        android:textSize="14sp" >
+    </TextView>
+</RelativeLayout>

app/src/main/res/layout/app_grid_view.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="match_parent">
+    <GridView
+        android:id="@+id/fragmentView"
+        android:layout_width="fill_parent"
+        android:layout_height="fill_parent"
+        android:numColumns="auto_fit"
+        android:columnWidth="160dp"
+        android:stretchMode="spacingWidth"
+        android:focusable="true"
+        android:focusableInTouchMode="true"
+        android:gravity="center"/>
+</LinearLayout>

app/src/main/res/layout/app_grid_view_small.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="match_parent">
+    <GridView
+        android:id="@+id/fragmentView"
+        android:layout_width="fill_parent"
+        android:layout_height="fill_parent"
+        android:numColumns="auto_fit"
+        android:columnWidth="105dp"
+        android:stretchMode="spacingWidth"
+        android:focusable="true"
+        android:focusableInTouchMode="true"
+        android:gravity="center"/>
+</LinearLayout>

app/src/main/res/layout/list_view.xml

@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="match_parent">
+
+    <ListView
+        android:id="@+id/fragmentView"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:background="@drawable/list_view_unselected"
+        android:fastScrollEnabled="true"
+        android:longClickable="false"
+        android:focusable="true"
+        android:focusableInTouchMode="true"
+        android:stackFromBottom="false" >
+
+    </ListView>
+
+</LinearLayout>

app/src/main/res/layout/pc_grid_item_small.xml

@@ -0,0 +1,37 @@
+<?xml version="1.0" encoding="utf-8"?>
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:padding="15dp">
+    <RelativeLayout
+        android:id="@+id/grid_image_layout"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content">
+        <ImageView
+            android:id="@+id/grid_image"
+            android:layout_centerHorizontal="true"
+            android:layout_width="100dp"
+            android:layout_height="67dp">
+        </ImageView>
+        <ImageView
+            android:id="@+id/grid_overlay"
+            android:layout_marginTop="10dp"
+            android:layout_marginLeft="42dp"
+            android:layout_marginStart="42dp"
+            android:layout_marginRight="13dp"
+            android:layout_marginEnd="13dp"
+            android:layout_width="33dp"
+            android:layout_height="33dp">
+        </ImageView>
+    </RelativeLayout>
+    <TextView
+        android:id="@+id/grid_text"
+        android:layout_width="100dp"
+        android:layout_height="wrap_content"
+        android:layout_below="@id/grid_image_layout"
+        android:layout_marginTop="10dp"
+        android:layout_centerHorizontal="true"
+        android:gravity="center"
+        android:textSize="14sp" >
+    </TextView>
+</RelativeLayout>

app/src/main/res/layout/pc_grid_view.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="match_parent">
+    <GridView
+        android:id="@+id/fragmentView"
+        android:layout_width="fill_parent"
+        android:layout_height="fill_parent"
+        android:numColumns="auto_fit"
+        android:columnWidth="160dp"
+        android:focusable="true"
+        android:focusableInTouchMode="true"
+        android:gravity="center"/>
+</LinearLayout>

app/src/main/res/layout/pc_grid_view_small.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="match_parent">
+    <GridView
+        android:id="@+id/fragmentView"
+        android:layout_width="fill_parent"
+        android:layout_height="fill_parent"
+        android:numColumns="auto_fit"
+        android:columnWidth="105dp"
+        android:focusable="true"
+        android:focusableInTouchMode="true"
+        android:gravity="center"/>
+</LinearLayout>

app/src/main/res/layout/simple_row.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:layout_width="fill_parent"
+    android:padding="10dp"
+    android:layout_height="wrap_content">
+
+    <TextView
+        android:id="@+id/grid_text"
+        android:layout_width="fill_parent"
+        android:layout_height="wrap_content"
+        android:textIsSelectable="false"
+        android:textSize="16sp" >
+    </TextView>
+
+</LinearLayout>

app/src/main/res/values-it/strings.xml

@@ -56,6 +56,9 @@
     <!-- General strings -->
     <string name="ip_hint">Indirizzo IP del PC</string>
     <string name="searching_pc">Ricerca PC in corso…</string>
+    <string name="yes">Sì</string>
+    <string name="no">No</string>
+    <string name="lost_connection">Connessione con il PC persa</string>
 
     <!-- AppList activity -->
     <string name="title_applist">Applicazioni su</string>
@@ -70,6 +73,7 @@
     <string name="applist_quit_app">Chiusura in corso…</string>
     <string name="applist_quit_success">Sessione chiusa con successo</string>
     <string name="applist_quit_fail">Chiusura sessione fallita</string>
+    <string name="applist_quit_confirmation">Sei sicuro di voler chiudere l\'applicazione avviata? Tutti i dati non salvati saranno persi.</string>
 
     <!-- Add computer manually activity -->
     <string name="title_add_pc">Aggiungi PC Manualmente</string>
@@ -87,20 +91,30 @@
     <string name="summary_seekbar_bitrate">Abbassa il bitrate per ridurre lo stuttering; alza il bitrate per aumenteare la qualità dell\'immagine</string>
     <string name="suffix_seekbar_bitrate">Mbps</string>
     <string name="title_checkbox_stretch_video">Forza video in full-screen</string>
-    <string name="title_checkbox_disable_warnings">Disabilita i messaggi di warning</string>
+    <string name="title_checkbox_disable_warnings">Disabilita messaggi di warning</string>
     <string name="summary_checkbox_disable_warnings">Disabilita i messaggi di warning sullo schermo durante lo streaming</string>
 
     <string name="category_gamepad_settings">Impostazioni Gamepad</string>
+    <string name="title_checkbox_multi_controller">Supporto controller multipli</string>
+    <string name="summary_checkbox_multi_controller">Quando disabilitato, tutti i controllers appaiono come uno solo</string>
     <string name="title_seekbar_deadzone">Aggiusta deadzone degli stick analogici</string>
     <string name="suffix_seekbar_deadzone">%</string>
 
+    <string name="category_ui_settings">Impostazioni Interfaccia</string>
+    <string name="title_language_list">Lingua</string>
+    <string name="summary_language_list">Lingua da usare in Limelight</string>
+    <string name="title_checkbox_list_mode">Usa lista invece della griglia</string>
+    <string name="summary_checkbox_list_mode">Visualizza applicazioni e computers in una lista invece di una griglia</string>
+    <string name="title_checkbox_small_icon_mode">Usa icone piccole</string>
+    <string name="summary_checkbox_small_icon_mode">Usa icone piccole nella vista a griglia per avere più oggetti sullo schermo</string>
+
     <string name="category_host_settings">Impostazioni Host</string>
     <string name="title_checkbox_enable_sops">Ottimizza le impostazioni dei giochi</string>
     <string name="summary_checkbox_enable_sops">Permetti a GFE di modificare le impostazioni dei giochi per uno streaming ottimale</string>
     <string name="title_checkbox_host_audio">Riproduci audio sul PC</string>
-    <string name="summary_checkbox_host_audio">Riproduci l\'audio sul computer e su questo dispositivo. Richiede GFE 2.1.2+</string>
+    <string name="summary_checkbox_host_audio">Riproduci l\'audio sul computer e su questo dispositivo</string>
 
     <string name="category_advanced_settings">Impostazioni Avanzate</string>
     <string name="title_decoder_list">Cambia decoder</string>
-    <string name="summary_decoder_list">Il decoder software può aumentare la latenza video quando si usano impostazioni streaming basse</string>
+    <string name="summary_decoder_list">Il decoder software può ridurre la latenza video quando si usano impostazioni streaming basse</string>
 </resources>

app/src/main/res/values/arrays.xml

@@ -13,6 +13,17 @@
         <item>1080p60</item>
     </string-array>
 
+    <string-array name="language_names" translatable="false">
+            <item>Default</item>
+            <item>English</item>
+            <item>Italiano</item>
+    </string-array>
+    <string-array name="language_values" translatable="false">
+            <item>default</item>
+            <item>en</item>
+            <item>it</item>
+    </string-array>
+
     <string-array name="decoder_names">
         <item>Auto-select Decoder</item>
         <item>Force Software Decoding</item>

app/src/main/res/values/strings.xml

@@ -56,6 +56,9 @@
     <!-- General strings -->
     <string name="ip_hint">IP address of GeForce PC</string>
     <string name="searching_pc">Searching for PCs…</string>
+    <string name="yes">Yes</string>
+    <string name="no">No</string>
+    <string name="lost_connection">Lost connection to PC</string>
 
     <!-- AppList activity -->
     <string name="title_applist">Apps on</string>
@@ -70,6 +73,7 @@
     <string name="applist_quit_app">Quitting</string>
     <string name="applist_quit_success">Successfully quit</string>
     <string name="applist_quit_fail">Failed to quit</string>
+    <string name="applist_quit_confirmation">Are you sure you want to quit the running app? All unsaved data will be lost.</string>
 
     <!-- Add computer manually activity -->
     <string name="title_add_pc">Add PC Manually</string>
@@ -91,14 +95,24 @@
     <string name="summary_checkbox_disable_warnings">Disable on-screen connection warning messages while streaming</string>
 
     <string name="category_gamepad_settings">Gamepad Settings</string>
+    <string name="title_checkbox_multi_controller">Multiple controller support</string>
+    <string name="summary_checkbox_multi_controller">When unchecked, all controllers appear as one</string>
     <string name="title_seekbar_deadzone">Adjust analog stick deadzone</string>
     <string name="suffix_seekbar_deadzone">%</string>
 
+    <string name="category_ui_settings">UI Settings</string>
+    <string name="title_language_list">Language</string>
+    <string name="summary_language_list">Language to use for Limelight</string>
+    <string name="title_checkbox_list_mode">Use lists instead of grids</string>
+    <string name="summary_checkbox_list_mode">Display apps and PCs in lists instead of grids</string>
+    <string name="title_checkbox_small_icon_mode">Use small icons</string>
+    <string name="summary_checkbox_small_icon_mode">Use small icons in grid items to allow more items on screen</string>
+
     <string name="category_host_settings">Host Settings</string>
     <string name="title_checkbox_enable_sops">Optimize game settings</string>
     <string name="summary_checkbox_enable_sops">Allow GFE to modify game settings for optimal streaming</string>
     <string name="title_checkbox_host_audio">Play audio on PC</string>
-    <string name="summary_checkbox_host_audio">Play audio from the computer and this device. Requires GFE 2.1.2+</string>
+    <string name="summary_checkbox_host_audio">Play audio from the computer and this device</string>
 
     <string name="category_advanced_settings">Advanced Settings</string>
     <string name="title_decoder_list">Change decoder</string>

app/src/main/res/xml/preferences.xml

@@ -26,14 +26,19 @@
             android:summary="@string/summary_checkbox_disable_warnings"
             android:defaultValue="false" />
     </PreferenceCategory>
-    <!--PreferenceCategory android:title="@string/category_gamepad_settings">
-        <com.limelight.preferences.SeekBarPreference
+    <PreferenceCategory android:title="@string/category_gamepad_settings">
+        <!--com.limelight.preferences.SeekBarPreference
             android:key="seekbar_deadzone"
             android:defaultValue="15"
             android:max="50"
             android:text="@string/suffix_seekbar_deadzone"
-            android:title="@string/title_seekbar_deadzone"/>
-    </PreferenceCategory-->
+            android:title="@string/title_seekbar_deadzone"/-->
+        <CheckBoxPreference
+            android:key="checkbox_multi_controller"
+            android:title="@string/title_checkbox_multi_controller"
+            android:summary="@string/summary_checkbox_multi_controller"
+            android:defaultValue="true" />
+    </PreferenceCategory>
     <PreferenceCategory android:title="@string/category_host_settings">
         <CheckBoxPreference
             android:key="checkbox_enable_sops"
@@ -46,6 +51,24 @@
             android:summary="@string/summary_checkbox_host_audio"
             android:defaultValue="false" />
     </PreferenceCategory>
+    <PreferenceCategory android:title="@string/category_ui_settings">
+        <ListPreference
+            android:key="list_languages"
+            android:title="@string/title_language_list"
+            android:entries="@array/language_names"
+            android:entryValues="@array/language_values"
+            android:summary="@string/summary_language_list"
+            android:defaultValue="default" />
+        <com.limelight.preferences.SmallIconCheckboxPreference
+            android:key="checkbox_small_icon_mode"
+            android:title="@string/title_checkbox_small_icon_mode"
+            android:summary="@string/summary_checkbox_small_icon_mode" />
+        <CheckBoxPreference
+            android:key="checkbox_list_mode"
+            android:title="@string/title_checkbox_list_mode"
+            android:summary="@string/summary_checkbox_list_mode"
+            android:defaultValue="false" />
+    </PreferenceCategory>
     <PreferenceCategory android:title="@string/category_advanced_settings">
         <ListPreference
             android:key="list_decoders"

app/src/nonRoot/AndroidManifest.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    package="com.limelight" >
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
 
     <!-- Non-root application name -->
     <application android:label="Limelight" />

app/src/root/AndroidManifest.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    package="com.limelight.root" >
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
 
     <!-- Root permissions -->
     <uses-permission android:name="android.permission.ACCESS_SUPERUSER" />

build.gradle

@@ -4,7 +4,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:0.13.2'
+        classpath 'com.android.tools.build:gradle:1.1.0'
     }
 }
 

decoder-errata.txt

@@ -4,12 +4,12 @@ This file serves to document some of the decoder errata when using MediaCodec ha
 	- Affected decoders: TI OMAP4, Allwinner A20
 	
 2. Some decoders have a huge per-frame latency with the unmodified SPS sent from NVENC. Setting max_dec_frame_buffering fixes this latency issue.
-	- Affected decoders: NVIDIA Tegra 3 and 4
+	- Affected decoders: NVIDIA Tegra 3 and 4, Broadcom VideoCore IV
 
 3. Some decoders strictly require that you pass BUFFER_FLAG_CODEC_CONFIG and crash upon the IDR frame if you don't
 	- Affected decoders: TI OMAP4
 
-4. Some decoders require num_ref_frames=1 and max_dec_frame_buffering=1 to avoid crashing on SPS or first I-frame
+4. Some decoders require num_ref_frames=1 and max_dec_frame_buffering=1 to avoid crashing on SPS on first I-frame
 	- Affected decoders: Qualcomm in GS3 on 4.3+, Exynos 4 at 1080p only
 
 5. Some decoders will hang if max_dec_frame_buffering is not present

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
-#Wed Apr 10 15:27:10 PDT 2013
+#Sun Dec 07 22:52:07 PST 2014
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-2.1-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-2.2.1-all.zip