/hg/icedtea-web: Provide names for unnamed threads for better de...

[adomurad at icedtea.classpath.org]

changeset ed4ecccd83e6 in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=ed4ecccd83e6
author: Adam Domurad <adomurad at redhat.com>
date: Wed Feb 06 09:18:38 2013 -0500

	Provide names for unnamed threads for better debugging


diffstat:

 ChangeLog                                                        |  17 ++++++++++
 netx/net/sourceforge/jnlp/NetxPanel.java                         |   2 +-
 netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java               |   2 +-
 plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java      |   6 ++-
 plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java |   1 +
 plugin/icedteanp/java/sun/applet/PluginStreamHandler.java        |   4 +-
 6 files changed, 27 insertions(+), 5 deletions(-)

diffs (97 lines):

diff -r fd01cd1c2bbc -r ed4ecccd83e6 ChangeLog
--- a/ChangeLog	Tue Feb 05 09:51:44 2013 +0100
+++ b/ChangeLog	Wed Feb 06 09:18:38 2013 -0500
@@ -1,3 +1,20 @@
+2013-02-06  Adam Domurad <adomurad at redhat.com>
+
+	Name threads for easier debugging/tooling. Remove 2 erroneous
+	VoidPluginCallRequest header comments.
+	* netx/net/sourceforge/jnlp/NetxPanel.java:
+	Provide name for thread that calls (run).
+	* netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java:
+	Provide name for shutdown hook thread.
+	* plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java:
+	Remove erroneous VoidPluginCallRequest comment. Provide name for
+	ConsumerThread thread.
+	* plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java:
+	Provide name for worker thread.
+	* plugin/icedteanp/java/sun/applet/PluginStreamHandler.java:
+	Remove erroneous VoidPluginCallRequest comment. Provide name for
+	stream listener thread.
+
 2013-02-03  Jiri Vanek  <jvanek at redhat.com>
 
 	Another renamed conflict file for case insensitive systems
diff -r fd01cd1c2bbc -r ed4ecccd83e6 netx/net/sourceforge/jnlp/NetxPanel.java
--- a/netx/net/sourceforge/jnlp/NetxPanel.java	Tue Feb 05 09:51:44 2013 +0100
+++ b/netx/net/sourceforge/jnlp/NetxPanel.java	Wed Feb 06 09:18:38 2013 -0500
@@ -181,7 +181,7 @@
             }
         }
 
-        handler = new Thread(getThreadGroup(), this);
+        handler = new Thread(getThreadGroup(), this, "NetxPanelThread@" + this.documentURL);
         handler.start();
     }
 
diff -r fd01cd1c2bbc -r ed4ecccd83e6 netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Tue Feb 05 09:51:44 2013 +0100
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Wed Feb 06 09:18:38 2013 -0500
@@ -738,7 +738,7 @@
             e.printStackTrace();
         }
 
-        Runtime.getRuntime().addShutdownHook(new Thread() {
+        Runtime.getRuntime().addShutdownHook(new Thread("JNLPRuntimeShutdownHookThread") {
             public void run() {
                 markNetxStopped();
                 CacheUtil.cleanCache();
diff -r fd01cd1c2bbc -r ed4ecccd83e6 plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java	Tue Feb 05 09:51:44 2013 +0100
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java	Wed Feb 06 09:18:38 2013 -0500
@@ -1,4 +1,4 @@
-/* VoidPluginCallRequest -- represent Java-to-JavaScript requests
+/* 
    Copyright (C) 2008  Red Hat
 
 This file is part of IcedTea.
@@ -131,6 +131,10 @@
 
     protected class ConsumerThread extends Thread {
 
+        public ConsumerThread() {
+            super("PluginMessageConsumer.ConsumerThread");
+        }
+
         /**
          * Scans the readQueue for priority messages and brings them to the front
          */
diff -r fd01cd1c2bbc -r ed4ecccd83e6 plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java	Tue Feb 05 09:51:44 2013 +0100
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java	Wed Feb 06 09:18:38 2013 -0500
@@ -50,6 +50,7 @@
                 PluginMessageConsumer consumer,
                 PluginStreamHandler streamHandler, int id,
                 boolean isPriorityWorker) {
+        super("PluginMessageHandlerWorker" + id);
 
         this.id = id;
         this.streamHandler = streamHandler;
diff -r fd01cd1c2bbc -r ed4ecccd83e6 plugin/icedteanp/java/sun/applet/PluginStreamHandler.java
--- a/plugin/icedteanp/java/sun/applet/PluginStreamHandler.java	Tue Feb 05 09:51:44 2013 +0100
+++ b/plugin/icedteanp/java/sun/applet/PluginStreamHandler.java	Wed Feb 06 09:18:38 2013 -0500
@@ -1,4 +1,4 @@
-/* VoidPluginCallRequest -- represent Java-to-JavaScript requests
+/*
    Copyright (C) 2008  Red Hat
 
 This file is part of IcedTea.
@@ -81,7 +81,7 @@
 
     public void startProcessing() {
 
-        Thread listenerThread = new Thread() {
+        Thread listenerThread = new Thread("PluginStreamHandlerListenerThread") {
 
             public void run() {