A quick fix for Issue 15531 and 10953...

A quick fix for Issue 15531 and 10953
This issue is caused by my another mistake that I forgot setting the modifier-key state in creating a Char event in RenderWidgetHostViewGtkWidget::ForwardCharEvent().
Since the GtkIMContext signal-handlers don't use GdkEventKey objects and cannot get the modififer-key state, this change save the state in RenderWidgetHostViewGtkWidget::KeyPressReleaseEvent() before dispatching a key event to the GtkIMContext object.

Also, this change adds gtk_im_context_focus_in() and gtk_im_context_focus_out() calls to fix compatibility problems with ibus.

BUG=10953
"IME support"
BUG=15531
"regression: shift-space doesn't scroll the page backwards"
TEST=Open a test page in <http://crbug.com/15531>;, press shift+space keys, and observe ev.shiftKey is true.  

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19707

[Hironori Bono, 2009-06-29 09:57:47]

To fix compatibility problems with IMEs (such as ibus), I have added a
gtk_im_context_focus_in() call to RenderWidgetHostViewGtkWidget::OnFocusIn() and
a gtk_im_context_focus_out() call in
RenderWidgetHostViewGtkWidget::OnFocusOut().
Also, it seems some IMEs get confuses because the current IME code calls
gtk_im_context_set_client_window() too often. To solve this issue, I moved
gtk_im_context_set_client_window() calls to
RenderWidgetHostViewGtkWidget::OnFocusIn() and
RenderWidgetHostViewGtkWidget::OnFocusOut().

[Evan Martin, 2009-06-29 14:45:48]

LGTM

[Evan Stade, 2009-06-29 17:29:56]

lg

http://codereview.chromium.org/151010/diff/17/1007
File chrome/browser/renderer_host/render_widget_host_view_gtk.cc (right):

http://codereview.chromium.org/151010/diff/17/1007#newcode191
Line 191: // Notify this focus-in event to the GtkIMContext object and
nit: "Notify the GtkIMContext object of this focus-in event" (here and below)