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

Issue 8543002: Remove fake context support. (Closed)

Created:
9 years, 1 month ago by Yusuke Sato
Modified:
9 years, 1 month ago
Reviewers:
Peng
CC:
chromium-reviews, tfarina, James Su, penghuang+watch_chromium.org, dhollowa
Visibility:
Public.

Description

Remove fake context support. The fake context support in views::InputMethodIBus is no longer necessary because on Chrome OS, all input method hot keys are now handled inside Chrome. On Linux, views::InputMethodIBus might be used in the future, but it will also be okay because on the OS, the panel process for IBus, /usr/share/ibus/ui/gtk/main.py, will handle all input method hot keys. BUG=chromium-os:22905 TEST=manual Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109842

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -94 lines) Patch
M views/ime/input_method_ibus.h View 5 chunks +2 lines, -14 lines 0 comments Download
M views/ime/input_method_ibus.cc View 12 chunks +12 lines, -80 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Yusuke Sato
9 years, 1 month ago (2011-11-13 07:45:40 UTC) #1
Peng
On 2011/11/13 07:45:40, Yusuke Sato wrote: lgtm.
9 years, 1 month ago (2011-11-14 01:13:49 UTC) #2
commit-bot: I haz the power
No LGTM from valid reviewers yet.
9 years, 1 month ago (2011-11-14 01:49:22 UTC) #3
Yusuke Sato
9 years, 1 month ago (2011-11-14 01:51:49 UTC) #4
On 2011/11/14 01:49:22, I haz the power (commit-bot) wrote:
> No LGTM from valid reviewers yet.

hmm.. weird. Peng should already be a Chromium committer. I'll submit the change
manually for now.

Powered by Google App Engine
This is Rietveld 408576698