projects / browser-switch / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Command-line utility: check that browser is installed before reporting it as the...
[browser-switch] / config-ui / browser-switchboard-cp.c
diff --git a/config-ui/browser-switchboard-cp.c b/config-ui/browser-switchboard-cp.c
index d4fcc8d..93f8e0a 100644 (file)
--- a/config-ui/browser-switchboard-cp.c
+++ b/config-ui/browser-switchboard-cp.c
@@ -52,6 +52,7 @@
 #endif /* HILDON */
 
 #include "config.h"
+#include "browsers.h"
 
 #define CONTINUOUS_MODE_DEFAULT 0
 
@@ -59,25 +60,10 @@
 #define _HILDON_SIZE_DEFAULT (HILDON_SIZE_AUTO_WIDTH|HILDON_SIZE_FINGER_HEIGHT)
 #endif
 
-struct browser_entry {
-       char *config;
-       char *displayname;
-};
-struct browser_entry browsers[] = {
-       { "microb", "MicroB (stock browser)" }, /* First entry is the default! */
-       { "tear", "Tear" },
-       { "fennec", "Mobile Firefox (Fennec)" },
-       { "opera", "Opera Mobile" },
-       { "midori", "Midori" },
-       { "other", "Other" },
-       { NULL, NULL },
-};
-
 struct swb_config orig_cfg;
 
 struct config_widgets {
 #if defined(HILDON) && defined(FREMANTLE)
-       GtkWidget *continuous_mode_selector;
        GtkWidget *default_browser_selector;
 #else
        GtkWidget *continuous_mode_off_radio;
@@ -91,21 +77,29 @@ struct config_widgets cw;
 GtkWidget *dialog;
 
 
+struct browser_entry *installed_browsers;
+void init_installed_browsers(void) {
+       struct browser_entry *cur = browsers;
+       unsigned int count = 0;
+
+       installed_browsers = calloc(sizeof browsers, 1);
+       if (!installed_browsers)
+               exit(1);
+
+       count = 0;
+       for (; cur->config; ++cur)
+               if (!cur->binary || !access(cur->binary, X_OK))
+                       installed_browsers[count++] = *cur;
+}
+
 /**********************************************************************
  * Configuration routines
  **********************************************************************/
 
 #if defined(HILDON) && defined(FREMANTLE)
 
-static inline int get_continuous_mode(void) {
-       return hildon_touch_selector_get_active(HILDON_TOUCH_SELECTOR(cw.continuous_mode_selector), 0);
-}
-static inline void set_continuous_mode(int state) {
-       hildon_touch_selector_set_active(HILDON_TOUCH_SELECTOR(cw.continuous_mode_selector), 0, state);
-}
-
 static inline char *get_default_browser(void) {
-       return browsers[hildon_touch_selector_get_active(HILDON_TOUCH_SELECTOR(cw.default_browser_selector), 0)].config;
+       return installed_browsers[hildon_touch_selector_get_active(HILDON_TOUCH_SELECTOR(cw.default_browser_selector), 0)].config;
 }
 
 #else /* !defined(HILDON) || !defined(FREMANTLE) */
@@ -121,7 +115,7 @@ static inline void set_continuous_mode(int state) {
 }
 
 static inline char *get_default_browser(void) {
-       return browsers[gtk_combo_box_get_active(GTK_COMBO_BOX(cw.default_browser_combo))].config;
+       return installed_browsers[gtk_combo_box_get_active(GTK_COMBO_BOX(cw.default_browser_combo))].config;
 }
 
 #endif /* defined(HILDON) && defined(FREMANTLE) */
@@ -130,10 +124,11 @@ static void set_default_browser(char *browser) {
        gint i;
 
        /* Loop through browsers looking for a match */
-       for (i = 0; browsers[i].config && strcmp(browsers[i].config, browser);
-                       ++i);
+       for (i = 0;
+            installed_browsers[i].config && strcmp(installed_browsers[i].config, browser);
+            ++i);
 
-       if (!browsers[i].config)
+       if (!installed_browsers[i].config)
                /* No match found, set to the default browser */
                i = 0;
 
@@ -156,7 +151,9 @@ static void load_config(void) {
        
        swb_config_load(&orig_cfg);
 
+#ifndef FREMANTLE
        set_continuous_mode(orig_cfg.continuous_mode);
+#endif
        set_default_browser(orig_cfg.default_browser);
        if (orig_cfg.other_browser_cmd)
                set_other_browser_cmd(orig_cfg.other_browser_cmd);
@@ -167,18 +164,22 @@ static void save_config(void) {
 
        swb_config_copy(&new_cfg, &orig_cfg);
 
+#ifndef FREMANTLE
        if (get_continuous_mode() != orig_cfg.continuous_mode) {
                new_cfg.continuous_mode = get_continuous_mode();
                new_cfg.flags |= SWB_CONFIG_CONTINUOUS_MODE_SET;
        }
+#endif
        if (strcmp(get_default_browser(), orig_cfg.default_browser)) {
                new_cfg.default_browser = get_default_browser();
                new_cfg.flags |= SWB_CONFIG_DEFAULT_BROWSER_SET;
        }
-       if ((orig_cfg.other_browser_cmd &&
-            strcmp(get_other_browser_cmd(), orig_cfg.other_browser_cmd)) ||
-           (!orig_cfg.other_browser_cmd &&
-            strlen(get_other_browser_cmd()) > 0)) {
+       if (strlen(get_other_browser_cmd()) == 0) {
+               new_cfg.other_browser_cmd = NULL;
+               new_cfg.flags &= ~SWB_CONFIG_OTHER_BROWSER_CMD_SET;
+       } else if (!(orig_cfg.other_browser_cmd &&
+                    !strcmp(get_other_browser_cmd(),
+                            orig_cfg.other_browser_cmd))) {
                new_cfg.other_browser_cmd = get_other_browser_cmd();
                new_cfg.flags |= SWB_CONFIG_OTHER_BROWSER_CMD_SET;
        }
@@ -230,7 +231,6 @@ static GtkDialog *swb_config_dialog(gpointer cp_window) {
        GtkWidget *dialog_vbox;
 
        GtkWidget *default_browser_selector_button;
-       GtkWidget *continuous_mode_selector_button;
        int i;
        HildonGtkInputMode input_mode;
 
@@ -247,9 +247,10 @@ static GtkDialog *swb_config_dialog(gpointer cp_window) {
        dialog_vbox = GTK_DIALOG(dialog)->vbox;
 
        /* Config options */
+       init_installed_browsers();
        cw.default_browser_selector = hildon_touch_selector_new_text();
-       for (i = 0; browsers[i].config; ++i)
-               hildon_touch_selector_append_text(HILDON_TOUCH_SELECTOR(cw.default_browser_selector), browsers[i].displayname);
+       for (i = 0; installed_browsers[i].config; ++i)
+               hildon_touch_selector_append_text(HILDON_TOUCH_SELECTOR(cw.default_browser_selector), installed_browsers[i].displayname);
        hildon_touch_selector_set_active(HILDON_TOUCH_SELECTOR(cw.default_browser_selector), 0, 0);
        default_browser_selector_button = hildon_picker_button_new(_HILDON_SIZE_DEFAULT, HILDON_BUTTON_ARRANGEMENT_HORIZONTAL);
        hildon_button_set_title(HILDON_BUTTON(default_browser_selector_button),
@@ -279,20 +280,6 @@ static GtkDialog *swb_config_dialog(gpointer cp_window) {
        gtk_box_pack_start(GTK_BOX(dialog_vbox),
                           cw.other_browser_cmd_entry_label, FALSE, FALSE, 0);
 
-       cw.continuous_mode_selector = hildon_touch_selector_new_text();
-       hildon_touch_selector_append_text(HILDON_TOUCH_SELECTOR(cw.continuous_mode_selector), "Lower memory usage");
-       hildon_touch_selector_append_text(HILDON_TOUCH_SELECTOR(cw.continuous_mode_selector), "Faster browser startup time");
-
-       continuous_mode_selector_button = hildon_picker_button_new(_HILDON_SIZE_DEFAULT, HILDON_BUTTON_ARRANGEMENT_VERTICAL);
-       hildon_button_set_title(HILDON_BUTTON(continuous_mode_selector_button),
-                               "Optimize Browser Switchboard for:");
-       hildon_picker_button_set_selector(HILDON_PICKER_BUTTON(continuous_mode_selector_button), HILDON_TOUCH_SELECTOR(cw.continuous_mode_selector));
-       hildon_button_set_alignment(HILDON_BUTTON(continuous_mode_selector_button),
-                                   0, 0, 0, 0);
-       set_continuous_mode(CONTINUOUS_MODE_DEFAULT);
-       gtk_box_pack_start(GTK_BOX(dialog_vbox),
-                          continuous_mode_selector_button, FALSE, FALSE, 0);
-
        gtk_widget_show_all(dialog);
        return GTK_DIALOG(dialog);
 }
@@ -329,10 +316,11 @@ static GtkDialog *swb_config_dialog(gpointer cp_window) {
        gtk_table_set_row_spacings(GTK_TABLE(options_table), 5);
        gtk_box_pack_start(GTK_BOX(dialog_vbox), options_table, FALSE, FALSE, 0);
 
+       init_installed_browsers();
        cw.default_browser_combo = gtk_combo_box_new_text();
-       for (i = 0; browsers[i].config; ++i)
+       for (i = 0; installed_browsers[i].config; ++i)
                gtk_combo_box_append_text(GTK_COMBO_BOX(cw.default_browser_combo),
-                                         browsers[i].displayname);
+                                         installed_browsers[i].displayname);
        gtk_combo_box_set_active(GTK_COMBO_BOX(cw.default_browser_combo), 0);
        default_browser_combo_label = gtk_label_new("Default browser:");
        gtk_misc_set_alignment(GTK_MISC(default_browser_combo_label), 1, 0.5);
Unnamed repository; edit this file to name it for gitweb.