gtk+ r22214 - in trunk: . gtk

[matthiasc svn gnome org]

Author: matthiasc
Date: Sun Jan 25 03:38:27 2009
New Revision: 22214
URL: http://svn.gnome.org/viewvc/gtk+?rev=22214&view=rev

Log:
        Revert changes for bug 567124. Instead of trying to avoid
        setting up the im context before the widget is realized, just
        reset it when the client window is set.

        * gtk/gtkimmulticontext.c: Reset the slave when a client window
        is set.

        * gtk/gtkimmodule.c
        * gtk/gtktextview.c: Revert changes for bug 567124.


Modified:
   trunk/ChangeLog
   trunk/gtk/gtkimmodule.c
   trunk/gtk/gtkimmulticontext.c
   trunk/gtk/gtktextview.c

Modified: trunk/gtk/gtkimmodule.c
==============================================================================
--- trunk/gtk/gtkimmodule.c	(original)
+++ trunk/gtk/gtkimmodule.c	Sun Jan 25 03:38:27 2009
@@ -671,9 +671,6 @@
   GdkScreen *screen;
   GtkSettings *settings;
       
-  /* assertion to make sure all of the unexpected invocation is really gone. */
-  g_return_val_if_fail (client_window != NULL, SIMPLE_ID);
-
   if (!contexts_hash)
     gtk_im_module_initialize ();
 

Modified: trunk/gtk/gtkimmulticontext.c
==============================================================================
--- trunk/gtk/gtkimmulticontext.c	(original)
+++ trunk/gtk/gtkimmulticontext.c	Sun Jan 25 03:38:27 2009
@@ -226,7 +226,7 @@
   if (!multicontext->slave)
     {
       GtkIMContext *slave;
-
+      
       g_free (multicontext->context_id);
        
       if (multicontext->priv->context_id)
@@ -258,17 +258,13 @@
 				       GdkWindow    *window)
 {
   GtkIMMulticontext *multicontext = GTK_IM_MULTICONTEXT (context);
-  GtkIMContext *slave; 
   GdkScreen *screen; 
   GtkSettings *settings;
   gboolean connected;
 
   multicontext->priv->client_window = window;
 
-  slave = gtk_im_multicontext_get_slave (multicontext);
-
-  if (slave)
-    gtk_im_context_set_client_window (slave, window);
+  gtk_im_multicontext_set_slave (multicontext, NULL, FALSE);
 
   if (window == NULL) 
     return;

Modified: trunk/gtk/gtktextview.c
==============================================================================
--- trunk/gtk/gtktextview.c	(original)
+++ trunk/gtk/gtktextview.c	Sun Jan 25 03:38:27 2009
@@ -2076,8 +2076,7 @@
    */
   area.width = 0;
 
-  if (GTK_WIDGET_REALIZED (text_view))
-    gtk_im_context_set_cursor_location (text_view->im_context, &area);
+  gtk_im_context_set_cursor_location (text_view->im_context, &area);
 }
 
 static gboolean
@@ -3840,7 +3839,7 @@
     }
 
   /* Ensure updating the spot location. */
-  gtk_text_view_update_im_spot_location(text_view);
+  gtk_text_view_update_im_spot_location (text_view);
 }
 
 static void