[NOT FOR COMMIT] Ignore inserting characters of Keys with modifiers on CrOS

ui/views/controls/textfield/textfield.cc

Index: ui/views/controls/textfield/textfield.cc
diff --git a/ui/views/controls/textfield/textfield.cc b/ui/views/controls/textfield/textfield.cc
index 4ae5f1acbc167a1f0a5d38ddf55cdc471cec525a..cb64f04f96ff0908ddc9d2a202cfbc4f2cd72cce 100644
--- a/ui/views/controls/textfield/textfield.cc
+++ b/ui/views/controls/textfield/textfield.cc
@@ -1449,17 +1449,19 @@ void Textfield::InsertText(const base::string16& new_text) {
 }
 
 void Textfield::InsertChar(base::char16 ch, int flags) {
+#if defined(OS_CHROMEOS)
   const int kControlModifierMask = ui::EF_CONTROL_DOWN | ui::EF_ALT_DOWN |
-                                   ui::EF_COMMAND_DOWN | ui::EF_ALTGR_DOWN |
-                                   ui::EF_MOD3_DOWN;
+                                   ui::EF_COMMAND_DOWN;
+#else
+  const int kControlModifierMask = ui::EF_ALT_DOWN;
+#endif  // defined(OS_CHROMEOS)
 
   // Filter out all control characters, including tab and new line characters,
-  // and all characters with Alt modifier. But allow characters with the AltGr
-  // modifier. On Windows AltGr is represented by Alt+Ctrl, and on Linux it's a
-  // different flag that we don't care about.
-  const bool should_insert_char =
-      ((ch >= 0x20 && ch < 0x7F) || ch > 0x9F) &&
-      (flags & kControlModifierMask) != ui::EF_ALT_DOWN;
+  // and all characters with Alt (also Ctrl and Search on CrOs) modifiers. But
+  // allow characters with the AltGr modifier. On Windows AltGr is represented
+  // by Alt+Ctrl, and on Linux it's a different flag that we don't care about.
+  const bool should_insert_char = ((ch >= 0x20 && ch < 0x7F) || ch > 0x9F) &&
+                                  (flags & kControlModifierMask) == 0;
   if (GetTextInputType() == ui::TEXT_INPUT_TYPE_NONE || !should_insert_char)
     return;