commit 7c4a15f2d49f018c6fe7e45935443419c96c5062
author Nicolò Mazzucato <nicomazz@google.com> Thu Sep 05 11:45:41 2019 +0100
committer Nicolò Mazzucato <nicomazz@google.com> Thu Sep 05 11:45:41 2019 +0100
tree f1b723dfafab7b72d3f7f492aec4f1db61f922fa
parent b1de0c8d6a36893d4b441b47429c9096945df0d6

Show devices in target selection dropdown & improve error handling

Thanks to this CL we are able to show the android devices
connected in the target selection dropdown list.
Now the last error from the record session is displayed, and after
an error the state of the UI is resetted, so that to be able to
start another trace.

Change-Id: I4093dd33dbe6436bbf3df3e856d575dc5bc8f403

ui/src/controller/adb_jsdomtest.ts

diff --git a/ui/src/controller/adb_jsdomtest.ts b/ui/src/controller/adb_jsdomtest.ts
index f2b757f..83d05fd 100644
--- a/ui/src/controller/adb_jsdomtest.ts
+++ b/ui/src/controller/adb_jsdomtest.ts
@@ -19,7 +19,6 @@
   AdbOverWebUsb,
   AdbState,
   DEFAULT_MAX_PAYLOAD_BYTES,
-  VERSION_NO_CHECKSUM,
   VERSION_WITH_CHECKSUM
 } from './adb';
 
@@ -34,10 +33,10 @@
 
   const expectedAuthMessage = AdbMsgImpl.create({
     cmd: 'CNXN',
-    arg0: VERSION_NO_CHECKSUM,
+    arg0: VERSION_WITH_CHECKSUM,
     arg1: DEFAULT_MAX_PAYLOAD_BYTES,
     data: 'host:1:UsbADB',
-    useChecksum: false
+    useChecksum: true
   });
   await adb.startAuthentication();
 
@@ -60,7 +59,7 @@
     arg0: VERSION_WITH_CHECKSUM,
     arg1: expectedMaxPayload,
     data: new TextEncoder().encode('device'),
-    useChecksum: false
+    useChecksum: true
   });
   await adb.onMessage(connectedMsg);