Request for Review 6882654

[Michael McMahon]

Looks fine to me.

Thanks,
Michael.

Christopher Hegarty - Sun Microsystems Ireland wrote:
>
> 6882654: Remove dependency on java.util.concurrent from KeepAlive 
> implementaion
>
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6882654/webrev.0/webrev/
>
> The HTTP keep alive implementation has unnecessary dependencies on 
> java.util.concurrent classes.
>
> KeepAliveCache subclasses ConcurrentHashMap even though all its 
> methods are synchronized, except run with synchronizes on the this 
> object before attempting to access and of the classes data.
>
> KeepAliveStreamCleaner subclasses LinkedBlockingQueue, but we can 
> easily rewrite it since it only uses a simple timed poll.
>
> -Chris.