Blocked volume up/down events that originated from remapped function keys.

ui/events/x/events_x_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <cstring>
 
 #include <X11/extensions/XInput2.h>
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
 #include <X11/XKBlib.h>
@@ skipping 41 matching lines @@
 
   // We don't bother setting fields that the event code doesn't use, such as
   // x_root/y_root and window/root/subwindow.
   XKeyEvent* key_event = &(event->xkey);
   key_event->display = display;
   key_event->type = is_press ? KeyPress : KeyRelease;
   key_event->keycode = keycode;
   key_event->state = state;
 }
 
+bool HasFunctionKeyFlagSet(Display* display, int x_keysym) {
+  XEvent event;
+  int x_keycode = XKeysymToKeycode(display, x_keysym);
+  // Exclude keysyms for which the server has no corresponding keycode.
+  if (x_keycode) {
+    InitKeyEvent(display, &event, true, x_keycode, 0);
+    ui::KeyEvent ui_key_event(&event, false);
+    return static_cast<bool>(ui_key_event.flags() & ui::EF_FUNCTION_KEY);

[sadrul, 2014/05/30 21:23:21]

Can you just do 'return (flags & ui::EF_FUNCTION_KEY);'?

If not, how about 'return !!(flags & ui::EF_FUNCTION_KEY)' instead of
static_cast<>?

[bruthig, 2014/06/02 14:22:01]

Done.

+  }
+  return false;

[sadrul, 2014/05/30 21:23:21]

Call the function HasFunctionKeyFlagSetIfSupported(), and return true if
x_keycode == 0;

[bruthig, 2014/06/02 14:22:01]

Done.

+}
+
 }  // namespace
 
 TEST(EventsXTest, ButtonEvents) {
   XEvent event;
   gfx::Point location(5, 10);
   gfx::Vector2d offset;
 
   InitButtonEvent(&event, true, location, 1, 0);
   EXPECT_EQ(ui::ET_MOUSE_PRESSED, ui::EventTypeFromNative(&event));
   EXPECT_EQ(ui::EF_LEFT_MOUSE_BUTTON, ui::EventFlagsFromNative(&event));
@@ skipping 291 matching lines @@
       InitKeyEvent(display, &event, true, x_keycode, 0);
       // int keysym = XLookupKeysym(&event.xkey, 0);
       // if (keysym) {
       ui::KeyEvent ui_key_event(&event, false);
       EXPECT_EQ(keys[k].is_numpad_key ? ui::EF_NUMPAD_KEY : 0,
                 ui_key_event.flags() & ui::EF_NUMPAD_KEY);
     }
   }
 }
 
+TEST(EventsXTest, FunctionKeyEvents) {
+  Display* display = gfx::GetXDisplay();
+
+  // Min  function key code minus 1.
+  EXPECT_FALSE(HasFunctionKeyFlagSet(display, XK_F1 - 1));
+  // All function keys.
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F1));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F2));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F3));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F4));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F5));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F6));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F7));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F8));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F9));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F10));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F11));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F12));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F13));

[bruthig, 2014/05/30 19:10:03]

The EXPECT_TRUE for XK_F13 to XK_F35 now fails because XKeysymToKeycode returns
a value of 0 for these KeySyms.  I'm not familiar enough with X11, KeySyms and
KeyCodes to know what to do here.  Please advise.

[oshima, 2014/05/30 21:13:06]

Humm, maybe we can just skip or return true for such case.

sadrul, what do you think?

[sadrul, 2014/05/30 21:23:21]

I agree. I left a suggestion above for this.

[bruthig, 2014/06/02 14:22:01]

Done.

+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F14));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F15));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F16));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F17));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F18));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F19));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F20));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F21));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F22));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F23));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F24));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F25));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F26));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F27));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F28));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F29));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F30));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F31));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F32));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F33));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F34));
+  EXPECT_TRUE(HasFunctionKeyFlagSet(display, XK_F35));
+  // Max function key code plus 1.
+  EXPECT_FALSE(HasFunctionKeyFlagSet(display, XK_F35 + 1));
+}
+
 }  // namespace ui