Adapt python scripts to use the new Forwarder2

tools/android/forwarder2/host_controller.cc

Index: tools/android/forwarder2/host_controller.cc
diff --git a/tools/android/forwarder2/host_controller.cc b/tools/android/forwarder2/host_controller.cc
index 0507c114a44deda8396e0cbcd6b2579c35d85b69..8ed89ae381605ee231b7b7d06fddcb371f385a1b 100644
--- a/tools/android/forwarder2/host_controller.cc
+++ b/tools/android/forwarder2/host_controller.cc
@@ -68,7 +68,7 @@ bool HostController::Connect() {
     return false;
   }
   // Send the command to the device start listening to the "device_forward_port"
-  SendCommand(command::LISTEN, device_port_, &adb_control_socket_);
+  CHECK(SendCommand(command::LISTEN, device_port_, &adb_control_socket_));

[bulach, 2012/09/26 15:13:11]

nit: it's fairly uncommon to call non-const methods inside CHECKs, i.e., to do
work there.. I think a more common idiom is:
bool ret = SendCommand(...);
CHECK(ret);

[felipeg, 2012/09/26 18:06:03]

Done.

   int device_port_allocated;
   command::Type command;
   if (!ReadCommand(&adb_control_socket_, &device_port_allocated, &command) ||