RFR [9] 8153372: Remove sun.misc.ManagedLocalsThread from jdk.httpserver

[Alan Bateman]

This looks fine.

On 18/04/2016 06:53, Chris Hegarty wrote:
> 8056152 added a new constructor to java.lang.Thread to constructing Threads that
> do not  inherit inheritable-thread-local initial values. Given there is now a supported
> API for creating such threads, other areas of the JDK should be updated to use it.
>
> This change updates the code in the jdk.httpserver to use the new Thread constructor.
>
> diff --git a/src/jdk.httpserver/share/classes/module-info.java b/src/jdk.httpserver/share/classes/module-info.java
> --- a/src/jdk.httpserver/share/classes/module-info.java
> +++ b/src/jdk.httpserver/share/classes/module-info.java
> @@ -25,8 +25,7 @@
>   
>   module jdk.httpserver {
>       requires java.logging;
> -    // 8153372
> -    requires jdk.unsupported;
> +
>       exports com.sun.net.httpserver;
>       exports com.sun.net.httpserver.spi;
>       uses com.sun.net.httpserver.spi.HttpServerProvider;
> diff --git a/src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java b/src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java
> --- a/src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java
> +++ b/src/jdk.httpserver/share/classes/sun/net/httpserver/ServerImpl.java
> @@ -36,7 +36,6 @@
>   import com.sun.net.httpserver.*;
>   import java.security.AccessController;
>   import java.security.PrivilegedAction;
> -import sun.misc.ManagedLocalsThread;
>   import sun.net.httpserver.HttpConnection.State;
>   
>   /**
> @@ -143,7 +142,7 @@
>           if (executor == null) {
>               executor = new DefaultExecutor();
>           }
> -        dispatcherThread = new ManagedLocalsThread(dispatcher);
> +        dispatcherThread = new Thread(null, dispatcher, "HTTP-Dispatcher", 0, false);
>           started = true;
>           dispatcherThread.start();
>       }
>
> -Chris.