Only run KDE browser fallback when necessary...

scripts/xdg-settings.in

 #!/bin/sh
 #---------------------------------------------
 #   xdg-settings
 #
 #   Utility script to get various settings from the desktop environment.
 #
 #   Refer to the usage() function below for usage.
 #
 #   Copyright 2009, Google Inc.
 #
@@ skipping 195 matching lines @@
       *)
         echo "$browser"
         ;;
     esac
 }
 
 # Reads the KDE browser setting, compensating for a bug in some versions of kreadconfig.
 read_kde_browser()
 {
     browser="`kreadconfig --file kdeglobals --group General --key BrowserApplication`"
-    if [ "$browser" ]; then
+    if [ x"$browser" != x ]; then
         echo "$browser"
+    else
+        # kreadconfig in KDE 4 may not notice Key[$*]=... localized settings, so
+        # check by hand if it didn't find anything (oddly kwriteconfig works
+        # fine though).
+        kdeglobals_dir=`kde${KDE_SESSION_VERSION}-config --path config  | cut -d ':' -f 1`
+        kdeglobals="$kdeglobals_dir/kdeglobals"
+        [ ! -f "$kdeglobals" ] && return
+        # This will only take the first value if there is more than one.
+        grep '^BrowserApplication\[$[^]=]*\]=' "$kdeglobals" | head -n 1 | cut -d= -f 2-
     fi
-    # kreadconfig in KDE 4 may not notice Key[$*]=... localized settings, so check
-    # by hand if it didn't find anything (oddly kwriteconfig works fine though).
-    kdeglobals_dir=`kde${KDE_SESSION_VERSION}-config --path config  | cut -d ':' -f 1`
-    kdeglobals="$kdeglobals_dir/kdeglobals"
-    [ ! -f "$kdeglobals" ] && return
-    # This will only take the first value if there is more than one.
-    grep '^BrowserApplication\[$[^]=]*\]=' "$kdeglobals" | head -n 1 | cut -d= -f 2-
 }
 
 get_browser_kde()
 {
     browser="`read_kde_browser`"
     if [ x"$browser" = x ]; then
         # No explicit default browser; KDE will use the MIME type text/html.
         get_browser_mime
     else
         resolve_kde_browser_desktop
@@ skipping 316 matching lines @@
     ;;
 
     generic)
     dispatch_generic "$@"
     ;;
 
     *)
     exit_failure_operation_impossible "unknown desktop environment"
     ;;
 esac