Makes it easier to switch to an alaternative Web browser in the 'New version is available' dialog on Windows.

src/net/java/sip/communicator/plugin/update/CheckForUpdatesMenuItemComponent.java

@@ -51,15 +51,16 @@ public JMenuItem getComponent()
                 = new JMenuItem(
                         Resources.getResources().getI18NString(
                                 "plugin.updatechecker.UPDATE_MENU_ENTRY"));
-            checkForUpdatesMenuItem.addActionListener(new ActionListener()
-            {
-                public void actionPerformed(ActionEvent e)
-                {
-                    UpdateActivator.getUpdateService().checkForUpdates(true);
-                }
-            });
+            checkForUpdatesMenuItem.addActionListener(
+                    new ActionListener()
+                    {
+                        public void actionPerformed(ActionEvent e)
+                        {
+                            UpdateActivator.getUpdateService().checkForUpdates(
+                                    true);
+                        }
+                    });
         }
-
         return checkForUpdatesMenuItem;
     }
 

src/net/java/sip/communicator/plugin/update/UpdateActivator.java

@@ -178,7 +178,7 @@ public void start(Object dependentService)
 
         if (OSUtils.IS_WINDOWS)
         {
-            updateService = new Update();
+            updateService = new UpdateServiceImpl();
 
             bundleContext.registerService(
                 UpdateService.class.getName(),

src/net/java/sip/communicator/plugin/update/UpdateServiceImpl.java

@@ -16,17 +16,16 @@
 import javax.swing.*;
 import javax.swing.text.*;
 
+import net.java.sip.communicator.plugin.desktoputil.*;
 import net.java.sip.communicator.service.gui.*;
 import net.java.sip.communicator.service.httputil.*;
 import net.java.sip.communicator.service.update.*;
 import net.java.sip.communicator.util.*;
 import net.java.sip.communicator.util.Logger;
-import net.java.sip.communicator.plugin.desktoputil.*;
 
 import org.jitsi.service.resources.*;
 import org.jitsi.service.version.*;
 import org.jitsi.util.*;
-// disambiguation
 
 /**
  * Implements checking for software updates, downloading and applying them i.e.
@@ -83,6 +82,53 @@ public class UpdateServiceImpl
     private static final String PROP_UPDATE_LINK
         = "net.java.sip.communicator.UPDATE_LINK";
 
+    /**
+     * Initializes a new Web browser <tt>Component</tt> instance and navigates
+     * it to a specific URL.
+     *
+     * @param url the URL to navigate the new Web browser <tt>Component</tt>
+     * instance
+     * @return the new Web browser <tt>Component</tt> instance which has been
+     * navigated to the specified <tt>url</tt>
+     */
+    private static Component createBrowser(String url)
+    {
+        // Initialize the user interface.
+        JEditorPane editorPane = new JEditorPane();
+
+        editorPane.setBorder(BorderFactory.createLoweredBevelBorder());
+        editorPane.setContentType("text/html");
+        editorPane.setEditable(false);
+
+        JScrollPane scrollPane = new JScrollPane();
+
+        scrollPane.setViewportView(editorPane);
+
+        // Navigate the user interface to the specified URL. 
+        try
+        {
+            Document document = editorPane.getDocument();
+
+            if (document instanceof AbstractDocument)
+                ((AbstractDocument) document).setAsynchronousLoadPriority(0);
+
+            editorPane.setPage(new URL(url));
+        }
+        catch (Throwable t)
+        {
+            if (t instanceof ThreadDeath)
+                throw (ThreadDeath) t;
+            else
+            {
+                logger.error(
+                        "Failed to navigate the Web browser to: " + url,
+                        t);
+            }
+        }
+
+        return editorPane;
+    }
+
     /**
      * Tries to create a new <tt>FileOutputStream</tt> for a temporary file into
      * which the setup is to be downloaded. Because temporary files generally
@@ -712,28 +758,12 @@ protected void close(boolean escaped)
         contentPane.setBorder(BorderFactory.createEmptyBorder(0, 10, 0, 10));
         contentPane.add(contentMessage, BorderLayout.NORTH);
 
-        JScrollPane scrollChanges = new JScrollPane();
-        scrollChanges.setPreferredSize(new Dimension(550, 200));
-        JEditorPane changesHtml = new JEditorPane();
-        changesHtml.setContentType("text/html");
-        changesHtml.setEditable(false);
-        changesHtml.setBorder(BorderFactory.createLoweredBevelBorder());
-        scrollChanges.setViewportView(changesHtml);
-        contentPane.add(scrollChanges, BorderLayout.CENTER);
-        try
-        {
-            Document changesHtmlDocument = changesHtml.getDocument();
+        Component browser = createBrowser(changesLink);
 
-            if (changesHtmlDocument instanceof AbstractDocument)
-            {
-                ((AbstractDocument) changesHtmlDocument)
-                    .setAsynchronousLoadPriority(0);
-            }
-            changesHtml.setPage(new URL(changesLink));
-        }
-        catch (Exception e)
+        if (browser != null)
         {
-            logger.error("Cannot set changes Page", e);
+            browser.setPreferredSize(new Dimension(550, 200));
+            contentPane.add(browser, BorderLayout.CENTER);
         }
 
         JPanel buttonPanel