Issue metadata
Sign in to add a comment
|
Ctrl-T doesn't work |
||||||||||||||||||||||
Issue descriptionChrome Version: 59.0.3048.0 OS: macOS 10.12 What steps will reproduce the problem? (1) Open http://textarea.org/ (2) Type two letters, "ab" (3) Move the caret between "a" and "b" (4) Type Ctrl-T What is the expected result? The editing text will be "ba" What happens instead? The whole text is selected. Please use labels and text to provide additional information. Not reproducible with Google Chrome 58 beta.
,
Mar 23 2017
Route to Mac team, it seems key binding is changed on Mac platform.
Observation:
- The sample[1] verifies execCommand('transpose') on Canary and ToT.
- "Transpose" command is routed from [2], which isn't modified last 12 months.
[1] https://jsfiddle.net/bqpx2awj/
[2] https://cs.chromium.org/chromium/src/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper.mm
,
Mar 23 2017
,
Mar 23 2017
Able to reproduce the issue on Mac-10.12.3 using chrome canary 59.0.3048.0. This is regression issue broken in M59.Please find the bisect information as below Narrow Bisect:: =============== Good::59.0.3043.0 -- (build revision 457297) Bad ::59.0.3044.0 -- (build revision 457648) After executing the per-revision bisect script , i got the following CL's between good and bad build versions =========================================== https://chromium.googlesource.com/chromium/src/+log/853496a78ae997c2d8b80f3cd8fabf9423fb3361..ef8ab61bac3cd1a4d714fbc4c1b9e5a042df3422 Review-Url: https://codereview.chromium.org/2747943005 chongz@could you please look into this issue if it is related to your change,else please help us in finding the appropriate owner for this issue. Note:Issue is specific to Mac Thanks,
,
Mar 23 2017
,
Mar 23 2017
Working on the fix.
,
Mar 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/34e20090704e62dbb27e402b116e5a0259c385ee commit 34e20090704e62dbb27e402b116e5a0259c385ee Author: chongz <chongz@chromium.org> Date: Fri Mar 24 15:04:28 2017 [InputEvent] Allow transpose and yank in plain-text field BUG= 704036 Review-Url: https://codereview.chromium.org/2768303003 Cr-Commit-Position: refs/heads/master@{#459424} [modify] https://crrev.com/34e20090704e62dbb27e402b116e5a0259c385ee/third_party/WebKit/LayoutTests/fast/events/inputevents/inputevent-transpose.html [modify] https://crrev.com/34e20090704e62dbb27e402b116e5a0259c385ee/third_party/WebKit/LayoutTests/fast/events/inputevents/inputevent-yank.html [modify] https://crrev.com/34e20090704e62dbb27e402b116e5a0259c385ee/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
,
Mar 24 2017
,
Mar 28 2017
Tested the issue on Mac-10.12.3 using chrome version# 59.0.3053.3 with the steps mentioned in comment#0. Observed that the fix is working as expected. Hence adding TE-Verified labels. Please find the attached screen cast for the same. Thanks!!
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by tkent@chromium.org
, Mar 22 2017