Skip to content
This repository has been archived by the owner on Aug 20, 2020. It is now read-only.

Backspace removes two characters #419

Open
mcatanzaro opened this issue Sep 8, 2015 · 8 comments
Open

Backspace removes two characters #419

mcatanzaro opened this issue Sep 8, 2015 · 8 comments
Labels

Comments

@mcatanzaro
Copy link
Contributor

The backspace key removes two characters instead of one.

@michaelforney
Copy link
Contributor

I am also observing this behavior.

@kalyankondapally
Copy link
Contributor

Yes, I observe the same. It would be interesting to check the behavior on x11 to check if this is ozone-wayland specific

@michaelforney
Copy link
Contributor

I dug into this a little bit.

It is because in DesktopWindowTreeHostOzone::DispatchEvent, you call SendEventToProcessor(event) for key events. DesktopWindowTreeHostX11 does not. I believe this was just an error in commit 25315fc.

Note in https://codereview.chromium.org/1236923003/diff/140001/ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc that they not only started calling GetInputMethod()->DispatchKeyEvent, but they also stopped calling SendEventToProcessor for key events.

@kalyankondapally
Copy link
Contributor

Thx for looking into this. Will check this, do you have a patch already ?

@jbrianceau
Copy link
Contributor

Seems to be the same problem I bumped into in #453
So this issue and #453 are very likely to be duplicates.

@liveopp
Copy link

liveopp commented Apr 11, 2016

Is this fixed?

@jbrianceau
Copy link
Contributor

As far as I know, it is not fixed yet @liveopp
Is this patch fixing the issue for you : jbrianceau@ec897cd ?

@jbrianceau
Copy link
Contributor

Should be fixed in master branch now : 607abb6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants