/hg/icedtea-web: Added support for chromium binary (along with o...

[jvanek at icedtea.classpath.org]

changeset cb57d0303e8d in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=cb57d0303e8d
author: Jiri Vanek <jvanek at redhat.com>
date: Mon Oct 20 17:28:45 2014 +0200

	Added support for chromium binary (along with older chromium-browser one)


diffstat:

 ChangeLog                                                               |   7 ++++
 tests/test-extensions/net/sourceforge/jnlp/browsertesting/Browsers.java |  17 +++++++++-
 2 files changed, 23 insertions(+), 1 deletions(-)

diffs (55 lines):

diff -r 72a1ff50858b -r cb57d0303e8d ChangeLog
--- a/ChangeLog	Mon Oct 20 15:33:38 2014 +0200
+++ b/ChangeLog	Mon Oct 20 17:28:45 2014 +0200
@@ -1,3 +1,10 @@
+2014-10-20  Jiri Vanek  <jvanek at redhat.com>
+
+	Added support for chromium binary (along with older chromium-browser one)
+	* tests/test-extensions/net/sourceforge/jnlp/browsertesting/Browsers.java:
+	(static) check if legacy chromium-browser do exists. If so use it, otherwise
+	use newer chromium only
+
 2014-10-20  Jiri Vanek  <jvanek at redhat.com>
 
 	Support only @BOLD_OPEN@ and @BOLD_CLOSE@ in documentation
diff -r 72a1ff50858b -r cb57d0303e8d tests/test-extensions/net/sourceforge/jnlp/browsertesting/Browsers.java
--- a/tests/test-extensions/net/sourceforge/jnlp/browsertesting/Browsers.java	Mon Oct 20 15:33:38 2014 +0200
+++ b/tests/test-extensions/net/sourceforge/jnlp/browsertesting/Browsers.java	Mon Oct 20 17:28:45 2014 +0200
@@ -37,6 +37,9 @@
 
 package net.sourceforge.jnlp.browsertesting;
 
+import java.io.File;
+import net.sourceforge.jnlp.browsertesting.browsers.LinuxBrowser;
+
 /**
  * When all represent all configured browser, one represens one random
  * (the first found) configured browser. Each other represents inidivdual browsers
@@ -46,6 +49,18 @@
 
    none, all, one, opera, googleChrome, chromiumBrowser, firefox, midori,epiphany;
 
+    public static final String CHROMIUM;
+
+    static {
+        final String def = "chromium";
+        final String alt = "chromium-browser";
+        if (new File(LinuxBrowser.DEFAULT_BIN_PATH, alt).exists()) {
+            CHROMIUM = alt;
+        } else {
+            CHROMIUM = def;
+        }
+    }
+
     public String toExec() {
         switch (this) {
             case opera:
@@ -53,7 +68,7 @@
             case googleChrome:
                 return "google-chrome";
             case chromiumBrowser:
-                return "chromium-browser";
+                return CHROMIUM;
             case firefox:
                 return "firefox";
             case midori: