[PATCH] Move timeout from ConnOpener::connect to ConnOpener::timeout

The attached patch removes the timeout checking code from
ConnOpener::connect and changes ConnOpener::timeout to actually force
the connection attempt to be terminated with 'a timeout error' instead
of invoking the connect method. The main motivation for this is that a
timeout occuring for a 'server first' SSL bumped connection attempt
would otherwise trigger an assert in forward.cc because of the lack
of either 'an I/O result' or an error state after the
comm_connect_addr routine invoked from connect after the connect call
from timeout had erroneously signalled that the connection was
successfully established to its caller.