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

Issue 253183002: Revert 266967 "Merge 266651 "linux-aura: Supports Compose key wi..." (Closed)

Created:
6 years, 7 months ago by Elliot Glaysher
Modified:
6 years, 7 months ago
Reviewers:
Elliot Glaysher
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 266967 "Merge 266651 "linux-aura: Supports Compose key wi..." Compile failure: ../../ui/events/x/events_x.cc: In function ‘int {anonymous}::GetEventFlagsFromXKeyEvent(XEvent*)’: ../../ui/events/x/events_x.cc:172:51: error: ‘IsKeypadKey’ was not declared in this scope > Merge 266651 "linux-aura: Supports Compose key with XIM." > > > linux-aura: Supports Compose key with XIM. > > > > Supports Compose key with XIM and makes .XCompose settings effective with Linux Aura build. > > > > BUG=351230, 360388, 349163 > > TEST=Done manually. Tested with GTK_IM_MODULE=xim XMODIFIERS=@im=none and ~/.XCompose with custom contents. > > R=erg@chromium.org, sadrul@chromium.org > > > > Review URL: https://codereview.chromium.org/243143002 > > TBR=piman@chromium.org,yukishiino@chromium.org > > Review URL: https://codereview.chromium.org/260573002 TBR=erg@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266978

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -48 lines) Patch
M branches/1916/src/chrome/browser/ui/libgtk2ui/x11_input_method_context_impl_gtk2.h View 2 chunks +0 lines, -4 lines 0 comments Download
M branches/1916/src/chrome/browser/ui/libgtk2ui/x11_input_method_context_impl_gtk2.cc View 3 chunks +2 lines, -7 lines 0 comments Download
M branches/1916/src/ui/aura/window_targeter.cc View 1 chunk +1 line, -2 lines 0 comments Download
M branches/1916/src/ui/base/ime/input_method_auralinux.cc View 1 chunk +1 line, -2 lines 0 comments Download
M branches/1916/src/ui/events/event_constants.h View 1 chunk +0 lines, -8 lines 0 comments Download
M branches/1916/src/ui/events/x/events_x.cc View 1 chunk +0 lines, -25 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Elliot Glaysher
6 years, 7 months ago (2014-04-29 22:19:56 UTC) #1
Elliot Glaysher
6 years, 7 months ago (2014-04-29 22:20:24 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r266978.

Powered by Google App Engine
This is Rietveld 408576698