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

Unified Diff: ash/common/accelerators/accelerator_controller.cc

Issue 2604093003: Don't mark virtual keyboard events as synthesized or distinguish between (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/extensions/api/virtual_keyboard_private/chrome_virtual_keyboard_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/accelerators/accelerator_controller.cc
diff --git a/ash/common/accelerators/accelerator_controller.cc b/ash/common/accelerators/accelerator_controller.cc
index db032bb3a8cafe407d68f3628775a7d7165a25af..285140764e39702945feaf1d354b5804d3ff111e 100644
--- a/ash/common/accelerators/accelerator_controller.cc
+++ b/ash/common/accelerators/accelerator_controller.cc
@@ -173,8 +173,7 @@ bool CanHandleCycleMru(const ui::Accelerator& accelerator) {
// users have better window switching options. See http://crbug.com/638269
keyboard::KeyboardController* keyboard_controller =
keyboard::KeyboardController::GetInstance();
- return !(keyboard_controller && keyboard_controller->keyboard_visible() &&
- (accelerator.modifiers() & ui::EF_IS_SYNTHESIZED));
+ return !(keyboard_controller && keyboard_controller->keyboard_visible());
oshima 2017/01/05 22:11:35 Hmm that's unfortunate. Could you please add comm
Evan Stade 2017/01/06 19:37:51 Done.
}
// We must avoid showing the Deprecated NEXT_IME notification erronously.
« no previous file with comments | « no previous file | chrome/browser/extensions/api/virtual_keyboard_private/chrome_virtual_keyboard_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698