/hg/release/icedtea-web-1.3: Provide names for unnamed threads f...

[adomurad at icedtea.classpath.org]

changeset c89c2bf6707b in /hg/release/icedtea-web-1.3
details: http://icedtea.classpath.org/hg/release/icedtea-web-1.3?cmd=changeset;node=c89c2bf6707b
author: Adam Domurad <adomurad at redhat.com>
date: Wed Feb 06 09:22:51 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 b619cda99360 -r c89c2bf6707b ChangeLog
--- a/ChangeLog	Wed Jan 16 15:34:44 2013 -0500
+++ b/ChangeLog	Wed Feb 06 09:22:51 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-01-16  Deepak Bhole <dbhole at redhat.com>
 
 	PR1260: IcedTea-Web should not rely on GTK
diff -r b619cda99360 -r c89c2bf6707b netx/net/sourceforge/jnlp/NetxPanel.java
--- a/netx/net/sourceforge/jnlp/NetxPanel.java	Wed Jan 16 15:34:44 2013 -0500
+++ b/netx/net/sourceforge/jnlp/NetxPanel.java	Wed Feb 06 09:22:51 2013 -0500
@@ -199,7 +199,7 @@
             }
         }
 
-        handler = new Thread(getThreadGroup(), this);
+        handler = new Thread(getThreadGroup(), this, "NetxPanelThread@" + this.documentURL);
         handler.start();
     }
 
diff -r b619cda99360 -r c89c2bf6707b netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Wed Jan 16 15:34:44 2013 -0500
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Wed Feb 06 09:22:51 2013 -0500
@@ -737,7 +737,7 @@
             e.printStackTrace();
         }
 
-        Runtime.getRuntime().addShutdownHook(new Thread() {
+        Runtime.getRuntime().addShutdownHook(new Thread("JNLPRuntimeShutdownHookThread") {
             public void run() {
                 markNetxStopped();
                 CacheUtil.cleanCache();
diff -r b619cda99360 -r c89c2bf6707b plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java	Wed Jan 16 15:34:44 2013 -0500
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java	Wed Feb 06 09:22:51 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 b619cda99360 -r c89c2bf6707b plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java	Wed Jan 16 15:34:44 2013 -0500
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java	Wed Feb 06 09:22:51 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 b619cda99360 -r c89c2bf6707b plugin/icedteanp/java/sun/applet/PluginStreamHandler.java
--- a/plugin/icedteanp/java/sun/applet/PluginStreamHandler.java	Wed Jan 16 15:34:44 2013 -0500
+++ b/plugin/icedteanp/java/sun/applet/PluginStreamHandler.java	Wed Feb 06 09:22:51 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() {