[Windows Host] Ctrl+Alt+Del does not work reliably |
||
Issue descriptionWe have ongoing reports of this issue and I suspect it is related to stuck keys on the keyboard. Instead of using individual keypresses, it might be better for the client to send a single 'execute' message which invokes the platform API for the action instead of injecting each individual key down/up event and using that to trigger the action.
,
Nov 16
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b511e4a9f8c6976affd959c0c8d310986f6e2df1 commit b511e4a9f8c6976affd959c0c8d310986f6e2df1 Author: Joe Downing <joedow@chromium.org> Date: Fri Nov 16 21:10:13 2018 Adding a DataChannelHandler for Action messages This change adds a DataChannelHandler for Action messages and ties it into ClientSession. When a new handler is created, it will receive an ActionExecutor (added in a previous CL) which is used to invoke the requested action. BUG: 892434 Change-Id: I56b60fdc56b89900297943eafe78f50982b7681f Reviewed-on: https://chromium-review.googlesource.com/c/1294715 Reviewed-by: Jamie Walch <jamiewalch@chromium.org> Commit-Queue: Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#608941} [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/BUILD.gn [add] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/action_message_handler.cc [add] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/action_message_handler.h [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/client_session.cc [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/client_session.h [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/me2me_desktop_environment.cc [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/host/win/session_action_executor.cc [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/proto/action.proto [modify] https://crrev.com/b511e4a9f8c6976affd959c0c8d310986f6e2df1/remoting/protocol/capability_names.h
,
Nov 21
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Oct 17