Make GtkItemFactory produce GtkSeparatorMenuItems.

https://bugzilla.gnome.org/show_bug.cgi?id=694077
Comment out a file chooser test that doesn't pass
release: update NEWS for 2.24.16
release: doc updates for 2.24.16
release: post-release version bump
filechooser: Tighten up some assertions when actually selecting files in the treeview

The model must be fully loaded, and it must be inserted in the treeview.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>
filechooser: When stopping/clearing BROWSE model, really ensure that we end up in LOAD_EMPTY state

Previously we could end up in a situation where browse_list_model==NULL, and yet load_state==LOAD_FINISHED.
This is not a valid state.  So, when we get rid of the list model, really ensure that we end up
in LOAD_EMPTY so nothing assumes that there is a valid list model around.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>
filechooserbutton: Add tests for cancelling the dialog via closing it (delete-event), instead of by simulating the Cancel button (response cancel)

Signed-off-by: Federico Mena Quintero <federico@gnome.org>
filechooserbutton: In the tests, run through the dialog more than one time

This ensures that data maintained by the button while the dialog opens/closes remains consistent.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>
filechooserbutton: Don't emit file-set signal when the change is not the result of a user action

We only emit that signal when the user confirms the button's internal GtkFileChooserDialog,
or when he drags-and-drops stuff into the button.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>
filechooserbutton: In tests, sleep after opening the dialog

This may ensure that the dialog is actually done initializing.  We need to kill this
sleeping business and really use signals, sigh...

Signed-off-by: Federico Mena Quintero <federico@gnome.org>