widget: Insist that widgets are rooted to be focusable
authorBenjamin Otte <otte@redhat.com>
Tue, 15 Oct 2019 14:11:03 +0000 (16:11 +0200)
committerBenjamin Otte <otte@redhat.com>
Tue, 15 Oct 2019 14:33:05 +0000 (16:33 +0200)
This check was in the vfunc before, but it's a general rule, so apply it
before.

gtk/gtkwidget.c

index 592754c2679708aee4f73b02ce76a52d723da4fd..c3e0e288d2477fa12eec144a05c948de556eef23 100644 (file)
@@ -5425,7 +5425,8 @@ gtk_widget_grab_focus (GtkWidget *widget)
 {
   g_return_val_if_fail (GTK_IS_WIDGET (widget), FALSE);
 
-  if (!gtk_widget_is_sensitive (widget))
+  if (!gtk_widget_is_sensitive (widget) ||
+      widget->priv->root == NULL)
     return FALSE;
 
   return GTK_WIDGET_GET_CLASS (widget)->grab_focus (widget);
@@ -5436,9 +5437,6 @@ gtk_widget_real_grab_focus (GtkWidget *focus_widget)
 {
   GtkWidgetPrivate *priv = gtk_widget_get_instance_private (focus_widget);
 
-  if (!priv->root)
-    return FALSE;
-
   gtk_root_set_focus (priv->root, focus_widget);
   return TRUE;
 }