Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(218)

Unified Diff: third_party/xdg-utils/scripts/xdg-settings.in

Issue 159738: Linux: skip invisible desktop files when looking up the default browser. (Closed) Base URL: svn://chrome-svn/chrome/trunk/deps/
Patch Set: Created 11 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/xdg-utils/scripts/xdg-settings ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/xdg-utils/scripts/xdg-settings.in
===================================================================
--- third_party/xdg-utils/scripts/xdg-settings.in (revision 22205)
+++ third_party/xdg-utils/scripts/xdg-settings.in (working copy)
@@ -215,6 +215,8 @@
[ -r "$file" ] || continue
# Check to make sure it's worth the processing
grep -q "^Exec.*$base" "$file" || continue
+ # Make sure it's a visible desktop file (e.g. not "preferred-web-browser.desktop")
+ grep -q "^NoDisplay=true" "$file" && continue
command="`grep "^Exec\(\[[^]=]*]\)\?=" "$file" | cut -d= -f 2- | first_word`"
full="`which "$command"`"
if [ x"$full" = x"$binary" ]; then
« no previous file with comments | « third_party/xdg-utils/scripts/xdg-settings ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698