Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2104)

Unified Diff: chrome/browser/renderer_host/gtk_im_context_wrapper.h

Issue 160577: Cleanup GtkIMContext related code by splitting it into separated source file. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/renderer_host/gtk_im_context_wrapper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/gtk_im_context_wrapper.h
===================================================================
--- chrome/browser/renderer_host/gtk_im_context_wrapper.h (revision 0)
+++ chrome/browser/renderer_host/gtk_im_context_wrapper.h (revision 0)
@@ -0,0 +1,158 @@
+// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_RENDERER_HOST_GTK_IM_CONTEXT_WRAPPER_H_
+#define CHROME_BROWSER_RENDERER_HOST_GTK_IM_CONTEXT_WRAPPER_H_
+
+#include <gdk/gdk.h>
+
+#include "base/basictypes.h"
+#include "base/string16.h"
+
+namespace gfx {
+class Rect;
+}
+
+class RenderWidgetHostViewGtk;
+class NativeWebKeyboardEvent;
+typedef struct _GtkIMContext GtkIMContext;
+
+// This class is a convenience wrapper for GtkIMContext.
+// It creates and manages two GtkIMContext instances, one is GtkIMMulticontext,
+// for plain text input box, another is GtkIMContextSimple, for password input
+// box.
+//
+// This class is in charge of dispatching key events to these two GtkIMContext
+// instances and handling signals emitted by them. Key events then will be
+// forwarded to renderer along with input method results via corresponding host
+// view.
+//
+// This class is used solely by RenderWidgetHostViewGtk.
+class GtkIMContextWrapper {
+ public:
+ explicit GtkIMContextWrapper(RenderWidgetHostViewGtk* host_view);
+ ~GtkIMContextWrapper();
+
+ // Processes a gdk key event received by |host_view|.
+ void ProcessKeyEvent(GdkEventKey* event);
+
+ // Updates IME status and caret position.
+ void UpdateStatus(int control, const gfx::Rect& caret_rect);
+ void OnFocusIn();
+ void OnFocusOut();
+
+ private:
+ // Check if a text needs commit by forwarding a char event instead of
+ // by confirming as a composition text.
+ bool NeedCommitByForwardingCharEvent();
+
+ void ProcessFilteredKeyPressEvent(NativeWebKeyboardEvent* wke);
+ void ProcessUnfilteredKeyPressEvent(NativeWebKeyboardEvent* wke);
+
+ // Processes result returned from input method after filtering a key event.
+ // |filtered| indicates if the key event was filtered by the input method.
+ void ProcessInputMethodResult(const GdkEventKey* event, bool filtered);
+
+ void CompleteComposition();
+
+ // Real code of "commit" signal handler.
+ void HandleCommit(const string16& text);
+
+ // Real code of "preedit-start" signal handler.
+ void HandlePreeditStart();
+
+ // Real code of "preedit-changed" signal handler.
+ void HandlePreeditChanged(const string16& text, int cursor_position);
+
+ // Real code of "preedit-end" signal handler.
+ void HandlePreeditEnd();
+
+ // Signal handlers of GtkIMContext object.
+ static void HandleCommitThunk(GtkIMContext* context, gchar* text,
+ GtkIMContextWrapper* self);
+ static void HandlePreeditStartThunk(GtkIMContext* context,
+ GtkIMContextWrapper* self);
+ static void HandlePreeditChangedThunk(GtkIMContext* context,
+ GtkIMContextWrapper* self);
+ static void HandlePreeditEndThunk(GtkIMContext* context,
+ GtkIMContextWrapper* self);
+
+ // The parent object.
+ RenderWidgetHostViewGtk* host_view_;
+
+ // The GtkIMContext object.
+ // In terms of the DOM event specification Appendix A
+ // <http://www.w3.org/TR/DOM-Level-3-Events/keyset.html>,
+ // GTK uses a GtkIMContext object for the following two purposes:
+ // 1. Composing Latin characters (A.1.2), and;
+ // 2. Composing CJK characters with an IME (A.1.3).
+ // Many JavaScript pages assume composed Latin characters are dispatched to
+ // their onkeypress() handlers but not dispatched CJK characters composed
+ // with an IME. To emulate this behavior, we should monitor the status of
+ // this GtkIMContext object and prevent sending Char events when a
+ // GtkIMContext object sends a "commit" signal with the CJK characters
+ // composed by an IME.
+ GtkIMContext* context_;
+
+ // A GtkIMContextSimple object, for supporting dead/compose keys when input
+ // method is disabled, eg. in password input box.
+ GtkIMContext* context_simple_;
+
+ // Whether or not this widget is focused.
+ bool is_focused_;
+
+ // Whether or not the above GtkIMContext is composing a text with an IME.
+ // This flag is used in "commit" signal handler of the GtkIMContext object,
+ // which determines how to submit the result text to WebKit according to this
+ // flag.
+ // If this flag is true or there are more than one characters in the result,
+ // then the result text will be committed to WebKit as a confirmed
+ // composition. Otherwise, it'll be forwarded as a key event.
+ //
+ // The GtkIMContext object sends a "preedit_start" before it starts composing
+ // a text and a "preedit_end" signal after it finishes composing it.
+ // "preedit_start" signal is monitored to turn it on.
+ // We don't monitor "preedit_end" signal to turn it off, because an input
+ // method may fire "preedit_end" signal before "commit" signal.
+ // A buggy input method may not fire "preedit_start" and/or "preedit_end"
+ // at all, so this flag will also be set to true when "preedit_changed" signal
+ // is fired with non-empty preedit text.
+ bool is_composing_text_;
+
+ // Whether or not the IME is enabled.
+ // This flag is actually controlled by RenderWidget.
+ // It shall be set to false when an ImeUpdateStatus message with control ==
+ // IME_DISABLE is received, and shall be set to true if control ==
+ // IME_COMPLETE_COMPOSITION or IME_MOVE_WINDOWS.
+ // When this flag is false, keyboard events shall be dispatched directly
+ // instead of sending to context_.
+ bool is_enabled_;
+
+ // Whether or not it's currently running inside key event handler.
+ // If it's true, then preedit-changed and commit handler will backup the
+ // preedit or commit text instead of sending them down to webkit.
+ // key event handler will send them later.
+ bool is_in_key_event_handler_;
+
+ // Stores a copy of the most recent preedit text retrieved from context_.
+ // When an ImeUpdateStatus message with control == IME_COMPLETE_COMPOSITION
+ // is received, this stored preedit text (if not empty) shall be committed,
+ // and context_ shall be reset.
+ string16 preedit_text_;
+
+ // Stores the cursor position in the stored preedit text.
+ int preedit_cursor_position_;
+
+ // Whether or not the preedit has been changed since last key event.
+ bool is_preedit_changed_;
+
+ // Stores a copy of the most recent commit text received by commit signal
+ // handler.
+ string16 commit_text_;
+
+ DISALLOW_COPY_AND_ASSIGN(GtkIMContextWrapper);
+};
+
+
+#endif // CHROME_BROWSER_RENDERER_HOST_GTK_IM_CONTEXT_WRAPPER_H_
Property changes on: chrome/browser/renderer_host/gtk_im_context_wrapper.h
___________________________________________________________________
Name: svn:eol-style
+ LF
« no previous file with comments | « no previous file | chrome/browser/renderer_host/gtk_im_context_wrapper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698