Formatted source
[jalview.git] / src / jalview / util / BrowserLauncher.java
index a36ea27..12d6f02 100755 (executable)
@@ -1,20 +1,32 @@
-/********************
- * 2004 Jalview Reengineered
- * Barton Group
- * Dundee University
- *
- * AM Waterhouse
- *******************/
-
-package jalview.util;
-
-import java.io.File;
-import java.io.IOException;
-import java.lang.reflect.Constructor;
-import java.lang.reflect.Field;
-import java.lang.reflect.InvocationTargetException;
-import java.lang.reflect.Method;
-
+/*
+* Jalview - A Sequence Alignment Editor and Viewer
+* Copyright (C) 2005 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
+*
+* This program is free software; you can redistribute it and/or
+* modify it under the terms of the GNU General Public License
+* as published by the Free Software Foundation; either version 2
+* of the License, or (at your option) any later version.
+*
+* This program is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+* GNU General Public License for more details.
+*
+* You should have received a copy of the GNU General Public License
+* along with this program; if not, write to the Free Software
+* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
+*/\r
+package jalview.util;\r
+\r
+import java.io.File;\r
+import java.io.IOException;\r
+\r
+import java.lang.reflect.Constructor;\r
+import java.lang.reflect.Field;\r
+import java.lang.reflect.InvocationTargetException;\r
+import java.lang.reflect.Method;\r
+\r
+\r
 /**
  * BrowserLauncher is a class that provides one static method, openURL, which opens the default
  * web browser for the current user of the system to the given URL.  It may support other
@@ -58,540 +70,698 @@ import java.lang.reflect.Method;
  *
  * @author Eric Albert (<a href="mailto:ejalbert@cs.stanford.edu">ejalbert@cs.stanford.edu</a>)
  * @version 1.4b1 (Released June 20, 2001)
- */
-public class BrowserLauncher {
-
-        /**
-         * The Java virtual machine that we are running on.  Actually, in most cases we only care
-         * about the operating system, but some operating systems require us to switch on the VM. */
-        private static int jvm;
-
-        /** The browser for the system */
-        private static Object browser;
-
-        /**
-         * Caches whether any classes, methods, and fields that are not part of the JDK and need to
-         * be dynamically loaded at runtime loaded successfully.
-         * <p>
-         * Note that if this is <code>false</code>, <code>openURL()</code> will always return an
-         * IOException.
-         */
-        private static boolean loadedWithoutErrors;
-
-        /** The com.apple.mrj.MRJFileUtils class */
-        private static Class mrjFileUtilsClass;
-
-        /** The com.apple.mrj.MRJOSType class */
-        private static Class mrjOSTypeClass;
-
-        /** The com.apple.MacOS.AEDesc class */
-        private static Class aeDescClass;
-
-        /** The &lt;init&gt;(int) method of com.apple.MacOS.AETarget */
-        private static Constructor aeTargetConstructor;
-
-        /** The &lt;init&gt;(int, int, int) method of com.apple.MacOS.AppleEvent */
-        private static Constructor appleEventConstructor;
-
-        /** The &lt;init&gt;(String) method of com.apple.MacOS.AEDesc */
-        private static Constructor aeDescConstructor;
-
-        /** The findFolder method of com.apple.mrj.MRJFileUtils */
-        private static Method findFolder;
-
-        /** The getFileCreator method of com.apple.mrj.MRJFileUtils */
-        private static Method getFileCreator;
-
-        /** The getFileType method of com.apple.mrj.MRJFileUtils */
-        private static Method getFileType;
-
-        /** The openURL method of com.apple.mrj.MRJFileUtils */
-        private static Method openURL;
-
-        /** The makeOSType method of com.apple.MacOS.OSUtils */
-        private static Method makeOSType;
-
-        /** The putParameter method of com.apple.MacOS.AppleEvent */
-        private static Method putParameter;
-
-        /** The sendNoReply method of com.apple.MacOS.AppleEvent */
-        private static Method sendNoReply;
-
-        /** Actually an MRJOSType pointing to the System Folder on a Macintosh */
-        private static Object kSystemFolderType;
-
-        /** The keyDirectObject AppleEvent parameter type */
-        private static Integer keyDirectObject;
-
-        /** The kAutoGenerateReturnID AppleEvent code */
-        private static Integer kAutoGenerateReturnID;
-
-        /** The kAnyTransactionID AppleEvent code */
-        private static Integer kAnyTransactionID;
-
-        /** The linkage object required for JDirect 3 on Mac OS X. */
-        private static Object linkage;
-
-        /** The framework to reference on Mac OS X */
-        private static final String JDirect_MacOSX = "/System/Library/Frameworks/Carbon.framework/Frameworks/HIToolbox.framework/HIToolbox";
-
-        /** JVM constant for MRJ 2.0 */
-        private static final int MRJ_2_0 = 0;
-
-        /** JVM constant for MRJ 2.1 or later */
-        private static final int MRJ_2_1 = 1;
-
-        /** JVM constant for Java on Mac OS X 10.0 (MRJ 3.0) */
-        private static final int MRJ_3_0 = 3;
-
-        /** JVM constant for MRJ 3.1 */
-        private static final int MRJ_3_1 = 4;
-
-        /** JVM constant for any Windows NT JVM */
-        private static final int WINDOWS_NT = 5;
-
-        /** JVM constant for any Windows 9x JVM */
-        private static final int WINDOWS_9x = 6;
-
-        /** JVM constant for any other platform */
-        private static final int OTHER = -1;
-
-        /**
-         * The file type of the Finder on a Macintosh.  Hardcoding "Finder" would keep non-U.S. English
-         * systems from working properly.
-         */
-        private static final String FINDER_TYPE = "FNDR";
-
-        /**
-         * The creator code of the Finder on a Macintosh, which is needed to send AppleEvents to the
-         * application.
-         */
-        private static final String FINDER_CREATOR = "MACS";
-
-        /** The name for the AppleEvent type corresponding to a GetURL event. */
-        private static final String GURL_EVENT = "GURL";
-
-        /**
-         * The first parameter that needs to be passed into Runtime.exec() to open the default web
-         * browser on Windows.
-         */
-    private static final String FIRST_WINDOWS_PARAMETER = "/c";
-
-    /** The second parameter for Runtime.exec() on Windows. */
-    private static final String SECOND_WINDOWS_PARAMETER = "start";
-
+ */\r
+public class BrowserLauncher {\r
+    /**
+ * The Java virtual machine that we are running on.  Actually, in most cases we only care
+ * about the operating system, but some operating systems require us to switch on the VM. */\r
+    private static int jvm;\r
+\r
+    /** The browser for the system */\r
+    private static Object browser;\r
+\r
+    /**
+ * Caches whether any classes, methods, and fields that are not part of the JDK and need to
+ * be dynamically loaded at runtime loaded successfully.
+ * <p>
+ * Note that if this is <code>false</code>, <code>openURL()</code> will always return an
+ * IOException.
+ */\r
+    private static boolean loadedWithoutErrors;\r
+\r
+    /** The com.apple.mrj.MRJFileUtils class */\r
+    private static Class mrjFileUtilsClass;\r
+\r
+    /** The com.apple.mrj.MRJOSType class */\r
+    private static Class mrjOSTypeClass;\r
+\r
+    /** The com.apple.MacOS.AEDesc class */\r
+    private static Class aeDescClass;\r
+\r
+    /** The &lt;init&gt;(int) method of com.apple.MacOS.AETarget */\r
+    private static Constructor aeTargetConstructor;\r
+\r
+    /** The &lt;init&gt;(int, int, int) method of com.apple.MacOS.AppleEvent */\r
+    private static Constructor appleEventConstructor;\r
+\r
+    /** The &lt;init&gt;(String) method of com.apple.MacOS.AEDesc */\r
+    private static Constructor aeDescConstructor;\r
+\r
+    /** The findFolder method of com.apple.mrj.MRJFileUtils */\r
+    private static Method findFolder;\r
+\r
+    /** The getFileCreator method of com.apple.mrj.MRJFileUtils */\r
+    private static Method getFileCreator;\r
+\r
+    /** The getFileType method of com.apple.mrj.MRJFileUtils */\r
+    private static Method getFileType;\r
+\r
+    /** The openURL method of com.apple.mrj.MRJFileUtils */\r
+    private static Method openURL;\r
+\r
+    /** The makeOSType method of com.apple.MacOS.OSUtils */\r
+    private static Method makeOSType;\r
+\r
+    /** The putParameter method of com.apple.MacOS.AppleEvent */\r
+    private static Method putParameter;\r
+\r
+    /** The sendNoReply method of com.apple.MacOS.AppleEvent */\r
+    private static Method sendNoReply;\r
+\r
+    /** Actually an MRJOSType pointing to the System Folder on a Macintosh */\r
+    private static Object kSystemFolderType;\r
+\r
+    /** The keyDirectObject AppleEvent parameter type */\r
+    private static Integer keyDirectObject;\r
+\r
+    /** The kAutoGenerateReturnID AppleEvent code */\r
+    private static Integer kAutoGenerateReturnID;\r
+\r
+    /** The kAnyTransactionID AppleEvent code */\r
+    private static Integer kAnyTransactionID;\r
+\r
+    /** The linkage object required for JDirect 3 on Mac OS X. */\r
+    private static Object linkage;\r
+\r
+    /** The framework to reference on Mac OS X */\r
+    private static final String JDirect_MacOSX = "/System/Library/Frameworks/Carbon.framework/Frameworks/HIToolbox.framework/HIToolbox";\r
+\r
+    /** JVM constant for MRJ 2.0 */\r
+    private static final int MRJ_2_0 = 0;\r
+\r
+    /** JVM constant for MRJ 2.1 or later */\r
+    private static final int MRJ_2_1 = 1;\r
+\r
+    /** JVM constant for Java on Mac OS X 10.0 (MRJ 3.0) */\r
+    private static final int MRJ_3_0 = 3;\r
+\r
+    /** JVM constant for MRJ 3.1 */\r
+    private static final int MRJ_3_1 = 4;\r
+\r
+    /** JVM constant for any Windows NT JVM */\r
+    private static final int WINDOWS_NT = 5;\r
+\r
+    /** JVM constant for any Windows 9x JVM */\r
+    private static final int WINDOWS_9x = 6;\r
+\r
+    /** JVM constant for any other platform */\r
+    private static final int OTHER = -1;\r
+\r
+    /**
+ * The file type of the Finder on a Macintosh.  Hardcoding "Finder" would keep non-U.S. English
+ * systems from working properly.
+ */\r
+    private static final String FINDER_TYPE = "FNDR";\r
+\r
+    /**
+ * The creator code of the Finder on a Macintosh, which is needed to send AppleEvents to the
+ * application.
+ */\r
+    private static final String FINDER_CREATOR = "MACS";\r
+\r
+    /** The name for the AppleEvent type corresponding to a GetURL event. */\r
+    private static final String GURL_EVENT = "GURL";\r
+\r
+    /**
+ * The first parameter that needs to be passed into Runtime.exec() to open the default web
+ * browser on Windows.
+ */\r
+    private static final String FIRST_WINDOWS_PARAMETER = "/c";\r
+\r
+    /** The second parameter for Runtime.exec() on Windows. */\r
+    private static final String SECOND_WINDOWS_PARAMETER = "start";\r
+\r
+    /**
+ * The third parameter for Runtime.exec() on Windows.  This is a "title"
+ * parameter that the command line expects.  Setting this parameter allows
+ * URLs containing spaces to work.
+ */\r
+    private static final String THIRD_WINDOWS_PARAMETER = "\"\"";\r
+\r
+    /**
+ * The shell parameters for Netscape that opens a given URL in an already-open copy of Netscape
+ * on many command-line systems.
+ */\r
+    private static final String NETSCAPE_REMOTE_PARAMETER = "-remote";\r
+    private static final String NETSCAPE_OPEN_PARAMETER_START = "openURL(";\r
+    private static final String NETSCAPE_OPEN_NEW_WINDOW = ", new-window";\r
+    private static final String NETSCAPE_OPEN_PARAMETER_END = ")";\r
+\r
+    /**
+ * The message from any exception thrown throughout the initialization process.
+ */\r
+    private static String errorMessage;\r
+\r
+    /**
+ * An initialization block that determines the operating system and loads the necessary
+ * runtime data.
+ */\r
+    static {\r
+        loadedWithoutErrors = true;\r
+\r
+        String osName = System.getProperty("os.name");\r
+\r
+        if (osName.startsWith("Mac OS")) {\r
+            String mrjVersion = System.getProperty("mrj.version");\r
+            String majorMRJVersion = mrjVersion.substring(0, 3);\r
+\r
+            try {\r
+                double version = Double.valueOf(majorMRJVersion).doubleValue();\r
+\r
+                if (version == 2) {\r
+                    jvm = MRJ_2_0;\r
+                } else if ((version >= 2.1) && (version < 3)) {\r
+                    // Assume that all 2.x versions of MRJ work the same.  MRJ 2.1 actually\r
+                    // works via Runtime.exec() and 2.2 supports that but has an openURL() method\r
+                    // as well that we currently ignore.\r
+                    jvm = MRJ_2_1;\r
+                } else if (version == 3.0) {\r
+                    jvm = MRJ_3_0;\r
+                } else if (version >= 3.1) {\r
+                    // Assume that all 3.1 and later versions of MRJ work the same.\r
+                    jvm = MRJ_3_1;\r
+                } else {\r
+                    loadedWithoutErrors = false;\r
+                    errorMessage = "Unsupported MRJ version: " + version;\r
+                }\r
+            } catch (NumberFormatException nfe) {\r
+                loadedWithoutErrors = false;\r
+                errorMessage = "Invalid MRJ version: " + mrjVersion;\r
+            }\r
+        } else if (osName.startsWith("Windows")) {\r
+            if (osName.indexOf("9") != -1) {\r
+                jvm = WINDOWS_9x;\r
+            } else {\r
+                jvm = WINDOWS_NT;\r
+            }\r
+        } else {\r
+            jvm = OTHER;\r
+        }\r
+\r
+        if (loadedWithoutErrors) { // if we haven't hit any errors yet\r
+            loadedWithoutErrors = loadClasses();\r
+        }\r
+    }\r
+\r
+    /**
+ * This class should be never be instantiated; this just ensures so.
+ */\r
+    private BrowserLauncher() {\r
+    }\r
+\r
+    /**
+ * Called by a static initializer to load any classes, fields, and methods required at runtime
+ * to locate the user's web browser.
+ * @return <code>true</code> if all intialization succeeded
+ *                        <code>false</code> if any portion of the initialization failed
+ */\r
+    private static boolean loadClasses() {\r
+        switch (jvm) {\r
+        case MRJ_2_0:\r
+\r
+            try {\r
+                Class aeTargetClass = Class.forName("com.apple.MacOS.AETarget");\r
+                Class osUtilsClass = Class.forName("com.apple.MacOS.OSUtils");\r
+                Class appleEventClass = Class.forName(\r
+                        "com.apple.MacOS.AppleEvent");\r
+                Class aeClass = Class.forName("com.apple.MacOS.ae");\r
+                aeDescClass = Class.forName("com.apple.MacOS.AEDesc");\r
+\r
+                aeTargetConstructor = aeTargetClass.getDeclaredConstructor(new Class[] {\r
+                            int.class\r
+                        });\r
+                appleEventConstructor = appleEventClass.getDeclaredConstructor(new Class[] {\r
+                            int.class, int.class, aeTargetClass, int.class,\r
+                            int.class\r
+                        });\r
+                aeDescConstructor = aeDescClass.getDeclaredConstructor(new Class[] {\r
+                            String.class\r
+                        });\r
+\r
+                makeOSType = osUtilsClass.getDeclaredMethod("makeOSType",\r
+                        new Class[] { String.class });\r
+                putParameter = appleEventClass.getDeclaredMethod("putParameter",\r
+                        new Class[] { int.class, aeDescClass });\r
+                sendNoReply = appleEventClass.getDeclaredMethod("sendNoReply",\r
+                        new Class[] {  });\r
+\r
+                Field keyDirectObjectField = aeClass.getDeclaredField(\r
+                        "keyDirectObject");\r
+                keyDirectObject = (Integer) keyDirectObjectField.get(null);\r
+\r
+                Field autoGenerateReturnIDField = appleEventClass.getDeclaredField(\r
+                        "kAutoGenerateReturnID");\r
+                kAutoGenerateReturnID = (Integer) autoGenerateReturnIDField.get(null);\r
+\r
+                Field anyTransactionIDField = appleEventClass.getDeclaredField(\r
+                        "kAnyTransactionID");\r
+                kAnyTransactionID = (Integer) anyTransactionIDField.get(null);\r
+            } catch (ClassNotFoundException cnfe) {\r
+                errorMessage = cnfe.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchMethodException nsme) {\r
+                errorMessage = nsme.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchFieldException nsfe) {\r
+                errorMessage = nsfe.getMessage();\r
+\r
+                return false;\r
+            } catch (IllegalAccessException iae) {\r
+                errorMessage = iae.getMessage();\r
+\r
+                return false;\r
+            }\r
+\r
+            break;\r
+\r
+        case MRJ_2_1:\r
+\r
+            try {\r
+                mrjFileUtilsClass = Class.forName("com.apple.mrj.MRJFileUtils");\r
+                mrjOSTypeClass = Class.forName("com.apple.mrj.MRJOSType");\r
+\r
+                Field systemFolderField = mrjFileUtilsClass.getDeclaredField(\r
+                        "kSystemFolderType");\r
+                kSystemFolderType = systemFolderField.get(null);\r
+                findFolder = mrjFileUtilsClass.getDeclaredMethod("findFolder",\r
+                        new Class[] { mrjOSTypeClass });\r
+                getFileCreator = mrjFileUtilsClass.getDeclaredMethod("getFileCreator",\r
+                        new Class[] { File.class });\r
+                getFileType = mrjFileUtilsClass.getDeclaredMethod("getFileType",\r
+                        new Class[] { File.class });\r
+            } catch (ClassNotFoundException cnfe) {\r
+                errorMessage = cnfe.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchFieldException nsfe) {\r
+                errorMessage = nsfe.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchMethodException nsme) {\r
+                errorMessage = nsme.getMessage();\r
+\r
+                return false;\r
+            } catch (SecurityException se) {\r
+                errorMessage = se.getMessage();\r
+\r
+                return false;\r
+            } catch (IllegalAccessException iae) {\r
+                errorMessage = iae.getMessage();\r
+\r
+                return false;\r
+            }\r
+\r
+            break;\r
+\r
+        case MRJ_3_0:\r
+\r
+            try {\r
+                Class linker = Class.forName("com.apple.mrj.jdirect.Linker");\r
+                Constructor constructor = linker.getConstructor(new Class[] {\r
+                            Class.class\r
+                        });\r
+                linkage = constructor.newInstance(new Object[] {\r
+                            BrowserLauncher.class\r
+                        });\r
+            } catch (ClassNotFoundException cnfe) {\r
+                errorMessage = cnfe.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchMethodException nsme) {\r
+                errorMessage = nsme.getMessage();\r
+\r
+                return false;\r
+            } catch (InvocationTargetException ite) {\r
+                errorMessage = ite.getMessage();\r
+\r
+                return false;\r
+            } catch (InstantiationException ie) {\r
+                errorMessage = ie.getMessage();\r
+\r
+                return false;\r
+            } catch (IllegalAccessException iae) {\r
+                errorMessage = iae.getMessage();\r
+\r
+                return false;\r
+            }\r
+\r
+            break;\r
+\r
+        case MRJ_3_1:\r
+\r
+            try {\r
+                mrjFileUtilsClass = Class.forName("com.apple.mrj.MRJFileUtils");\r
+                openURL = mrjFileUtilsClass.getDeclaredMethod("openURL",\r
+                        new Class[] { String.class });\r
+            } catch (ClassNotFoundException cnfe) {\r
+                errorMessage = cnfe.getMessage();\r
+\r
+                return false;\r
+            } catch (NoSuchMethodException nsme) {\r
+                errorMessage = nsme.getMessage();\r
+\r
+                return false;\r
+            }\r
+\r
+            break;\r
+\r
+        default:\r
+            break;\r
+        }\r
+\r
+        return true;\r
+    }\r
+\r
+    /**
+ * Attempts to locate the default web browser on the local system.  Caches results so it
+ * only locates the browser once for each use of this class per JVM instance.
+ * @return The browser for the system.  Note that this may not be what you would consider
+ *                        to be a standard web browser; instead, it's the application that gets called to
+ *                        open the default web browser.  In some cases, this will be a non-String object
+ *                        that provides the means of calling the default browser.
+ */\r
+    private static Object locateBrowser() {\r
+        if (browser != null) {\r
+            return browser;\r
+        }\r
+\r
+        switch (jvm) {\r
+        case MRJ_2_0:\r
+\r
+            try {\r
+                Integer finderCreatorCode = (Integer) makeOSType.invoke(null,\r
+                        new Object[] { FINDER_CREATOR });\r
+                Object aeTarget = aeTargetConstructor.newInstance(new Object[] {\r
+                            finderCreatorCode\r
+                        });\r
+                Integer gurlType = (Integer) makeOSType.invoke(null,\r
+                        new Object[] { GURL_EVENT });\r
+                Object appleEvent = appleEventConstructor.newInstance(new Object[] {\r
+                            gurlType, gurlType, aeTarget, kAutoGenerateReturnID,\r
+                            kAnyTransactionID\r
+                        });\r
+\r
+                // Don't set browser = appleEvent because then the next time we call\r
+                // locateBrowser(), we'll get the same AppleEvent, to which we'll already have\r
+                // added the relevant parameter. Instead, regenerate the AppleEvent every time.\r
+                // There's probably a way to do this better; if any has any ideas, please let\r
+                // me know.\r
+                return appleEvent;\r
+            } catch (IllegalAccessException iae) {\r
+                browser = null;\r
+                errorMessage = iae.getMessage();\r
+\r
+                return browser;\r
+            } catch (InstantiationException ie) {\r
+                browser = null;\r
+                errorMessage = ie.getMessage();\r
+\r
+                return browser;\r
+            } catch (InvocationTargetException ite) {\r
+                browser = null;\r
+                errorMessage = ite.getMessage();\r
+\r
+                return browser;\r
+            }\r
+\r
+        case MRJ_2_1:\r
+\r
+            File systemFolder;\r
+\r
+            try {\r
+                systemFolder = (File) findFolder.invoke(null,\r
+                        new Object[] { kSystemFolderType });\r
+            } catch (IllegalArgumentException iare) {\r
+                browser = null;\r
+                errorMessage = iare.getMessage();\r
+\r
+                return browser;\r
+            } catch (IllegalAccessException iae) {\r
+                browser = null;\r
+                errorMessage = iae.getMessage();\r
+\r
+                return browser;\r
+            } catch (InvocationTargetException ite) {\r
+                browser = null;\r
+                errorMessage = ite.getTargetException().getClass() + ": " +\r
+                    ite.getTargetException().getMessage();\r
+\r
+                return browser;\r
+            }\r
+\r
+            String[] systemFolderFiles = systemFolder.list();\r
+\r
+            // Avoid a FilenameFilter because that can't be stopped mid-list\r
+            for (int i = 0; i < systemFolderFiles.length; i++) {\r
+                try {\r
+                    File file = new File(systemFolder, systemFolderFiles[i]);\r
+\r
+                    if (!file.isFile()) {\r
+                        continue;\r
+                    }\r
+\r
+                    // We're looking for a file with a creator code of 'MACS' and\r
+                    // a type of 'FNDR'.  Only requiring the type results in non-Finder\r
+                    // applications being picked up on certain Mac OS 9 systems,\r
+                    // especially German ones, and sending a GURL event to those\r
+                    // applications results in a logout under Multiple Users.\r
+                    Object fileType = getFileType.invoke(null,\r
+                            new Object[] { file });\r
+\r
+                    if (FINDER_TYPE.equals(fileType.toString())) {\r
+                        Object fileCreator = getFileCreator.invoke(null,\r
+                                new Object[] { file });\r
+\r
+                        if (FINDER_CREATOR.equals(fileCreator.toString())) {\r
+                            browser = file.toString(); // Actually the Finder, but that's OK\r
+\r
+                            return browser;\r
+                        }\r
+                    }\r
+                } catch (IllegalArgumentException iare) {\r
+                    browser = browser;\r
+                    errorMessage = iare.getMessage();\r
+\r
+                    return null;\r
+                } catch (IllegalAccessException iae) {\r
+                    browser = null;\r
+                    errorMessage = iae.getMessage();\r
+\r
+                    return browser;\r
+                } catch (InvocationTargetException ite) {\r
+                    browser = null;\r
+                    errorMessage = ite.getTargetException().getClass() + ": " +\r
+                        ite.getTargetException().getMessage();\r
+\r
+                    return browser;\r
+                }\r
+            }\r
+\r
+            browser = null;\r
+\r
+            break;\r
+\r
+        case MRJ_3_0:\r
+        case MRJ_3_1:\r
+            browser = ""; // Return something non-null\r
+\r
+            break;\r
+\r
+        case WINDOWS_NT:\r
+            browser = "cmd.exe";\r
+\r
+            break;\r
+\r
+        case WINDOWS_9x:\r
+            browser = "command.com";\r
+\r
+            break;\r
+\r
+        case OTHER:default:\r
+            browser = jalview.bin.Cache.applicationProperties.getProperty(\r
+                    "jalview.browser");\r
+\r
+            if (browser == null) {\r
+                // hope firefox exists :-/\r
+                browser = "firefox";\r
+            }\r
+\r
+            break;\r
+        }\r
+\r
+        return browser;\r
+    }\r
+\r
+    /**
+ * Attempts to open the default web browser to the given URL.
+ * @param url The URL to open
+ * @throws IOException If the web browser could not be located or does not run
+ */\r
+    public static void openURL(String url) throws IOException {\r
+        if (!loadedWithoutErrors) {\r
+            throw new IOException("Exception in finding browser: " +\r
+                errorMessage);\r
+        }\r
+\r
+        Object browser = locateBrowser();\r
+\r
+        if (browser == null) {\r
+            throw new IOException("Unable to locate browser: " + errorMessage);\r
+        }\r
+\r
+        switch (jvm) {\r
+        case MRJ_2_0:\r
+\r
+            Object aeDesc = null;\r
+\r
+            try {\r
+                aeDesc = aeDescConstructor.newInstance(new Object[] { url });\r
+                putParameter.invoke(browser,\r
+                    new Object[] { keyDirectObject, aeDesc });\r
+                sendNoReply.invoke(browser, new Object[] {  });\r
+            } catch (InvocationTargetException ite) {\r
+                throw new IOException(\r
+                    "InvocationTargetException while creating AEDesc: " +\r
+                    ite.getMessage());\r
+            } catch (IllegalAccessException iae) {\r
+                throw new IOException(\r
+                    "IllegalAccessException while building AppleEvent: " +\r
+                    iae.getMessage());\r
+            } catch (InstantiationException ie) {\r
+                throw new IOException(\r
+                    "InstantiationException while creating AEDesc: " +\r
+                    ie.getMessage());\r
+            } finally {\r
+                aeDesc = null; // Encourage it to get disposed if it was created\r
+                browser = null; // Ditto\r
+            }\r
+\r
+            break;\r
+\r
+        case MRJ_2_1:\r
+            Runtime.getRuntime().exec(new String[] { (String) browser, url });\r
+\r
+            break;\r
+\r
+        case MRJ_3_0:\r
+\r
+            int[] instance = new int[1];\r
+            int result = ICStart(instance, 0);\r
+\r
+            if (result == 0) {\r
+                int[] selectionStart = new int[] { 0 };\r
+                byte[] urlBytes = url.getBytes();\r
+                int[] selectionEnd = new int[] { urlBytes.length };\r
+                result = ICLaunchURL(instance[0], new byte[] { 0 }, urlBytes,\r
+                        urlBytes.length, selectionStart, selectionEnd);\r
+\r
+                if (result == 0) {\r
+                    // Ignore the return value; the URL was launched successfully\r
+                    // regardless of what happens here.\r
+                    ICStop(instance);\r
+                } else {\r
+                    throw new IOException("Unable to launch URL: " + result);\r
+                }\r
+            } else {\r
+                throw new IOException(\r
+                    "Unable to create an Internet Config instance: " + result);\r
+            }\r
+\r
+            break;\r
+\r
+        case MRJ_3_1:\r
+\r
+            try {\r
+                openURL.invoke(null, new Object[] { url });\r
+            } catch (InvocationTargetException ite) {\r
+                throw new IOException(\r
+                    "InvocationTargetException while calling openURL: " +\r
+                    ite.getMessage());\r
+            } catch (IllegalAccessException iae) {\r
+                throw new IOException(\r
+                    "IllegalAccessException while calling openURL: " +\r
+                    iae.getMessage());\r
+            }\r
+\r
+            break;\r
+\r
+        case WINDOWS_NT:\r
+        case WINDOWS_9x:\r
+\r
+            // Add quotes around the URL to allow ampersands and other special\r
+            // characters to work.\r
+            Process process = Runtime.getRuntime().exec(new String[] {\r
+                        (String) browser, FIRST_WINDOWS_PARAMETER,\r
+                        SECOND_WINDOWS_PARAMETER, THIRD_WINDOWS_PARAMETER,\r
+                        '"' + url + '"'\r
+                    });\r
+\r
+            // This avoids a memory leak on some versions of Java on Windows.\r
+            // That's hinted at in <http://developer.java.sun.com/developer/qow/archive/68/>.\r
+            try {\r
+                process.waitFor();\r
+                process.exitValue();\r
+            } catch (InterruptedException ie) {\r
+                throw new IOException(\r
+                    "InterruptedException while launching browser: " +\r
+                    ie.getMessage());\r
+            }\r
+\r
+            break;\r
+\r
+        case OTHER:\r
+\r
+            // Assume that we're on Unix and that Netscape (actually Firefox) is installed\r
+            // First, attempt to open the URL in a currently running session of Netscape\r
+            // JBPNote log debug\r
+\r
+            /* System.out.println("Executing : "+browser+" "+
+                                                                                                    NETSCAPE_REMOTE_PARAMETER+" "+
+                                                                                                    NETSCAPE_OPEN_PARAMETER_START +
+                                                                                                    url +
+                                                                                                    NETSCAPE_OPEN_NEW_WINDOW +
+                                                                                                    NETSCAPE_OPEN_PARAMETER_END);
+ */\r
+            process = Runtime.getRuntime().exec(new String[] {\r
+                        (String) browser, NETSCAPE_REMOTE_PARAMETER,\r
+                        \r
+                    NETSCAPE_OPEN_PARAMETER_START + url +\r
+                        NETSCAPE_OPEN_NEW_WINDOW + NETSCAPE_OPEN_PARAMETER_END\r
+                    });\r
+\r
+            try {\r
+                int exitCode = process.waitFor();\r
+\r
+                if (exitCode != 0) { // if Netscape was not open\r
+                    Runtime.getRuntime().exec(new String[] { (String) browser, url });\r
+                }\r
+            } catch (InterruptedException ie) {\r
+                throw new IOException(\r
+                    "InterruptedException while launching browser: " +\r
+                    ie.getMessage());\r
+            }\r
+\r
+            break;\r
+\r
+        default:\r
+\r
+            // This should never occur, but if it does, we'll try the simplest thing possible\r
+            Runtime.getRuntime().exec(new String[] { (String) browser, url });\r
+\r
+            break;\r
+        }\r
+    }\r
+\r
     /**
-     * The third parameter for Runtime.exec() on Windows.  This is a "title"
-     * parameter that the command line expects.  Setting this parameter allows
-     * URLs containing spaces to work.
-     */
-    private static final String THIRD_WINDOWS_PARAMETER = "\"\"";
-
-        /**
-         * The shell parameters for Netscape that opens a given URL in an already-open copy of Netscape
-         * on many command-line systems.
-         */
-        private static final String NETSCAPE_REMOTE_PARAMETER = "-remote";
-        private static final String NETSCAPE_OPEN_PARAMETER_START = "openURL(";
-        private static final String NETSCAPE_OPEN_NEW_WINDOW = ", new-window";
-        private static final String NETSCAPE_OPEN_PARAMETER_END = ")";
-
-        /**
-         * The message from any exception thrown throughout the initialization process.
-         */
-        private static String errorMessage;
-
-        /**
-         * An initialization block that determines the operating system and loads the necessary
-         * runtime data.
-         */
-        static {
-                loadedWithoutErrors = true;
-                String osName = System.getProperty("os.name");
-                if (osName.startsWith("Mac OS")) {
-                        String mrjVersion = System.getProperty("mrj.version");
-                        String majorMRJVersion = mrjVersion.substring(0, 3);
-                        try {
-                                double version = Double.valueOf(majorMRJVersion).doubleValue();
-                                if (version == 2) {
-                                        jvm = MRJ_2_0;
-                                } else if (version >= 2.1 && version < 3) {
-                                        // Assume that all 2.x versions of MRJ work the same.  MRJ 2.1 actually
-                                        // works via Runtime.exec() and 2.2 supports that but has an openURL() method
-                                        // as well that we currently ignore.
-                                        jvm = MRJ_2_1;
-                                } else if (version == 3.0) {
-                                        jvm = MRJ_3_0;
-                                } else if (version >= 3.1) {
-                                        // Assume that all 3.1 and later versions of MRJ work the same.
-                                        jvm = MRJ_3_1;
-                                } else {
-                                        loadedWithoutErrors = false;
-                                        errorMessage = "Unsupported MRJ version: " + version;
-                                }
-                        } catch (NumberFormatException nfe) {
-                                loadedWithoutErrors = false;
-                                errorMessage = "Invalid MRJ version: " + mrjVersion;
-                        }
-                } else if (osName.startsWith("Windows")) {
-                        if (osName.indexOf("9") != -1) {
-                                jvm = WINDOWS_9x;
-                        } else {
-                                jvm = WINDOWS_NT;
-                        }
-                } else {
-                        jvm = OTHER;
-                }
-
-                if (loadedWithoutErrors) {     // if we haven't hit any errors yet
-                        loadedWithoutErrors = loadClasses();
-                }
-        }
-
-        /**
-         * This class should be never be instantiated; this just ensures so.
-         */
-        private BrowserLauncher() { }
-
-        /**
-         * Called by a static initializer to load any classes, fields, and methods required at runtime
-         * to locate the user's web browser.
-         * @return <code>true</code> if all intialization succeeded
-         *                     <code>false</code> if any portion of the initialization failed
-         */
-        private static boolean loadClasses() {
-                switch (jvm) {
-                        case MRJ_2_0:
-                                try {
-                                        Class aeTargetClass = Class.forName("com.apple.MacOS.AETarget");
-                                        Class osUtilsClass = Class.forName("com.apple.MacOS.OSUtils");
-                                        Class appleEventClass = Class.forName("com.apple.MacOS.AppleEvent");
-                                        Class aeClass = Class.forName("com.apple.MacOS.ae");
-                                        aeDescClass = Class.forName("com.apple.MacOS.AEDesc");
-
-                                        aeTargetConstructor = aeTargetClass.getDeclaredConstructor(new Class [] { int.class });
-                                        appleEventConstructor = appleEventClass.getDeclaredConstructor(new Class[] { int.class, int.class, aeTargetClass, int.class, int.class });
-                                        aeDescConstructor = aeDescClass.getDeclaredConstructor(new Class[] { String.class });
-
-                                        makeOSType = osUtilsClass.getDeclaredMethod("makeOSType", new Class [] { String.class });
-                                        putParameter = appleEventClass.getDeclaredMethod("putParameter", new Class[] { int.class, aeDescClass });
-                                        sendNoReply = appleEventClass.getDeclaredMethod("sendNoReply", new Class[] { });
-
-                                        Field keyDirectObjectField = aeClass.getDeclaredField("keyDirectObject");
-                                        keyDirectObject = (Integer) keyDirectObjectField.get(null);
-                                        Field autoGenerateReturnIDField = appleEventClass.getDeclaredField("kAutoGenerateReturnID");
-                                        kAutoGenerateReturnID = (Integer) autoGenerateReturnIDField.get(null);
-                                        Field anyTransactionIDField = appleEventClass.getDeclaredField("kAnyTransactionID");
-                                        kAnyTransactionID = (Integer) anyTransactionIDField.get(null);
-                                } catch (ClassNotFoundException cnfe) {
-                                        errorMessage = cnfe.getMessage();
-                                        return false;
-                                } catch (NoSuchMethodException nsme) {
-                                        errorMessage = nsme.getMessage();
-                                        return false;
-                                } catch (NoSuchFieldException nsfe) {
-                                        errorMessage = nsfe.getMessage();
-                                        return false;
-                                } catch (IllegalAccessException iae) {
-                                        errorMessage = iae.getMessage();
-                                        return false;
-                                }
-                                break;
-                        case MRJ_2_1:
-                                try {
-                                        mrjFileUtilsClass = Class.forName("com.apple.mrj.MRJFileUtils");
-                                        mrjOSTypeClass = Class.forName("com.apple.mrj.MRJOSType");
-                                        Field systemFolderField = mrjFileUtilsClass.getDeclaredField("kSystemFolderType");
-                                        kSystemFolderType = systemFolderField.get(null);
-                                        findFolder = mrjFileUtilsClass.getDeclaredMethod("findFolder", new Class[] { mrjOSTypeClass });
-                                        getFileCreator = mrjFileUtilsClass.getDeclaredMethod("getFileCreator", new Class[] { File.class });
-                                        getFileType = mrjFileUtilsClass.getDeclaredMethod("getFileType", new Class[] { File.class });
-                                } catch (ClassNotFoundException cnfe) {
-                                        errorMessage = cnfe.getMessage();
-                                        return false;
-                                } catch (NoSuchFieldException nsfe) {
-                                        errorMessage = nsfe.getMessage();
-                                        return false;
-                                } catch (NoSuchMethodException nsme) {
-                                        errorMessage = nsme.getMessage();
-                                        return false;
-                                } catch (SecurityException se) {
-                                        errorMessage = se.getMessage();
-                                        return false;
-                                } catch (IllegalAccessException iae) {
-                                        errorMessage = iae.getMessage();
-                                        return false;
-                                }
-                                break;
-                        case MRJ_3_0:
-                            try {
-                                        Class linker = Class.forName("com.apple.mrj.jdirect.Linker");
-                                        Constructor constructor = linker.getConstructor(new Class[]{ Class.class });
-                                        linkage = constructor.newInstance(new Object[] { BrowserLauncher.class });
-                                } catch (ClassNotFoundException cnfe) {
-                                        errorMessage = cnfe.getMessage();
-                                        return false;
-                                } catch (NoSuchMethodException nsme) {
-                                        errorMessage = nsme.getMessage();
-                                        return false;
-                                } catch (InvocationTargetException ite) {
-                                        errorMessage = ite.getMessage();
-                                        return false;
-                                } catch (InstantiationException ie) {
-                                        errorMessage = ie.getMessage();
-                                        return false;
-                                } catch (IllegalAccessException iae) {
-                                        errorMessage = iae.getMessage();
-                                        return false;
-                                }
-                                break;
-                        case MRJ_3_1:
-                                try {
-                                        mrjFileUtilsClass = Class.forName("com.apple.mrj.MRJFileUtils");
-                                        openURL = mrjFileUtilsClass.getDeclaredMethod("openURL", new Class[] { String.class });
-                                } catch (ClassNotFoundException cnfe) {
-                                        errorMessage = cnfe.getMessage();
-                                        return false;
-                                } catch (NoSuchMethodException nsme) {
-                                        errorMessage = nsme.getMessage();
-                                        return false;
-                                }
-                                break;
-                        default:
-                            break;
-                }
-                return true;
-        }
-
-        /**
-         * Attempts to locate the default web browser on the local system.  Caches results so it
-         * only locates the browser once for each use of this class per JVM instance.
-         * @return The browser for the system.  Note that this may not be what you would consider
-         *                     to be a standard web browser; instead, it's the application that gets called to
-         *                     open the default web browser.  In some cases, this will be a non-String object
-         *                     that provides the means of calling the default browser.
-         */
-        private static Object locateBrowser() {
-                if (browser != null) {
-                        return browser;
-                }
-                switch (jvm) {
-                        case MRJ_2_0:
-                                try {
-                                        Integer finderCreatorCode = (Integer) makeOSType.invoke(null, new Object[] { FINDER_CREATOR });
-                                        Object aeTarget = aeTargetConstructor.newInstance(new Object[] { finderCreatorCode });
-                                        Integer gurlType = (Integer) makeOSType.invoke(null, new Object[] { GURL_EVENT });
-                                        Object appleEvent = appleEventConstructor.newInstance(new Object[] { gurlType, gurlType, aeTarget, kAutoGenerateReturnID, kAnyTransactionID });
-                                        // Don't set browser = appleEvent because then the next time we call
-                                        // locateBrowser(), we'll get the same AppleEvent, to which we'll already have
-                                        // added the relevant parameter. Instead, regenerate the AppleEvent every time.
-                                        // There's probably a way to do this better; if any has any ideas, please let
-                                        // me know.
-                                        return appleEvent;
-                                } catch (IllegalAccessException iae) {
-                                        browser = null;
-                                        errorMessage = iae.getMessage();
-                                        return browser;
-                                } catch (InstantiationException ie) {
-                                        browser = null;
-                                        errorMessage = ie.getMessage();
-                                        return browser;
-                                } catch (InvocationTargetException ite) {
-                                        browser = null;
-                                        errorMessage = ite.getMessage();
-                                        return browser;
-                                }
-                        case MRJ_2_1:
-                                File systemFolder;
-                                try {
-                                        systemFolder = (File) findFolder.invoke(null, new Object[] { kSystemFolderType });
-                                } catch (IllegalArgumentException iare) {
-                                        browser = null;
-                                        errorMessage = iare.getMessage();
-                                        return browser;
-                                } catch (IllegalAccessException iae) {
-                                        browser = null;
-                                        errorMessage = iae.getMessage();
-                                        return browser;
-                                } catch (InvocationTargetException ite) {
-                                        browser = null;
-                                        errorMessage = ite.getTargetException().getClass() + ": " + ite.getTargetException().getMessage();
-                                        return browser;
-                                }
-                                String[] systemFolderFiles = systemFolder.list();
-                                // Avoid a FilenameFilter because that can't be stopped mid-list
-                                for(int i = 0; i < systemFolderFiles.length; i++) {
-                                        try {
-                                                File file = new File(systemFolder, systemFolderFiles[i]);
-                                                if (!file.isFile()) {
-                                                        continue;
-                                                }
-                                                // We're looking for a file with a creator code of 'MACS' and
-                                                // a type of 'FNDR'.  Only requiring the type results in non-Finder
-                                                // applications being picked up on certain Mac OS 9 systems,
-                                                // especially German ones, and sending a GURL event to those
-                                                // applications results in a logout under Multiple Users.
-                                                Object fileType = getFileType.invoke(null, new Object[] { file });
-                                                if (FINDER_TYPE.equals(fileType.toString())) {
-                                                        Object fileCreator = getFileCreator.invoke(null, new Object[] { file });
-                                                        if (FINDER_CREATOR.equals(fileCreator.toString())) {
-                                                                browser = file.toString();     // Actually the Finder, but that's OK
-                                                                return browser;
-                                                        }
-                                                }
-                                        } catch (IllegalArgumentException iare) {
-                                                browser = browser;
-                                                errorMessage = iare.getMessage();
-                                                return null;
-                                        } catch (IllegalAccessException iae) {
-                                                browser = null;
-                                                errorMessage = iae.getMessage();
-                                                return browser;
-                                        } catch (InvocationTargetException ite) {
-                                                browser = null;
-                                                errorMessage = ite.getTargetException().getClass() + ": " + ite.getTargetException().getMessage();
-                                                return browser;
-                                        }
-                                }
-                                browser = null;
-                                break;
-                        case MRJ_3_0:
-                        case MRJ_3_1:
-                                browser = "";  // Return something non-null
-                                break;
-                        case WINDOWS_NT:
-                                browser = "cmd.exe";
-                                break;
-                        case WINDOWS_9x:
-                                browser = "command.com";
-                                break;
-                        case OTHER:
-                        default:
-                                browser = jalview.bin.Cache.applicationProperties.getProperty("jalview.browser");
-                                if (browser==null) {
-                                  // hope firefox exists :-/
-                                  browser = "firefox";
-                                }
-                                break;
-                }
-                return browser;
-        }
-
-        /**
-         * Attempts to open the default web browser to the given URL.
-         * @param url The URL to open
-         * @throws IOException If the web browser could not be located or does not run
-         */
-        public static void openURL(String url) throws IOException {
-                if (!loadedWithoutErrors) {
-                        throw new IOException("Exception in finding browser: " + errorMessage);
-                }
-                Object browser = locateBrowser();
-                if (browser == null) {
-                        throw new IOException("Unable to locate browser: " + errorMessage);
-                }
-
-                switch (jvm) {
-                        case MRJ_2_0:
-                                Object aeDesc = null;
-                                try {
-                                        aeDesc = aeDescConstructor.newInstance(new Object[] { url });
-                                        putParameter.invoke(browser, new Object[] { keyDirectObject, aeDesc });
-                                        sendNoReply.invoke(browser, new Object[] { });
-                                } catch (InvocationTargetException ite) {
-                                        throw new IOException("InvocationTargetException while creating AEDesc: " + ite.getMessage());
-                                } catch (IllegalAccessException iae) {
-                                        throw new IOException("IllegalAccessException while building AppleEvent: " + iae.getMessage());
-                                } catch (InstantiationException ie) {
-                                        throw new IOException("InstantiationException while creating AEDesc: " + ie.getMessage());
-                                } finally {
-                                        aeDesc = null; // Encourage it to get disposed if it was created
-                                        browser = null;        // Ditto
-                                }
-                                break;
-                        case MRJ_2_1:
-                                Runtime.getRuntime().exec(new String[] { (String) browser, url } );
-                                break;
-                        case MRJ_3_0:
-                                int[] instance = new int[1];
-                                int result = ICStart(instance, 0);
-                                if (result == 0) {
-                                        int[] selectionStart = new int[] { 0 };
-                                        byte[] urlBytes = url.getBytes();
-                                        int[] selectionEnd = new int[] { urlBytes.length };
-                                        result = ICLaunchURL(instance[0], new byte[] { 0 }, urlBytes,
-                                                                                        urlBytes.length, selectionStart,
-                                                                                        selectionEnd);
-                                        if (result == 0) {
-                                                // Ignore the return value; the URL was launched successfully
-                                                // regardless of what happens here.
-                                                ICStop(instance);
-                                        } else {
-                                                throw new IOException("Unable to launch URL: " + result);
-                                        }
-                                } else {
-                                        throw new IOException("Unable to create an Internet Config instance: " + result);
-                                }
-                                break;
-                        case MRJ_3_1:
-                                try {
-                                        openURL.invoke(null, new Object[] { url });
-                                } catch (InvocationTargetException ite) {
-                                        throw new IOException("InvocationTargetException while calling openURL: " + ite.getMessage());
-                                } catch (IllegalAccessException iae) {
-                                        throw new IOException("IllegalAccessException while calling openURL: " + iae.getMessage());
-                                }
-                                break;
-                    case WINDOWS_NT:
-                    case WINDOWS_9x:
-                            // Add quotes around the URL to allow ampersands and other special
-                            // characters to work.
-                                Process process = Runtime.getRuntime().exec(new String[] { (String) browser,
-                                                                                                                                FIRST_WINDOWS_PARAMETER,
-                                                                                                                                SECOND_WINDOWS_PARAMETER,
-                                                                                                                                THIRD_WINDOWS_PARAMETER,
-                                                                                                                                '"' + url + '"' });
-                                // This avoids a memory leak on some versions of Java on Windows.
-                                // That's hinted at in <http://developer.java.sun.com/developer/qow/archive/68/>.
-                                try {
-                                        process.waitFor();
-                                        process.exitValue();
-                                } catch (InterruptedException ie) {
-                                        throw new IOException("InterruptedException while launching browser: " + ie.getMessage());
-                                }
-                                break;
-                        case OTHER:
-                                // Assume that we're on Unix and that Netscape (actually Firefox) is installed
-
-                                // First, attempt to open the URL in a currently running session of Netscape
-                                process = Runtime.getRuntime().exec(new String[] { (String) browser,
-                                                                                                        NETSCAPE_REMOTE_PARAMETER,
-                                                                                                        NETSCAPE_OPEN_PARAMETER_START +
-                                                                                                        url +
-                                                                                                        NETSCAPE_OPEN_NEW_WINDOW +
-                                                                                                        NETSCAPE_OPEN_PARAMETER_END });
-                                try {
-                                        int exitCode = process.waitFor();
-                                        if (exitCode != 0) {   // if Netscape was not open
-                                                Runtime.getRuntime().exec(new String[] { (String) browser, url });
-                                        }
-                                } catch (InterruptedException ie) {
-                                        throw new IOException("InterruptedException while launching browser: " + ie.getMessage());
-                                }
-                                break;
-                        default:
-                                // This should never occur, but if it does, we'll try the simplest thing possible
-                                Runtime.getRuntime().exec(new String[] { (String) browser, url });
-                                break;
-                }
-        }
-
-        /**
-         * Methods required for Mac OS X.  The presence of native methods does not cause
-         * any problems on other platforms.
-         */
-        private native static int ICStart(int[] instance, int signature);
-        private native static int ICStop(int[] instance);
-        private native static int ICLaunchURL(int instance, byte[] hint, byte[] data, int len,
-                                                                                        int[] selectionStart, int[] selectionEnd);
-}
+ * Methods required for Mac OS X.  The presence of native methods does not cause
+ * any problems on other platforms.
+ */\r
+    private native static int ICStart(int[] instance, int signature);\r
+\r
+    private native static int ICStop(int[] instance);\r
+\r
+    private native static int ICLaunchURL(int instance, byte[] hint,\r
+        byte[] data, int len, int[] selectionStart, int[] selectionEnd);\r
+}\r