file chooser: Do less work
authorMatthias Clasen <mclasen@redhat.com>
Thu, 9 Jul 2015 01:09:23 +0000 (21:09 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Sat, 11 Jul 2015 05:15:42 +0000 (00:15 -0500)
We only need to update the sensitivity of the context menu
items rihgt before we are going to pop it up. Everything else
is pointless work.

gtk/gtkfilechooserwidget.c

index 6caae6c005d7f15895a79e745bd2e92e00d3cd95..d6fe28144e802714594da11d0a96ac471e6f76df 100644 (file)
@@ -2239,8 +2239,6 @@ file_list_build_popup_menu (GtkFileChooserWidget *impl)
 
   priv->sort_directories_item
     = file_list_add_check_menu_item (impl, _("Sort _Folders before Files"), G_CALLBACK (sort_directories_toggled_cb));
-
-  check_file_list_menu_sensitivity (impl);
 }
 
 /* Updates the popup menu for the file list, creating it if necessary */
@@ -2250,6 +2248,7 @@ file_list_update_popup_menu (GtkFileChooserWidget *impl)
   GtkFileChooserWidgetPrivate *priv = impl->priv;
 
   file_list_build_popup_menu (impl);
+  check_file_list_menu_sensitivity (impl);
 
   /* The sensitivity of the Add to Bookmarks item is set in
    * bookmarks_check_add_sensitivity()
@@ -7610,7 +7609,6 @@ list_selection_changed (GtkTreeSelection     *selection,
   location_bar_update (impl);
 
   check_preview_change (impl);
-  check_file_list_menu_sensitivity (impl);
 
   g_signal_emit_by_name (impl, "selection-changed", 0);
 }