diff options
| author | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-31 17:54:03 +0000 |
|---|---|---|
| committer | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-31 17:54:03 +0000 |
| commit | c4bb35a31da8476fc2f9b19c0bb4832082c8d528 (patch) | |
| tree | 1b66cb91d1476809e33baaa50c0f176e99f2959d /chrome/renderer | |
| parent | b1b5a304d200fa38157eb003576f8e79f2be459d (diff) | |
| download | chromium_src-c4bb35a31da8476fc2f9b19c0bb4832082c8d528.zip chromium_src-c4bb35a31da8476fc2f9b19c0bb4832082c8d528.tar.gz chromium_src-c4bb35a31da8476fc2f9b19c0bb4832082c8d528.tar.bz2 | |
A fix for Issue 3234 in chromium "Input type password: IME is not disabled."
This issue is caused by my bonehead mistake that I forgot removing my legacy code, which raises a false event to Chrome. Sorry.
This change also fixes a crash that happens in Google Sites.
Review URL: http://codereview.chromium.org/8745
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4301 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer')
| -rw-r--r-- | chrome/renderer/render_widget.cc | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/chrome/renderer/render_widget.cc b/chrome/renderer/render_widget.cc index abfe528..874768b 100644 --- a/chrome/renderer/render_widget.cc +++ b/chrome/renderer/render_widget.cc @@ -652,14 +652,10 @@ void RenderWidget::GetRootWindowRect(WebWidget* webwidget, gfx::Rect* rect) { } void RenderWidget::OnImeSetInputMode(bool is_active) { - // A renderer process may move its input focus and the caret position - // while a browser process stop receiving IPC messages. - // Thus, when a browser process requests for a renderer process to send - // IPC messages, it has to check whether or not a renderer process moves - // its input focus and send an IPC message if they are updated. + // To prevent this renderer process from sending unnecessary IPC messages to + // a browser process, we permit the renderer process to send IPC messages + // only during the IME attached to the browser process is active. ime_is_active_ = is_active; - ime_control_updated_ = true; - ime_control_new_state_ = true; } void RenderWidget::OnImeSetComposition(int string_type, |